• Open Menu Close Menu
  • Apple
  • Shopping Bag
  • Apple
  • Mac
  • iPad
  • iPhone
  • Watch
  • TV
  • Music
  • Support
  • Search apple.com
  • Shopping Bag

Lists

Open Menu Close Menu
  • Terms and Conditions
  • Lists hosted on this site
  • Email the Postmaster
  • Tips for posting to public mailing lists
Re: Does this caution need fixed? (newb)
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: Does this caution need fixed? (newb)


  • Subject: Re: Does this caution need fixed? (newb)
  • From: Michael Watson <email@hidden>
  • Date: Thu, 3 Jul 2008 15:29:49 -0400

Well, look at what you're doing:

1. First, you declare a pointer to an NSMutableString object called "theSettings".

2. You then allocate and initialize an empty NSMutableString object. A pointer to some address in memory is returned, and you assign theSettings to this value. Now theSettings is a pointer to the address of this NSMutableString object you just allocated.

3. Inside your loop, you call a method that returns a pointer to a new object, reassigning theSettings to that pointer, which overwrites the original value of theSettings. You never use the NSMutableString object you allocated before the loop. (And now you can't release it, if you're not using garbage collection. You'll leak that object.)

You should pick up a copy of K&R and read up on pointers. In the meantime, there's also Uli Kusterer's excellent Masters of the Void tutorials, which contains a good chapter on this stuff too:

http://masters-of-the-void.com/


-- m-s


On 03 Jul, 2008, at 14:57, Chris Paveglio wrote:

My code is like this:

NSMutableString *theSettings;
theSettings = [[NSMutableString alloc] init];

//myPrefs is an array of strings, each item is like "Library/Safari"

int i;
for (i = 0; i < 8; i++
{
theSettings = [NSHomeDirectory() stringByAppendingPathComponent: [myPrefs objectAtIndex:i]];
....
}


Thinking about it, do I need the alloc and init commands? Sometimes I am unsure about what needs alloc or init versus what I can just declare as a variable without doing that.

Thanks for your help!
Chris




_______________________________________________

Cocoa-dev mailing list (email@hidden)

Please do not post admin requests or moderator comments to the list.
Contact the moderators at cocoa-dev-admins(at)lists.apple.com

Help/Unsubscribe/Update your Subscription:
@bungie.org


This email sent to email@hidden

_______________________________________________

Cocoa-dev mailing list (email@hidden)

Please do not post admin requests or moderator comments to the list.
Contact the moderators at cocoa-dev-admins(at)lists.apple.com

Help/Unsubscribe/Update your Subscription:
This email sent to email@hidden


References: 
 >Re: Does this caution need fixed? (newb) (From: Chris Paveglio <email@hidden>)

  • Prev by Date: Re: How to support dictionary service in a custom text view?
  • Next by Date: Re: Does this caution need fixed? (newb)
  • Previous by thread: Re: Does this caution need fixed? (newb)
  • Next by thread: Re: Does this caution need fixed? (newb)
  • Index(es):
    • Date
    • Thread