Common mistakes with delegation in Cocoa

Q: One of the delegate methods in my Cocoa application isn't being invoked. What is probably wrong?

A: When a delegate message isn't being received as you intended, it is usually a sign of one of two things, either:

  1. the method selectors don't match, often because of capitialization or a missing colon, or

  2. the delegate outlet has not been set or is simply set to the wrong object.

To illustrate further, let's say that you want to handle attempts by the user to close the window myWindow in your application. When the user clicks on the close box, the window object will send the equivalent of [delegate respondsToSelector:@selector(windowShouldClose:)] to see if it should even send the delegate message. If the receiver is nil, or the respondsToSelector: test returns NO, the delegate message will not be sent.

Verify that the method signatures match

Be sure you've implemented the right method. It is not uncommon for a method selector like -windowShouldClose: to either be misspelled (e.g. -windowShoulfClose:), miscapitalized (e.g. -windowShouldclose:) or be missing the colon (-windowShouldClose). These all describe a different method, and the runtime test will fail.

Perhaps the safest habit to develop is to simply always cut and paste the method selector. In earlier releases of Mac OS X you can find the delegate methods on the Class Reference page. In more recent releases, you'll also find delegate methods grouped together as an informal protocol (a category on NSObject). For example, the window delegate methods are declared in NSWindow.h in the NSObject(NSWindowDelegate) category.

Note: Capitalization errors are often the hardest to catch. Any of the methods of the NSTableDataSource informal protocol are likely candidates for subtle miscapitalization. For example, numberOfRowsInTableView: is often mistakenly entered as numberOfRowsInTableview:.

Ensure that the delegate is set properly

Even if you've implemented the correct method, it can be easy to overlook assigning the delegate object. Sometimes a connection may have become disconnected in the nib file or be made to the wrong object. So be sure to check and make sure that the instance variable (most often an IBOutlet named delegate) is connected correctly in Interface Builder, or alternatively, that the appropriate accessor method is being used to set the instance variable at runtime.

Listing 1: If you don't, or can't, assign a delegate in Interface Builder, you can set it in code.

- (void) awakeFromNib {
    [[NSFontManager sharedManager] setDelegate:self];
}

Prior to release, consider the use of assertions to ensure connections were established correctly.

Listing 2: Use of NSAssert() during debugging to test for a delegate connection.

#if ! defined(NDEBUG)



- (void) awakeFromNib {
    // let's make sure that connection in Interface Builder was made, and made to us
    NSAssert([myWindow delegate] == self, @"You forgot to connect the window delegate.!");

   // ...
}

#endif

Delegates and memory management

Note that delegating objects keep a weak reference to their delegates. In other words, delegates are not retained by the objects they respond to: they are free to come and go at any time. When using garbage collection, this weak reference means you don't need to do anything about this relationship when the delegate is collected. However, when using retain counting it is essential to explicitly clear this relationship when the object is deallocated.

Listing 3: When not under garbage-collection, a delegate must clear itself before deallocation.


- (void) dealloc {
    // unhook ourselves so the window won't try to message us in the future
    [myWindow setDelegate:nil];
    [super dealloc];
}

For More Information

See the Cocoa documentation for more on Delegates and Data Sources.

Document Revision History

DateNotes
2008-02-27First Version

Posted: 2008-02-27


Did this document help you?
Yes: Tell us what works for you.
It’s good, but: Report typos, inaccuracies, and so forth.
It wasn’t helpful: Tell us what would have helped.