我应该通过 unique_ptr<t> 在这里参考？</t>

Question

我最初发现了一些（令人讨厌的）代码来更新指向指针的指针，这样做：

void func(X** ptr2ptr, X* oldx){

    X* x2 = new X();
    x2->a(oldx->a);
    // etc

    delete *ptr2ptr;
    *ptr2ptr = x2;
    oldx = *ptr2ptr;
}

你可以想象，这太可怕了。

我重构了上面的方法并从外部包装器调用它，然后是另一个使用更新指针的方法（见下文）。 但是，似乎我的更新 memory 在调用 anotherMethod() 之前被删除了，因为我遇到了段错误：

void wrapper(std::unique_ptr<X>& x){
    func(x);
    anotherMethod(x);
}

void func(std::unique_ptr<X>& x){
    std::unique_ptr<X> x2(new X());
    // same assignment as before

    x = std::move(x2);
}

void anotherMethod(std::unique_ptr<X>& x){
    // Seg fault occurs here whilst accessing x class member
}

有人可以帮忙吗？ 我认为我正在使用 std::move() 并通过引用传递 unique_ptr 来做正确的事情。

Answer 1

旧代码不仅仅是“移动”指针：它将a成员保留在结构中。

尝试这样的事情：

void func(std::unique_ptr<X>& x){
    auto old_a = x->a;

    x = new X;

    x->a = old_a;
}