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

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

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

1. PHP (5703)
2. C++ (2740)
3. Куча (2339)
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. 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] (подробнее)

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