Я просматриваю какой-то старый код в одном из моих приложений и исправляю код в областях, которые могут быть потенциально проблематичными.
Я вижу много старого кода, используя...
NSRange range = //determine range here....
if(range.length > 0)
{
//do stuff
}
Является ли этот код "прекрасным" или мне нужно его изменить?
NSRange range = //determine range here....
if(range.location != NSNotFound)
{
//do stuff
}
Являются ли эти два метода идентичными, по существу, или нет?
Две проверки не всегда идентичны. Это зависит от того, как был создан диапазон. Пример:
NSRegularExpression *re = [NSRegularExpression
regularExpressionWithPattern:@"(?= )" options:0 error:NULL];
NSTextCheckingResult *result = [re firstMatchInString:@"hello world"
options:0 range:NSMakeRange(0, 11)];
NSLog(@"range = %@", NSStringFromRange(result.range));
Длина диапазона равна 0, но его местоположение равно 5, а не NSNotFound.
Ответ зависит от используемой функции/метода. NSRange - это просто структура, поэтому вам нужно прочитать документацию для функции/метода, который вы вызываете.
<сильные > Примеры:
NSRangeFromString
Возвращает диапазон из текстового представления.
... Если aString не содержит целых чисел, эта функция возвращает Структура NSRange, значения местоположения и длины которой равны 0.
В этом случае проверка на NSNotFound не будет работать.
... Возвращает {NSNotFound, 0}, если aString не найден или пуст (@"").
Здесь указано, что местоположение будет NSNotFound, а length будет 0, поэтому любая из проверок будет работать, однако я бы рекомендовал проверить местоположение на NSNotFound.
NSNotFound определяется как "NSIntegerMax". Даже если они дают тот же результат, ваш второй дисплей намного читабельнее и самодокументируется. Может быть, вы не должны находить их все и менять, а просто переключаться на это, двигаясь вперед.