1. C++ / Говнокод #11983

    +30

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

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

    Запостил: bazhenovc, 24 Октября 2012

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

    • ava WGH 24.10.2012 12:43 # +1

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

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

    • ava ErmineMD 24.10.2012 13:02 # +9

      А почему тут ||, а не &&?
      Ответить

      • ava TarasB 24.10.2012 13:07 # 0

        По аналогии с 
        str[0] == '<' || str[0] == '?' || str[0] == 'x' || str[0] == 'm' || str[0] == 'l'
        Ответить

        • ava ErmineMD 24.10.2012 13:08 # +1 

          if( str[0] == '<' && str[1] == '?' && str[2] == 'x' && str[3] == 'm' && str[4] == 'l') {...}
          должно быть, нет?
          Ответить

          • ava bormand 24.10.2012 13:18 # +1

            Да. Ну или для обратного:

            if( str[0] != '<' || str[1] != '?' || str[2] != 'x' || str[3] != 'm' || str[4] != 'l') {...}
            Ответить

      • ava bazhenovc 24.10.2012 14:35 # +1

        Я решил запостить без изменений :)
        Ответить

      • ava absolut 25.10.2012 10:53 # +4

        >А почему тут ||, а не &&?
        Это проверка на "подозрение на xml-ность" :)
        Ответить

    • ava i-- 24.10.2012 14:10 # +4

      <?xmlGovno version="0.1">
      Ответить

    • ava Steve_Brown 24.10.2012 16:02 # +4 

      if (*(long *)str == '<?xm' || *(long *)str == 'mx?<')
  ...

      Close enough.
      Ответить

      • ava defecate-plusplus 24.10.2012 16:14 # +2

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

      • ava WGH 24.10.2012 17:19 # +1

        >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 25.10.2012 10:05 # 0

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

      • ava Elvenfighter 24.10.2012 23:36 # 0

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


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

        • ava bormand 25.10.2012 06:39 # +2

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

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

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

    • ava bot 25.10.2012 19:14 # 0

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

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