iphoneアプリ開発についての質問です

質問

tableviewcellにチェックマークを付けるだけの簡単なコードですが、なぜかチェックマークを付けたセルとは別のセルにまでチェックマークがついてしまっています。

また、激しくスクロールするとさっきまで付いていたチェックマークが外れていたりとさらに酷い状態に・・・。

- (void)tableView:(UITableView *)tableView didSelectRowAtIndexPath:(NSIndexPath *)indexPath {
[tableView cellForRowAtIndexPath:indexPath].accessoryType = UITableViewCellAccessoryCheckmark;
UITableViewCell *cell = [tableView cellForRowAtIndexPath:indexPath];
if (cell.accessoryType == UITableViewCellAccessoryCheckmark)
cell.accessoryType = UITableViewCellAccessoryNone;
else
cell.accessoryType = UITableViewCellAccessoryCheckmark;

[tableView deselectRowAtIndexPath:indexPath animated:YES];
}

いろんなサイトで見ても上記のコードが最もよく見るコードでした。

- (UITableViewCell *)tableView:(UITableView *)tableView cellForRowAtIndexPath:(NSIndexPath *)indexPath {
NSString *CellIdentifier = @"ButtonCell";

UITableViewCell *cell = [tableView dequeueReusableCellWithIdentifier:CellIdentifier forIndexPath:indexPath];

[self updateCell:cell :CellIdentifier atIndexPath:indexPath];

return cell;
}

- (void)updateCell:(UITableViewCell *)cell :(NSString *)CellIdentifier atIndexPath:(NSIndexPath *)indexPath {
BButton *type = (BButton *)[cell viewWithTag:1];
//Buttonはライブラリを使用
[ReturnButton typeButton:type :self.showArr[indexPath.row]];
}

チェックマーク以外はUIButtonなども正常に表示されました。
心当たりがあればお願いします。

回答No.2

harawo · ベストアンサー

No.1の回答を、サンプルコードの形にすると、こうなります。

#define CELL @"Cell"

#import "TableViewController.h"

@interface TableViewController ()

@end

@implementation TableViewController {
NSMutableArray *dataArray;
}

- (void)viewDidLoad {
[super viewDidLoad];

NSMutableDictionary *item;
dataArray = [NSMutableArray arrayWithCapacity: 12];
item = [@{@"month": @"January", @"accessory": @(UITableViewCellAccessoryNone)} mutableCopy];
[dataArray addObject: item];
item = [@{@"month": @"Febtruary", @"accessory": @(UITableViewCellAccessoryNone)} mutableCopy];
[dataArray addObject: item];
item = [@{@"month": @"March", @"accessory": @(UITableViewCellAccessoryNone)} mutableCopy];
[dataArray addObject: item];
item = [@{@"month": @"April", @"accessory": @(UITableViewCellAccessoryNone)} mutableCopy];
[dataArray addObject: item];
item = [@{@"month": @"May", @"accessory": @(UITableViewCellAccessoryNone)} mutableCopy];
[dataArray addObject: item];
item = [@{@"month": @"June", @"accessory": @(UITableViewCellAccessoryNone)} mutableCopy];
[dataArray addObject: item];
item = [@{@"month": @"July", @"accessory": @(UITableViewCellAccessoryNone)} mutableCopy];
[dataArray addObject: item];
item = [@{@"month": @"August", @"accessory": @(UITableViewCellAccessoryNone)} mutableCopy];
[dataArray addObject: item];
item = [@{@"month": @"September", @"accessory": @(UITableViewCellAccessoryNone)} mutableCopy];
[dataArray addObject: item];
item = [@{@"month": @"October", @"accessory": @(UITableViewCellAccessoryNone)} mutableCopy];
[dataArray addObject: item];
item = [@{@"month": @"November", @"accessory": @(UITableViewCellAccessoryNone)} mutableCopy];
[dataArray addObject: item];
item = [@{@"month": @"December", @"accessory": @(UITableViewCellAccessoryNone)} mutableCopy];
[dataArray addObject: item];
}

#pragma mark - Table view data source

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

- (NSInteger)tableView:(UITableView *)tableView numberOfRowsInSection:(NSInteger)section {
return dataArray.count;
}

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

cell.textLabel.text = dataArray[indexPath.row][@"month"];
cell.accessoryType = (UITableViewCellAccessoryType)[dataArray[indexPath.row][@"accessory"] intValue];

return cell;
}

- (void) tableView:(UITableView *)tableView didSelectRowAtIndexPath:(NSIndexPath *)indexPath {
UITableViewCell *cell = [tableView cellForRowAtIndexPath: indexPath];
if (cell.accessoryType == UITableViewCellAccessoryNone) {
cell.accessoryType = UITableViewCellAccessoryCheckmark;
dataArray[indexPath.row][@"accessory"] = @(UITableViewCellAccessoryCheckmark);
} else {
cell.accessoryType = UITableViewCellAccessoryNone;
dataArray[indexPath.row][@"accessory"] = @(UITableViewCellAccessoryNone);
}
}

@end

お礼

非常に丁重な解説とサンプルコードで分かりやすかったです。
おかげで問題解決いたしました、ありがとうございます。

回答No.3

harawo

回答No.2のサンプルコードの、いちぶ修正です。

- (void) tableView:(UITableView *)tableView didSelectRowAtIndexPath:(NSIndexPath *)indexPath {
UITableViewCell *cell = [tableView cellForRowAtIndexPath: indexPath];
// if (cell.accessoryType == UITableViewCellAccessoryNone) { // この行を変更
if ([dataArray[indexPath.row][@"accessory"] intValue] == UITableViewCellAccessoryNone) {
cell.accessoryType = UITableViewCellAccessoryCheckmark;
dataArray[indexPath.row][@"accessory"] = @(UITableViewCellAccessoryCheckmark);
} else {
cell.accessoryType = UITableViewCellAccessoryNone;
dataArray[indexPath.row][@"accessory"] = @(UITableViewCellAccessoryNone);
}
}

「MVCデザインパターンを準拠しよう」といっている手前、ビュー（テーブルセル）に依存したプログラムでは、統一性がありませんからね。

回答No.1

harawo

まず、MVCデザインパターンについて説明します。これはウィキペディアで、「Model View Controller」という項目で記載がありますから、あわせて読んでください。
ひとことでいうと、プログラムの構成は、データ（Model）と表示（View）を別々にしましょう、という考え方の指針です。インターフェイスを変更したら、データ構造も作り直さないといけないという設計では、むだなコストがかかりすぎるという理由で、このような考え方が生まれました。ここでMVCデザインパターンの詳細を語るつもりはないので、これ以上は説明しませんが、iOSのUIKitは、このMVCデザインパターンに準拠した構成になっています。UIKitのクラスに「Controller」と付く名前が多数ありますが、MVCデザインパターンを準拠している証です。Controllerは、データを参照しながら、ビューを制御する役割を担います。
UIControlのサブクラス、スライダーや、スイッチ、そしてUILabelなどは、そのインスタンスにデータを保持することができます。スライダーはvalueというプロパティ、スイッチならonというプロパティ、ラベルならtextというプロパティ、それぞれがデータを保持するプロパティです。なので、これらのオブジェクトを使えば、MVCデザインパターンに準拠しないプログラムを作ることは可能です。しかしながらテーブルとテーブルセルは、それ自身にデータを保持することができません。加えて、テーブルセルはテーブルの行数分の数存在するわけではないので、その理由だけでもテーブルセルにデータを保持することは不可能です。
テーブルセルのアクセサリの状態も、テーブル、テーブルセルから独立した、なんらかのデータに保存します。一般的には、NSArrayインスタンスを使うことになるでしょう。テキストラベルのテキスト、アクセサリの状態……と、項目ごとに配列を作っていたら、管理がうっとうしくなるので、複数の項目を保持したNSDictionaryインスタンスを、NSArrayに格納するという方向が、現実的でしょう。