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

    −17

    1. 1
    if(0){}else for (... тут всякий код...) и тут всякий код

    Из реализации foreach в Qt. Не ГК. Кто первый скажет почему не ГК получит пирожок с полочки.

    Запостил: bormand, 17 Мая 2012

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

    • ava krypt 17.05.2012 10:09 # 0

      Исправляет область видимости "всякого кода"? Гугл подсказывает о баге, с ней связанным, в старых версиях Visual Studio.
      Ответить

      • ava bormand 17.05.2012 10:18 # 0

        Да, именно так.
        Ответить

        • ava krypt 17.05.2012 10:21 # 0

          А почему не
          if (1) for ...

          или вообще не

          { for (...)
          {
          }}

          ?
          Ответить

          • ava krypt 17.05.2012 10:24 # +1

            Хотя да, 2е негья исправить дефайном, а 1е может иметь эффекты с другими условиями
            Ответить

    • ava Arigato 17.05.2012 22:10 # 0

      Говнецо, однозначно!
      Ответить

      • ava bormand 18.05.2012 05:10 # 0

        Да ну ? )
        Ответить

      • ava krypt 18.05.2012 11:20 # 0

        Ненене, это не говнецо. Говнецо в VS, а это классическая подпорка, в простонародье именуемая "костылём" )
        Ответить

    • ava absolut 18.05.2012 08:43 # 0

      помнится BCB6 тоже таким грешил
      Ответить

    • ava TheCalligrapher 19.05.2012 19:48 # +9

      Ну, так избитая классика жанра - в С++ исправление scope переменной, объявленной в заголовке цикла 'for'. Т.е. в

      for (int i = 0; ... )


      переменная 'i' не должна существовать после тела 'for'. А в старом VC++ она продолжала жить и далее.

      Только в явно выписанном коде подобный трюк с 'if;else' не имеет никакого смысла - можно было просто взять этот 'for' в дополнительные скобки '{}' и дело с концом. Из поста не ясно, было ли это в оригинале макросом, или все таки было выписано посередине обыкновенного кода. В последнем случае - есть элемент "говнокодовости".

      Так вот, этот трюк с 'if;else' применяется именно для написания макроса/макросов, решающего данную проблему. В частности, самым радикальным вариантом будет просто сделать

      #define for if (false); else for


      т.е. фактически нагло "переопределить" ключевое слово 'for' и вопрос решен.

      А сам трюк с 'if;else' (вне зависимости от проблемы с 'for') произрастает из техники правильного написания многострочных (т.е. много-stetement-ных) макросов. Правильно написанный многострочный макрос должен быть либо заооболочен в скобки 'do{ ... } while (false)', либо в скобки 'if (false); else { ... }'. Подобная "оболочка" вокруг макроса делает его поведение правильным при использовании в ветках внешних if-ов.
      Ответить

      • ava HaskellGovno 19.05.2012 22:57 # +2

        У Вас нет своего блога или статей по программированию? Я бы с большим удовольствием почитал. :)
        Ответить

      • ava bormand 20.05.2012 05:34 # 0

        > Из поста не ясно, было ли это в оригинале макросом, или все таки было выписано посередине обыкновенного кода.

        Да, код взят из макроса foreach.
        Ответить

      • ava HaskellGovno 20.05.2012 09:26 # 0

        А чем 
        #define for if (true) for

        не угодил?
        Ответить

        • ava TheCalligrapher 20.05.2012 10:10 # +11

          Надо чтобы макрос вел себя правильно во внешних if-ах, как я говорил выше. Если сделать по-вашему, то вот в таком коде

          if (условие)
   for (тра-ля-ля) {
     чего-то делаем;
   }
else {
  чего-то делаем;
}


          ветка 'else' будет проассоциирована с "невидимым" if-ом спрятанным внутри макроса 'for'. Это, разумеется, будет большой неожиданностью для автора кода и доставит ему массу счастливых часов недоуменной отладки.
          Ответить

          • ava HaskellGovno 20.05.2012 12:40 # 0

            Очень годно. Мужик! Уважаю!
            Ответить

          • ava JavaGovno 20.05.2012 13:13 # +1

            ждем MacroGovno
            Ответить

          • ava SemaReal 14.10.2017 01:34 # 0

            Мне всегда было интересно: а вот наступающие на эти шутки с макросами люди -- они что, не могут прогнать препроцессор и поглядеть что получилось?
            Ответить

            • ava bormand 14.10.2017 07:26 # 0

              Могут, но об этом же ещё вспомнить надо...
              Ответить

    • ava inho 12.10.2017 13:09 # 0

      Говно
      Ответить

    • ava nyTuH_nugop_pidor 28.03.2026 18:12 # 0

      Делают do {...} while(false) угадайте для чего?
      Ответить

      • ava guest6 28.03.2026 18:15 # 0

        дай-ка угадаю: Макроёбы делают?
        Ответить

        • ava nyTuH_nugop_pidor 28.03.2026 18:20 # 0

          Знаешь для чего?
          Ответить

          • ava guest6 28.03.2026 18:22 # 0

            https://govnokod.xyz/_10306/#comment-137822

            >>
            А сам трюк с 'if;else' (вне зависимости от проблемы с 'for') произрастает из техники правильного написания многострочных (т.е. много-stetement-ных) макросов. Правильно написанный многострочный макрос должен быть либо заооболочен в скобки 'do{ ... } while (false)', либо в скобки 'if (false); else { ... }'. Подобная "оболочка" вокруг макроса делает его поведение правильным при использовании в ветках внешних if-ов.
            >>>
            Ответить

    • ava nyTuH_nugop_pidor 28.03.2026 18:21 # 0

      Я — nugop.
      Ответить

    • ava guest6 28.03.2026 19:51 # 0

      Поскольку я ничего не умею, то вряд ли я могу найти работу побогаче. Правда, иногда попадаются вакансии, где предлагают в два раза больше с теми же требованиями. Но я вряд ли составлю туда конкуренцию. Наверняка туда уже молодых 50 человек понабежало. А я, с точки зрения работодателя, если за 8 лет в фронтенде не научился тесты писать и никогда вебпак не настраивал, то, видимо, и не научусь.
      Ответить

    • ava guest6 28.03.2026 19:53 # 0

      да ну его нафиг. Это залупа какая то. Вместо человеческого cntrl+s каждый раз этот танец с бубном делать. Пошли они **** со своим гитом. Это как покакать и вместо того чтобы просто подтереться и помыть руки с грязными начинаешь в тетрадку записывать. Какашка отправлена, бумага получена. Прям непередаваемая жесть.

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

      • ava X3-KTO 28.03.2026 20:05 # 0

        Ура! Ротоеб вернулся!
        Ответить

        • ava guest6 28.03.2026 23:19 # 0

          ты думаешь я шучу что ли?

          Это вообще крутой пацанчик
          https://habr.com/ru/articles/1016136/comments/#comment_29737752
          или вот
          https://habr.com/ru/articles/1014102/comments/#comment_29728984

          прямо с сайта пхпклуб из 2001-го года к нам пожаловал
          Ответить

          • ava guest6 28.03.2026 23:20 # 0

            > Мне проще снова его написать чем КАЖДЫЙ РАЗ вместо обычного cntrl+s вот эту вот всю бороду проделывать.

            Действительно, охуенный.
            Ответить

            • ava guest6 28.03.2026 23:57 # 0

              и вот еще питух смешной, никогда про KUnit не слышавший, но зачем-то решивший выбенутьсчя
              https://habr.com/ru/articles/1014102/#comment_29721198
              Ответить

            • ava guest6 29.03.2026 00:00 # 0

              давайте его на сайт позовем?? ну пазяззя
              https://habr.com/ru/articles/1014102/#comment_29728690
              Ответить

              • ava guest6 29.03.2026 00:20 # 0

                Я тоже против юнит-тестов и против протоколирования «я посрал». Я просто пишу говнокод и теку.
                Ответить

    • ava guest6 28.03.2026 21:07 # 0

      https://habr.com/ru/articles/850590/
      побдитель конкурса "ненужная статья"
      Ответить

    • ava guest6 28.03.2026 23:35 # 0

      Гологубы столетней давности были намного креативнее.

      Смотрите, какую классную карикатуру нарисовал немецкий противник понаехов сто лет назад, буквально миллион деталей
      https://theins.ru/storage/content_image/original/69c/69c28bf2e4f4c3.59688035/YCf1fwoGhNFOFWRcUruQaSMFkAqRDqBHuxvAmMfy.jpg
      Ответить

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