消息发送到解除分配的实例问题

时间:2011-05-29 01:46:53

标签: iphone objective-c ipad uitableview nsindexpath

过去几个小时我一直在调试以下问题,似乎无法得出一个好的结论。

  

- [NSIndexPath row]:发送到解除分配的实例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 个答案:

答案 0 :(得分:2)

    NSIndexPath* indexPath = [self.table indexPathForRowAtPoint:location];

返回一个自动释放的实例。所以,是的,你对global的任务需要保留对象。 copy有效地返回保留的实例。

不要泄漏。如果您这样做:

global = [indexPath copy];

并且您不会首先释放global的原始值,否则会泄漏。

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