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

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

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

1. PHP (5700)
2. C++ (2734)
3. Куча (2331)
4. JavaScript (2028)
5. C# (1926)
6. Java (1501)
7. Си (1098)
8. Pascal (647)
9. Python (582)
10. 1C (533)
11. SQL (432)
12. ActionScript (292)
13. Objective C (202)
14. bash (198)
15. Perl (194)
16. Assembler (146)
17. Ruby (142)
18. VisualBasic (138)
19. Lua (49)
20. Go (29)
21. Swift (27)
22. Haskell (26)
23. Kotlin (13)

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

   +133

   1. 1

   Удалено по просьбе автора

   Запостил: alt0, 08 Августа 2014

   Tweet

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

   • TauSigma 08.08.2014 17:29 # 0

     Чем чаще в коде встречаются List<T>, тем больше в нём говна...

     Ответить

   • kegdan 08.08.2014 18:06 # +1

     return GetData(w => w.UserName == CurrentUserLogin).FirstOrDefault();

     
     
     же

     Ответить

     • TauSigma 08.08.2014 18:18 # 0

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

       String userLogin=(CurrentUserLogin??String.Empty).Trim().RemoveInvalidChars();
       return GetData(w => String.Equals(w.UserName, userLogin, StringComparsion.OrdinalIgnoreCase).FirstOrDefault();

       
       
       Ну а если по существу, такой код получения юзера по полю, лучше вызвать с SingleOrDefault(), дабы, не дай боже, не получить дубль...

       Ответить

   • wvxvw 08.08.2014 18:52 # −1

     А что какой-нибудь find нету? Зачем пролистывать весь список?
     Ну и тип у метода охуенный, только ObjectDataControler забыли.
     
     И, почти наверняка, w - инстанс какого-нибудь remote object, привязаного к табличке со 100500 связей к другим табличкам, так, что этот невинный код, перебирающий пользователей по одному, почти наверняка нагенерит немало запросов. Ну, или еще хуже, пользователи хранятся в памяти приложения, вместо базы.

     Ответить

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

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

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

   Я, guest, находясь в здравом уме и твердой памяти, торжественно заявляю:

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

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

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