Говнокод: по колено в коде.

• Все
• Лучший
• Сток
• Глупый поиск
• Наговнокодить!
• Oтзывы

Нашли или выдавили из себя код, который нельзя назвать нормальным, на который без улыбки не взглянешь? Не торопитесь его удалять или рефакторить, — запостите его на говнокод.ру, посмеёмся вместе!

1. PHP (5698)
2. C++ (2733)
3. Куча (2324)
4. JavaScript (2027)
5. C# (1925)
6. Java (1501)
7. Си (1095)
8. Pascal (647)
9. Python (582)
10. 1C (531)
11. SQL (431)
12. ActionScript (292)
13. Objective C (202)
14. bash (196)
15. Perl (194)
16. Assembler (146)
17. Ruby (142)
18. VisualBasic (138)
19. Lua (49)
20. Go (28)
21. Swift (27)
22. Haskell (26)
23. Kotlin (13)

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

   +58

   1. 1

   #define CHK( x ) {do{if (x == MBRES_ERR){goto __gotoFAIL;}} while (FALSE);}

   Из SDK корейского сканера

   Запостил: snw, 08 Августа 2014

   Tweet

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

   • Soul_re@ver 08.08.2014 15:35 # +3

     Полное непонимание зачем нужен и как делать do .. while(0) в макросах.
     
     Использование FALSE вместо false в крестах тоже радует.

     Ответить

     • inkanus-gray 08.08.2014 15:41 # 0

       А ещё goto в макросе, да к тому же в крестах...

       Ответить

       • bormand 08.08.2014 15:56 # +1

         > крестах
         Да не кресты это. Это C/C++ (когда автор пишет на крестах как на си).

         Ответить

         • roman-kashitsyn 08.08.2014 16:04 # +3

           > C/C++ (когда автор пишет на крестах как на си)
           Самое отвратительное сочетание

           Ответить

           • inkanus-gray 08.08.2014 17:01 # 0

             А с Objective C, значит, лучше?

             Ответить

             • kegdan 08.08.2014 17:04 # 0

               почему?

               Ответить

               • inkanus-gray 08.08.2014 17:08 # 0

                 Потому что у C с C++ самое отвратительное сочетание.

                 Ответить

                 • kegdan 08.08.2014 17:13 # −2

                   Там свои сорта говна, в том мире уже нет понятие хуже - лучше. Там скорее пожухлое - свежее, бурое - серое

                   Ответить

           • Psionic 08.08.2014 17:30 # 0

             О ужас, ну тогда любая прикладная прога говно, API то что у Винды, что у Линупса, шо у Юникса сишный. И если ты решил что-то написать на них под С++ - то пиздравляю. Вроде как у макоси есть обьекто ориентированый API, но печенью чую что просто прослойка над сишним ядром UNIX.

             Ответить

             • roman-kashitsyn 08.08.2014 17:35 # +2

               > И если ты решил что-то написать на них под С++ - то пиздравляю.
               
               Нет никаких проблем реализовать полноценную C++-обёртку (читай - с поддержкой RAII) над сишным апи и спокойно писать прикладуху на идеоматичном C++.
               
               Я просто насмотрелся на проги, которые используют сырые низкоуровневые ресурсы и malloc/free/new/delete вперемешку с исключенями и goto. Это леденящий душу 3.14здец.

               Ответить

               • defecate-plusplus 08.08.2014 17:38 # +6

                 фу какой у тебя грубый 3.14159265здец

                 Ответить

               • Psionic 08.08.2014 19:09 # 0

                 >>>malloc/free
                 А какой в них смысл если доступны new/delete? Я бы вообще в свежий стандарт
                 чистого Си (если он выходит сейчас) бы перенес их вместе с пространствами имен и каким-то простейшим полиморфизмом для сишных функций.

                 Ответить

                 • bormand 08.08.2014 19:41 # 0

                   > А какой в них смысл если доступны new/delete
                   К примеру, если какой-то сишной либе нужно отдать указатель, который она потом освободит через free. Ну или наоборот - либа возвращает malloc'нутый указатель, который мне нужно будет освободить через free.

                   Ответить

         • kegdan 08.08.2014 16:45 # 0

           ака Си с классами

           Ответить

           • roman-kashitsyn 08.08.2014 16:47 # 0

             и с исключениями

             Ответить

             • myaut 08.08.2014 16:48 # 0

               и с goto

               Ответить

               • bormand 08.08.2014 17:11 # 0

                 void foo() {
                     try { operation1(); } catch (...) { goto cleanup; }
                     try { operation2(); } catch (...) { goto cleanup; }
                     // ...
                 cleanup:
                     // ...
                 }

                 Ответить

             • kegdan 08.08.2014 16:48 # 0

               "Я запилю свой си! С классами! И исключениями!" Сказал как то мертвый страус и понеслось....

               Ответить

               • Elvenfighter 08.08.2014 17:16 # 0

                 Нет, он просто дал эту гениальную вещь макакам, которые с ней обращаться-то не умеют.

                 Ответить

                 • kegdan 08.08.2014 17:23 # 0

                   гениальную? и в чем гениальность плюсов?

                   Ответить

                   • inkanus-gray 08.08.2014 19:57 # +2

                     В том, что валидность кода может зависеть от значения константы.

                     Ответить

                     • kegdan 08.08.2014 20:01 # +3

                       вспомнил бородатое
                       
                       
                       Известны 10 преимуществ Паскаля перед Си
                       Я приведу только одно, но самое важное.
                       На Си вы можете написать:
                       for(;P("\n").R-;P("\ "))for(e=3DC;e-;P("_ "+(*u++/8)%2))P("| "+ (*u/4)%2);
                       На Паскале Вы НЕ МОЖЕТЕ такого написать

                       Ответить

                       • defecate-plusplus 08.08.2014 20:04 # +3

                         > 3DC
                         что это?

                         Ответить

                         • kegdan 08.08.2014 20:16 # 0

                           я хз, просто скопипастил. видимо без С просто тройка даблов

                           Ответить

                           • bormand 08.08.2014 20:52 # +2

                             > тройка даблов
                             Туз интов.

                             Ответить

                             • kegdan 08.08.2014 20:57 # +2

                               Король стригнов

                               Ответить

                         • kegdan 08.08.2014 20:17 # +1

                           for(;P("\n"),R--;P("|"))for(e=C;e--;P("_"+(*u++/8)%2))P("| "+(*u/4)%2);
                           
                           нашел православный вариант

                           Ответить

                           • bormand 08.08.2014 20:53 # 0

                             Этому коду не хватает определений P и R.

                             Ответить

                             • inkanus-gray 09.08.2014 15:02 # +1

                               Для стиля K&R пойдёт и без определения P (подразумевается int P(int), что само по себе говно, но работает). Нужно только определить переменные C, u, R, e.

                               Ответить

                   • Psionic 08.08.2014 23:01 # 0

                     В том что он находится на стыке двух миров - мира процессорных инструкций и мира высокоуровневых языков. Чтоб хорошо писать на Си нужно с одной сторы иметь хотябы примерное предствление как твой код превращается в процессорные инструкции, а с другой тебе вовсе не нужно писать на громостком ассеблере.

                     Ответить

                     • kegdan 08.08.2014 23:03 # +1

                       Это гениальность С. А в чем гениальность плюсов?)

                       Ответить

                       • Psionic 08.08.2014 23:23 # 0

                         Все тоже самое, но естьООП (но его считают некоторые личности эрзацем) и еще пачка современных ништячков.

                         Ответить

                         • kegdan 08.08.2014 23:35 # 0

                           Это не делает его гениальным
                           
                           А С с ООП так и вообще смотрится адово

                           Ответить

                   • guest 09.08.2014 12:25 # 0

                     Мощнейшая система изнасилования типом в компалтайме.
                     
                     Что дает сочетание максимального быстродействия рантайма, экономичности и безопасности.

                     Ответить

                   • roman-kashitsyn 09.08.2014 13:46 # +4

                     > в чем гениальность плюсов?
                     
                     1. Универсальный механизм управления ресурсами - объекты + идиома RAII.
                     2. Встроенный функциональный язык для метапрограммирования с паттерн-матчингом - шаблоны. Если подсел на шаблоны, мир уже никогда больше не станет прежним.
                     3. Возможность выбрать стиль программирования, наиболее подходящий к задаче - процедурный, ООП, функциональщина, метапрограммирование. Можно сочетать.
                     4. Возможность напрямую дёргать системные вызовы и низкоуровневые сишные апи, не дожидаясь, пока их красиво обернут в популярной платформе. Особенно полезно в сочетании с пунктом 1.
                     5. Нет пауз gc. Возможность работы в условиях real-time.
                     6. Быстрый старт приложений.

                     Ответить

                     • kegdan 09.08.2014 14:17 # 0

                       1 благодаря чему программы текут как сучки
                       2 А потом начинается шаблоноблядство - вообще хуй че прочитаешь
                       3 лучше иметь кошерный набор инструментов, чем один, который может все, в том числе и стрелять тебе по ногам, сношать тебя анально и течь
                       4 в резулятате чего ось ложиться на лопатки - ибо дергается за яйца без какой либо защиты (зато быстро)
                       5. По дефолту нет. А вообще можно подцепить
                       
                       так о чем это я? о том, что язык полезен, но он полон подводных камней, и подходить к нему надо осторожно и в сознательном возрасте (не раньше 25 лет). Неоправданно высокий пород вхождения, монструозность и всеобьемнимость. Си гениален тем, что прост как 3 рубля, но эффективен и красив. А тут... печаль - тоска. Как по мне - так плюсы должны быть малопопулярны (как хаски), дабы люди, решившие на них кодить, понимали че это, и что язык этот - удел извращенцев и гиков.
                       
                       Почему я ненавижу язык. люто, бегшено? - потому что во всех вузах (кроме ТГУ ибо у нах пох на чем сдавать) и полувузах его преподают как основной язык. Человек не знает что такое граф - но его уже садят на крестоблядскую машину. Это прискорбно

                       Ответить

                       • bormand 09.08.2014 14:28 # +4

                         > благодаря чему программы текут как сучки
                         Только у неосиляторов, которые все еще освобождают память вручную.
                         
                         У меня в небольшом проектике было одно delete на 90+ классов, и то просто лень было выпиливать... И ничего не текло, да. А с move семантикой можно еще и кучу new убрать...
                         
                         > преподают как основной язык
                         Уверен? Вроде в вузах преподают C/C++ (аля си с классами). И это прискорбно, да. Как раз из-за такого обучения проги у нубов текут как сучки...

                         Ответить

                         • kegdan 09.08.2014 15:05 # 0

                           хуй знает, чем им преподают, но они бегают и кричат - мне нужна крестобала! мне нужна крестобала!

                           Ответить

                           • 1024-- 09.08.2014 15:12 # 0

                             Кажется, копипастер попался!

                             Ответить

                             • kegdan 09.08.2014 15:14 # 0

                               не царское это дело - повторы писать

                               Ответить

                       • Abbath 09.08.2014 14:31 # 0

                         Ты неправ.

                         Ответить

                         • kegdan 09.08.2014 15:04 # +1

                           ты изменил мою жизнь

                           Ответить

                       • bormand 09.08.2014 14:39 # +2

                         > в резулятате чего ось ложиться на лопатки
                         Если прога может взять и положить ось на лопатки - значит это была хуёвая ось (или просто ты купил мало оперативки и она упала в своп).
                         
                         > дергается за яйца без какой либо защиты
                         Ты под досом что-ли работаешь? :)

                         Ответить

                         • 1024-- 09.08.2014 14:48 # 0

                           А знаете... Дуракам везёт. Особенно - на баги в осях.

                           Ответить

                           • bormand 09.08.2014 14:58 # 0

                             У меня был битый ico файл, который ронял XP в BSOD, если его открыть в irfan view...
                             
                             А баги в ядре и дровах они ведь на любые языки одинаково влияют. Сишка/кресты тут на равных со всеми остальными.

                             Ответить

                           • TauSigma 11.08.2014 16:14 # 0

                             Ещё реализация Biff8 в Office 2003 очень хорошо ложит ось на лопатки.
                             Там букет разнообразных ароматов достигается уже после 2-3 недель реверсирования...

                             Ответить

                         • TauSigma 11.08.2014 16:11 # 0

                           >значит это была хуёвая ось
                           Либо стало слишком много фиговых писателей...

                           Ответить

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

   Ошибка компиляции комментария:

   1. Гости могут высказаться только в понедельник, среду, четверг или воскресение

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

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

   • [b]жирный[/b] — жирный
   • [i]курсив[/i] — курсив
   • [u]подчеркнутый[/u] — подчеркнутый
   • [s]перечеркнутый[/s] — перечеркнутый
   • [blink]мигающий[/blink] — мигающий
   • [color=red]цвет[/color] — цвет (подробнее)
   • [size=20]размер[/size] — размер (подробнее)
   • [code=<language>]some code[/code] (подробнее)

   Проверочный код: *