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

0

  1. 1
  2. 2
  3. 3
// Всем привет. Я тоже принёс говнокода, но в необычном формате.
// А именно, я написал мини-книгу "60 антипаттернов для С++ программиста".
// https://pvs-studio.ru/ru/blog/posts/cpp/1053/

Там вы найдёте и реальный C++ говнокод и просто вредные советы в духе "Пишите код так, как будто его будет читать председатель жюри IOCCC и он знает, где вы живёте (чтоб приехать и вручить вам приз)".

Если сразу не понятно почему "совет" вреден, то там есть соответствующий разбор.

Готов подискутировать про написанное. Ну и приглашаю накидывать в комментариях аналогичные советы.

P.S. Предупреждаю: там много букв. Сразу запасайтесь кофе/энергетиком. Или попкорном :)

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

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

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

    >Вредный совет N6. Невидимые символы
    >
    >Используйте при написании кода невидимые символы. Пусть ваш код работает магическим образом. Это прикольно.

    Я б этот вредный совет расширил до "Используйте побольше символов за пределами ASCII"
    Ответить

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

      Смайлик говна какое-то время (а может и сейчас) был легальным символом в идентификаторах
      Ответить

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

    >Вредный совет N31. Всё в h-файлах
    >
    >Побольше кода в заголовочных файлах, ведь так гораздо удобнее, а время компиляции возрастает очень незначительно.
    >
    >В эпоху моды на header-only библиотеки этот совет не кажется таким уж и вредным. В конце концов, существует даже "A curated list of awesome header-only C++ libraries".
    >
    >Но одно дело – маленькие библиотеки. А другое – большой проект, в который вовлечены десятки людей и который развивается многие годы. В какой-то момент время компиляции вырастет с минут до часов, а сделать что-то с этим уже будет сложно. Не рефакторить же сотни и тысячи файлов, перенося реализацию функций из *.h в cpp-файлы? А если рефакторить, то не проще ли сразу было делать нормально? 🙂

    А это уже проблема скорее компилятора, который не может быстро это делать. Допустим, есть некий .h файл где реализована некая шаблонная хренотень, есть куча .cpp файлов которые этот .h файл инклудят и каким-то образом используют, компилятор каждый раз с нуля все парсит в хедерах и специализирует, как это можно было бы исправить чтоб был быстро? Например, скомпилировать сам .h файл в некий промежуточный байткод, чтобы каждый раз не тратить время на его парсинг. Если во множестве файлов некий шаблон из хедера специализируется одним и тем же, это тоже можно как-нибудь закешировать и значительно ускорить компиляцию. Под что-то такое даже пилился специальный кеширующий компилятор zapcc https://github.com/yrnkrn/zapcc (основан на clang), но что-то не взлетело.
    Ответить

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

      У многих компиляторов есть PCH — precompiled C headers, но реализован он страшно неудобно: хоть один из хедеров поменятся, и PCH собирается заново с нуля. И исправить это невозможно из-за самой природы макросов, потому что один хедер может портить своими макросами другой.
      Ответить

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

    Про «UB» я бы вторым советом вынес. Абсолютно невыносимы питухи, которые пытаются оправдывать свои UB, а уж питухи, которые при этом с умным видом рассуждают, какой -O не «ломает их программу» — это вообще пиздец.
    Ответить

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

        У Царя некоторые программы работали исключительно с -O2, а с -O0, -O1, -O3 и с прочими параметрами оптимизации падали, потому что с его точки зрения -O2 — единственно правильный ключ.
        Ответить

    • ava guest # 0 ⇈

      Не знаю, у меня на компьютере все работает
      Ответить

  • ava Soul_re@ver # Ссылка на оригинал 0

    > Табуляция [в строковых литералах] в процессе рефакторинга или использования утилит автоформатирования кода может превратиться в пробелы, что повлияет на результат работы программы.

    Утилиты автоформатирования, которы трогают строковые литералы — то ещё говно.
    Ответить

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

      Это уровень пиздец-оффтопа уже, они вообще не должны туда лазить, литерал — одна неделимая нода в parsing tree.
      Ответить

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

      > Табуляция [в строковых литералах] в процессе рефакторинга или использования утилит автоформатирования кода может превратиться в пробелы, что повлияет на результат работы программы.

      Кривая утилита автоформатирования может и несколько пробелов скукожить в один(или заменить на таб), так что теперь, пробелы в строковых литералах не использовать?
      Ответить

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

        > несколько пробелов скукожить в один
        Приоткрывается завеса на чём написана эта утялита.
        Ответить

    • ava guest # 0 ⇈

      >> Утилиты автоформатирования, которы трогают строковые литералы — то ещё говно.


      Это еще ничего. Некоторые утилиты автоформатирования и бизнес-логику править могут
      Ответить

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

        Как вы такое юзаете? Если всё через жопу, то какой шанс на то, что сделает красиво?

        Единственный вариант вижу: немного кривой выхлоп утялиты случайно совпадает с немного кривым видением смузихлёба, а потом немного кривой стартап и целая стая приматов с этим все живет от получки до получки
        Ответить

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

    Уважаемый Вячеслав Владимирович! Убедительно просим Вас обратить внимание на нашу проблему: с. Новая, Таволжанка, ул. Волчанская. Во многие дома проникают военнослужащие РФ(хотя режима ЧС не введено и они не имеют права заходить в частные домовладения). Они живут в наших домах, ведут безобразный образ жизни, алкоголь и прочее остаётся в виде мусора и грязи, загажены туалеты и дома,воруются личные вещи и имущество. Из нашего гаража был украден квадроцикл, который теперь часто видят в округах Новой Таволжанки. Кроме этого люди жаловались на украденный прицеп. Теперь он катается вместе с нашим квадроциклом. Мы очень просим помочь в нашем вопросе. Мы не хотим, чтобы наши дома, которые итак пострадали от действий ВСУ, теперь были ещё пристанищем для безобразий наших защитников!

    https://vk.com/wall639631882_932848?reply=932944

    Унитаз воровать невыгодно через 3... 2...
    Ответить

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

      а тем у кого нагадили в доме? Куда смотрит власть???? Куда стучать???? Шойгу твои военные у нас воруют! Российские военные у российских гражданских людей воруют и там же срут!!! Позор власти, Позор Шойгу, Позор русскому солдату!!!

      п-р сркн
      Ответить

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

      Наталья, квадроцикл с прицепом даже в Ютуб попал. Там и правда, военные на нем. https://youtu.be/830rPQvFyIc на 3 мин.13 сек. можно увидеть.
      Ответить

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

      К сожалению, люди слишком поздно понимают, что силовики, менты и проч - это легальные бандиты, на службе государства. Враги с положительным зарядом. На их оружии кровь, так же, как у бандитов и убийц.
      И им решительно похуй, кого перемолоть, если невовремя подвернуться под руку.
      Ответить

  • ava guest # 0

    Хорошие советы, но некоторые очень уж юниорски (типа глобальных переменных)
    Буду писать по комментарию про совет раз в час
    Ответить

  • ava guest # 0

    Совет 1:

    >>
    Плохо, когда начинают использовать этот язык только потому, что это "круто" или это единственный язык, с которым хорошо знакома команда.
    >>

    Я бы разделил эти два понятия. "Круто" -- это для школьников, а вот выбирать язык потому, что с ним хорошо знакома ваша команда -- вполне логично.

    Я чаще вижу обратную проблему: люди берут какой-нить Python "потому что нужно быстро", а потом у них все тормозит.
    Ответить

  • ava guest # 0

    Совет 2:

    >>случается, что программист вместо того, чтобы использовать '\t', не задумываясь, просто нажимает кнопку TAB.

    Это странный аргумент. Случается так, что программист вместо "gcc" случайно нажимет "rm / -rf", но это не повод не пользоваться gcc.

    >>На самом деле в литерале могут оказаться не табы, а пробелы. Это зависит от настроек редактора.

    Пожалуйста всегда имейте одинаковые настройки у всех программистов проекта, и храние их в VCS, иначе у вас отбивка в каждой функции будет зависеть от предпочтений программиста

    >>Явные символы табуляции использовались в перемешку с пробелами.
    кошмар))) но я на лабах такое видел
    Ответить

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

      Есть текстовые редакторы, которые заменяют табы на пробелы или пробелы на табы. Например, IDE на Turbo Vision так делали (там в настройках была галочка, причём вариант «ничего не заменять» отсутствовал).
      Ответить

  • ava guest # 0

    Совет 3.

    >>Вложенные макросы

    Мне кажется, что макросы нужны только для совместимости с си. В плюсах макросы не нужны почти никогда.
    Ответить

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

      Действительно, если в сишке ещё использовали #define для эмуляции именованных констант, то в крестах есть const.

      Параметрические макросы? В крестах есть шаблонные функции. Ничем не хуже, заодно добавится проверка типов аргументов (мы же не на скриптовом языке пишем).

      Что остаётся? #ifdef да #include. Остальные возможности макросов разве что для отладки.

      Чуть не забыл про constexpr и consteval, которые могут в компилтайме выполнить то, что раньше делали макросами.
      Ответить

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

        В макросах ещё остаётся ещё стрингификация идентификаторов.

        Чтобы делать что-то типа:

        LOG(huita)

        >> Value of "huita" is: 42
        Ответить

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

          А, вот что, оказывается, означает «для кривой мумуляции компайлтайм-рефлексии».
          Ответить

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

      К слову, какой ещё ЯП, кроме сишки, придумали Керниган и Ритчи? Они ещё предлагали на нём писать макросы.
      Ответить

      • ava guest # 0 ⇈

        B:)

        А еще awk: папу перла, дедушку руби
        Ответить

          • ava guest # 0 ⇈

            ааа, точно))) и как раз для макросов же
            Ответить

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

          Глянул статейки про CPL и BCPL, которые почему-то считаются предшественниками Би и Си. Ничего общего. В CPL и BCPL был паттерн-матчинг и другие интересные штуки, которые сишкой были похоронены на долгие годы, потому что K&R не смогли сделать эффективную реализацию и урезали всё, что могли.
          Ответить

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

      Пока ещё нужны для кривой мумуляции компайлтайм-рефлексии. Когда её завезут по-настоящему — тогда можно будет хоронить, да.
      Ответить

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

        Макросы позволяют делать хуйню, которую принципиально без макросов пока никак не сделать, и которая к компилтайм-рефлексии прямого отношения не имеет.
        Вот взять пример из https://gcc.gnu.org/onlinedocs/cpp/Stringizing.html :
        #define WARN_IF(EXP) \
do { if (EXP) \
        fprintf (stderr, "Warning: " #EXP "\n"); } \
while (0)

        Как это переписать без макроса, и при чем тут компилтайм-рефлексия?
        Ответить

        • ava guest # 0 ⇈

          >> и причем тут компилтайм-рефлексия?
          >>#EXP
          Ответить

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

            И как стрингификация относится к компилтайм-рефлексии?
            Ответить

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

              Потому что стрингификация это частный случай рефлексии.
              Ответить

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

                Почему это стрингификация является частным случаем рефлексии?
                Ответить

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

                  По определению, рефлексия – это:

                  Отражение (рефлексия; холоним интроспекции, англ. reflection) — процесс, во время которого программа может отслеживать и модифицировать собственную структуру и поведение во время выполнения (мы говорим про рефлексию времени компиляции, к слову). Парадигма программирования, положенная в основу отражения, является одной из форм метапрограммирования[1] и называется рефлексивным программированием.

                  Таким образом, раз программа во время компиляции отслеживает собственную структуру (имена переменных), то это – частный случай рефлексии.
                  Ответить

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

                    Макроговно никакие "имена переменных" не отслеживает, ни про какие имена переменных оно не знает, оно делает тупую подстановку текста по шаблону, т.е. рефлексией это не является.
                    Ответить

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

                      Макроговно поддерживает операцию: "получить имя переменной", это – частный случай рефлексии, т.к. отслеживаются метаинформация программы.
                      Ответить

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

                        никакое "имя переменной" оно не получает, оно тупо стрингифицирует некую хуету
                        Ответить

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

                          То, что ты описал – это детали реализации. И это не меняет того факта, что в C есть интерфейс для получения имени переменной из исходного кода, что и является частным случаем метапрограммирования.
                          Ответить

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

                            Нет, никакого получения имени переменной там нет. Там есть стрингификация.
                            Ответить

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

                              Я и не говорил, что оно есть. Я сказал, что существует интерфейс для получения имени переменной во время сборки. То, что он реализован с помощью подстановок и макросов – это деталь реализации, и вовсе не главное.

                              Ты не прав, а я – прав. Потому что я выучил сишку.

                              Ты сейчас похож на человека, который не верит, что на C можно писать в ООП парадигме, т.к. в C отсутствуют т.н. "классы".
                              Ответить

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

                              Он, как попка, – «Си» да «Си»,
                              Ну а сам все налегает
                              На селедку иваси!
                              Ответить

                              • ava guest # 0 ⇈

                                Помолюсь, как говорится, Аллаху
                                И рубаю в маринаде салаку.

                                А на утро я от жажды мычу,
                                И хоть воду мне давай, хоть мочу!
                                Ответить

  • ava guest # 0

    * Вредный совет N4. Выключить предупреждения

    ну тут без комментариев. За заметание мусора под ковёр нужно гнать из профессии, да

    >>Старайтесь, чтобы при компиляции проекта у вас не выдавалось ни одного предупреждения.
    охох, может быть вам еще и 40 мигающих тестов не нравятся?:)

    >>Вредный совет N5. Чем короче имя переменной, тем лучше
    А вот в языке Го принято ИНОГДА давать корткие имена переменным (заместо this)
    https://github.com/golang/go/wiki/CodeReviewComments#variable-names

    Кроме того, есть всякие for(i и пр

    Сокращать имена можно (arr и vec вместо array и vector, например)

    >>Вредный совет N6. Невидимые символы
    ну тут я могу только троллфейс нарисовать
    Ответить

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

    . 
 * p v s * p v s * p v s * p v s * p v s * p v s * 
 p                                               p  
 v /     \             \            /    \       v  
 s|       |             \          |      |      s  
 *|       `.             |         |       :     *  
 p`        |             |        \|       |     p  
 v \       | /       /  \\\   --__ \\       :    v  
 s  \      \/   _--~~          ~--__| \     |    s  
 *   \      \_-~                    ~-_\    |    *  
 p    \_     \        _.--------.______\|   |    p  
 v      \     \______// _ ___ _ (_(__P  \   |    v  
 s       \   .  C ___)  ______ (_(____V  |  /    s  
 *       /\ |   + ____)/ viva \ (_____S  |_/     *  
 p      / /\|   +_____)   64  |  (___>   /  \    p  
 v     |   (   23_____)\______/  // _/ /     \   v  
 s     |    \  |__   \\_________// (__/       |  s  
 *    | \    \____)   `----   --'             |  *  
 p    |  \_          ___\       /_          _/ | p  
 v   |              /    |     |  \            | v  
 s   |             |    /       \  \           | s  
 *   |          / /    |  U   B  |  \           |*  
 p   |         / /      \__/\___/    |          |p  
 v  |         / /        |    |       |         |v  
 s  |          |         |    |       |         |s  
 * p v s * p v s * p v s * p v s * p v s * p v s *
    Ответить

    • ava guest # 0 ⇈

      Выглядит забавно. Жаль, что пиздежь.
      Ответить

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

    > Вредный совет N48. У каждого свой стиль

    Мне искренне жаль автора, если он даже про утилиту "Indent" не слышал.
    Ответить

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

Я, guest, находясь в здравом уме и твердой памяти, торжественно заявляю:

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

    ↑
    8
    ↓