Question

我最初发现了一些（讨厌的）代码来更新指向指针的指针，执行此操作：

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

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

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

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

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

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;
}

我应该在这里通过引用传递unique_ptr <t>吗？

1 个答案: