Почему мое приложение завершает работу со следующей ошибкой?

#objective-c #cocoa-touch #ios #memory-management

#objective-c #cocoa-touch #iOS #управление памятью

Вопрос:

Когда я прокручиваю свой табличный вид вверх и вниз, примерно через 6-8 раз происходит сбой моего приложения, и я получаю следующее в окне отладки:

 myapp[250:207] *** -[NSIndexPath row]: message sent to deallocated instance 0xdd0eab0
  

Вот мой код:

 - (NSInteger)tableView:(UITableView *)tableView numberOfRowsInSection:(NSInteger)section {
    // Return the number of rows in the section.
    return [remoteRecipientItems count];
}


// Customize the appearance of table view cells.
- (UITableViewCell *)tableView:(UITableView *)tableView cellForRowAtIndexPath:(NSIndexPath *)indexPath {

    static NSString *CellIdentifier = @"RemoteRecipientItem";

    UITableViewCell *cell = [tableView dequeueReusableCellWithIdentifier:CellIdentifier];
    if (cell == nil) {
        cell = [[[UITableViewCell alloc] initWithStyle:UITableViewCellStyleDefault reuseIdentifier:CellIdentifier] autorelease];
    }




    NSUInteger row = [indexPath row];

    NSUInteger oldRow = [lastIndexPath row];

    // Configure the cell...

    [[cell textLabel]setText:[remoteRecipientItems objectAtIndex:[indexPath row]]];

    cell.accessoryType = (row == oldRow amp;amp; lastIndexPath !=nil)? UITableViewCellAccessoryCheckmark : UITableViewCellAccessoryNone; 

    return cell;
}





#pragma mark -
#pragma mark Table view delegate

- (void)tableView:(UITableView *)tableView didSelectRowAtIndexPath:(NSIndexPath *)indexPath {



    int newRow = [indexPath row];
    int oldRow = (lastIndexPath !=nil)?[lastIndexPath row]:-1;

    if (newRow != oldRow) {
        UITableViewCell *newCell = [tableView cellForRowAtIndexPath:indexPath];
        newCell.accessoryType = UITableViewCellAccessoryCheckmark;

        UITableViewCell *oldCell = [tableView cellForRowAtIndexPath:lastIndexPath];

        oldCell.accessoryType = UITableViewCellAccessoryNone;

        lastIndexPath = indexPath;


    }

    [tableView deselectRowAtIndexPath:indexPath animated:YES];


}


#pragma mark -
#pragma mark Memory management

- (void)didReceiveMemoryWarning {
    // Releases the view if it doesn't have a superview.
    [super didReceiveMemoryWarning];

    // Relinquish ownership any cached data, images, etc. that aren't in use.
}

- (void)viewDidUnload {
    // Relinquish ownership of anything that can be recreated in viewDidLoad or on demand.
    // For example: self.myOutlet = nil;


    remoteRecipientItems = nil;
    remoteRecipientID = nil;
    xmlData = nil;
    lastIndexPath = nil;

}


- (void)dealloc {

    [remoteRecipientItems release];
    [remoteRecipientID release];
    [xmlData release];

    [lastIndexPath release];

    [super dealloc];
}
  

Ответ №1:

Вы не являетесь владельцем indexPath переменной, поэтому вам нужно сохранить ее.

Попробуйте заменить это:

 lastIndexPath = indexPath;
  

С этим:

 lastIndexPath = [indexPath retain];
  

Комментарии:

1. так это значит, что я должен его выпустить? Где я должен его выпустить?

2. разве сохранение не происходит при каждом нажатии на просмотр таблицы? И в dealloc я бы выпустил его только один раз? Не приведет ли это к искажению количества сохранений / выпусков?

3. Я думаю, вы можете просто выпустить его в dealloc таком виде, как у вас есть. Вы также можете попробовать освободить его прямо перед сохранением indexPath . Просто убедитесь, что проверьте, имеет ли lastIndexPath значение nil, и если это не так, освободите его, а затем сохраните новое.

4. упс. @edc1591 прав. Вы должны освободить его непосредственно перед выполнением следующего сохранения. Что касается nil , сообщения на nil не учитываются.

5. Создайте lastIndexPath как свойство / синхронизируйте его, и среда выполнения автоматически выполнит за вас всю работу по сохранению / освобождению, когда вы присвоите ей новое значение. Затем вы можете просто освободить свойство в dealloc.