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

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

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

1. PHP (5703)
2. C++ (2737)
3. Куча (2343)
4. JavaScript (2028)
5. C# (1928)
6. Java (1502)
7. Си (1108)
8. Pascal (647)
9. Python (583)
10. 1C (535)
11. SQL (432)
12. ActionScript (292)
13. Objective C (202)
14. bash (198)
15. Perl (194)
16. Assembler (146)
17. Ruby (145)
18. VisualBasic (138)
19. Lua (49)
20. Go (30)
21. Swift (27)
22. Haskell (26)
23. Kotlin (14)

1. PHP / Говнокод #16479

   +160

   1. 1

   ($isLead) ? $customerBalance = 0 : $customerBalance = $customerMapper->calculateBalance($customer);

   Код от мастера Йода

   Запостил: andr435, 05 Августа 2014

   Tweet

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

   • Elvenfighter 05.08.2014 15:53 # 0

     А вот Perl бы проучил засранца: http://ideone.com/GJj8h3

     Ответить

     • 1024-- 05.08.2014 17:47 # −1

       А что случилось? Perl не знаю, но выглядит логично.
       (Или PHP недостаточно знаю, может в PHP этот код творит нелогичные непотребства.)

       Ответить

       • kegdan 05.08.2014 18:03 # −1

         0 в перле - тру
         Как и в руби впрочем

         Ответить

         • 1024-- 05.08.2014 18:08 # −1

           А, 0 - false. Понял.
           Elvenfighter выбрал неудачное число для примера (или удачное для выявления перлонубов).

           Ответить

           • kegdan 05.08.2014 18:13 # −1

             даже я запутался.
             В перле и правда 0 - false. а я думал как в руби.

             Ответить

             • Elvenfighter 06.08.2014 00:26 # 0

               0 -- это false в перле. И теперь лёгким движением руки:
               

               - my $isLead = 0;
               + my $isLead = 1;

               
               этот кусок хм... кода превращается... превращается в http://ideone.com/N8ox5H WAIT, OH SHI~

               Ответить

               • kegdan 06.08.2014 00:33 # −1

                 WTF?

                 Ответить

             • Elvenfighter 06.08.2014 00:41 # −1

               0, undef, пустая строка -- false
               пусой массив -- тоже false, так как в скалярном контексте возвращает свою длинну (0)
               пустой хеш -- тоже false, так как в скалярном контексте тоже возвращает 0.
               
               Всё остальное -- true. Т.е. любой другой скаляр (кроме специально запрограммированного) и непустой массив/хеш -- это true.

               Ответить

               • kegdan 06.08.2014 00:51 # −1

                 так почему $isLead = 1; он все равно идет по ложной ветке?

                 Ответить

                 • inkanus-gray 06.08.2014 00:53 # −1

                   Идёт как раз по нужной ветке. Введём две переменные и убедимся:

                   ($isLead) ? $customerBalance1 = 0 : $customerBalance2 = $customerMapper->calculateBalance($customer);

                   Ответить

                   • kegdan 06.08.2014 00:55 # 0

                     тогда у меня один вопрос - WHY?!!!!

                     Ответить

                     • inkanus-gray 06.08.2014 01:02 # −1

                       Подсказка: ветки имеют совсем не ту длину, которую ожидает сишник.

                       Ответить

                       • kegdan 06.08.2014 01:05 # −1

                         (($isLead) ? $customerBalance = 0 : $customerBalance) = $customerMapper->calculateBalance($customer);
                         
                         Так что ле?)

                         Ответить

                         • inkanus-gray 06.08.2014 01:11 # −1

                           Именно так.

                           Ответить

                           • kegdan 06.08.2014 01:14 # −1

                             Да, Ларри умеет шокировать.
                             
                             Например шестым перлом

                             Ответить

                         • 1024-- 06.08.2014 10:45 # 0

                           Блджад... Какой приоритет )))

                           Ответить

               • inkanus-gray 06.08.2014 01:05 # −1

                 А как же строка, состоящая из одного нолика?

                 Ответить

                 • kegdan 06.08.2014 01:12 # −1

                   то ли дело в руби - false да nil - остальное труъ

                   Ответить

                   • 1024-- 06.08.2014 10:49 # −1

                     Да ничего хорошего. Приходится из-за такого строгого разделения использовать дополнительные операторы, универсальность теряется.
                     Где можно было написать if(str), приходится писать if(!isNullOrEmpty(str)).
                     Где можно было написать if(x), приходится if(x == 0).
                     Становится длинно, надо много писать и читать.

                     Ответить

                     • kegdan 06.08.2014 11:06 # −1

                       Зато логика железобетонная. и если должен быть уверен, что обьект не null тебе придется проверять на что именно false - на null или на 0 или на "0". в руби сомнений нет

                       Ответить

                       • 1024-- 06.08.2014 11:17 # −1

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

                         if(!x) return; // x не установлен!
                         if(isString(x)) return 'hello ' + x;
                         return 'Plohoe value!!!111';

                         
                         Если же тип известен, нет нужды явно прописывать.
                         

                         if(evilListLength) // evilListLength, очевидно, целое число

                         Ответить

                         • kegdan 06.08.2014 12:23 # −1

                           !x выдаст тру на х = "0"

                           Ответить

                           • 1024-- 06.08.2014 12:56 # −1

                             В PHP выдаст, но у нас же не только один идеальный PHP, вокруг которого крутятся все планеты!
                             В C/C++ это будет аналог isNull, в JS - isNullOrEmpty. А !"0" будет false, никаких беспорядков.

                             Ответить

                             • roman-kashitsyn 06.08.2014 13:11 # 0

                               import Data.Maybe (isJust)
                                                 
                               class Boolable a where
                                   true :: a -> Bool
                               
                               instance Boolable Bool where
                                   true = id
                               
                               instance Boolable Int where
                                   true = (/=) 0
                               
                               instance Boolable Integer where
                                   true = (/=) 0
                                  
                               instance Boolable [a] where
                                   true = not . null
                                         
                               instance Boolable (Maybe a) where
                                   true = isJust
                                         
                               main :: IO ()
                               main = print
                                 [ true True
                                 , true False
                                 , true (1 :: Integer)
                                 , true (0 :: Int)
                                 , true "lol"
                                 , true ""
                                 , true $ Just 5
                                 , true Nothing
                                 ]

                               
                               
                               > C++
                               operator bool() const

                               Ответить

                               • kegdan 06.08.2014 13:20 # −1

                                 и причем тут хаски?)

                                 Ответить

                                 • roman-kashitsyn 06.08.2014 13:22 # −1

                                   забавные у вас проблемки в забавных язычках

                                   Ответить

                                   • kegdan 06.08.2014 13:31 # −1

                                     Какие проблемы, нет проблем.
                                     
                                     Имя функции некошерное. toBool было бы лучше. Имхо.

                                     Ответить

                                     • 1024-- 06.08.2014 13:34 # −1

                                       А если бы назвали "bool", можно было бы писать "(bool) 3".

                                       Ответить

                                       • kegdan 06.08.2014 13:35 # −1

                                         зачем скобки? Типа сишкапародия?

                                         Ответить

                                         • 1024-- 06.08.2014 13:36 # −1

                                           Конечно. Сишка - сила.

                                           Ответить

                                           • kegdan 06.08.2014 13:37 # −1

                                             Тебя царь покусал?

                                             Ответить

                                             • 1024-- 06.08.2014 13:57 # −1

                                               Нет, просто уж очень красиво сделано. Скобочки, значочки, ...

                                               Ответить

                                               • roman-kashitsyn 06.08.2014 14:15 # 0

                                                 В чём красота? Чтобы вопсользоваться результатом каста в общем случае нужны ещё одни скобки:
                                                 ((T) x)->callMe();
                                                 неудобно же

                                                 Ответить

                                                 • 1024-- 06.08.2014 14:25 # 0

                                                   Красота общая :) Если не всматриваться в подобные досадные детали (да, их может быть бесконечно много, но всё проходит мимо влюблённых глаз поклонника сишки).
                                                   Ну, у (type) достаточно высокий приоритет, чтобы не всегда были лишних скобки. А если учесть, что в других языках могут потребовать писать if(toBool(x)) вместо if(x), то не так обидно.

                                                   Ответить

                                                   • kegdan 06.08.2014 14:28 # −1

                                                     мне кажется нет ничего красивее функции. разве что функция высшего порядка

                                                     Ответить

                                                     • 1024-- 06.08.2014 14:36 # −1

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

                                                       Ответить

                             • kegdan 06.08.2014 13:20 # −1

                               я конкретно про перл говорил

                               Ответить

                               • 1024-- 06.08.2014 13:23 # −1

                                 А я про !x в целом и возможность выбрать годные умолчания.
                                 А с вышеупомянутым operator bool из C++ и valueOf из JS можно пилить свои !x со своими удобными правилами.
                                 P.S. Хотя, нет. В JS придётся !+x, чтобы valueOf сработал.

                                 Ответить

                                 • kegdan 06.08.2014 13:34 # −1

                                   Да так везде можно. Ввел свою функцию или класс с функцией как машинариум и радуйся. Я о том. что мне больше нравиться дефолт в руби .

                                   Ответить

                 • Elvenfighter 03.09.2014 00:19 # 0

                   Да в перле "все строка" по этому не стал это упоминать. Ведь тогда придётся объяснять "0E0" и "0 but true".

                   Ответить

                   • bormand 03.09.2014 05:59 # 0

                     В пёрле все-таки касты повменяемей, чем в пыхе. Там же == и eq - разные операторы, поэтому всё предсказуемо.

                     Ответить

   • andr435 14.08.2014 11:19 # +1

     Народ что с вами?
     Код написн правильно, просто по человечески было бы написать:
     
     $customerBalance = ($isLead) ? 0 : $customerMapper->calculateBalance($customer);

     Ответить

     • 1024-- 14.08.2014 11:39 # −1

       > Народ что с вами?
       Тут всегда так.

       Ответить

     • kegdan 14.08.2014 12:56 # −1

       А что ты хотел от сайта, где маскот - каког багог?

       Ответить

       • andr435 14.08.2014 13:48 # −1

         А, точно, извините забылся

         Ответить

         • kegdan 14.08.2014 14:12 # −1

           Будь внимателен!

           Ответить

           • anonimb84a2f6fd141 14.08.2014 16:27 # −3

             ага, будь внимателен, чтобы ни капли мимо не пролетело. Все в рот, в рот!..
             p.s.
             Въебал минус за быдлоганство.

             Ответить

   • anonimb84a2f6fd141 14.08.2014 16:30 # −4

     Красным минусы горят - это Путин виноват
     Въебал всем по минусу.

     Ответить

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

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

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

   Где здесь C++, guest?!

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

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

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