Question

I hope someone can tell me why I’m wrong. Here is a snippet of my code:

TimeLogAppDelegate *appDelegate = (TimeLogAppDelegate *)[[UIApplication sharedApplication] delegate];

PickFromListViewController * pl = [[PickFromListViewController alloc] initWithNibName:@"PickList" bundle:nil];
pickList = pl;
[pickList setSearchItems:[appDelegate tableListMutableArray:type ] :NSLocalizedString(type,nil)];
pickList.callingViewController = self;
[pl release];
pickList.responseSelector = [[type lowercaseString] stringByAppendingString: @"Selected:"];
pickList.includeNone = YES;
pickList.includeNew = YES;
[self.navigationController pushViewController:pickList animated:YES];

As you can see, I am releasing pl half-way through, just to create the problem. ‘pickList’ is obviously a PickFromListViewController and is declared in the Header. I set it up as a property (@property (nonatomic, retain) PickFromListViewController *pickList;) and I @synthesize it.

My problem is:

after pl is release I get a BAD ACCESS error accessing pickList indicating that the pointer is no longer available, but I thought the fact that pickList is synthesized, it would be retained until I release it at dealloc?

Can someone tell me why I am wrong?

Many thanks

Answer 1

Setting up your pickList property as (nonatomic, retain) only applies if you access the property via a property accessor, as follows:

self.pickList = P1;

or

[self setPickList:P1];

If you simply assign a value to the member variable:

pickList = P1;

You are bypassing the property accessor methods, so no retain message is sent.