0

我有一个看起来像这样的类结构:

class Bar
{
public:
    Bar & AddPart(int size, const string & name)
    {
        partitions.insert(pair<string, int>(name,size));
        return *this;
    }
    void PrintParts(ostream & os) const
    {
        map<string, int>::const_iterator it;

        for (it = partitions.begin(); it != partitions.end(); ++it)
        {
            os << it->first << " " << it->second << endl;
        }
    }
private:
    std::map<string, int> parts;
}

class Foo
{
public:
    Foo & AddElement(Bar &a)
    {
        elements.push_back(&a);
        return *this;
    }
private:
    vector<Bar *> elements;
};

我返回当前类实例的原因是能够链接方法调用,如下所示:

Foo f();
f.AddElement(Bar().AddPart(10,"abcd").AddPart(5,"xyz").AddPart(20, "jklm"));

问题是,当我调用这些链接方法然后尝试打印数据时,它已损坏。而不是得到预期的输出,应该是这样的:

abcd 10
xyz 5
jklm 20

我得到甚至无法显示的随机字符。

我尝试通过 valgrind 运行它,但我遇到了一堆错误,与这个非常相似:

Invalid read of size 8 ==3347== at 0x4EC6690: std::basic_ostream<char, std::char_traits<char> >& std::operator<< <char, std::char_traits<char>, std::allocator<char> >(std::basic_ostream<char, std::char_traits<char> >&, std::basic_string<char, std::char_traits<char>, std::allocator<char> > const&) (in /usr/lib/x86_64-linux-gnu/libstdc++.so.6.0.17)

我怎样才能解决这个问题?

4

2 回答 2

5

您的实例在Bar之后立即被销毁f.AddElement(Bar().AddPart(10,"abcd").AddPart(5,"xyz").AddPart(20, "jklm"));但是您正在保存指针

于 2013-04-14T14:51:19.523 回答
2

再想想当你把一个指向 rvalue ( Bar())的指针放入 时会发生什么elements

Foo & AddElement(Bar &a)
{
    elements.push_back(&a); // <--- Here you take address of temporary Bar()
    return *this;
}

f.AddElement(Bar().AddPart(10,"abcd").AddPart(5,"xyz").AddPart(20, "jklm"));
于 2013-04-14T14:51:38.993 回答