For instance:
if ( thing != NULL )
{
thing->doSomething();
thing->doSomethingElse();
thing->doSomethingMore();
}
... can become ...
if ( !thing )
return;
thing->doSomething();
thing->doSomethingElse();
thing->doSomethingMore();
... and ...
if (value == otherValue)
{
thing->doSomething();
}
else
{
thing->doSomethingElse();
}
... can become ...
if (value == otherValue)
{
thing->doSomething();
return;
}
thing->doSomethingElse();
This is why I cringe when I see this in Apple's tutorial documentation:
-(id)init
{
self = [super init];
if (self != nil) {
// initialization code
} else {
[self release];
return nil;
}
return self;
}
With this, I get to write my entire init method in two levels of indentation just so I can happily execute one return statement after the end paren.
This is also in some of XCode's class templates, and in many cocoa books that I see. I wish they would change it to this. It looks far calmer to me.
-(id)init
{
if ((self = [super init]) == nil )
{
[self release];
return nil;
}
// initialization code
return self;
}
I realize this is a small thing, but after looking over page after page of highly indented code, I get a little fatigued. The init method will likely be the first method beginning Objective-C coders see, and they will likely copy that style, never re-thinking how they indent code.
1 comment:
You disgust me.
Post a Comment