C ++:为什么这个sync()在这个Composition模式中不起作用?

时间:2012-05-19 20:11:10

标签: c++ algorithm composition

我正在尝试通过使用看起来像合成模式的东西来构建一个可以拥有任意数量的子进程条的进度条类。

假设我有这个课程pbar

class pbar
{
    public:
        pbar(const int w) { width = w; } // already sets the
        ~pbar() {}

         void setwidth(const int w) { width = w; } // set the width to w
         void show() const;
         void sync();

         void add(const pbar bar)
         {
              // add's a subbar
              subbars.pushback(bar);
         }

     private:
         std::vector<pbar> subbars; // the sub-process progressbars
         int width;                 // onscreen width of the pbar
};

正如您所看到的,pbar有两个成员:宽度和子进程条(它们本身是pbars)。我一直在努力实现一个sync函数,它会更改pbarssubbars的所有宽度,以匹配调用它的pbar的宽度:

void pbar::sync()
{
    for ( pbar bar : subbars )
    {
         bar.setwidth(width);  // first set the width of the subbar
         bar.sync();           // secondly make it sync up it's subbars
    }
}

但这似乎不起作用。我尝试过使用这个测试程序:

int main()
{
    pbar a(1);
    pbar b(2);
    pbar c(3);
    pbar d(4);

    c.add(d);
    b.add(c);
    a.add(b);

    a.show();
    std::cout << "syncing" << std::endl;
    a.sync();
    a.show();
}

show函数定义为:

void pbar::show() const
{
    std::cout << w << std::endl;
    for ( pbar bar : subbars )
    {
         bar.show();
    }
}

预期输出为:

1
1
1
1

但它是:

1
2
3
4

奇怪的是,show()函数正确迭代到所有子栏,但看起来sync()没有(事实上,使用cout我已经确认实际确实,但似乎没有效果。)

我的代码出了什么问题?它不是使用c++0x类型的循环,因为我尝试使用较旧的迭代器循环。我找不到我犯的错误。我认为这与我在pbar中使用setwidth时更改错误sync的事实有关。

免责声明:这实际上是一个较大项目的一部分,而且这个类比这里显示的要复杂得多,但我设法使用上面的代码重现了不需要的行为(顺便说一句,这不是复制粘贴,可能包含拼写错误)

4 个答案:

答案 0 :(得分:5)

您遇到的问题是您在sync()方法的循环中使用了局部变量“bar”。这是制作每个子栏的副本,并操纵副本而不是原始版本(保留在向量中)。这就是为什么在稍后调用show()方法时没有看到更改“粘滞”的原因。

您可以通过使用引用而不是常规变量来解决此问题。尝试:

for ( pbar &bar : subbars )
{
    ...
}

您可能希望在addSubBar()方法中进行类似的更改,因为您还要在向量中保存另一个副本之前复制您传入的值。您可以通过将其参数作为参考来跳过一个副本。避免使用第二个副本需要更多的小心处理内存(我将留下另一个问题)。

答案 1 :(得分:4)

您应该存储指向子pbars的指针。在目前的情况下,您只是存储子pbars的副本。因此,虽然它们(内部副本)已更改,但外部对象不会被修改。

答案 2 :(得分:3)

  

循环

不是使用c ++ 0x类型

实际上,根据您真正想做的事情,它可能只是您正在使用的基于范围的for循环。

正如问题中所述,subbars向量存储了添加到其中的对象的副本 - 可能是您想要的也可能不是。让我们假设它是你想要的。您现在在pbar::sync()中使用的基于范围的for循环:

for ( pbar bar : subbars )
{
    // ...
}

迭代subbars向量,但在这种情况下bar变量本身就是subbars向量中每个元素的副本。因此,在for循环的每次迭代后,您对该变量所做的任何更改都会丢失。

但是,如果你更改基于范围的for循环:

for ( pbar& bar : subbars ) // note the `&`
{
    // ...
}

现在,bar是对subbars向量中对象的引用,对其进行的更改将“坚持”。

但请注意,由于subbars包含添加到其中的对象的副本,因此这些更改不会传播到添加的原始对象。这是否是你想要的取决于你想要什么。如果您希望更改一直传播到原始文件,那么您需要将指针(或智能指针)存储到原始文件而不是副本,mentioned in visier's answer也是如此。

答案 3 :(得分:0)

您应该存储pbars的引用而不是实例副本。