deleteRowsAtIndexPaths 崩溃

背景

公司项目有一个消息的列表页，点击自定义cell上的删除按钮执行删除操作

#import "MyCell.h"

、、、、、

-(void)rowDelete:(UIButton *)btn{
    if (self.delegate && [self.delegate respondsToSelector:@selector(deleteNewsBtnClicked:)]) {
        [self.delegate deleteNewsBtnClicked:self.buttonTag];
    }
}

#import "ViewController.h"

、、、、、、

-(UITableViewCell *)tableView:(UITableView *)tableView cellForRowAtIndexPath:(NSIndexPath *)indexPath{
    MyCell *cell = [tableView dequeueReusableCellWithIdentifier:@"Cell"];
    if (!cell) {
        cell = [[MyCell alloc] initWithStyle:UITableViewCellStyleDefault reuseIdentifier:@"Cell"];
        cell.delegate = self;
    }
    cell.titleLab.text = self.dataSource[indexPath.row];
    cell.buttonTag = indexPath.row;
    return cell;
}

、、、、、、

- (void)deleteBtnClicked:(NSInteger)button{
    NSLog(@"----------正在删除:%zd行",button);
    [self.dataSource removeObjectAtIndex:button];
    //错误的写法:
 NSIndexPath* path =[NSIndexPath indexPathForRow:button inSection:0];
    NSMutableArray* deleteArray=[NSMutableArray arrayWithObject:path];
    [self.tableView deleteRowsAtIndexPaths:deleteArray withRowAnimation:UITableViewRowAnimationFade];
}

造成的后果

• 崩溃 ：如果所有cell在屏幕的高度范围内可以全部展示出来，这时候删除第一条数据，再删除最后一条数据，会造成崩溃，报错信息是数组越界
• 删除数据不对：如果删除第一个数据后，接着删除第二条数据，会发现实际上是第三条数据被删除了

原因分析

执行deleteRowsAtIndexPaths只会重新执行numberOfRowsInSection方法，而cellForRowAtIndexPath不会重新执行，这使得每个cell的buttontag不变，不会根据删除后的dataSource重新赋值。这时候数据源已经减少，如果点击最后一行的删除按钮，必然会引起数组越界。
如果列表较长，删除数据时需要滑动，这时候就不会出现上述的问题，因为滑动会走cellForRowAtIndexPath方法，会使cell的buttontag属性值及时得到更新。

解决办法一

使用

- (void)deleteNewsBtnClicked:(NSInteger)button{
    NSLog(@"----------正在删除:%zd行",button);
    [self.dataSource removeObjectAtIndex:button];
    // 表格刷新
    [self.tableView reloadData];
}

采用这种方法的缺点就是：

• 没有自带删除动画
• reload会消耗更多资源

解决办法二（推荐）

将cell通过代理传出来，再通过cell找到其indexpath

#import <UIKit/UIKit.h>
@class MyCell;

@protocol LKLNewsDelegate <NSObject>
- (void)deleteNewsBtnClicked:(MyCell *)cell;
@end

@interface MyCell : UITableViewCell

@property (nonatomic, assign)id<LKLNewsDelegate>delegate;
@property(nonatomic,strong)UILabel *titleLab;

@end

#import "MyCell.h"
    、、、、、、
-(void)rowDelete:(UIButton *)btn{
    if (self.delegate && [self.delegate respondsToSelector:@selector(deleteNewsBtnClicked:)]) {
        [self.delegate deleteNewsBtnClicked:self];
    }
}

#import "ViewController.h"
       、、、、、
-(void)deleteNewsBtnClicked:(MyCell *)cell{
    NSIndexPath *indexPath = [self.tableView indexPathForCell:cell];
    [self.dataSource removeObjectAtIndex:indexPath.row];
    [self.tableView deleteRowsAtIndexPaths:@[indexPath] withRowAnimation:UITableViewRowAnimationFade];    
}

这种方法的优点就是没有方法一的缺点 哈哈哈