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

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

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

1. PHP (5698)
2. C++ (2733)
3. Куча (2324)
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. C# / Говнокод #310

   +849.2

   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

   switch (driver.Status)
         {
           case ClientStatus.Unknown:
             return m_driverStatusNames[ClientStatus.Unknown];
           case ClientStatus.Free:
             return m_driverStatusNames[ClientStatus.Free];
           case ClientStatus.Busy:
             return m_driverStatusNames[ClientStatus.Busy];
           case ClientStatus.InWay:
             return m_driverStatusNames[ClientStatus.InWay];
           case ClientStatus.Work:
             return m_driverStatusNames[ClientStatus.Work];
           case ClientStatus.Break:
             return m_driverStatusNames[ClientStatus.Break];
           case ClientStatus.Alarm:
             return m_driverStatusNames[ClientStatus.Alarm];
         }

   :)

   Запостил: guest, 30 Декабря 2008

   Tweet

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

   • guest 30.12.2008 21:52 # 0

     А в С++ имело бы право на жизнь - если автор боится driver.Status, выходящего за границы массива :)

     Ответить

     • guest8 16.09.2019 21:38 # −999

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

       Ответить

       • guest8 21.09.2019 15:06 # −999

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

         Ответить

         • guest8 21.09.2019 15:11 # −999

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

           Ответить

           • guest8 24.09.2019 14:24 # −999

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

             Ответить

             • guest8 25.09.2019 16:13 # −999

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

               Ответить

               • guest8 25.09.2019 22:41 # −999

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

                 Ответить

                 • guest8 26.09.2019 16:53 # −999

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

                   Ответить

                   • guest8 26.09.2019 20:28 # −999

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

                     Ответить

   • guest 31.12.2008 08:21 # 0

     Член:
     Жоско!

     Ответить

     • guest8 16.09.2019 21:38 # −999

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

       Ответить

   • guest 10.01.2009 22:17 # −1

     BlackHeretic:
     А помоему нормально, если в энуме есть еще элементы.

     Ответить

     • guest8 16.09.2019 21:37 # −999

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

       Ответить

   • guest 13.01.2009 12:33 # 0

     Crazy_W:
     По моему проще было бы проверить на отсутствующие элементы если их больше или присутствующие, если больше их. Но зачем на каждую писать то?

     Ответить

     • guest8 16.09.2019 21:37 # −999

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

       Ответить

   • guest 18.01.2009 14:29 # +1

     J:
     

     switch (driver.Status)
           {
             case ClientStatus.Unknown:
             case ClientStatus.Free:
             case ClientStatus.Busy:
             case ClientStatus.InWay:
             case ClientStatus.Work:
             case ClientStatus.Break:
             case ClientStatus.Alarm:
                 return m_driverStatusNames[driver.Status];
           }

     
     если многие элементы энума отсутствуют

     Ответить

     • guest8 16.09.2019 21:36 # −999

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

       Ответить

   • guest 06.03.2009 15:27 # 0

     ahmed:
     

     List<En> exc = new List<En> {ClientStatus.Unknown, ClientStatus.Free ...};
     if (exc.Contains(driver.Status)) throw new Exception();
     return m_driverStatusNames[driver.Status];

     Ответить

     • guest8 16.09.2019 21:36 # −999

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

       Ответить

   • guest 01.04.2009 23:48 # 0

     !00b:
     Любителям велосипедов советую ознакомиться с Enum.IsDefined и вообще возможностями System.Enum здесь http://msdn.microsoft.com/en-us/library/system.enum.aspx

     Ответить

     • guest8 16.09.2019 21:36 # −999

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

       Ответить

   • guest 10.08.2009 17:15 # −1

     А если это строковое представление enum? Всегда приходится чтото колхозить в таком случае.

     Ответить

     • guest8 16.09.2019 21:36 # −999

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

       Ответить

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

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

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

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

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

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

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