2

我有一个类Bar引用了它的一个成员（Bar::foo）的内部：

#include<vector>
#include<algorithm>

struct Foo{
    int x, y;
};

struct Bar{
    Foo foo;
    int &x, &y;
    Bar(): x(foo.x), y(foo.y){}
    // copy constructor and assignment operator
    Bar(const Bar& other): foo(other.foo), x(foo.x), y(foo.y){}
    Bar& operator=(const Bar& other){ foo=other.foo; return *this; }
};

int main(void){
    std::vector<Bar> a, b;
    Bar p; p.x=0; p.y=0;
    a.push_back(p);
    std::copy(a.begin(),a.end(),b.begin());
}

使用（没有任何特殊选项）进行编译g++，我在赋值运算符中遇到崩溃。为什么？

Program received signal SIGSEGV, Segmentation fault.
0x0000000000400b29 in Bar::operator= (this=0x0, other=...) at ref2.cpp:14
14      Bar& operator=(const Bar& other){ foo=other.foo; return *this; }
(gdb) bt
#0  0x0000000000400b29 in Bar::operator= (this=0x0, other=...) at ref2.cpp:14
#1  0x00000000004016a0 in std::__copy_move<false, false, std::random_access_iterator_tag>::__copy_m<Bar*, Bar*> (
__first=0x405010, __last=0x405030, __result=0x0) at /usr/include/c++/4.6/bits/stl_algobase.h:329
#2  0x000000000040148d in std::__copy_move_a<false, Bar*, Bar*> (__first=0x405010, __last=0x405030, __result=0x0)
at /usr/include/c++/4.6/bits/stl_algobase.h:384
#3  0x0000000000401157 in std::__copy_move_a2<false, __gnu_cxx::__normal_iterator<Bar*, std::vector<Bar, std::allocator<Bar> > >, __gnu_cxx::__normal_iterator<Bar*, std::vector<Bar, std::allocator<Bar> > > > (__first=..., __last=..., 
__result=...) at /usr/include/c++/4.6/bits/stl_algobase.h:422
#4  0x0000000000400cc0 in std::copy<__gnu_cxx::__normal_iterator<Bar*, std::vector<Bar, std::allocator<Bar> > >, __gnu_cxx::__normal_iterator<Bar*, std::vector<Bar, std::allocator<Bar> > > > (__first=..., __last=..., __result=...)
at /usr/include/c++/4.6/bits/stl_algobase.h:454
#5  0x00000000004009f4 in main () at ref2.cpp:21
4

2 回答 2

4

这合法吗？

不，因为匿名结构是不合法的。如果您命名它，它将调用未定义的行为，除非 foo首先正确初始化（特别是如果您初始化，则无法读取xandy成员data）。

一旦我解决了所有这些问题，这还会导致什么其他问题？

嗯，首先，拥有任何类型的引用成员都会抑制赋值运算符的生成。这对你来说可能重要也可能不重要。然后，编译器生成的复制构造函数会咬你：它只是盲目地复制所有成员，以成员的副本foo和引用原始对象foo的引用成员结束。如果这个原始对象的生命周期较短，就会发生不好的事情。您可能想要编写一个复制构造函数，或者考虑没有引用成员的替代设计。

于 2012-03-19T23:12:41.990 回答
0

首先初始化您的工会成员，否则它是 IIRC 未定义的行为。然后您也可以初始化对您的工会成员的引用。

于 2012-03-19T23:12:15.573 回答