Hi,
besides what Ken Thomases and Graham Cox already said, I have a remark concerning programming style:
            if ( colorSwitch == YES) {
                [colorBox setState:NSOnState];
            }
            else {
                [colorBox setState:NSOffState];
            }


It is bad to test a Boolean value with "colorSwitch==NO" and it is a very very bad habit to test a Boolean value with "== YES" (there is a longer thread in this forum). It is not that difficult to smoothly deal with Boolean expressions. It's not a question of efficiency but a question of style,
so the above snippet is equivalent (and more Boolean-like):

[colorBox setState: colorSwitch ? NSOnState : NSOffState];

Heinrich

--
Heinrich Giesen
[EMAIL PROTECTED]

_______________________________________________

Cocoa-dev mailing list (Cocoa-dev@lists.apple.com)

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:
http://lists.apple.com/mailman/options/cocoa-dev/archive%40mail-archive.com

This email sent to [EMAIL PROTECTED]

Reply via email to