ドハマりしました

UITableViewの行をdeleteRowsAtIndexPaths:withRowAnimation:メソッドで削除しようとしたら、何度やってもエラーになってしまってハマりました。

UITableViewの行数とデータソースの要素数を合わせる必要があるということは知っていましたが、行を削除する時は必ず少しの間は不整合が発生するので、どっちを先に削除すればいいのだろうと悩んでしまいました。(悩んだあげく適当に消してしまったので、どんどん悩みが深まっていった…)

というわけで失敗の過程と解決策について説明します。

最初の失敗

最初の失敗は、手動削除前にデータソースを更新していて、かつUITableViewもリロードしていました。

なので、当然削除しようとした時には、すでにその行事態が存在しておらず、エラーになってしまいました。バカですね。

コードがゴチャゴチャしていたせいで、どこでデータソースが更新されて、どこでUITableViewがリロードされるか分かりづらくなっていたのが原因でした。バカですね。

次の失敗

次の失敗は、データソースもUITableViewも更新しないようにしていました。 データソースの更新は行を削除した後でもいいのかなと思っていたのですが、逆でした。なので当然のようにクラッシュしました。

解決策

解決策は、手動削除前にデータソースを更新しておき、TableViewの行はそのままの状態にすることです。

そうすることで、deleteRowsAtIndexPaths:withRowAnimation:メソッドが削除対象の行を削除してデータソースとTableViewの行の帳尻が合います。

よく考えると当たり前の話ですが、TableViewの行とデータソースの矛盾のエラーは分かりづらいので、解決するのにいつも苦労してしまいます。

サンプルコード

汚い。


#import "ViewController.h"

@interface ViewController ()

@property (nonatomic, readwrite, strong) NSMutableArray *array;

@end

@implementation ViewController

- (void)viewDidLoad
{
    [super viewDidLoad];

    self.array = [NSMutableArray arrayWithCapacity:0];
    [_array addObject:@"0"];
    [_array addObject:@"1"];
    [_array addObject:@"2"];
    [_array addObject:@"3"];
    [_array addObject:@"4"];
}

- (NSInteger)numberOfSectionsInTableView:(UITableView *)tableView
{
    return 1;
}

- (NSInteger)tableView:(UITableView *)tableView numberOfRowsInSection:(NSInteger)section
{
    return [_array count];
}

- (UITableViewCell *)tableView:(UITableView *)tableView cellForRowAtIndexPath:(NSIndexPath *)indexPath
{
    UITableViewCell *cell = [tableView cellForRowAtIndexPath:indexPath];

    if (!cell) {
        cell = [[UITableViewCell alloc] initWithStyle:UITableViewCellStyleDefault reuseIdentifier:@"Cell"];
    }

    cell.textLabel.text = [_array objectAtIndex:indexPath.row];

    return cell;
}

- (void)tableView:(UITableView *)tableView didSelectRowAtIndexPath:(NSIndexPath *)indexPath
{
    [tableView deselectRowAtIndexPath:indexPath animated:YES];

    // 先にデータソースを削除する
    [_array removeObjectAtIndex:indexPath.row];

    NSArray *deleteArray = [NSArray arrayWithObject:indexPath];
    // UITableViewの行を削除する
    [tableView deleteRowsAtIndexPaths:deleteArray withRowAnimation:UITableViewRowAnimationAutomatic];
}

参考

iPhoneのTableView のセルを削除するときにはまったところ