这个指针和 QSharedPointer

Question

我有一个类似 class 的树节点，名为Message ，如下所示：

class Message
{
public:
    using Ptr = QSharedPointer<Message>;

public:
    explicit Message();
    explicit Message(Message::Ptr parentPtr);
    explicit Message(const Data &data, Message::Ptr parentPtr = Message::Ptr());

    void setParent(Message::Ptr parentPtr);
    Message::Ptr parent() const;
    bool hasParent() const;

    QSet<Message::Ptr> children() const;
    void setChildren(const QSet<Message::Ptr> &children);
    bool hasChildren() const;

    Data data() const;
    void setData(const Data &data);

private:
    void addChild(Message::Ptr childPtr);
    void removeChild(Message::Ptr childPtr);

private:
    Message::Ptr m_parentPtr;
    QSet<Message::Ptr> m_children;
    Data m_data;
};

这个 class 可以有一个父母和一组孩子。 我在实现addChild和setParent成员函数时遇到问题：

void Message::addChild(Message::Ptr childPtr)
{
    if (!m_children.contains(childPtr)) {
        m_children.insert(childPtr);
    }

    Message::Ptr thisPtr(this);

    if (childPtr->parent() != thisPtr) {
        childPtr->setParent(thisPtr);
    }
}

void Message::setParent(Message::Ptr parentPtr)
{
    if (m_parentPtr != parentPtr) {
        m_parentPtr = parentPtr;

        m_parentPtr->addChild(Message::Ptr(this));
    }
}

我期望会发生的事情：

1. Message::addChild被调用
2. thisPtr创建时引用计数为 1
3. childPtr->parent() != thisPtr将被解析为true
4. childPtr->setParent(thisPtr); , Message::setParent被执行并且thisPtr引用计数将随着创建共享指针的副本而增加 1。 现在thisPtr的引用计数为 2
5. 当Message::setParent被执行时， m_parentPtr = parentPtr; 会将m_parentPtr 、 parentPtr和thisPtr引用计数增加 1； 这 3 个智能指针现在的引用计数为 3。
6. 执行退出Message::setParent并销毁parentPtr将m_parentPtr和thisPtr的引用计数减 1
7. 执行返回到Message::addChild 。 现在thisPtr的引用计数是 2。

实际发生了什么：

当执行退出Message::addChild中的if语句时， thisPtr引用计数再次减少 1，使thisPtr的引用计数为 1。这使得一切都中断，因为当执行存在Message::addChild时， thisPtr被销毁，因此this被删除。

我的问题：

为什么thisPtr引用计数会在执行退出Message::addChild中的if语句或那里实际发生的情况时再次减少？...

以下是它在调试器中的运行方式：

Answer 1

5. 当Message::setParent被执行时， m_parentPtr = parentPtr; 将增加m_parentPtr ， parentPtr ，因此thisPtr引用计数增加1； 这3个智能指针现在的引用计数为3。

5.1。 然后， setParent构造一个指向引用计数为1的子级的临时共享指针，并在父级上调用addChild ：

m_parentPtr->addChild(Message::Ptr(this));

5.2。 addChild创建一个指向引用计数为1的父级的共享指针：

Message::Ptr thisPtr(this);

5.3。 addChild返回，破坏了5.2的共享指针，破坏了父节点，这破坏了父节点的QSet<Message::Ptr> m_children成员。

5.4。 5.1的临时共享指针被破坏，这将破坏子代。

一般来说，您有一个循环引用：父母拥有孩子，而孩子拥有父母，这是内存泄漏和删除后使用错误的秘诀。 构造拥有其他共享指针已经拥有的原始指针的新共享指针是双重删除和使用后删除错误的诀窍。 共享的指针不会相互了解，它们的引用计数将独立变化。 您应该研究QWeakPointer来打破循环，并研究QEnableSharedFromThis以安全地获取指向*this的共享指针。

Answer 2

请参阅Qt文档中的 QEnableSharedFromThis。 这应该可以解决您的问题。