плохой доступ после синхронизации NSUserDefault

Question

У меня есть UITableView с некоторыми данными. У каждой ячейки есть кнопка, и если вы нажмете ее, вы удалите выбранную ячейку. После этого я должен сохранить свои данные с помощью NSUserDefault. Вот некоторые код:

- (void)buttonPressed:(UIButton *)sender{ 
    int tag=sender.tag; 
    [myArray removeObjectAtIndex:tag]; 

    [self saveData]; 
    [self.myTableView reloadData]; 
} 

метод buttonPressed вызывает saveData метод:

-(void)saveData{ 
    dataString=[[NSString alloc]init]; //defined in .h file 

    for(int i=0; i<([myArray count]); i++){ 
     ClassObject *aObject=[[ClassObject alloc]init]; 
     aObject=[myArray objectAtIndex:i]; 
     dataString=[dataString stringByAppendingString:aObject.idObject]; 
     dataString=[dataString stringByAppendingString:@"$"]; 
     dataString=[dataString stringByAppendingString:aObject.description1]; 
     dataString=[dataString stringByAppendingString:@"$"]; 
     dataString=[dataString stringByAppendingString:aObject.description2]; 
     dataString=[dataString stringByAppendingString:@"?"]; 
     [aObject release]; 
    } 


    NSUserDefaults *dataDefault=[NSUserDefaults standardUserDefaults]; 
    [dataDefault setObject:dataString forKey:@"myDataString"]; 
    [dataDefault synchronize]; 
    [dataString release]; 
} 

в режиме отладки все идет хорошо, пока метод saveData не заканчивается. Когда он заканчивает отладочные возвраты по методу buttonPressed, он перезагружает табличное представление, а затем приложение сбой. И я не знаю, почему.

Некоторые идеи? Благодарю.

Answer 1

Где aEquipaggio происходит из вашего приложения saveData? Почему вы выпускаете его на каждой итерации цикла? Это не имеет смысла, и это, скорее всего, слишком поздно, что приведет к сбою позже.

Edit: Теперь, когда вы изменили код в ваш вопрос, aEquipaggio больше не существует, но release в конце цикла все еще неправильно, и первая строка в цикле совершенно бессмысленно (и утечки), потому что вы никогда не используете экземпляр ClassObject, который вы выделяете.

Вы также не должны выпускать dataString в конце метода. В этот момент dataString больше не содержит тот же экземпляр, что и вы в alloc - init в начале, потому что вы заменили его экземпляром с автореализацией в цикле. ([[NSString alloc]init] бессмысленно в любом случае, просто используйте @"").

Answer 2

Дикая догадка, может быть, это как-то связано с освобождением dataString. Попробуйте его оптимизировать.

Answer 3

ClassObject *aObject=[[ClassObject alloc]init]; 
aObject=[arrayEquipaggio objectAtIndex:i]; 
// ... 
[aObject release]; 

Firstly, and most importantly, it's rather anObject.

Во-вторых, это и утечек памяти (вы потеряете указатель на выделенную, например, когда вы переназначить [arrayEquipaggio objectAtIndex:i]; к переменной) и над-релизах aObject (который теперь, как я просто объяснил, указывает на объект внутри массива, поэтому вы освобождаете объект, который у вас нет).

В общем, удалите строку с Alloc-INIT и что с выходом (вы делаете ту же самую ошибку с dataString переменной тоже) и:

Read this before continuing with development! Иначе вы будете иметь серьезные проблема с чем-либо в Objective-C. В настоящее время вы, похоже, не знаете, как работает управление памятью. (Также было бы полезно изучить C, прежде чем пытаться использовать Objective-C, но, к сожалению, это слишком большое ожидание ...)

Answer 4

У вас неправильное распределение памяти. Это то, что вы делаете:

dataString = [[NSString alloc] init]; 
dataString = [dataString stringByAppendingString:@"$"]; 
[dataString release]; 

Сначала вы назначаете новую строку, которую вы должны выпустить позже. Ничего страшного. Затем вы сохраняете другой указатель на переменную dataString, протекая предыдущую строку.И затем вы освобождаете строку с автореализацией, создаваемую -stringByAppendingString, так что вскоре после этого объект будет высвобожден.