从 NSArray 中找到最低的 NSInteger

Question

我试图返回数组中的最低数字。

参数：arrayOfNumbers - NSNumbers 数组。

返回：数组中最小的数字作为 NSInteger。

到目前为止，我的代码没有给我任何错误，但没有通过单元测试。 我究竟做错了什么？

- (NSInteger) lowestNumberInArray:(NSArray *)arrayOfNumbers {

    NSNumber* smallest = [arrayOfNumbers valueForKeyPath:@"@min.self"];
    for (NSInteger i = 0; i < arrayOfNumbers.count; i++) {
        if (arrayOfNumbers[i] < smallest) {
            smallest = arrayOfNumbers[i];
        }
    }

    NSInteger smallestValue = [smallest integerValue];
    return smallestValue;

}

这是单元测试：

- (void) testThatLowestNumberIsReturned {
    NSInteger lowestNumber = [self.handler lowestNumberInArray:@[@3, @8, @-4, @0]];
    XCTAssertEqual(lowestNumber, -4, @"Lowest number should be -4.");

    lowestNumber = [self.handler lowestNumberInArray:@[@83, @124, @422, @953, @1004, @9532, @-1000]];
    XCTAssertEqual(lowestNumber, -1000, @"Lowest number should be -1000.");    
}

Answer 1

这种方法

NSNumber* smallest = [arrayOfNumbers valueForKeyPath:@"@min.self"];

将已经确定数组中的最小数字，因此该方法内的循环是多余的（除了完全错误之外，正如@vikingosegundo 所注意到的那样）。

Answer 2

您正在将对象与 c 类型进行比较，导致 im 指针地址与 int 进行比较。

除了您最小的已经是最小的这一事实之外，因为您使用了KVC 集合运算符@min.self （请参阅 Glorfindel 答案），以下代码显示了正确的比较

if (arrayOfNumbers[i] < smallest)

应该

if ([arrayOfNumbers[i] compare:smallest] == NSOrderingAscending)

或者

if ([arrayOfNumbers[i] integerValue] < [smallest integerValue])