#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.