Cocoa, varför fungerar ena metoden men ej andra?

Undersöker möjligheten att modifiera fönster och andra visuella klasser i Cocoa. Jag skrev en metod medan jag labbade med NSColorPanel. Den fungerar någon gång, sedan kraschar det.

- (void)awakeFromNib skapar färgpanelen, sätter MyController som target och meddelar färpanelen namnet på dess action.

- (void)actionChangeColor hämtar i båda fallen färgen från färgpanelen och ställer in bakgrundsfärgen på myWindow till den hämtade färgen.

#import "MyController.h"

@implementation MyController
- (void)awakeFromNib
{
	myColorPanel = [NSColorPanel sharedColorPanel];
	[myColorPanel setTarget:self];
	[myColorPanel setAction: NSSelectorFromString(@actionChangeColor)];
	[myColorPanel orderFront:nil];
	
}
- (void)actionChangeColor
{
	NSColor *myColor = [[NSColor alloc] init];
	myColor = [myColorPanel color];
	[myWindow setBackgroundColor: myColor];
	[myWindow display];
	[myColor release];
	
}	

@end

Cocoa har legat nere för mig ett bra tag, och jag har väl gjort något enkelt fel, för om jag snyggar upp - (void)actionChangeColor så den ser ut som den ska (?)

- (void)actionChangeColor
{
	[myWindow setBackgroundColor: [myColorPanel color]];
	[myWindow display];	
}	

Fungerar det utmärkt.

Vad har är det jag inte har förstått?

Vänligen, Ylan

Men när jag körde exakt samma metod fast kallad med en knapptryckning (IBAction), fungerade det utmärkt.

Vänligen, Ylan, som är oerhört imponerad av den snabba responsen.

Ledsen att jag var otydlig. Jag menade

- (void)actionChangeColor
{
	NSColor *myColor = [[NSColor alloc] init];
	myColor = [myColorPanel color];
	[myWindow setBackgroundColor: myColor];
	[myWindow display];
	[myColor release];
	
}	

Fast då hette den - (IBAction)changeColor

Vänligen, Ylan

Nu är jag med. Det visade sig att programmet inte nödvändigtvis måste krascha direkt, utan att metoden kan fungera några gånger innan det kraschar. När jag drar i punkten i färgväljaren, skickar den sin action många gånger, och jag fick intrycket av att den kraschade direkt (och jag hade inte klickat många gånger på knappen när jag testade tidigare).

Jag skulle helt enkelt strunta i att allokera minne för myColor? Eller dra en kopia av [myColorPanel color] och lägga på myColors allokerade minnesplats?

Då kommer ju frågan om när det blir så dyrt att sända meddelanden att det lönar sig att ha en egen kopia (om man använder informationen i en loop t. ex.)

Jag antar att man tar en egen kopia först när man märker att man har prestandaproblem.

Vänligen, Ylan

Tack alla, metoden

- (IBAction)changeColor:(id)sender
{
	NSColor *myColor;
	myColor = [myColorPanel color];
	[myWindow setBackgroundColor: myColor];
	[myWindow display];
}

fungerar mycket riktigt. Min, i efterhand, solklara miss pekar på några besvärligheter kring (min?) minneshantering:
Om något fungerar när jag testat att klicka en gång, för att senare ej fungera, kan jag ej se felet. Min kod fungerade ju nyss! Sedan fungerar inte med självklarhet något många gånger, bara för att det fungerar en gång. (Jag har fortfarande inte lyckats få det att krascha med endast ett klick.)

Vänligen, Ylan