сообщение отправлено на освобождение освобожденного экземпляра - PullRequest
2 голосов
/ 29 мая 2011

За последние пару часов я отлаживал следующую проблему и, похоже, не могу прийти к хорошему выводу.

- [строка NSIndexPath]: сообщение отправлено освобожденному экземпляру 0x506cf90

Я реализую распознаватель пролистывания в каждой ячейке UITableView.Всякий раз, когда обнаруживается пролистывание, вызывается приведенный ниже код.Что он делает, чтобы показать / скрыть подпредставление в ячейке.Если подпредставление уже показывает, то оно скрывается, в противном случае оно показывает.Если в ячейке 5 уже отображается подпредставление, а в ячейке 3 обнаружен пролистывание, то сначала мы удаляем это подпредставление, а затем добавляем подпредставление в ячейку 3.

Я попробовал приведенный ниже код и, похоже, он работает с некоторыми ячейками,Тем не менее, в нижней части UITableView есть ячейка, в которой, если я пытаюсь скрыть подпредставление, появляется ошибка, описанная выше.Он отлично работает для другой ячейки, только ячейки внизу.Почему это так?

Код выглядит следующим образом:

- (void)swipe:(UISwipeGestureRecognizer *)recognizer direction:(UISwipeGestureRecognizerDirection)direction
{
    if (recognizer && recognizer.state == UIGestureRecognizerStateEnded)
    {
        // Get the table view cell where the swipe occured
        CGPoint location = [recognizer locationInView:self.table];
        NSIndexPath* indexPath = [self.table indexPathForRowAtPoint:location];
        ConvoreCell* cell = (ConvoreCell *) [self.table cellForRowAtIndexPath:indexPath];

        [self.table beginUpdates];

        NSLog(@"ROW is %d", global.row);
        //removing the options view at the other cell before adding a new one
        if (global != nil && global.row != indexPath.row){
            [sideSwipeView removeFromSuperview];
            [sideSwipeView release];
            sideSwipeView = nil;
        }

        //options already exist, we need to remove it
        if (sideSwipeView != nil){
            [sideSwipeView removeFromSuperview];
            [sideSwipeView release];
            sideSwipeView = nil;
            slide = NO;
        } else {
            //options do not exist and therefore we need to add it
            NSArray * buttonData = [[NSArray arrayWithObjects:
                                     [NSDictionary dictionaryWithObjectsAndKeys:@"Mark Read", @"title", @"mark.png", @"image", nil],
                                     [NSDictionary dictionaryWithObjectsAndKeys:@"Track", @"title", @"play.png", @"image", nil],
                                     [NSDictionary dictionaryWithObjectsAndKeys:@"Leave", @"title", @"delete.png", @"image", nil],
                                     nil] retain];

            NSMutableArray * buttons = [[NSMutableArray alloc] initWithCapacity:buttonData.count];
            sideSwipeView = [[UIView alloc] initWithFrame:CGRectMake(0, cell.frame.size.height-25, 320, 25)];
            [sideSwipeView setAutoresizingMask:UIViewAutoresizingFlexibleTopMargin];
            [sideSwipeView setBackgroundColor:[UIColor colorWithPatternImage: [UIImage imageNamed:@"dotted-pattern.png"]]];
            [sideSwipeView setTag:-10];

            CGFloat leftEdge = BUTTON_LEFT_MARGIN;
            for (NSDictionary* buttonInfo in buttonData)
            {
                if (!([[buttonInfo objectForKey:@"title"] isEqualToString:@"Mark Read"] && [[[groups objectAtIndex:indexPath.row] unread] intValue] == 0))
                {

                    UIButton* button = [UIButton buttonWithType:UIButtonTypeCustom];

                    button.autoresizingMask = UIViewAutoresizingFlexibleRightMargin | UIViewAutoresizingFlexibleLeftMargin | UIViewAutoresizingFlexibleBottomMargin | UIViewAutoresizingFlexibleTopMargin;

                    UIImage* buttonImage = [UIImage imageNamed:[buttonInfo objectForKey:@"image"]];
                    if ([[[groups objectAtIndex:indexPath.row] tracked] intValue] == 1 && [[buttonInfo objectForKey:@"title"] isEqualToString:@"Track"]){
                        buttonImage = [UIImage imageNamed:@"pause.png"];
                        [button setSelected:YES];
                    } else 
                        [button setSelected:NO];

                    button.frame = CGRectMake(leftEdge, 0, buttonImage.size.width, buttonImage.size.height);

                    UIImage* grayImage = [self imageFilledWith:[UIColor colorWithWhite:0.9 alpha:1.0] using:buttonImage];
                    [button setImage:grayImage forState:UIControlStateNormal];

                    if ([[buttonInfo objectForKey:@"title"] isEqualToString:@"Mark Read"]){
                        [button addTarget:self action:@selector(markRead:) forControlEvents:UIControlEventTouchUpInside];
                    } else if ([[buttonInfo objectForKey:@"title"] isEqualToString:@"Track"]){
                        [button addTarget:self action:@selector(track:) forControlEvents:UIControlEventTouchUpInside];
                    } else if ([[buttonInfo objectForKey:@"title"] isEqualToString:@"Leave"]){
                        [button addTarget:self action:@selector(leave:) forControlEvents:UIControlEventTouchUpInside];
                    }
                    [button setTag:indexPath.row];
                    [buttons addObject:button];

                    [sideSwipeView addSubview:button];
                    leftEdge = leftEdge + buttonImage.size.width + BUTTON_SPACING;
                }
            }

            [cell.contentView addSubview:sideSwipeView];
            [buttons release];
            [buttonData release];
            global = indexPath;
            slide = YES;

        }
        [self.table endUpdates]; 
        [self.table deselectRowAtIndexPath:indexPath animated:YES];        

    }
}

Теперь проблема в том, что указатель на indexPath каким-то образом освобожден.Я смог обойти это, сделав копию indexPath вместо того, чтобы просто ссылаться на указатель.Поэтому я сделал:

global = [indexPath copy];

по какой-то причине после вызова этого метода он освобождает indexPath, и я не уверен, кто это делает .. Я думаю, что iOS это делает ... это правда?

1 Ответ

2 голосов
/ 29 мая 2011
    NSIndexPath* indexPath = [self.table indexPathForRowAtPoint:location];

Возвращает автоматически выпущенный экземпляр. Итак, да, ваше назначение global должно сохранить объект. copy эффективно возвращает сохраненный экземпляр.

Не просачивайся. Если вы просто сделаете:

global = [indexPath copy];

И вы сначала не выпустите исходное значение global, вы утечете.

О, и: http://developer.apple.com/library/mac/#documentation/Cocoa/Conceptual/MemoryMgmt/MemoryMgmt.html

...