两相结构具有以下形状:
- struct something {
- something ()
- : p1(NULL),p2(NULL)
- { }
- ~something () {
- if (p1) delete p1;
- if (p2) delete p2;
- }
- void initialize () {
- p1 = new int(2);
- p2 = new int(5); // May throw if allocation fails!
- }
- int* p1;
- int* p2;
- };
其中一个天真的构造函数(不注意分配失败)将泄漏内存:从不调用部分构造的对象的析构函数.
我的问题:以下代码是安全的,并且通过证据,智能指针是否可以避免两阶段构造?
- struct something {
- something ()
- : p1(new int(2)),p2(new int(5))
- { }
- std::unique_ptr<int> p1;
- std::unique_ptr<int> p2;
- };
解决方法
是的,你的新代码很好.但请注意,在更复杂的情况下可能会有微妙的情况:
- #include <memory>
- struct foo {
- foo(std::shared_ptr<int> a,std::shared_ptr<int> b) { }
- };
- struct bar {
- foo f;
- bar() : f(std::shared_ptr<int>(new int),std::shared_ptr<int>(new int)) { }
- };
- int main() {
- bar b;
- }
但是不安全,因为未指定在bar的初始化列表中评估foo的构造函数的参数的顺序.符合标准的编译器可能会选择执行深度或广度的第一个评估顺序(或其他任何东西,只要它们最终都被正确评估).这意味着如果第一个新int成功,但第二个新int在构造shared_ptr对象之前抛出,则第一个要执行的分配仍然可能泄漏.
如果您发现自己想要这样做,除了回到两阶段构造之外还有两种可能的解决方案:第一种可能是重构,第二种是在f之前首先单独构造shared_ptrs作为bar的成员.其中哪一项最合适的是我认为需要根据具体情况进行判断.