views:

3781

answers:

3

Hi all,

I'm surfing through the wonderful blog maintained by Scott Stevenson, and I'm trying to understand a fundamental Objective-C concept of assigning delegates the 'assign' property vs 'retain'. Note, the both are the same in a garbage collected environment. I'm mostly concerned with a non-GC based environment (eg: iPhone).

Directly from Scott's blog:

"The assign keyword will generate a setter which assigns the value to the instance variable directly, rather than copying or retaining it. This is best for primitive types like NSInteger and CGFloat, or objects you don't directly own, such as delegates."

What does it mean that you don't directly own the delegate object? I typically retain my delegates, because if I don't want them to go away into the abyss, retain will take care of that for me. I usually abstract UITableViewController away from its respective dataSource and delegate also. I also retain that particular object. I want to make sure it never goes away so my UITableView always has its delegate around.

Can someone further explain where/why I'm wrong, so I can understand this common paradigm in Objective-C 2.0 programming of using the assign property on delegates instead of retain?

Thanks!

+27  A: 

The reason that you avoid retaining delegates is that you need to avoid a retain loop:

A creates B A sets itself as B's delegate … A is released by its owner

If B had retained A, A wouldn't be released, as B owns A, thus A's dealloc would never get called, causing both A and B to leak.

You shouldn't worry about A going away b/c it owns B and thus gets rid of it in dealloc.

Andrew Pouliot
In other words it's a classic circular reference, which as we all know causes problems with reference-counted memory management (retain/release). Plus there is not really going to be a situation where the delegate is released but the child doing the delegation is still retained by someone else.
Mike Weller
I don't agree, Mike. I just found a problem where a modal has a delegate that dismisses the modal. But when I do a memory warning in the modal, it releases the delegate. Then when I go to dismiss my modal, the delegate is nil. Crash.
Paul Shapiro
Ok so I disagree, but you're right that it's a design flaw. I found out that I was passing on my dismiss call through a child class of the real dismisser. That child class got released and couldn't pass onto the modal's container delegate to dismiss. I changed it to pass on a pointer to the final delegate and that one didn't get released at memory warning and all is good.
Paul Shapiro
+8  A: 

Because the object sending the delegate messages does not own the delegate.

Many times, it's the other way around, as when a controller sets itself as the delegate of a view or window: the controller owns the view/window, so if the view/window owned its delegate, both objects would be owning each other. This, of course, is a retain cycle, similar to a leak with the same consequence (objects that should be dead remain alive).

Other times, the objects are peers: neither one owns the other, probably because they are both owned by the same third object.

Either way, the object with the delegate should not retain its delegate.

(There's at least one exception, by the way. I don't remember what it was, and I don't think there was a good reason for it.)

Peter Hosey
+2  A: 

Note that when you have a delegate that's assign, it makes it very important to always set that delegate value to nil whenever the object is going to be deallocated - so an object should always be careful to nil out delegate references in dealloc if it has not done so elsewhere.

Kendall Helmstetter Gelner
“Note that when you have a delegate that's assign, it makes it very important to always set that delegate value to nil whenever the object is going to be deallocated” Why?
Peter Hosey
Because any reference left set, will be invalid after an object is deallocated (pointing at memory no longer allocated to the kind of object expected) - and thus cause a crash if you try to use it.A sign of this in the debugger is when the debugger claims some variable has a type that seems totally wrong from what the variable is actually declared as.
Kendall Helmstetter Gelner
This is only necessary if the object that you are a delegate of is being retained by another source, such as a timer or other asynchronous callback. Otherwise, it will be dealloced after you release it and won't be trying to call delegate methods.
Andrew Pouliot
@Andrew: That's true, but if you always make it a practice to nil out delegates then you won't forget when it matters, or if you accidentally over-retain a held object and it stays around anyway. If you nil out the delegate then the result is just a leak, instead of a leak followed by a crash.
Kendall Helmstetter Gelner