1. Си / Говнокод #19177

    −94

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    if(Validate_Value((void *)pchStream, "st::r:[0\\-9]+:", &bValid) == ERR_NONE)
{
    if(bValid) 
    {
        (void)sscanf(pchStream, "%hd", pusTextID);
    }

    из встроенщины медицинской. проверяем валидность числа регулярками. а вы думали такое только в пыхе...

    Запостил: Dummy00001, 14 Декабря 2015

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

    • ava imihajlov 14.12.2015 11:51 # +1

      Небось сертификацию софт прошел, исходники куда-нибудь отправляли или как там это делается.
      Ответить

      • ava Dummy00001 14.12.2015 11:58 # +2

        официально - да.

        но подкол в том что сертифицируют те же люди что и пишут. клиент зааутсорсил все одному подрядчику.
        Ответить

        • ava Vasiliy 14.12.2015 12:11 # +4

          потом удивляются чего у нас спутники падают.
          Ответить

    • ava codemonkey 14.12.2015 12:07 # −3

      А не легче:
      if (sscanf(buf, "%hd", &number) != 1) 
    return ERR_INPUT;
      Ответить

      • ava Dummy00001 14.12.2015 12:12 # +4

        нет. правильней будет `strtol()` + проверка `endptr` что были все символы обработаны.
        Ответить

        • ava Xom94ok 14.12.2015 21:39 # 0

          Он ещё, ЕМНИП, выставляет errno при переполнении.
          Ответить

    • ava 3_dar 17.12.2015 23:32 # +2

      А зачем там приведение к void?
      Ответить

      • ava bormand 18.12.2015 08:54 # 0

        Давит ворнинг про неиспользуемый результат.
        Ответить

        • ava TarasB 18.12.2015 10:39 # 0

          В студии на w4 такого варнинга нет, это где такой?
          Ответить

          • ava roman-kashitsyn 18.12.2015 11:13 # +2

            В GCC такой есть и включен по дефолту, но но только если функция помечена атрибутом warn_unused_result и только когда компиляешь с оптимизацией.
            Ответить

            • ava Dummy00001 18.12.2015 11:41 # 0

              некоторые типа меня еще пользуются этим (вместо комментария) для того что бы в коде пометить что возвращаемое значение пофиг.
              Ответить

              • ava roman-kashitsyn 18.12.2015 21:09 # 0

                Они только как комментарий и могут работать - эти касты никак не влияют на ворнинги.
                Ответить

                • ava Dummy00001 18.12.2015 21:14 # 0

                  это древняя практика еще с тех времен когда компилеры таких ворнингов не умели делать.
                  Ответить

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