我想知道在这个使用std :: experimental :: optional和union类型的简单示例中可能导致段错误的原因。令人好奇的是,clang和gcc都发生了段错误,但是在两个不同的地方。
我也对从下面的日志中看到的大量复制和破坏行为感到困惑,想知道是否有更好/惯用的方法来避免这么多明显多余的操作?在这种情况下,假设这里的所有对象都按值传递和访问,是否有助于切换所有构造函数以获取右值引用并使用std :: move到处?
#include <iostream>
#include <vector>
// https://github.com/akrzemi1/Optional
#include "Optional/optional.hpp"
using std::cout;
using std::vector;
using std::experimental::optional;
struct X {
int y;
X(int y) : y(y) { cout << "X::X(int)\n"; }
X(const X& x) : y(x.y) { cout << "X::X(const X&)\n"; }
~X() noexcept { cout << "X::~X()\n"; }
};
struct A {
vector<X> x;
A(const vector<X>& x) : x(x) { cout << "A::A(const vector<X>&)\n"; }
A(const A& a) : x(a.x) { cout << "A::A(const A&)\n"; }
~A() noexcept { cout << "A::~A()\n"; }
static optional<A> get() {
cout << "A::get()\n";
return A({ X(1), X(2) });
}
};
struct M {
union { A a; };
M(A a) : a(a) {cout << "M::M(A)\n";}
M(const M &m) { a = m.a; }
~M() noexcept {
cout << "M::~M()\n";
(&a)->A::~A();
}
static optional<M> get() {
cout << "M::get()\n";
auto a = A::get();
return M(*a);
}
};
struct P {
vector<M> m;
P(const vector<M>& m) : m(m) { cout << "P::P(const vector<M>&)\n"; }
P(const P& p) : m(p.m) { cout << "P::P(const P&)\n"; }
static optional<P> get() {
cout << "P::get()\n";
auto m1 = M::get();
auto m2 = M::get();
vector<M> m;
cout << "push message 1\n";
m.push_back(*m1);
cout << "push message 2\n";
m.push_back(*m2);
return P(m);
}
};
int main() {
auto p = P::get();
cout << (*p).m[1].a.x[0].y << "\n";
}
GCC失败了:
P::get()
M::get()
A::get()
X::X(int)
X::X(int)
X::X(const X&)
X::X(const X&)
X::X(const X&)
X::X(const X&)
A::A(const vector<X>&)
X::X(const X&)
X::X(const X&)
A::A(const A&)
A::~A()
X::~X()
X::~X()
X::~X()
X::~X()
X::~X()
X::~X()
X::X(const X&)
X::X(const X&)
A::A(const A&)
M::M(A)
X::X(const X&)
X::X(const X&)
A::A(const A&)
'./a.out' terminated by signal SIGBUS (Misaligned address error)
#0 0x0000000100003c59 in X* std::__copy_move<false, false, std::random_access_iterator_tag>::__copy_m<X const*, X*>(X const*, X const*, X*) ()
#1 0x000000010000364e in X* std::__copy_move_a<false, X const*, X*>(X const*, X const*, X*) ()
#2 0x0000000100002f3c in __gnu_cxx::__normal_iterator<X*, std::vector<X, std::allocator<X> > > std::__copy_move_a2<false, __gnu_cxx::__normal_iterator<X const*, std::vector<X, std::allocator<X> > >, __gnu_cxx::__normal_iterator<X*, std::vector<X, std::allocator<X> > > >(__gnu_cxx::__normal_iterator<X const*, std::vector<X, std::allocator<X> > >, __gnu_cxx::__normal_iterator<X const*, std::vector<X, std::allocator<X> > >, __gnu_cxx::__normal_iterator<X*, std::vector<X, std::allocator<X> > >) ()
#3 0x00000001000025f8 in __gnu_cxx::__normal_iterator<X*, std::vector<X, std::allocator<X> > > std::copy<__gnu_cxx::__normal_iterator<X const*, std::vector<X, std::allocator<X> > >, __gnu_cxx::__normal_iterator<X*, std::vector<X, std::allocator<X> > > >(__gnu_cxx::__normal_iterator<X const*, std::vector<X, std::allocator<X> > >, __gnu_cxx::__normal_iterator<X const*, std::vector<X, std::allocator<X> > >, __gnu_cxx::__normal_iterator<X*, std::vector<X, std::allocator<X> > >) ()
#4 0x0000000100001d19 in std::vector<X, std::allocator<X> >::operator=(std::vector<X, std::allocator<X> > const&) ()
#5 0x00000001000012ad in A::operator=(A const&) ()
#6 0x00000001000012d7 in M::M(M const&) ()
#7 0x0000000100001356 in std::experimental::storage_t<M>::storage_t<M>(M&&) ()
#8 0x0000000100001393 in std::experimental::optional_base<M>::optional_base(M&&) ()
#9 0x00000001000013c4 in std::experimental::optional<M>::optional(M&&) ()
#10 0x0000000100001456 in M::get() ()
#11 0x00000001000016a8 in P::get() ()
#12 0x0000000100000db1 in main ()
而clang有时不会崩溃,有时会这样:
P::get()
M::get()
A::get()
X::X(int)
X::X(int)
X::X(const X&)
X::X(const X&)
X::X(const X&)
X::X(const X&)
A::A(const vector<X>&)
X::X(const X&)
X::X(const X&)
A::A(const A&)
A::~A()
X::~X()
X::~X()
X::~X()
X::~X()
X::~X()
X::~X()
X::X(const X&)
X::X(const X&)
A::A(const A&)
M::M(A)
X::X(const X&)
X::X(const X&)
A::A(const A&)
X::X(const X&)
X::X(const X&)
M::~M()
A::~A()
X::~X()
X::~X()
A::~A()
X::~X()
X::~X()
A::~A()
X::~X()
X::~X()
M::get()
A::get()
X::X(int)
X::X(int)
X::X(const X&)
X::X(const X&)
X::X(const X&)
X::X(const X&)
A::A(const vector<X>&)
X::X(const X&)
X::X(const X&)
A::A(const A&)
A::~A()
X::~X()
X::~X()
X::~X()
X::~X()
X::~X()
X::~X()
X::X(const X&)
X::X(const X&)
A::A(const A&)
M::M(A)
X::X(const X&)
X::X(const X&)
A::A(const A&)
X::X(const X&)
X::X(const X&)
M::~M()
A::~A()
X::~X()
X::~X()
A::~A()
X::~X()
X::~X()
A::~A()
X::~X()
X::~X()
push message 1
'./a.out' terminated by signal SIGSEGV (Address boundary error)
答案 0 :(得分:6)
您没有在A
ctor案例中构建M const&
,只是在未初始化时才分配给它。
工会不会构建他们的内容。
这与可选项无关。
答案 1 :(得分:4)
问题(如果我没有错)不是由std::experimental::optional
引起的;它是由union { A a; }
中的struct M
引起的,并且已在M
复制构造函数中使用。
union { A a; }
是一个C ++ 11联合,具有非平凡的(具有非平凡的构造函数)。所以,如果我记得很清楚,那么构造函数(和析构函数等)就会被删除。
我们可以观察到M
M(const M &m) { a = m.a; }
没有初始化列表;所以联合中的a
元素未初始化。何时执行作业
a = m.a;
等号左边的a
未初始化(即使:如果我没错),程序的行为是未定义的。
解决方案(我希望):
1)在A
A() {};
2)并在复制构造函数
中的初始化列表中初始化a
M(const M &m) : a() { a = m.a; }
---编辑---
更好的解决方案(感谢Orbit中的Lightness Races):使用A
M(const M &m) : a(m.a) { }