用于比较UITableViewCell的快速​​枚举逻辑错误检查

时间:2015-04-01 20:24:56

标签: objective-c uitableview ios8.1 fast-enumeration

我有一个可变数组( 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

1 个答案:

答案 0 :(得分:0)

您可以使用containsObject:方法完全避免循环:

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