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

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

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

1. PHP (5698)
2. C++ (2733)
3. Куча (2323)
4. JavaScript (2027)
5. C# (1924)
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. Си / Говнокод #14137

   +135

   1. 01
   2. 02
   3. 03
   4. 04
   5. 05
   6. 06
   7. 07
   8. 08
   9. 09
   10. 10
   11. 11
   12. 12
   13. 13
   14. 14
   15. 15
   16. 16
   17. 17
   18. 18
   19. 19
   20. 20
   21. 21
   22. 22
   23. 23
   24. 24
   25. 25
   26. 26

   static int
   wl_iw_get_country(
           struct net_device *dev,
           struct iw_request_info *info,
           union iwreq_data *wrqu,
           char *extra
   )
   {
   	char *ccode;
   	int current_channels;
   	
   	WL_TRACE(("%s\n", __FUNCTION__));
   
   	ccode = dhd_bus_country_get(dev);
   	if(ccode){
   		if(0 == strcmp(ccode, "Q2"))
   			current_channels = 11;
   		else if(0 == strcmp(ccode, "EU"))
   			current_channels = 13;
   		else if(0 == strcmp(ccode, "JP"))
   			current_channels = 14;
   	}
   	sprintf(extra, "Scan-Channels = %d", current_channels);
   	printk("Get Channels return %d,(country code = %s)\n",current_channels, ccode);	
   	return 0;
   }

   На этот раз отличился броадком.
   Подсказка: Пользователи с кантри-кодом RU должны СТРАДАТЬ!

   Запостил: Necromant, 25 Ноября 2013

   Tweet

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

   • anonimb84a2f6fd141 26.11.2013 00:48 # −3

     Нипонял, а где default?

     Ответить

     • Necromant 26.11.2013 00:50 # −1

       его нет

       Ответить

       • anonimb84a2f6fd141 26.11.2013 01:11 # −3

         А что произойдет в таком случае?

         Ответить

         • guest 26.11.2013 01:19 # −2

           Всё будет как в старой притче про буратино.

           Ответить

           • anonimb84a2f6fd141 26.11.2013 01:21 # −2

             Опять день гостя? В какой притче?

             Ответить

             • Stertor 26.11.2013 12:52 # +5

               ~Притча про Буратино~
               Буратино прибегает к папе Карло и жалуется: 
               -Папа Карло, папа Карло, мне Мальвина не дает, говорит, у меня эта штучка шершавая.
               -Погоди, сейчас исправим, оботрем шкуркой... 
               А знаешь что, папа Карло, ну ее на хуй, эту Мальвину, потри лучше опять эту штучку шкуркой!..

               
               с уважением, Ваш кэп.

               Ответить

         • dwarf_with_beer 29.11.2013 07:54 # +3

           кто то не из JP Q2 или EU получит число каналов равное куску фрейма предыдущей функции

           Ответить

           • bormand 29.11.2013 13:40 # 0

             Ответить

             • guest 29.11.2013 15:55 # 0

               Борманд, вы же работали на сишном билдере, так скажите: скорость выполнения прог, наисанных на нем сколь-нибудь превосходит скорость дельфийных прог? Или билдер в целом - возможность покодить на дельфях сишникам?

               Ответить

               • bormand 29.11.2013 16:36 # 0

                 > превосходит скорость дельфийных прог
                 Я не знаю, задачи были не те, чтобы скорость задрачивать... Вот в скорости компиляции он точно проигрывает делфям ;) А оптимизатор, имхо, что в D7, что в CB6 дерьмо, перейдешь - поменяешь шило на мыло ;) В новых RAD studio вроде бы годный компилятор стал, но опять же, скорее всего, его улучшили и для паскаля и для сишечки, и профита от перехода с паскаля на си не будет.
                 
                 > билдер в целом - возможность покодить на дельфях сишникам
                 Именно так. Для тех, кто хочет пользоваться делфийскими компонентами, но на дух не переносит паскаль ;) Для других применений он бесполезен. А из-за испорченного языка даже вреден: не знаю как новые, но CB6, емнип, не умеет во множественное наследование из-за совместимости с delphi. И это не единственная нестандартность...

                 Ответить

                 • eth0 02.12.2013 20:23 # +2

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

                   Ответить

                   • Stertor 02.12.2013 20:26 # 0

                     я шутки ради компилировал в билдере проги на делфи - вполне успешно. Вот только редактировать форму не дает : "для дельфийных прог эта функция не подерживается". (ага, кто же тогда еще будет покупать делфи)
                     
                     Кстати. Популярная программа UltraIso написана именно на сиплюсном билдере.

                     Ответить

                     • bormand 02.12.2013 20:45 # +2

                       А в рад студии, в которую превратились делфи и билдер там вообще уже можно половину проекта на делфи писать, а половину на крестах ;)

                       Ответить

                       • LispGovno 02.12.2013 22:05 # +1

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

                         Ответить

                         • bormand 02.12.2013 22:20 # +2

                           В 64 битке. В 32 вроде бы старый. Или я туплю?

                           Ответить

                           • LispGovno 02.12.2013 22:24 # +1

                             Ну странно будет, если в 32битном один набор ошибок, а в 64хбитном другой набор фич.
                             
                             А так может ты прав. я хз

                             Ответить

                           • LispGovno 02.12.2013 22:29 # 0

                             Ты чего никак не зарегаешься на гейдеве?
                             
                             Пошли обсуждать С++1y:
                             
                             http://www.gamedev.ru/flame/forum/?id=183380
                             
                             >Тема повышенной сложности
                             азаза

                             Ответить

                             • bormand 02.12.2013 22:45 # +1

                               > Ты чего никак не зарегаешься на гейдеве?
                               Меня ж зобанят ;)

                               Ответить

                               • LispGovno 02.12.2013 22:51 # 0

                                 За что? За ботоводство? Привычки с говнокода неискоренимы.

                                 Ответить

                               • LispGovno 02.12.2013 22:52 # 0

                                 Я серьёзно. Без тебя там скучно.

                                 Ответить

                                 • defecate-plusplus 02.12.2013 23:09 # +3

                                   там нельзя материться, называть дельфинов дельфинами и обижать платных пользователей

                                   Ответить

                                   • bormand 02.12.2013 23:12 # +2

                                     > там нельзя материться, называть дельфинов дельфинами и обижать платных пользователей
                                     А за донат можно все это делать? :)

                                     Ответить

                                   • anonimb84a2f6fd141 02.12.2013 23:17 # +1

                                     На гейдеве есть платные пользователи?

                                     Ответить

                                     • Stertor 02.12.2013 23:20 # +2

                                       Ага. Только не деньгами, а натурой берут. Гей-клубДев все ж таки..

                                       Ответить

                                       • LispGovno 02.12.2013 23:55 # +4

                                         Да ладно вам посоны. Мы же там все затралим. Они начнут крыть матом в ответ, а мы поудаляем свои коменты. Их сразу всех в бан за мат. Так Гейцдев превратится в говнокод. Всё в говнокод. И даже небо и даже аллах.

                                         Ответить

                                     • defecate-plusplus 03.12.2013 00:15 # +3

                                       > платные пользователи
                                       именно так

                                       Ответить

                                       • LispGovno 03.12.2013 00:22 # 0

                                         Вот ты мне лучше скажи: Они что упоротые?
                                         Зачем в С++11 заменять:
                                         http://www.cplusplus.com/reference/functional/greater/
                                         

                                         template <class T> struct greater : binary_function <T,T,bool> {
                                           bool operator() (const T& x, const T& y) const {return x>y;}
                                         };

                                         на

                                         template <class T> struct greater {
                                           bool operator() (const T& x, const T& y) const {return x>y;}
                                           typedef T first_argument_type;
                                           typedef T second_argument_type;
                                           typedef bool result_type;
                                         };

                                         Ответить

                                         • bormand 03.12.2013 05:35 # 0

                                           > Зачем в С++11 заменять
                                           Поддержка "вывода типов"? :) Чтобы ты мог передать эта хуйню в какой-нибудь комбинатор, и он мог вычислить результирующий тип.

                                           Ответить

                                           • defecate-plusplus 03.12.2013 07:47 # +2

                                             дык в бинари-фанкшен эти тайпдефы и так были
                                             
                                             видать, воркэраунд от сраного компилятора от мс, который ограничивает число иерархий сильнее остальных сраных компиляторов

                                             Ответить

                                             • LispGovno 03.12.2013 08:07 # 0

                                               > ограничивает число иерархий сильнее
                                               
                                               А можно поподробнее?

                                               Ответить

                                               • defecate-plusplus 03.12.2013 08:47 # +3

                                                 http://msdn.microsoft.com/en-us/library/ft39hh4x.aspx
                                                 ну ладно, число наследований они заявляют стандартным
                                                 замечу, что в лимитах у мс никакого прогресса уже охулиард версий - в 2013 те же, что и в 2005 - молодцы, хорошо работают

                                                 Ответить

                                                 • LispGovno 03.12.2013 09:14 # +1

                                                   256 же за глаза? Или нет?

                                                   Ответить

                                                   • defecate-plusplus 03.12.2013 10:13 # +1

                                                     смотря чего
                                                     если мл пива, то это вообще ни о чем

                                                     Ответить

                   • bormand 02.12.2013 20:42 # +2

                     > Слишком много приходится перекомпилировать.
                     Да не. Просто в крестах нет модулей. И приходится слишком много ашек парсить, если PCH отключены. Там вон в билдере\делфи статистика показывается. В делфи компилятор обрабатывает столько строчек, сколько написано в модуле. В билдере в добавок к этому еще несколько десятков тысяч ;)
                     
                     > Свои - можно.
                     Хм. Ну если так - то ок.

                     Ответить

                     • LispGovno 02.12.2013 22:23 # 0

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

                       Ответить

                     • eth0 03.12.2013 20:31 # 0

                       Да, типа того. Давно я там не бывал, уже и подзабывать стал, что и как.

                       Ответить

   • govnomonad 26.11.2013 06:22 # 0

     WL_TRACE(("%s\n", __FUNCTION__));

     
     зачем две пары скобок? Сишка научилась в кортежи?
     
     

     wl_iw_get_country

     
     возвращает то функция не страну, а количество каналов. Причем в виде строки. Нахуя? Да ещё sprintf без проверок границ
     
     Лень искать, но откуда это? Из broadcom-sta или br-mac? Первый - эпически кривое поделка, которое требует выпилить CONFIG_WIRELESS

     Ответить

     • govnomonad 26.11.2013 06:28 # −1

       Линусу ещё бы раз не помешало применить свой спелл на сабжевой конторе

       Ответить

     • LispGovno 26.11.2013 06:39 # 0

       #define WL_TRACE(out) print out

       Ответить

       • LispGovno 26.11.2013 06:44 # −1

         *printf

         Ответить

     • Necromant 26.11.2013 09:19 # −1

       Дрова для bcm4329 из дерева рокчиповых сырцов.

       Ответить

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

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

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

   Помни, guest, за тобой могут следить!

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

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

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