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

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，还必须小心循环引用的问题。