UIswitch в ячейке таблицы повторно используется

Question

У меня проблема с uiswitch в моем UITableViewCell, что всякий раз, когда я изменяю значение переключателя в определенной ячейке, относящейся к определенному разделу. Все остальные ячейки секций, которые имеют одинаковые inexPath.row, меняются. Пожалуйста помоги ?

Вот мой код cellForRowAtIndexPath метода:

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

    //UISwitch *switchview = nil ; 

    if (cell == nil) 
    { 
     cell = [[UITableViewCell alloc] initWithStyle:UITableViewCellStyleDefault 
             reuseIdentifier:CellIdentifier]; 



    } 


    UISwitch *switchController = [[UISwitch alloc] initWithFrame:CGRectZero]; 
    CGRect switchFrame = switchController.frame; 
    NSString *str = [[self.SwitchArray objectAtIndex:indexPath.section] objectAtIndex:indexPath.row]; 
    NSLog(@"string in cell%@", str); 


    //set its x and y value, this you will have to determine how to space it on the left side 
    switchFrame.origin.x = 50.0f; 
    switchFrame.origin.y = 10.0f; 
    switchController.frame = switchFrame; 


    [switchController addTarget:self action:@selector(switchChanged:) forControlEvents:UIControlEventValueChanged]; 
    [cell addSubview:switchController ]; 
    [addSubview:switchController]; 


    if ([[[self.SwitchArray objectAtIndex:indexPath.section] objectAtIndex:indexPath.row ]isEqualToString:@"ON"]) 
{ 
     switchController.on=YES; 
} 
    else 
    { 
     switchController.on=NO; 
    } 


    return cell; 

} 

А вот SwitchChangedEvent:

-(void)switchChanged:(UISwitch *)sender 
{ 
    UITableViewCell *cell = (UITableViewCell *)[sender superview]; 
    NSIndexPath *index=[mainTableView indexPathForCell:cell]; 

    if (sender.on) 
    { 

     [[self.SwitchArray objectAtIndex:index.section] replaceObjectAtIndex:index.row withObject:@"ON"]; 
     NSString *word= [[self.mainArray objectAtIndex:index.section ] objectAtIndex:index.row]; 

    } 
    else 
    { 
     //call the first array by section 
     [[self.SwitchArray objectAtIndex:index.section] replaceObjectAtIndex:index.row withObject:@"OFF"]; 
     NSString *word= [[self.mainArray objectAtIndex:index.section ] objectAtIndex:index.row]; 


    } 

    [padFactoids setObject:[NSKeyedArchiver archivedDataWithRootObject:SwitchArray] forKey:@"savedArray"]; 
    [padFactoids synchronize]; 

} 

Answer 1

Если вы посмотрите, я думаю, вы обнаружите, что вы строите несколько коммутаторов и на каждом клетка.

Поскольку ячейки используются повторно, вы добавляете новый переключатель, но не удаляете старый. Я думаю, что это мое, по крайней мере, часть причины вашей проблемы.

---

UPDATE

Я думаю, что я бы справиться с этим немного иначе, чем другие. Я думаю, что @AlanoudAldrees будет проще удалить старый переключатель.

if (cell == nil) 
{ 
    cell = [[UITableViewCell alloc] initWithStyle:UITableViewCellStyleDefault 
            reuseIdentifier:CellIdentifier]; 
} 
else 
{ 
    for (UIView *view in cell.subviews) 
     if ([view isKindOfClass:[UISwitch class]]) 
      [view removeFromSuperview]; 
} 

Answer 2

У вас есть две основные проблемы:

1. Этот код является неправильным:

   [cell addSubview:switchController ]; 
   

   Никогда не добавляйте подвид в клетку; добавьте его только в ячейку contentView.

2. Вы добавляете переключатель каждый раз через cellForRowAtIndexPath:. Но эти ячейки повторно используются. Поэтому в некоторых ячейках уже есть есть переключатель. Таким образом, вы добавляете много переключателей в некоторые ячейки.

Answer 3

Как уже указывалось другими, вы добавляете несколько переключателей в каждую ячейку. FTFY:

UISwitch *switchController = nil; 
if (cell == nil) 
{ 
    cell = [[UITableViewCell alloc] initWithStyle:UITableViewCellStyleDefault 
            reuseIdentifier:CellIdentifier]; 
    switchController = [[UISwitch alloc] initWithFrame:CGRectZero]; 
    switchController.tag = 'swch'; 
    CGRect switchFrame = switchController.frame; 
    //set its x and y value, this you will have to determine how to space it on the left side 
    switchFrame.origin.x = 50.0f; 
    switchFrame.origin.y = 10.0f; 
    switchController.frame = switchFrame; 
    [switchController addTarget:self action:@selector(switchChanged:) forControlEvents:UIControlEventValueChanged]; 
    [cell.contentView addSubview:switchController]; 
} 
else 
{ 
    switchController = (UISwitch*)[cell.contentView viewWithTag: 'swch']; 
    NSAssert(switchController != nil, @"how?"); 
} 

NSString *str = [[self.SwitchArray objectAtIndex:indexPath.section] objectAtIndex:indexPath.row]; 
NSLog(@"string in cell%@", str); 

switchController.on = [str isEqualToString:@"ON"]; 

return cell;