內存管理問題

Question

我以爲我得到可可內存管理的竅門，但顯然我有很多東西要學。

看看這個類我寫道：

Word.h

#import <UIKit/UIKit.h> 


@interface Word : NSObject { 
    NSString *word; 
    NSMutableArray *wordArray; 
} 

@property (nonatomic ,retain) NSString *word; 
@property (nonatomic ,retain) NSMutableArray *wordArray; 

@end 

Word.m

#import "Word.h" 


@implementation Word 

@synthesize word, wordArray; 

- (void)setWord:(NSString *)aWord { 
    NSMutableArray *newValue = [[NSMutableArray alloc] init]; 
    self.wordArray = newValue; 
    [newValue release]; 

    for (int i = 0 ;i < [aWord length] ;i++) { 
     NSString *character = [[NSString alloc] init]; 
     character = [NSString stringWithFormat:@"%C",[aWord characterAtIndex:i]]; 
     [wordArray addObject:character]; 
     //[character release]; 
    } 
} 

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

@end 

正如你所看到的，當我設置字符串時，我正在填充一個數組。我通過拉出字符串的字符並將它們放入一個新的分配字符串中來對此進行for循環。 然後我把該字符串放在數組中，然後我必須釋放字符串。 它在這裏我做錯了，因爲我釋放了我放入數組的值。當後來我試圖使用數組時，它就消失了。

我應該怎麼辦？讓「角色」成爲財產並在dealloc中釋放它是錯誤的，因爲它只是暫時的。

Answer 1

首先，您不需要指定新的NSString到character，因爲您立即用另一個對象覆蓋它。記住，它是一個指針，所以當你指定它的時候，你並沒有改變它指向的對象，只是把它指向另一個對象。因此，當我們丟失了指向它的指針時，您分配並最初分配給character的對象將在下一行中泄漏。

現在，stringWithFormat返回一個自動釋放字符串，然後character指向。這個對象不應該被你釋放，但這就是在註釋掉的行中發生的事情。這樣循環會更好：

for (int i = 0 ;i < [aWord length] ;i++) { 
     NSString* character = [NSString stringWithFormat:@"%C",[aWord characterAtIndex:i]]; 
     [wordArray addObject:character]; 
} 

Answer 2

除了內存管理的關注，你知不知道你已經覆蓋了setter函數爲word財產？

當你@synthesize你的話的財產，都在幕後生成的setter和getter方法等有已經爲- (void)setWord:(NSString *)aWord;功能和- (NSString *)word;可以在Objective-C primer或Programming Language documentation瞭解這一點。

由於您已經創建了(void)setWord:(NSString *)aWord;函數，因此這將覆蓋生成的setter。由於您實際上未設置word變量，因此您的資產已「破損」。

舉個例子，也許把它改寫爲：

#import "Word.h" 

@implementation Word 

@synthesize word, wordArray; 

- (void)setWord:(NSString *)aWord { 
    // set the word iVar 
    if (aWord != word) { 
     [aWord retain]; // I prefer [aWord copy]; 
     [word release]; 
     word = aWord; 
    } 

    NSMutableArray *newValue = [[NSMutableArray alloc] init]; 

    for (int i = 0 ;i < [aWord length] ;i++) { 
     NSString *character = [NSString stringWithFormat:@"%C",[aWord characterAtIndex:i]]; 
     [wordArray addObject:character]; 

    } 
    // Now that you have created the newArray, set it to the property 
    self.wordArray = newValue; 
    [newValue release]; 
} 

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

@end 

編輯

爲什麼我寫的，我首選[aWord copy]超過[aWord retain]？因爲我會宣佈屬性爲copy而不是retain。

可變性可能是一種痛苦。讓我們假設代替傳遞NSString*到aWord而不是傳遞NSMutableString*。這是可能的，因爲NSMutableString是NSString的子類。讓我們假設程序的另一部分改變了這個字符串的值（畢竟它是可變的）。現在，我們從外部改變了Word類的屬性。這打破封裝。

對集合類使用@property (copy)…是有意義的。

您可以閱讀一個很好的答案here。