Почему алфавит разделен на несколько диапазонов в этом C-коде?

161

В пользовательской библиотеке я увидел реализацию:

inline int is_upper_alpha(char chValue)
{
    if (((chValue >= 'A') && (chValue <= 'I')) ||
        ((chValue >= 'J') && (chValue <= 'R')) ||
        ((chValue >= 'S') && (chValue <= 'Z')))
        return 1;
    return 0;
}

Это пасхальное яйцо или в чем преимущества стандартного метода Си / Си ++?

inline int is_upper_alpha(char chValue)
{
    return ((chValue >= 'A') && (chValue <= 'Z'));
}

c++ c character toupper

— Владимир Ч.
источник

Обратите внимание, что в EBCDIC диапазон символов для букв нижнего регистра стоит перед диапазоном символов для букв верхнего регистра, и оба идут перед цифрами - что в точности противоположно порядку в кодировках на основе ASCII (таких как 8859- серия x, или Unicode, или CP1252, или…).

— Джонатан Леффлер

Примечание: если 'J' - 'I'и 'S' - 'R'оба равны 1, то я ожидаю, что разумный оптимизатор превратит первое в последнее.

— Матье М.

Ответы:

214

Автор этого кода предположительно должен был поддерживать EBCDIC в какой - то точке, где числовые значения букв являются Непоследовательными (пробела существуют между I, Jи R, Sкак вы уже догадались).

Стоит отметить , что C и C ++ стандарты только гарантирует , что символы 0не 9имеют непрерывные числовые значения именно по этой причине, так что ни один из этих методов, строго соответствующий стандартам.

— Wintermute
источник

Настоящий WTF - это то, почему оригинальный автор не написал комментарий // In the EBCDIC coding, the alphabet has gaps between these values. See URL: xxxx for details. Тогда вам даже не придется задавать вопрос. У вас есть ответ, встроенный в код.

— Абеленки

@abelenky Если код изначально предназначался для системы, в которой обычно используется ebcdic, он мог казаться очевидным в то время и не нуждался в комментариях, к сожалению, вещи, которые кажутся хорошими в унаследованном коде, сейчас кажутся странными.

— Vality

@abelenky: Настоящий WTF - это то, почему оригинальный автор не использовал стандартную функциональность, то есть return ( isalpha( chValue ) && isupper( chValue ) )...

— DevSolar

@ Damon: это не проблема. Возможно, вам придется обрабатывать «чужую» кодировку даже в системе, которая не использует эту кодировку изначально. Таким образом, вы устанавливаете для своей локали заданную кодировку, а затем вы должны держать пальцы скрещенными, чтобы программист фактически использовал стандартные функции вместо выполнения «умного» кодирования, подобного описанному выше, думая, что он знает, что каждая кодировка, с которой когда-либо столкнется его программа ...

— DevSolar

Если он был написан для поддержки EBCDIC с 1970-х годов, была ли isalpha и isupper даже ANSI или поддерживалась большинством компиляторов тогда?

— Никал

Похоже, он пытается охватить как EBCDIC, так и ASCII. Ваш альтернативный метод не работает для EBCDIC (у него есть ложные срабатывания, но нет ложных отрицаний)

C и C ++ действительно требуют, '0'-'9'являются смежными.

Обратите внимание , что стандартные библиотечные вызовы действительно знают , запускать ли они на ASCII, EBCDIC или других систем, поэтому они более портативными и , возможно , более эффективным.

— MSalters
источник

std::isupperфактически запрашивает текущую установленную глобальную локаль C

— Линси

Да ты прав. Метод написан для покрытия обеих кодировок. Спасибо за ответ!

— Владимир Ч.

@Lingxi: Да, но это не значит, что вы можете переключить язык с ASCII на EBCDIC. 'A'должен остаться 'A'независимо от локали. ASCII в UTF-8, это было бы возможно.

— MSalters

@Lingxi: std::isupper запрашивает установленную глобальную локаль C, да, но фаза компиляции, которая интерпретирует символьные литералы, - нет.

— Гонки легкости на орбите

@Lingxi - Просто быстрое примечание. Сомнительно, std::isupperдействительно ли это необходимо в большинстве случаев. Он учитывает локали, используемые для ввода от пользователя. Но при разборе файлов, взаимодействии с базами данных вы обычно ожидаете какой-то другой язык. Более того, по крайней мере в Linux эти вызовы, связанные с локалью, очень медленные - например, std::isalphaдва раза вызывает dynamic_cast, чтобы «найти» правильную реализацию локали, прежде чем сравнивать один символ.

— ibre5041