How to manage custom cell label value in iOS?

Question

I’m building an iOS app using storyboards.I have used UITableViewController which has 6 custom cells each of which contains three IBOutlet buttons and one IBOutlet label.

When the user clicks on any button in one particular custom cell, then the value of only that particular cell label should change.

But what happens is, values of all labels in each custom cell get changed.

Here is my code:

- (UITableViewCell *)tableView:(UITableView *)tableView cellForRowAtIndexPath:(NSIndexPath *)indexPath {

    static NSString *Cellidentifier1 = @"List";
    Cell *cell1 = [tableView dequeueReusableCellWithIdentifier:Cellidentifier1 forIndexPath:indexPath];
     cell1.selectionStyle = UITableViewCellSelectionStyleNone;
    // Configure the cell...

    long row = [indexPath row];

    cell1.First.tag=row;//button iboutlet in cell
    cell1.Second.tag=row;//button iboutlet in cell
    cell1.Third.tag=row;//button iboutlet in cell

    cell1.Level.tag=row;

    cell1.Level.text=Skill;//label iboutlet in cell

    return cell1;
 }


-(IBAction)FirstAction:(id)sender{

     Skill=@"first";
     [self.tableView reloadData];

}

-(IBAction)SecondAction:(id)sender{

     Skill=@"Second";
     [self.tableView reloadData];

}


-(IBAction)ThirdAction:(id)sender{

     Skill=@"Third";
     [self.tableView reloadData];

}

Answer 1

A couple of issues:

1. You should have a model that reflects what's in your table view. Specifically, right now you have a single Skill value, but you have six rows. You want to maintain a model that is an array of values, something like:

   @property (nonatomic, strong) NSMutableArray *values;
   

   And

   - (void)viewDidLoad {
       [super viewDidLoad];
   
       self.values = [@[@"first", @"first", @"first", @"first", @"first", @"first"] mutableCopy];
   }
   

   And

   - (UITableViewCell *)tableView:(UITableView *)tableView cellForRowAtIndexPath:(NSIndexPath *)indexPath {
   
       static NSString *Cellidentifier1 = @"List";
       Cell *cell = [tableView dequeueReusableCellWithIdentifier:Cellidentifier1 forIndexPath:indexPath];
   
       NSString *value = self.values[indexPath.row];
   
       cell.level.text = value;
   
       return cell;
   }
   

   Note, no tag numbers needed.

2. When you tap on a button you must (a) identify what row in the table that corresponds to; (b) update that row in your model; and (c) reload that single row in the table (which will look up the value in the model):

   - (void)updateCell:(UIView *)cell withValue:(NSString *)value {
       NSParameterAssert(cell);
   
       NSIndexPath *indexPath = [self.tableView indexPathForCell:(UITableViewCell *)cell];
       if (cell) {
           self.values[indexPath.row] = value;
           [self.tableView reloadRowsAtIndexPaths:@[indexPath] withRowAnimation:UITableViewRowAnimationFade];
       }
   }
   
   - (IBAction)didTapFirstButton:(id)sender {
       [self updateCell:[[sender superview] superview] withValue:@"first"];
   }
   
   - (IBAction)didTapSecondButton:(id)sender{
       [self updateCell:[[sender superview] superview] withValue:@"second"];
   }
   
   - (IBAction)didTapThirdButton:(id)sender{
       [self updateCell:[[sender superview] superview] withValue:@"third"];
   }
   

   Note, the sender is the button. So I get the table view cell by grabbing the superview (which is the cell's content view) and then grab its superview (the cell itself). If your view hierarchy is different, then change that as appropriate, but hopefully this illustrates the idea.