Поиск самого низкого NSInteger от NSArray

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

вы сравниваете объекты с типами гр, в результате чего им-адреса указателя сравнивается с междунар.

Помимо того, ваш маленький уже самый маленький, как вы использовали KVC collection operator@min.self (см Глорфиндел ответ), следующий код показывает корректное сравнение

if (arrayOfNumbers[i] < smallest) 

должен быть

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

или

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