基类中 boost::shared_ptr 指向的对象永远不会被破坏

Question

我对 boost shared_ptr 有一个奇怪的问题：

class A
{

    A( )
    : m_myObjectPtr( new MyObject( ) )
    {

    }

    protected:

    boost::shared_ptr<MyObject> m_myObjectPtr; // MyObject class is a simple class with a constructor and destructor
};

class B : A
{
    B( )
    {

    }

    void CleanMyObject( )
    {
        m_myObjectPtr.reset( );
    }
};

class MyObject
{
    MyObject( )
    {
        cout << "Constructed MyObject" << endl;
    }

    ~MyObject( )
    {
        cout << "Destroyed MyObject" << endl;
    }
};

我的问题是当我调用 B::CleanMyObject() 时从未调用 MyObject 的析构函数。 “Destroyed MyObject”永远不会被打印出来。

我在 iOS 上使用 https://github.com/danoli3/ofxiOSBoost/blob/master/scripts/build-libc%2B%2B.sh 构建的 arm64 构建 boost 1_57 看到了这一点

有什么想法吗？

Answer 1

显而易见的答案是您有多个 shared_ptrs 引用单个对象，因此重置一个会减少引用计数但不会删除该对象。

即使shared_ptr 没有在A 和B 之外引用，也会发生这种情况。如果您分配 A 或 B 而不重载 operator= 或复制 A 或 B（例如，按值传递，按值返回）而不重载复制构造函数，则会出现这种情况。

您可以通过多种方式进行调查。

• 您可以检查 CleanMyObject 中的 boost::shared_ptr::use_count() 以查看它是否大于 1。
• 如果您不想共享和引用计数，可以将boost::shared_ptr 替换为std::unique_ptr 或boost::scoped_ptr。
• 如果您想确保不会意外复制或分配A 或B，可以从boost::noncopyable 派生A。

Answer 2

按照@Josh Kelley 的建议，使用 unique_ptr 为我解决了问题。

所以我将注意力转移到 boost::shared_ptr 以了解问题所在。事实证明，使用标志 BOOST_AC_USE_PTHREADS 和 BOOST_SP_USE_PTHREADS 为 iOS 构建提升是奇怪的最初原因。基于：

1) 安迪·温斯坦的回答：Boost threads: in IOS, thread_info object is being destructed before the thread finishes executing

2) 弱序CPU教程(http://preshing.com/20121019/this-is-why-they-call-it-a-weakly-ordered-cpu/)

很明显，我应该使用自旋锁，即 BOOST_SP_USE_SPINLOCK 标志来构建提升。

在我使用 BOOST_SP_USE_SPINLOCK 标志重建提升后，问题似乎得到了解决。

Answer 3

以下代码说明了一个工作示例。从 C++11 开始，它使用 std 命名空间中的 shared_ptr<T>，但您可以替换为 boost::shared_ptr<T>。

MyObject::~MyObject 在调用B::CleanMyObject 时被调用，就像您尝试做的那样。

#include <memory>
#include <iostream>
using namespace std;

class MyObject
{
public:
    MyObject()
    {
        cout << "Constructed MyObject" << endl;
    }

    ~MyObject()
    {
        cout << "Destroyed MyObject" << endl;
    }
};

class A
{
protected:
    A()
    : m_myObjectPtr(new MyObject())
    {

    }

    std::shared_ptr<MyObject> m_myObjectPtr; // MyObject class is a simple class with a constructor and destructor
};

class B : A
{
public:
    B()
    {

    }
    void CleanMyObject()
    {
        m_myObjectPtr.reset();
    }
};

int main() {
    B b;
    b.CleanMyObject();
}