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

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

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

1. PHP (5703)
2. C++ (2740)
3. Куча (2341)
4. JavaScript (2030)
5. C# (1930)
6. Java (1503)
7. Си (1103)
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 (32)
21. Swift (29)
22. Haskell (28)
23. Kotlin (15)

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

   +159

   1. 1

   $this->last_error = $success;

   кстати о названиях переменных :))

   Запостил: youROCK, 22 Февраля 2011

   Tweet

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

   • Sulik78 22.02.2011 18:47 # −1

     хде ГК???

     Ответить

   • Kirinyale 22.02.2011 18:54 # +4

     В виндовом апи тоже есть константа ERROR_SUCCESS. И чё?

     Ответить

     • youROCK 22.02.2011 19:07 # 0

       И что эта константа означает?

       Ответить

       • Kirinyale 22.02.2011 19:08 # 0

         "операция завершена успешно"

         Ответить

         • youROCK 22.02.2011 19:23 # −2

           Ну как бы вот и ответ на ваш вопрос :). Это не ошибка, какого хрена называть переменную, в которую записана ошибка, $success :)?

           Ответить

           • Kirinyale 22.02.2011 19:35 # 0

             Без полного контекста вполне можно предположить, что в переменной $success лежит код ошибки, означающий отсутствие ошибки. Правда, в этом случае логичнее было бы сделать её константой.

             Ответить

           • Sulik78 22.02.2011 19:38 # 0

             хахаха
             ну вы и шутник... оО
             наверное потому что success содержит либо ОК либо ERROR и пишет это в last_error?

             Ответить

             • youROCK 22.02.2011 19:40 # 0

               На самом деле, туда записывается либо 1 либо строка с ошибкой :)

               Ответить

               • tr00_gr1m_doomster 22.02.2011 20:09 # 0

                 где используется это поле?

                 Ответить

                 • youROCK 22.02.2011 20:14 # 0

                   игнорируется :). Ну это другая история

                   Ответить

               • Sulik78 22.02.2011 20:53 # 0

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

                 Ответить

           • Sulik78 23.02.2011 06:16 # +2

             > Это не ошибка, какого хрена называть переменную, в которую записана ошибка, $success
             а как надо то ?
             $this->last_error = $error;
             $this->last_success = $success;
             так ?

             Ответить

   • Lure Of Chaos 22.02.2011 21:44 # +1

     сам подход - говно

     Ответить

     • bober_maniac 23.02.2011 10:57 # 0

       Отличный подход для конца 80-х.

       Ответить

   • sectus 23.02.2011 05:24 # 0

     Может быть success заменить на result и не будет противоречия?

     Ответить

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

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

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

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

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

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

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