Question

if I run this code:

NSString *xml = [[NSString alloc] init];
xml=[NSString stringWithFormat:@"<%@>", @"msg"];
NSLog(@"%@\n",xml);
[xml release];

I got:
2010-09-07 11:45:15.523 test[1312:207] <msg>
2010-09-07 11:45:15.527 test[1312:207] *** -[CFString release]: message sent to deallocated instance 0x3d52ba0

I have red the memory guide and I think that:
if i create an object using the alloc & init methods, I should release that object, but the rule does not play well here, why?

UPDATED

it seems that this line xml=[NSString stringWithFormat:@"<%@>", @"msg"]; is the problem.
I replaced it by xml=@"something"; and it worked.

any idea why I can not use the stringWithFormat method here?

UPDATED
Thanks for the answers @(Douwe Maan, BoltClock, Rod)

I have updated the code, just testing another way:

NSString *xml = [[NSString alloc] initWithFormat,@"<%@>", @"msg"];
NSLog(@"%@\n",xml);
[xml release];

interesting: if i run that code with the debug option enabled i get a message saying:
*** -[CFString _cfTypeID]: message sent to deallocated instance 0x3954ab0

but if i run it without the debugging tool, it never sends a message saying that.

questions:
1. is it normal that the error message will not appear if i am not running the project with the debug tool?
2. so, what is the problem with initWithFormat?, is it the same that has stringWithFormat? (i think it does not create an autoreleased instance)
3. how may i know that some method returns an autoreleased instance?, is there some naming convention anywhere?

Answer 1

Both @"some string" (apparently not, read @BoltClock’s second comment) and [NSString stringWithFormat:@"some string with object: %@", someObject] return an autoreleased instance of NSString. You can use this object any way you like, but you don’t own it (since you didn’t alloc, copy or retain it), so you don’t have to and shouldn’t release it yourself.

Also, in your code, you actually have two NSString objects, the one you alloc‘d (on line 1) and the one that is alloc‘ed for you when you call stringWithFormat:, on line 2. On line 2 you’re re-pointing the variable xml to the second NSString, but the first one is still around in memory, so you’re creating a memory leak (as you haven’t released it)! You didn’t have to alloc this first one anyway, as your second NSString is the only one you’re actually using.

So the correct code would be, getting rid of the release call and the first NSString:

NSString *xml = [NSString stringWithFormat:@"<%@>", @"msg"];
NSLog(@"%@\n", xml);

---

Questions:

1. Hmm, I’m not sure why that error would occur in the first place…
2. init... should be called after alloc‘ing something, so it’s always [[Object alloc] initWithSomething]. The autorelease-version of this is [Object objectWithSomething], which does essentially this: return [[[Object alloc] initWithSomething] autorelease].
3. You don’t really need to know what does and does not return an autoreleased object, just what you own, and should thus release, and do not own. You own everything you alloc, copy or retain, and you do not own everything else.