views:

1146

answers:

3

Title says it all I guess.

Is there anything wrong with doing something like

NSString * string = [ [ NSString alloc ] init ];
...
[ string release ];

or is there any value (other than best practice) in also adding

string = nil;

?

+13  A: 

Not necessary, but good practice. If you were to inadvertently reference it after release, bad things could happen, but in Objective C there isn't any harm in referencing a nil.

Paul Tomblin
Cool. I figured as much, but I wanted to make sure there wasn't something crucial I was missing. Thanks.
LucasTizma
+2  A: 

Setting an instance variable to nil is more useful in a multi-threaded application than a single-threaded one, since with multiple threads you can't always guarantee that an instance variable will only be read before it's released.

I generally don't bother in single-threaded applications, unless there's some other compelling reason.

BJ Homer
A: 

Objective-C is really the same as C with a fancy preprocessor.

Setting a pointer to nil in Objective-C has no effect on what was once pointed to by that pointer.

Vagrant