UITableViewCellを取得するには、UIBarButtonItemをタップして背景色を変更しますか？

Question

私は、押されたときに、セルを強調表示し、強調表示を解除するために、tableViewの右隅にUIBarButtonItemを作成しようとしています。

言い換えれば、ユーザーがボタンを押すと、セルの範囲が背景色を白から黄色に変更します。

私はそのボタンを押すたびにアプリがクラッシュするので、私はそれを作ることに失敗しています。

- (void)viewDidLoad { 
    [super viewDidLoad]; 
    UIBarButtonItem *barButton; 
    barButton = [[[UIBarButtonItem alloc] initWithImage:[UIImage imageNamed:@"high.png"] style:UIBarButtonItemStyleBordered target:self action:@selector(colorCells:)] autorelease]; 
    self.navigationItem.rightBarButtonItem = barButton; 
} 

そしてここに、それはセルの範囲ハイライトにするために：私は失敗しています

- (void) colorCells:(id)sender 
{ 
    UITableViewCell *cell; 
    NSString *cellValue = cell.textLabel.text; 
    if ([cellValue isEqual: @"textTheCellShouldBeEqualTo"]){ 
     cell.backgroundColor = [UIColor colorWithRed:251/255.0f green:255/255.0f blue:192/255.0f alpha:1]; ; 
     cell.imageView.image = [UIImage imageNamed:@"hot.png"]; 

    } 
    else { 
     cell.imageView.image = nil; 
     cell.backgroundColor = [UIColor whiteColor]; 
    } 
} 

を

ここで私は、ボタンを作成するために使用しているコードですか？それはうまく動作するはずです。または？何か不足していますか？ビューはUITableViewControllerです。

EDIT

私はそうのように私のコードを変更：

- (void) colorCells:(id)sender{ 
    UITableViewCell *cell; 
    NSInteger nSections = [self.tableView numberOfSections]; 
    for (int j=0; j<nSections; j++) { 
     NSInteger nRows = [self.tableView numberOfRowsInSection:j]; 
     for (int i=0; i<nRows; i++) { 
      NSIndexPath *indexPath = [NSIndexPath indexPathForRow:i inSection:j]; 
      cell = [self.tableView cellForRowAtIndexPath:indexPath]; 
     } 
     NSString *cellValue = cell.textLabel.text; 
     if ([cellValue isEqual: @"textTheCellShouldBeEqualTo"]){ 
      cell.backgroundColor = [UIColor colorWithRed:251/255.0f green:255/255.0f blue:192/255.0f alpha:1]; ; 
      cell.imageView.image = [UIImage imageNamed:@"hot.png"]; 

     } 
     else { 
      cell.imageView.image = nil; 
      cell.backgroundColor = [UIColor whiteColor]; 
     } 
     [self.tableView reloadData]; 

    } 
} 

今ではクラッシュしないが、それはあまりにも背景に色をしませ​​ん。何か案は？

Answer 1

セル内のテキストの背景色を基準にすると、非常に壊れやすいようです。セルをハイライトする必要があると判断するものは何ですか？それは変化しますか？確かに、背景の色は、データソース内のオブジェクトの特定のプロパティに対応しています。より堅牢なアプローチは、クラスのNSMutableIndexSetプロパティを使用して行インデックスのセットを追跡し、バーボタンをタップして強調表示する必要があります。

この例を検討してください。私はrowsToHighlightがあなたのクラスで宣言されたNSMutableIndexSetのインスタンスであり、ハイライトを必要とする行のインデックスで設定されていると仮定しています。提供されたインデックスパスがrowsToHighlightのメンバであるかどうかに応じて、セルの背景色を調整するために-tableView:willDisplayCell:forRowAtIndexPath:を実装します。

- (void)tableView:(UITableView *)tableView willDisplayCell:(UITableViewCell *)cell forRowAtIndexPath:(NSIndexPath *)indexPath 
{ 
    if ([self.rowsToHighlight containsIndex:indexPath.row]) 
    { 
     cell.backgroundColor = [UIColor yellowColor]; 
    } 
} 

次に、バーボタンで発生したメソッドでは、空の更新ブロックを実行してテーブルをアニメーションでリロードするだけです。

- (void)colorCells:(id)sender 
{ 
    [self.tableView beginUpdates]; 
    [self.tableView endUpdates]; 
} 

Answer 2

セルを再表示するようにテーブルビューに指示しない限り、セルは再描画されません。いくつかの方法のうちの1つでこれを行うことができます。 [someTableView reloadData]またはone of these methodsと考えることができます。

また、セルオブジェクトはテーブルビュー内のオブジェクトではありません。これを考慮するとよいでしょう：

- (void) colorCells:(id)sender 
{ 
    UITableViewCell *cell; 

    //grab a cell 
    cell = [self tableView:self.tableView cellForRowAtIndexPath:[NSIndexPath indexPathForRow:someRow inSection:someSection]]; 
    NSString *cellValue = cell.textLabel.text; 
    if ([cellValue isEqual: @"textTheCellShouldBeEqualTo"]){ 
     cell.backgroundColor = [UIColor colorWithRed:251/255.0f green:255/255.0f blue:192/255.0f alpha:1]; ; 
     cell.imageView.image = [UIImage imageNamed:@"hot.png"]; 

    } 
    else { 
     cell.imageView.image = nil; 
     cell.backgroundColor = [UIColor whiteColor]; 
    } 

    //perhaps you need to reload the table as well 
    [self.tableView reloadData]; 
}