有人可以告訴我我是否正確使用了獨特的指針

Question

我正在嘗試更多地使用智能指針，但不確定我是否正確使用了它們。 我似乎需要到處使用 std::move 。 不太明白為什么，但它的工作原理。 我的示例代碼所做的並不重要（什么都沒發生:)），我更關心我如何使用這些指針的結構。 感謝您的任何建議。

class IBehaviour {
public: virtual void Process() = 0;
};

class BehaviourA : public IBehaviour {
    public: virtual void Process() override { /* do behaviour A stuff */ }
};

class BehaviourB : public IBehaviour {
    public: virtual void Process() override {/* do behaviour B stuff */ }
};

class BehaviourC : public IBehaviour {
    public: virtual void Process() override {/* do behaviour C stuff */ }
};

class BehaviourProcessor {
public:
    BehaviourProcessor(int m) {
        if (m == 1)     behaviour = std::make_unique<BehaviourA>();
        else if (m==2)  behaviour = std::make_unique<BehaviourB>();
        else            behaviour = std::make_unique<BehaviourC>();
    }

    std::unique_ptr<IBehaviour> getBehaviour() {
        return std::move(behaviour);
    }

    std::unique_ptr<IBehaviour> behaviour = nullptr;
};

class Container {
public:
    void setProcessor(std::unique_ptr<BehaviourProcessor> ada) {
        adaproc = std::move(ada);
    }

    std::unique_ptr<BehaviourProcessor> getProcessor() {
        return std::move(adaproc);
    }

    std::unique_ptr<BehaviourProcessor> adaproc = nullptr;
};

int main()
{
    Container c;
    std::unique_ptr<BehaviourProcessor> beh = std::make_unique<BehaviourProcessor>(1);
    c.setProcessor(std::move(beh));
    c.getProcessor()->getBehaviour()->Process();

    beh = std::make_unique<BehaviourProcessor>(2);
    c.setProcessor(std::move(beh));
    c.getProcessor()->getBehaviour()->Process();

    beh = std::make_unique<BehaviourProcessor>(3);
    c.setProcessor(std::move(beh));
    c.getProcessor()->getBehaviour()->Process();
}

Answer 1

class IBehaviour { public: virtual void Process() = 0; }; std::unique_ptr<IBehaviour> behaviour = nullptr; behaviour = std::make_unique<BehaviourA>();

這是錯誤的。 通過將智能指針指向基類 sub object，它將通過該基類指針進行刪除，這將導致未定義的行為，因為基類的析構函數不是虛擬的。 解決方案：聲明~IBehaviour虛擬。

---

 std::unique_ptr<IBehaviour> getBehaviour() { return std::move(behaviour); } std::unique_ptr<BehaviourProcessor> getProcessor() { return std::move(adaproc); }

這些都是可疑的。 非右值引用限定的 function 不應從成員中移動。 這可能不是調用者所期望的。

---

考慮到BehaviourProcessor不是基數 class，目前還不清楚為什么您對Container::adaproc使用動態分配。這可能是一種不必要的悲觀。 我建議考慮一個有價值的成員：

class Container {
public:
    BehaviourProcessor adaproc;

同樣，getter 和 setter 似乎在很大程度上是多余的。 這些類作為聚合可以簡單得多。