Question

I have UITableViewController which contains a list of items. Now, I want the list to be automatically scrolled to a item (index = bestOne ) once the view appears. Meanwhile I want the item to be colored into red and be labeled as Marked.

My code roughly achieves what I want. But, I actually see more than one red items iterating: every 10 items, there is a red item.

I am quite new to iphone development, I figure it might have something to do with reusable cells. But I am not exactly sure why. Can anybody suggest one way to solve this issue? Thanks in advance.

(void)viewDidAppear:(BOOL)animated
{
    if (self.bestOne != -1)
    {
        NSIndexPath *indexPath = [NSIndexPath indexPathForRow:self.bestOne inSection:0];
        [self.tableView scrollToRowAtIndexPath: atScrollPosition: animated:YES];
    }
}


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

    UITableViewCell *cell = [tableView dequeueReusableCellWithIdentifier:CellIdentifier];
    if (cell == nil) {
        cell = [[[UITableViewCell alloc] initWithStyle:reuseIdentifier:CellIdentifier] autorelease];
    }

    cell.textLabel.text = [self.array objectAtIndex:indexPath.row];
    if (indexPath.row == self.bestOne)
    {
        cell.detailTextLabel.text = @"Marked";
        cell.textLabel.textColor = [UIColor redColor];
    }
    return cell;
}

Answer 1

You are right about the reusable cells part.

Your code should be something like –

cell.textLabel.text = [self.array objectAtIndex:indexPath.row];
if (indexPath.row == self.bestOne)
{
    cell.detailTextLabel.text = @"Marked";
    cell.textLabel.textColor = [UIColor redColor];
}
else 
{
    cell.detailTextLabel.text = @"";
}

On reuse, you get the exact cell that you had set before. While other cells are undistinguishable, the marked cell stands out with its specifically set detailTextLabel. You need to reset it before you can use it as an unmarked cell.