Obj-C: appeler la libération/dealloc provoque EXC_BAD_ACCESS

Question

Voici ma classe personnalisée:

PolygonShape : NSObject { 
int numberOfSides; 
int minimumNumberOfSides; 
int maximumNumberOfSides; 
} 

Ma méthode init personnalisée:

- (id)initWithNumberOfSides:(int)sides minimumNumberOfSides:(int)min maximumNumberOfSides:(int)max { 
if (self = [super init]) { 
    [self setMinimumNumberOfSides:min]; 
    [self setMaximumNumberOfSides:max]; 
    [self setNumberOfSides:sides]; 
} 

return self; 
} 

Ma méthode dealloc:

- (void) dealloc { 
NSLog("Calling dealloc"); 
[super dealloc]; 
} 

Mon exemple de code qui plante:

PolygonShape *shape1 = [[PolygonShape alloc] initWithNumberOfSides:6 minimumNumberOfSides:5 maximumNumberOfSides:9]; 
[shape1 release]; 

J'ai alloué un PolygonShape en augmentant le nombre de retainer de 1, puis en le relâchant, je le décrémenterais à 0 et j'appellerai dealloc, en imprimant ce message à NSLog mais je viens d'obtenir EXC_BAD_ACESS. Je suis en mesure d'accéder et de modifier les champs de mon objet après l'avoir créé afin que tout fonctionne jusqu'à ce qu'il fonctionne. Merci beaucoup pour l'aide!

Answer 1

NSLog("Calling dealloc"); 

Vous passez une chaîne C régulière, plutôt qu'un NSString. Vous devez faire ceci:

NSLog(@"Calling dealloc"); 

Votre compilateur aurait dû vous prévenir de passer un pointeur incompatible.

Answer 2

Avez-vous ajouté ces derniers dans la @interface ...

@property(assign) int numberOfSides, minimumNumberOfSides, maximumNumberOfSides; 

et ceux-ci dans le @implementation?

@synthesize numberOfSides, minimumNumberOfSides, maximumNumberOfSides; 

Les setters ne seront pas générés automatiquement à moins que vous explicitement @synthesize il.

Answer 3

Sur quelle ligne se plante-t-il? L'erreur peut-elle être dans l'une des méthodes que vous appelez dans initWithNumberOfSides?

Si vous le faites « construire et déboguer » pour trouver où il se bloque ne fonctionne pas, je me retournais sur NSZombieEnabled pour trouver le problème (ne pas oublier de l'éteindre à nouveau!)