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

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

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

1. PHP (5703)
2. C++ (2740)
3. Куча (2337)
4. JavaScript (2030)
5. C# (1929)
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# / Говнокод #11556

   +131

   1. 1
   2. 2
   3. 3
   4. 4
   5. 5
   6. 6

   if (Cache[articlesRssData] != null)
   {
        return Cache[articlesRssData] as ArticleRssData?;
   }
   
   return null;

   Запостил: uusb, 08 Августа 2012

   Tweet

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

   • uusb 08.08.2012 17:40 # 0

     забыл сказать что это все находится в отдельной функции, послевызова которой проверяем на null

     Ответить

     • Sulik78 08.08.2012 18:34 # +3

       >> return null;
       мы поняли. КЭП.

       Ответить

   • Elvenfighter 08.08.2012 20:06 # 0

     Что значит знак вопроса тут: as ArticleRssData?;

     Ответить

     • bormand 08.08.2012 20:26 # +4

       The syntax T? is shorthand for Nullable<T>, where T is a value type. The two forms are interchangeable.
       
       http://msdn.microsoft.com/en-us/library/1t3y8s4s.aspx
       
       P.S. Знатоки c#, поясните, пожалуйста, зачем это здесь - ведь ArticleRssData не слишком похож на value-type.
       P.P.S. Кажется понял - он описан как struct.

       Ответить

       • absolut 08.08.2012 20:30 # +3

         А как на хинди вопрос будет, так и не узнали

         Ответить

   • phoenixx 09.08.2012 02:07 # +2

     Ну говнокод здесь в лишних проверках. В простом случае хватило бы
     

     return Cache[articlesRssData] as ArticleRssData?;

     
     Если там будет null, то он его и вернет.
     И в оригинальном случае будет еще два поиска по кешу.

     Ответить

     • bormand 09.08.2012 07:03 # +2

       s/еще два поиска/еще один поиск/

       Ответить

       • absolut 09.08.2012 09:23 # +2

         вероятно имелось в виду: ещё два поиска помимо проверки, а не ещё один поиск помимо уже одного поиска :)

         Ответить

       • phoenixx 09.08.2012 12:49 # +1

         спасибо, за фикс :)
         либо "два поиска вместо одного", либо "еще один поиск" :)

         Ответить

     • abatishchev 10.08.2012 15:35 # 0

       Cache - скорее всего словарь, поэтому запрос будет О(1)

       Ответить

       • phoenixx 10.08.2012 15:37 # +1

         Скорее всего еще не значит что так оно и есть. Это ж говнокод, вдруг это обвертка над List<>-ом на тысячи элементов.

         Ответить

   • vistefan 09.08.2012 15:21 # +1

     Типичный Нульшит, не?

     Ответить

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

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

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

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

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

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

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