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

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

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

1. PHP (5703)
2. C++ (2739)
3. Куча (2337)
4. JavaScript (2030)
5. C# (1929)
6. Java (1503)
7. Си (1102)
8. Pascal (649)
9. Python (585)
10. 1C (536)
11. SQL (434)
12. ActionScript (294)
13. Objective C (204)
14. bash (200)
15. Perl (196)
16. Assembler (148)
17. Ruby (144)
18. VisualBasic (140)
19. Lua (51)
20. Go (31)
21. Swift (29)
22. Haskell (28)
23. Kotlin (15)

1. C# / Говнокод #4810

   +118

   1. 1

   if (string.IsNullOrEmpty(email) != false)

   Запостил: zheka, 02 Декабря 2010

   Tweet

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

   • gegMOPO4 02.12.2010 19:04 # −4

     За != false и == true хочется душить. Только ведь рук не хватит.

     Ответить

     • KirAmp 02.12.2010 19:18 # −12

       показать все, что скрытоНу где-то лучше написать true == длиннаяСтрочка для понимания реальности происходящего

       Ответить

       • gegMOPO4 02.12.2010 19:34 # +6

         Переходите на синие таблетки.

         Ответить

       • Анонимус 02.12.2010 20:03 # −8

         показать все, что скрытострочка это объект. ее нельзя сравнивать с булевым типом true в языках со статической типизацией

         Ответить

         • KirAmp 02.12.2010 20:20 # −12

           показать все, что скрытопод длиннойСтрочкой я подразумевал длинную ссылку на некий бул
           
           можно также в AS3 написать Boolean(длиннаяСсылка), так же помогает осознать смысл условия в говно-структуре

           Ответить

           • Анонимус 02.12.2010 20:24 # −8

             показать все, что скрытоа такую нужно в переменную выносить
             см рефакторинг extract variable

             Ответить

             • KirAmp 02.12.2010 20:30 # −12

               показать все, что скрытоНу... это не всегда оправданно =) Бывает надо проверить 1 раз переменную в далеком-придалеком объекте, который больше не упоминается в этом классе

               Ответить

               • 3.14159265 06.12.2010 14:45 # −2

                 тред зла на говнокоде? любим. ценим.
                 
                 и миня, миня заминусуйте.
                 
                 PS. какой же я слоупок

                 Ответить

         • absolut 02.12.2010 22:32 # −12

           показать все, что скрытоЕсли есть преобразование из длиннаяСтрочка в bool, то будет работать и при статической типизации. В С++ точно.

           Ответить

           • absolut 03.12.2010 09:05 # −9

             показать все, что скрытоДля тех кто не понял: operator bool() в классе.

             Ответить

       • SemaReal 27.10.2017 00:38 # −1

         думаете?

         Ответить

         • AnalPerOral 27.10.2017 04:39 # 0

           Маму твою ебём.

           Ответить

     • eth0 03.12.2010 21:02 # −8

       показать все, что скрытоА за != true и == false?

       Ответить

       • gegMOPO4 03.12.2010 21:10 # −4

         Просто добродушно отпинать ногами.

         Ответить

       • brainstorm 06.12.2010 20:50 # −7

         показать все, что скрытону он же хотел подчеркнуть что не Ъ

         Ответить

     • banderror 04.12.2010 11:41 # −6

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

       Ответить

   • Govnocoder#0xFF 02.12.2010 20:10 # −10

     показать все, что скрытоНу написал человек
     

     If bool = true then
     {
      }
     Else {}

     
     
     Что с того? Может это такой стиль программирования?

     Ответить

     • KirAmp 02.12.2010 20:44 # −7

       показать все, что скрытов этом примере все куда запущенней, тут: bool != false

       Ответить

     • Анонимус 02.12.2010 20:55 # −9

       показать все, что скрыто>>} Else
       это что я за язык такой?)))

       Ответить

       • Govnocoder#0xFF 02.12.2010 21:04 # −8

         показать все, что скрытоДа так, на псевдокоде набросал. Никакой в общем.

         Ответить

         • KirAmp 02.12.2010 22:36 # −9

           показать все, что скрытоthen
           {
           }
           решает =)

           Ответить

       • Pauchok-Anaynckiy 03.12.2010 00:42 # −8

         показать все, что скрытоКомбинированный язык VB#

         Ответить

     • SemaReal 28.10.2017 02:12 # 0

       дурацкий стиль довольно

       Ответить

   • GLeBaTi 03.12.2010 00:25 # −16

     показать все, что скрытоТуплю наверное, но в чем "сок"?
     В том, что != false можно заменить на == true?)

     Ответить

     • KirAmp 03.12.2010 12:24 # −8

       показать все, что скрытоВерное условие: if (string.IsNullOrEmpty(email))

       Ответить

       • GLeBaTi 03.12.2010 13:53 # −13

         показать все, что скрытоА если использовать if (string.IsNullOrEmpty(email) == true)?
         Может для кого-то это более наглядно? Тем более ничего не проигрываем мы с этого...

         Ответить

         • TarasB 03.12.2010 14:17 # −5

           показать все, что скрытоНет, нагляднее всего, конечно же
           
           if !(false == string.IsNullOrEmpty(email))

           Ответить

           • Анонимус 03.12.2010 15:33 # −8

             показать все, что скрытоif ((false == string.IsNullOrEmpty(email)) == false)

             Ответить

         • andrewpey 14.12.2010 01:04 # −3

           Это усложняет логику чтения кода. Если в первом случае мы читаем как
           "Если строка пуста, тогда...",
           то в вашем получается
           "Если условие "строка пуста" равно истине, тогда..."

           Ответить

           • bugmenot 14.12.2010 02:13 # −2

             добро пожаловать в тред зла!

             Ответить

             • andrewpey 14.12.2010 11:02 # −2

               Спасибо )

               Ответить

   • Pauchok-Anaynckiy 03.12.2010 00:48 # −6

     показать все, что скрытоИз разряда:
     if (result == true)
     
     Надо бы уже ГК по категориям распределять, таких примеров тут уже порядочно набралось.
     
     Но из-за !=false немного прикольнее, чем просто == true )))). Хотя можа быть эта строчка просто случайно выжила после ряда преобразований.
     
     Пока воздерживаюсь.

     Ответить

     • brainstorm 06.12.2010 20:53 # −4

       ну.... у нас в PHP бывают функции которые могут и false вернуть а могут целочисленное :)
       http://ru.php.net/manual/en/function.strpos.php
       гы. там даже цветное предупреждение сделали :)

       Ответить

       • MAK 08.12.2010 10:08 # −4

         К языкам без строгой типизации вопросов нет, разное бывает. В С# это однозначный говнокод.

         Ответить

   • DropWorld 03.12.2010 22:06 # −8

     показать все, что скрытоТопики зла - теперь и на Govnokod.ru !

     Ответить

     • eth0 04.12.2010 12:29 # −10

       показать все, что скрытоПоддержим тред ненависти!

       Ответить

       • Govnoeb 06.12.2010 13:02 # −7

         показать все, что скрытоуже

         Ответить

         • inkanus-gray 06.12.2010 19:45 # −7

           показать все, что скрытоЭто не тред ненависти. Это секретный тред, а минусики — штатное средство убрать текст «под кат».

           Ответить

           • bugmenot 06.12.2010 19:52 # −4

             Это - говножаргон (из жежешечки?), на самом деле "под катом" означает умеренную эйфорию и возбуждение. Педивикия таки:

             Под его влиянием люди становятся очень разговорчивыми и кажутся неадекватными и эмоционально неустойчивыми. 
             Кат может провоцировать неадекватное поведение и гиперактивность.
             Кат сильно подавляет аппетит и его использование также может привести к запору.

             Ответить

   • inkanus-gray 06.12.2010 18:20 # −7

     показать все, что скрытоЯ что-то пропустил? Меня ещё не минусовали!!!!!

     Ответить

   • bugmenot 06.12.2010 19:07 # −5

     показать все, что скрытореквестую яростный клик на красненьком крестике!

     Ответить

     • inkanus-gray 06.12.2010 19:42 # −7

       показать все, что скрытоОцените мою аву, а я вам стикеры на фото и подарки в гостевую.
       Есть!

       Ответить

       • bugmenot 06.12.2010 19:54 # −5

         показать все, что скрытосм. также: букмарклет зла

         javascript:var nodes=document.getElementsByClassName('comment-vote-against');for(var i=0;i<nodes.length;i++)nodes[i].click()

         Ответить

         • Анонимус 06.12.2010 20:17 # −4

           >>getElementsByClassName
           чо?
           оно же в осле не пашет
           
           жуквери нуно

           Ответить

           • bugmenot 06.12.2010 22:28 # −5

             показать все, что скрыто> жуквери нуно
             фу, прекратите обезьяничать
             
             давайте, вместе, по слогам
             HTMLElement
             prototype
             getElementsByClassName
             function
             
             следуя спецификации http://www.whatwg.org/specs/web-apps/current-work/multipage/dom.html#dom-getelementsbyclassname :р

             Ответить

             • Анонимус 07.12.2010 15:21 # −6

               показать все, что скрытозачем лечить проблемы осла, когда есть jquery?
               хотя прототипировать конечно кошернее

               Ответить

               • bugmenot 07.12.2010 16:52 # −6

                 показать все, что скрытопотому что jquery говно, не?
                 тогда уж document.querySelector('a.comment-vote-against') и лечить его с помощью резиговских костылей

                 Ответить

                 • Анонимус 07.12.2010 16:56 # −6

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

                   Ответить

                   • bugmenot 07.12.2010 18:13 # −6

                     показать все, что скрытолюди? когда резиг перестал единолично тужиться, в его мешанине уже никто не мог разобраться =)
                     а потом придумали JIT компиляторы и все успокоились

                     Ответить

                     • Анонимус 07.12.2010 18:15 # −6

                       показать все, что скрытоя же в сырцы не заглядываю)
                       мне и представить страшно, какой jquery внутри говнокод.
                       но может быть Вы знаете хотя бы один неговнокодерский проект на JS?

                       Ответить

                       • bugmenot 07.12.2010 18:37 # −5

                         показать все, что скрытоя не настолько в сортах говна разбираюсь :)
                         meanwhile in japan:

                         javascript:$('a.comment-vote-against').each(function(){comments.vote(this)})

                         дальше ниасилил

                         Ответить

                         • Анонимус 07.12.2010 18:38 # −6

                           показать все, что скрытоhttp://api.jquery.com/jQuery.each/
                           )
                           
                           мне вообще JavaScript не нравится как язык. Я считаю мудизмом писать
                           

                           function MyClass {
                            var m_privateMember;
                            this.publicMember = ...
                           }

                           
                           
                           и jquery позволяет мне меньше писать на этом чудесном языке.
                           
                           вообще иногда правда проще на GWT написать, что бы избавица от javascriptерства)

                           Ответить

                           • bugmenot 07.12.2010 19:32 # −6

                             показать все, что скрытоя всё равно читать эту хрень не буду, хотя бы потому что они не знают где употреблять красивое слово API, а где воздержаться во имя здравого смысла
                             
                             таки JavaScript - не маленькая Java
                             
                             GWT это вообще AJAX

                             Ответить

                             • Анонимус 07.12.2010 19:34 # −5

                               показать все, что скрыто>>GWT это вообще AJAX
                               не обязательно совершенно
                               этоможет быть чисто клиентский код без запросов на сервер
                               просто на жабе
                               
                               потом гугол сделает по его мотивам жабаскрипт

                               Ответить

                               • bugmenot 07.12.2010 19:47 # −6

                                 показать все, что скрытомм, ну если гоогле вынесли говнотермин в первую строку описание, то это символизирует их целевую аудиторию, разве нет?

                                 Ответить

                                 • Анонимус 07.12.2010 19:49 # −5

                                   показать все, что скрытопросто это модное слово
                                   как web20))
                                   
                                   если мне (не дай Бог!!!) прийдется строить сложную бизнес-логику на клиенте -- я возьму GWT, что бы писать не на JS, а на Java, тестировать ее JUnitом итд...
                                   Даже если там не будет AJAX совсем)

                                   Ответить

                                   • bugmenot 08.12.2010 02:05 # −4

                                     а почему не подобное от Yahoo? оно вроде как поматурнее будет и дядьки там посерьезнее сидят

                                     Ответить

                                     • Анонимус 08.12.2010 13:23 # −4

                                       1) у яхи есть способ преобразовывать жабу в JS?
                                       2) яха по серьезнее гугла?) яха, где чуть ли не 9 лет работал клоун Лердорф?)

                                       Ответить

                                       • bugmenot 08.12.2010 14:14 # −4

                                         думаете колесо изобрели тоже в гоогле?
                                         а вот поименно павианов которые там еще работают не знаю

                                         Ответить

                                         • Анонимус 08.12.2010 14:22 # −4

                                           ну, во всяком случае говна про гугл я могу сказать меньше, чем про яху)

                                           Ответить

                         • inkanus-gray 07.12.2010 18:41 # −6

                           показать все, что скрытониработает

                           Ответить

                           • bugmenot 07.12.2010 19:25 # −6

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

                             javascript:void($('a.comment-vote-against').each(function(){comments.vote($(this))}))

                             
                             за необходимость void резига следует бросить в сортир

                             Ответить

                             • inkanus-gray 07.12.2010 19:52 # −4

                               Ура! Заработало!
                               Теперь все версии букмарклета зла нужно оформить как отдельный говнокод.

                               Ответить

                               • bugmenot 07.12.2010 20:11 # −2

                                 еще надо сделать букмарклет хаотической нейтральности =)

                                 Ответить

                             • 3.14159265 08.12.2010 13:18 # −3

                               не к добру подобные скрипты.
                               
                               хотя там бы еще проверку допилить c регэксом на contains("C++")...

                               Ответить

         • inkanus-gray 06.12.2010 20:18 # −7

           показать все, что скрытоclick не работает

           Ответить

           • brainstorm 06.12.2010 22:16 # −6

             показать все, что скрытоonClick() ?

             Ответить

             • bugmenot 06.12.2010 22:41 # −6

               показать все, что скрытоя честно говоря не знаю как добраться до эвентсинка куда функции прилепленные с помощью addEventListener попадают

               Ответить

           • bugmenot 06.12.2010 22:35 # −4

             и снова пользователям говнобраузеров не повезло :(
             насчет этой херни не ко мне, а к говнокодерам которые вместо починки pop-up blocker`а отключают метод с помощью которого его можно обойти
             мозилла такая секурная :р

             Ответить

         • brainstorm 06.12.2010 20:56 # −8

           показать все, что скрытоgetElementsByTagName() ?

           Ответить

           • inkanus-gray 06.12.2010 21:49 # −5

             показать все, что скрытоМожно. Тогда нужен вложенный цикл, чтобы это отфильтровать.

             Ответить

   • inkanus-gray 06.12.2010 21:54 # −5

     показать все, что скрытоПоскольку в этой теме всё равно минусуют, буду офтопить.
     Обнаружил зеркало Говнокода: http://govnokod.vfose.ru/
     Обновляется оно с задержкой, поэтому с помощью него есть шанс посмотреть удалённые говнокоды.

     Ответить

     • 3.14159265 13.12.2010 11:30 # −1

       ха! при переходе по прямой ссылке, на какой-нить рандомный говнокод
       http://govnokod.vfose.ru/3216
       
       Internal Interpreter Error
       Description: Notice: Undefined variable: body
       Filename: includes/models/mcacher.php Line: 914
       
       сыро пока что.

       Ответить

       • inkanus-gray 14.12.2010 03:41 # −2

         Эта ссылка уже работает.

         Ответить

         • 3.14159265 14.12.2010 12:19 # +1

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

           Ответить

   • bugmenot 07.12.2010 07:26 # −5

     показать все, что скрытоага, минурасты устали и отправились в люлю приблизительно 19:30 UTC
     совсем мышей не ловят, блжад

     Ответить

   • bugmenot 07.12.2010 16:54 # −7

     показать все, что скрытони одного минуса за сегодня; похоже, я сломал тред зла :-(

     Ответить

     • 3.14159265 07.12.2010 16:58 # −6

       показать все, что скрыто>ни одного минуса за сегодня
       special for you.
       evil thread continued...

       Ответить

   • eth0 12.12.2010 20:47 # −2

     -17 - официально зарегистрированный рекорд на 12 dec 2010 20:47 MSK

     Ответить

     • absolut 14.12.2010 00:46 # −1

       холодно, однако

       Ответить

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

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

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

   Семь раз отмерь — один отрежь, guest!

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

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

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