Objective-C :: использование метода для изменения объекта

Question

У меня есть класс под названием «CardSet», содержащий NSMutableArray * cardSet для хранения «карт», которые я расширяю, чтобы сделать «DeckCards». Я бы хотел, чтобы «CardSet» имел метод «(void) addCard: (Card *)» (и аналогичным образом метод «removeCard»). Я хотел бы добавить «addCard» к некоторым способам доступа и установки CardSet. Еще лучше я хотел бы использовать метод «addCard» для инициализации cardSet. Файл класса "CardSet.h" гласит:

#import <Cocoa/Cocoa.h> 

#import <Card.h> 

@interface CardSet : NSObject { 

    NSMutableArray* cardSet; 

} 

-(id)init; 

-(NSMutableArray*)getCardSet; 

-(void)setCardSet:(NSMutableArray *)new_cardset; 

-(Card*)getCard:(NSInteger) index; 

**-(void)addCard:(Card*) new_card;** 

-(void)removeCard:(Card*) old_card; 

-(void)dealloc; 

@property (readwrite, retain, getter=getCardSet, setter=setCardSet) NSMutableArray* cardSet; 

@end 

и файл метод читает:

#import "CardSet.h" 

@implementation CardSet 

-(id)init{ 
    if(self = [super init]){} //will add initialisations here later 
    return self; 
} 

-(NSMutableArray*)getCardSet{ 
    return cardSet; 
} 

-(void)setCardSet:(NSMutableArray *)new_cardSet{ 
    cardSet = new_cardSet; 
} 

-(Card*)getCard:(NSInteger)index{ 
    return [cardSet objectAtIndex:index]; 
} 

**-(void)addCard:(Card *)new_card{ 
    [cardSet addObject:new_card]; 
}** 

-(void)removeCard:(Card *)old_card{ 
    [cardSet removeObject:old_card]; 
} 

-(void)dealloc{ 
    [cardSet release]; 
    [super dealloc]; 
} 

@synthesize cardSet; 

@end 

Это компилируется нормально. Я хотел бы инициализировать экземпляр «DeckCards», используя его метод «addCard» 52 раза. Когда я вызываю addCard 52 раза в методе setter DeckCards и запрашиваю размер его «cardSet», я возвращаюсь 0.

Это проблема с областью или привилегиями? Может ли метод «addCard» иметь какие-либо привилегии setter? Должен ли параметр setter быть таким же, как возвращаемый и соответствующий тип члена?

[Я могу обойти выше, создав объект NSMutableArray «deck_cards_temp» за пределами «DeckCard», добавьте к нему 52 карты и передайте его, чтобы установить член моего экземпляра «DeckCards» через установщик, унаследованный от " CardSet». Это не очень хорошо!]

Что вы посоветуете? Большое спасибо за вашу помощь и терпение.

Answer 1

Вы никогда не создаете объект cardSet. Вы должны создать его в методе -init:

-(id)init 
{ 
    if(self = [super init]) 
    { 
     cardSet = [[NSMutableArray alloc] init]; 
    } 
    return self; 
} 

Потому что вы никогда на самом деле создать массив, все вызовы к -addCard: направляются к нулевому объекту.

Когда вы передаете массив в -setCardSet:, вы передаете инициализированный массив, поэтому массив больше не равен нулю, а вызовы -addCard: работают нормально.

Answer 2

CardSet.h

#import <Cocoa/Cocoa.h> 

// For know we just need to know there is a class named "Card" being used but implemented later 
@class Card; 

@interface CardSet : NSObject { 
    NSMutableArray *cardSet; 
} 

// Here are the methods according to "correct" naming conventions 
- (Card *)cardAtIndex:(NSInteger)index; 
- (void)addCard:(Card *)card; 
- (void)removeCard:(Card *)card; 

// This will help us and forget about writing the setter/getter 
@property (nonatomic, retain) NSMutableArray *cardSet; 

@end 

CardSet.m

#import "CardSet.h" 
// Now we tell the compiler what "Card" is and what methods etc. it has 
#import "Card.h" 

@implementation CardSet 

@synthesize cardSet; 

- (id)init { 
    if (self = [super init]) { 
     // If we don't create the cardSet, how are we able to work with it!? 
     NSMutableArray *anArray = [[NSMutableArray alloc] init]; 
     self.cardSet = anArray; 
     [anArray release]; 
    } 
    return self; 
} 

- (Card *)cardAtIndex:(NSInteger)index { 
    return [cardSet objectAtIndex:index]; 
} 

- (void)addCard:(Card *)card { 
    [cardSet addObject:card]; 
} 

- (void)removeCard:(Card *)card { 
    [cardSet removeObject:card]; 
} 

- (void)dealloc { 
    [cardSet release]; 
    [super dealloc]; 
} 

@end 

Как Abizern уже отмечалось: Именование массив такой же, как ваш класс плохо.

Answer 3

Я бы сократить этот метод инициализации:

- (id)init { 
    if (self = [super init]) { 
     // If we don't create the cardSet, how are we able to work with it!? 
     self.cardSet = [NSMutableArray array]; 
    } 
    return self; 
}