Re: Sanity Check
Re: Sanity Check
- Subject: Re: Sanity Check
- From: Kyle Sluder <email@hidden>
- Date: Thu, 8 Jul 2010 09:41:31 -0700
On Thu, Jul 8, 2010 at 8:57 AM, <email@hidden> wrote:
> - (IBAction)sewing:(id)sender {
>
> [[[SewingController alloc] initWithWindowNibName:@"Sewing and Color"
> andBFileName:&mBFilename] release];
This is entirely wrong. Why would you create an object just to
immediately release it? Please review the Cocoa Memory Management
Guide: http://developer.apple.com/mac/library/documentation/Cocoa/Conceptual/MemoryMgmt/MemoryMgmt.html
> }
>
>
>
> - (id) initWithWindowNibName:(NSString*)windowNibName
> andBFileName:(BFilename*)bfilename
> {
> self = [super initWithWindowNibName:windowNibName];
> if (self != nil)
> {
> [self retain];
This is insanity. Again, please reread the memory management guide.
> m_design = [[self window] contentView];
>
>
> [[self window] setDelegate:self];
Why don't you just make these outlets and wire them up in Interface
Builder? And even if you wanted to do this in code for some reason,
the appropriate place to do it would be in -windowDidLoad. Right now
you're loading the window from within the initializer, which is wrong.
> }
> return self;
> }
>
> - (void)windowWillClose:(NSNotification *)notification {
>
> if(m_design->m_dirtyDesign)
> NSLog(@"dirty message");
> else
> NSLog(@"clean, no message");
If you're writing a document-based application, NSDocument does dirty
tracking for you already.
> [self release];
Again, read the memory management guide. Perhaps consider investing in
Aaron Hillegass's Cocoa Programming for Mac OS X.
> }
--Kyle Sluder
_______________________________________________
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