快速枚举逻辑错误检查比较UITableViewCell

Question

我有一个可变数组（editedServiceParts）与其中的两个对象;我有一个约30个单元格的表格视图。我想检查是否有任何表视图单元格（cell.textLabel.text）出现在可变数组中;如果他们这样做，我想将cell.accessory设置为复选标记。

修订这是-cellForRowAtIndexPath我的代码：

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

// get the timeFormat 
NSUserDefaults *userDefaults = [NSUserDefaults standardUserDefaults]; 
NSMutableDictionary *preferencesDict = [[userDefaults dictionaryForKey:@"preferencesDictionary"] mutableCopy]; 
int iTimeFormat = [[preferencesDict objectForKey:@"timeFormat"] intValue]; // set timeFormat 
NSMutableArray *selectedServicesParts = [NSMutableArray new]; 
NSMutableArray *editedServiceParts = [NSMutableArray new]; 
NSString *service; 



if(tableView.tag == kServicesTableView) { // services array 

    static NSString *CellIdentifier = @"apptServicesCell"; 
    UITableViewCell *cell = [tableView dequeueReusableCellWithIdentifier:CellIdentifier]; 

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

    // Configure the cell 
    SingletonServicesArray *sharedServicesArray = [SingletonServicesArray sharedServicesArray]; // list of available services 
    [cell.textLabel setText:[sharedServicesArray.globalServicesArray objectAtIndex:indexPath.row]]; // move selected row to cell 

    // separate soServices.text into individual elements 
    NSString *cleanService; 
    if(soServices.text.length > 0) { 
     selectedServicesParts = [[soServices.text componentsSeparatedByString:@","] mutableCopy]; 

     for(int x = 0; x < selectedServicesParts.count; x++) { 
      cleanService = [selectedServicesParts[x] stringByReplacingOccurrencesOfString:@"," withString:@""]; 

      [editedServiceParts insertObject: cleanService atIndex: x]; 
     } 
    } 

    // now, take the editedServicesParts (w/o commas) and mark cell if appropriate 
    for (int i = 0; i < editedServiceParts.count; i++) { if ([editedServiceParts containsObject:cell.textLabel.text]) { 
     cell.accessoryType = UITableViewCellAccessoryCheckmark; 
    } else { 
     // Remove accessory mark of recycled cells 
     cell.accessoryType = UITableViewCellAccessoryNone; 
    } 
    } 
//  for (int i = 0; i < editedServiceParts.count; i++) { // total number of rows in list of all available services 
//    
//   for(service in editedServiceParts) { 
//    if([cell.textLabel.text isEqualToString: service]) { 
//     cell.accessoryType = UITableViewCellAccessoryCheckmark; 
//     break; 
       } 
//   } 
//  } 

    return cell; 

} 

发生了什么事是只有第一个比较被人发现，虽然有另一个应该已经被发现。我担心这里有一个逻辑问题，而对于我来说，我没有看到它。帮助将不胜感激！

SD

Answer 1

您可以使用containsObject:方法来避免循环干脆：

if ([editedServiceParts containsObject:cell.textLabel.text]) { 
    cell.accessoryType = UITableViewCellAccessoryCheckmark; 
} else { 
    // Remove accessory mark of recycled cells 
    cell.accessoryType = UITableViewCellAccessoryNone; 
}