• 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: Best practice question
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: Best practice question


  • Subject: Re: Best practice question
  • From: Chris Suter <email@hidden>
  • Date: Sat, 28 Oct 2006 08:31:34 +1000

We enable extra warnings (using the "-W -Wall" flags) which generates a warning for assignments in if statements, and do it like this:

- (id)init
{
  if ((self = [super init])) {
    ...
  }
  return self;
}

- Chris

When we're on the topic of "best practices", it could be worth noting that placing the constant value (nil in this case) to the left of the compare operator decreases the likelihood of an involuntary assignment.

Like Shawn, I also like to make the check for a nil value explicit, so in my case it looks like this:

- (id) init
{
	if (nil != (self = [super init]))
	{
		// Stuff
	}

	return self;
}

j o a r


_______________________________________________
Do not post admin requests to the list. They will be ignored.
Cocoa-dev mailing list (email@hidden)
Help/Unsubscribe/Update your Subscription:
systems.com


This email sent to email@hidden

_______________________________________________ Do not post admin requests to the list. They will be ignored. Cocoa-dev mailing list (email@hidden) Help/Unsubscribe/Update your Subscription: This email sent to email@hidden
  • Follow-Ups:
    • Re: Best practice question
      • From: Diederik Hoogenboom <email@hidden>
    • Re: Best practice question
      • From: Diederik Hoogenboom <email@hidden>
References: 
 >Best practice question (From: "Alan Smith" <email@hidden>)
 >Re: Best practice question (From: "Shawn Erickson" <email@hidden>)
 >Re: Best practice question (From: j o a r <email@hidden>)

  • Prev by Date: Small NSTableView
  • Next by Date: Re: Small NSTableView
  • Previous by thread: Re: Best practice question
  • Next by thread: Re: Best practice question
  • Index(es):
    • Date
    • Thread