Logo Questions Linux Laravel Mysql Ubuntu Git Menu
 

Why shouldn't you use references to smart pointers?

Tags:

I recall reading somewhere that using references to smart pointers can cause memory corruption. Is this simply because of using the reference of the smart pointer after its been destroyed? Or does the reference counting get messed up?

Thanks for clarifying

like image 543
Superpolock Avatar asked Oct 07 '08 15:10

Superpolock


3 Answers

Assuming you are talking about shared_ptr here...

Is this simply because of using the reference of the smart pointer after its been destroyed?

This is a good answer. You may not know absolutely the lifetime of the pointer your reference refers too.

To get around this, you'd want to look into boost::weak_ptr. It doesn't participate in reference counting. When you need to use it, it gives you a shared_ptr which goes away once your done with it. It will also let you know when the refered to pointer has been collected.

From the weak_ptr documentation

The weak_ptr class template stores a "weak reference" to an object that's already managed by a shared_ptr. To access the object, a weak_ptr can be converted to a shared_ptr using the shared_ptr constructor or the member function lock. When the last shared_ptr to the object goes away and the object is deleted, the attempt to obtain a shared_ptr from the weak_ptr instances that refer to the deleted object will fail: the constructor will throw an exception of type boost::bad_weak_ptr, and weak_ptr::lock will return an empty shared_ptr.

Note the method expired() will also tell you if your ptr is still around.

like image 90
Doug T. Avatar answered Sep 23 '22 17:09

Doug T.


When using smart pointers (or any allocation management object) you are counting on the behaviors defined in the constructor/destructor to manage refs/derefs/locks/unlocks. As a result, those types of objects must be true objects to perform properly. when using references to such objects (or pointers) you are bypassing the mechanism (and asking for a wedgie).

like image 22
Dan Hewett Avatar answered Sep 23 '22 17:09

Dan Hewett


There are still many cases in which a reference to a smart pointer is a good idea. An obvious example is the assignment method of the smart pointer class itself, which accept a reference to another smart pointer as its parameter.

Making a method that accepts a smart pointer reference means that the parameter doesn't increment the smart pointer's internal reference count. This can improve performance - but probably not a lot. Also, there are a lot of things the method cannot do with the reference - or the original smart pointer. If you know what these things are and avoid them, passing by reference works just fine. Of course, the purpose of smart pointers is to avoid having to know these things.

Also, if you have a method that will modify the value of a smart pointer parameter, passing as a reference is required, as it is with any other type.

like image 4
Jeff B Avatar answered Sep 22 '22 17:09

Jeff B