- 1
- 2
- 3
- 4
- 5
int Cparse_post::is_hex(char c)
{
if (strchr("0123456789abcdefABCDEF",c)!=NULL) return 1;
return 0;
}
Нашли или выдавили из себя код, который нельзя назвать нормальным, на который без улыбки не взглянешь? Не торопитесь его удалять или рефакторить, — запостите его на говнокод.ру, посмеёмся вместе!
+73.1
int Cparse_post::is_hex(char c)
{
if (strchr("0123456789abcdefABCDEF",c)!=NULL) return 1;
return 0;
}
Да... Народ библиотеки не зyает.
isxdigit()
ахахахаха
я тоже не юзаю говнобиблиоеки rtl. а сделал бы так.
if((c >= '0' && c <= '9') || (c >= 'a' && c <= 'f') || (c >= 'A' && c <= 'F')){}
djmans
это еще хуже :)а так да тут стандартная сишная функция :)
Отличный код! Забавный подход :-)
Надо добавить свистелок:
int Cparse_post::is_hex(char c)
{
return !(strchr("0123456789abcdefABCDEF",c) && 1);
}
2 djmans:
Вы привязываетесь к конкретным кодам символов и порядку их следования в таблице. Это не очень хорошо
Вроде бы стандарт гарантирует, что символы идут последовательно. Но можно было бы жаловаться, если бы было так:
if((c >= '0' && c <= '9') || ((c = c&~32) >= 'a' && c <= 'f')){}
>> if((c >= '0' && c <= '9') || ((c = c&~32) >= 'a' && c <= 'f')){}
и этот говнокод лучше исходного? помилуйте..
уж лушче strchr
А вы привязываетесь к конкретным буквам и порядку их следования в русском языке. Это тоже не очень хорошо. Перепишите.