Objective-C 丑陋的代码？

Question

我有一些 Objective-C 代码可以为我完成这项工作。但它丑陋（低效）吗？使用 for-each 循环可以更好地执行吗？

请看这段代码：

for (int i = 0; i < [careerIds count]; i++) {

    NSString *titleString = [[titles objectAtIndex:i] stringValue];
    if ([titleString isEqualToString:@""] || [titleString rangeOfString:@"Intresseanmälan"].location != NSNotFound) {
        // Don't add the id
    } else {
        [ids addObject:[[careerIds objectAtIndex:i] stringValue]];
    }

}

Answer 1

我不认为你的代码特别难看——像你所做的那样使用索引 for 循环没有任何问题。我唯一可能改变的就是颠倒if 语句的含义，这样你就可以避免空的// Don't add the id 行。这是一种方法：

for (int i = 0; i < [careerIds count]; i++) {

    NSString *titleString = [[titles objectAtIndex:i] stringValue];

    if (([titleString length] > 0) && 
        ([titleString rangeOfString:@"Intresseanmälan"].location == NSNotFound))
    {
        [ids addObject:[[careerIds objectAtIndex:i] stringValue]];
    }

}

要解决您问题的核心，不，我不认为可以使用for 循环的快速枚举版本同时迭代两个单独容器的内容。您可以将它与一个一起使用，但正如我在评论中指出的那样，您必须使用 -indexOfObject: 来恢复当前对象的索引，以便您可以使用 -objectAtIndex: 从另一个数组中获取相应的项目。

Answer 2

使用 for-each 循环可以更好地执行吗？

是的。

for (id title in careerIds) {

    NSString *titleString = [title stringValue];
    if ([titleString isEqualToString:@""] || [titleString rangeOfString:@"Intresseanmälan"].location != NSNotFound) {
        // Don't add the id
    } else {
        [ids addObject:titleString];
    }
}

或者如果你想成为真正的闪光：

for (NSString* titleString in [careerIds valueForKey: @"stringValue"]) {

    if ([titleString isEqualToString:@""] || [titleString rangeOfString:@"Intresseanmälan"].location != NSNotFound) {
        // Don't add the id
    } else {
        [ids addObject:titleString];
    }
}

没有。 :-)

Answer 3

假设 careerIds 是一个 NSArray 或 NSMutableArray 并且您要向其中添加 NSString 对象，您可以这样做：

for (NSString *titleString in careerIds) {

    if ([titleString isEqualToString:@""] || [titleString rangeOfString:@"Intresseanmälan"].location != NSNotFound) {
        // Don't add the id
    } else {
        [ids addObject: titleString];
    }
}

[ids addObject:[[careerIds objectAtIndex:i] stringValue]];