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

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

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

1. PHP (5700)
2. C++ (2734)
3. Куча (2331)
4. JavaScript (2028)
5. C# (1926)
6. Java (1501)
7. Си (1097)
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# / Говнокод #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] (подробнее)

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