[转]boost智能指针shared_ptr使用要注意的几个问题

转自http://blog.csdn.net/hongjiqin/article/details/4406805

　　boost库中的shared_ptr是个好东西，一度我认为可以像C#那样使用对象，完全不用考虑内存的释放问题。不过使用起来发现还是有许多要注意的地方。

　　首先，绝对不可以像这样使用：

int *p = new int (10);
boost::shared_ptr<int> sp1 (p);
boost::shared_ptr<int> sp2 (p);

　　这个看上去很简单，一般都不会这样犯错误。但如果在一个复杂的场景中呢？比如这样：

class CListener
{
public:
    virtual void TakeAction () = 0;
};
class CPublisher
{
public:
    CPublisher ()
    {
        cout << "CPublisher::CPublisher()" << endl;
    }
    ~CPublisher ()
    {
        cout << "CPublisher::~CPublisher()" << endl;
    }
    void SetListener (boost::shared_ptr<CListener> listener)
    {
        m_listener = listener;
        m_listener->TakeAction ();
    }
private:
    boost::shared_ptr<CListener> m_listener;
};
class CConcreteListener : public CListener
{
public:
    CConcreteListener (const boost::shared_ptr<CPublisher> publiser)
    {
        cout << "CConcreteListener::CConcreteListener ()" << endl;
        m_publisher = publiser;
        m_publisher->SetListener (boost::shared_ptr<CListener> (this));
    }
    ~CConcreteListener ()
    {
        cout << "CConcreteListener::~CConcreteListener()" << endl;
    }
    void TakeAction ()
    {
        cout << "CConcreteListener::TakeAction()" << endl;
    }
private:
    boost::shared_ptr<CPublisher> m_publisher;
};
void shared_ptr_test()
{
    boost::shared_ptr<CPublisher> publisher1 (new CPublisher);
    boost::shared_ptr<CConcreteListener> concreteListener1 (new CConcreteListener (publisher1));
}

在这里，我考虑设计一个简单的发布/定购的模式，为了简化内存管理，应用了shared_ptr。看上去好像没什么问题，
但实际并非如此，当shared_ptr_test()执行结束时会报错：

"Unhandled exception at ...: 0xC0000005: Access violation writing location 0xfeeefef2."

为什么？
这个错误太隐蔽了，问题出现在CConcreteListener的构造函数中。
这两行代码：
boost::shared_ptr<CConcreteListener> concreteListener1 (new CConcreteListener (publisher1));
和CConcreteListener构造函数中的
m_publisher->SetListener (boost::shared_ptr<CListener> (this));

如果展开用另一种方式看的话，相当于这样：
CConcreteListener *p1 = new CConcreteListener;
boost::shared_ptr<CListener> sp1 (p1);
publisher1->SetListener (sp1);
boost::shared_ptr<CConcreteListener> concreteListener1 (p1);

哦，看上去和上面那个例子犯了同样的错误，不过这里更隐蔽，更不容易发现。

那么我们改动一下，把
m_publisher->SetListener (boost::shared_ptr<CListener> (this));
从CConcreteListener的构造函数中移出去，然后在shared_ptr_test()中这样使用：

CConcreteListener (const boost::shared_ptr<CPublisher> publiser)
    {
        cout << "CConcreteListener::CConcreteListener ()" << endl;
        m_publisher = publiser;
        //m_publisher->SetListener (boost::shared_ptr<CListener> (this));
    }
//......
void shared_ptr_test()
{
    boost::shared_ptr<CPublisher> publisher1 (new CPublisher);
    boost::shared_ptr<CConcreteListener> concreteListener1 (new CConcreteListener (publisher1));
    publisher1->SetListener (concreteListener1);
}

也就是在外面显式调用SetListener，恩，这回上面那个问题消失了，程序不会崩溃了。
这回正确了？看看程序的输出把：
CPublisher::CPublisher()
CConcreteListener::CConcreteListener ()
CConcreteListener::TakeAction()

好像有点问题，shared_ptr_test()函数已经结束了，publisher1和concreteListener1都已经过了生命期，应该析构销毁了，
那他们内部的的CPublisher和CConcreteListener对象也应该被释放呀，但没看到这两个类的析构函数被调用。

如果跟踪函数的调用，会发现当publisher1和concreteListener1结束生命期析构后，内部的引用计数都是1，导致内部引用对象无法释放。
这是由于两个类CPublisher和CConcreteListener中存在循环引用。
由于shared_ptr是通过引用计数管理对象，他是无法处理循环引用的问题，导致有些对象永远无法从内存中释放。
所以，使用shared_ptr，还必须小心循环引用的问题。