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

• Все
• Лучший
• Сток
• Глупый поиск
• Наговнокодить!
• 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# / Говнокод #1598

   +139

   1. 01
   2. 02
   3. 03
   4. 04
   5. 05
   6. 06
   7. 07
   8. 08
   9. 09
   10. 10

   public bool IsEmpty()
           {
               int x = 0;
               x += EntityType.Length == 0 ? 1 : 0;
               x += EntityTypeName.Length == 0 ? 1 : 0;
               x += EntityDisplayName.Length == 0 ? 1 : 0;
               x += EntityId == Guid.Empty ? 1 : 0;
   
               return x == 0 ? false : true;
           }

   Аритметичный if :)

   Запостил: bugotrep, 16 Августа 2009

   Tweet

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

   • guest 16.08.2009 15:31 # 0

     и ?

     Ответить

   • ikovalyov 16.08.2009 15:52 # 0

     return (EntityType.Length == 0) && (EntityTypeName.Length == 0) && (EntityDisplayName.Length == 0) && (EntityId == Guid.Empty);

     Ответить

     • Dimarius 16.08.2009 17:14 # 0

       Неа. Метод возвращает true, когда хотя бы один пустой. Написано так, что нужно долго въезжать.
       Поэтому переписать можно так:
       return (EntityType.Length == 0) || (EntityTypeName.Length == 0) || (EntityDisplayName.Length == 0) || (EntityId == Guid.Empty);
       Или, полагая, что Length >= 0
       return (EntityType.Length * EntityTypeName.Length * EntityDisplayName.Length == 0) || (EntityId == Guid.Empty);

       Ответить

       • ikovalyov 16.08.2009 17:24 # 0

         я лох

         Ответить

       • пъянь 16.08.2009 18:07 # 0

         а почему не делать так:
         bool result = true;
         result = result && (EntityType.Length == 0);
         result = result && (EntityTypeName.Length == 0);
         ....
         return result;

         Ответить

         • пъянь 16.08.2009 18:29 # 0

           в сравнении с этим
           return (EntityType.Length == 0) && (EntityTypeName.Length == 0) && (EntityDisplayName.Length == 0) && (EntityId == Guid.Empty);

           Ответить

           • guest 18.08.2009 03:03 # 0

             В общем случае Ваша конструкция работает медленнее - почитайте про short-circuit evaluation

             Ответить

             • пъянь 18.08.2009 13:21 # 0

               тут вопрос не в оптимизации а в читабильности кода
               это в 100 раз важнее :)

               Ответить

               • guest 18.08.2009 16:05 # +2

                 Правильный код из ответа Dimarius'а можно записать и так:
                 return
                 (EntityType.Length == 0) ||
                 (EntityTypeName.Length == 0) ||
                 (EntityDisplayName.Length == 0) ||
                 (EntityId == Guid.Empty);
                 
                 Мне будет больно слышать, если это нечитаемо :)

                 Ответить

   • guest 10.09.2009 00:01 # 0

     как будто ассемблерщик писал, у них вечно вот такие вот костыли

     Ответить

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

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

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

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

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

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

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