Кресты / Говнокод #11983 Ссылка на оригинал

0

  1. 1
if( str[0] == '<' || str[1] == '?' || str[2] == 'x' || str[3] == 'm' || str[4] == 'l') {...}

Проверка на xml-ность

Запостил: bazhenovc bazhenovc, (Updated )

Комментарии (20) RSS

  • ava WGH # Ссылка на оригинал 0

    Интересно, как оно сделано в программе file? Есть подозрения, что примерно так же.

    Ну а что касается кода... ну пренебрегает автор strncmp, с кем не бывает.
    Ответить

    • ava defecate-plusplus # Ссылка на оригинал 0 ⇈

      1) надо брать int
      и 2) проверять на 2, 4 и 8 символьные литералы тоже, и на будущее 16-символьные тоже неплохо бы
      Ответить

    • ava WGH # Ссылка на оригинал 0 ⇈

      >An ordinary character literal that contains more than one c-char is a multicharacter literal . A multicharacter literal has type int and implementation-defined value.
      Не портабельно. Два наиболее разумных варианта предусмотрены, но всё равно!
      Ответить

      • ava Steve_Brown # Ссылка на оригинал 0 ⇈

        *размышляет, будет ли работать код на тупоконечной платформе, где int имеет ширину 64 бита, а многосимвольные литералы переводятся в число в смешанном порядке*
        Ответить

    • ava Elvenfighter # Ссылка на оригинал 0 ⇈

      Ну так тогда уже:
      if ( *(long *)str == (('<' << 24) | ('?' << 16) | ('x' << 8) | 'm')
    || /* ... */ )


      ... хотя - то же самое что в топике 🙂
      Ответить

      • ava bormand # Ссылка на оригинал 0 ⇈

        > то же самое что в топике 🙂
        [trollmode=on]
        Только топик работает на строках короче четырех символов, а ваш код - UB.

        str = "<?" - 3 байта, '<' '?' и нолик. Код из топика (если не забыть заменить || на &&) остановится на нолике и корректно выдаст false, ваш же код прочитает 1 лишний байт за буфером. Да, согласен, на 32битной платформе краш при этом маловероятен (Но если буфер начинался за 3 байта до конца странички, за которой следует невыделенная память - то таки возможен). Но это UB, а UB это как заряженное ружье висящее на стене, которое когда-нибудь выстрелит.

        К тому же на некоторых платформах этот код не сможет нормально работать из-за требований к выравниванию, endianess и размеру long'а.
        [/trollmode]
        Ответить

  • ava bot # Ссылка на оригинал 0

    Мне это жутко напоминает:
    if (str[j] != '+' || str[j] != '-' || str[j] != '*' || str[j] != '/') {
    Ответить

Добавить комментарий

Переведи на "PHP", guest!

    А не использовать ли нам bbcode?

    ↑
    8
    ↓