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

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

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

1. PHP (5706)
2. C++ (2739)
3. Куча (2371)
4. JavaScript (2030)
5. C# (1928)
6. Java (1502)
7. Си (1117)
8. Pascal (648)
9. Python (587)
10. 1C (539)
11. SQL (432)
12. ActionScript (292)
13. Objective C (202)
14. bash (198)
15. Perl (194)
16. Assembler (146)
17. Ruby (145)
18. VisualBasic (138)
19. Lua (49)
20. Go (31)
21. Haskell (29)
22. Swift (27)
23. Kotlin (14)

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

   −126

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

   public class MobileObjectCache<TKey, TItem> : ConcurrentDictionaryCache<TKey, TItem> where TItem : IIdentified<TKey>, IDataRecordLoadable, new()
   {
       public MobileObjectCache(IdSeries series)
           : base(series)
       {
           
       }
   
       protected override void OnUpdate(TKey key, TItem oldItem, TItem newItem)
       {
           MobileObject mo = oldItem as MobileObject;
           mo.Description = (newItem as MobileObject).Description;
       }
   }

   Главный разработчик знает как правильно расширять Generic-классы.

   Запостил: random12, 25 Марта 2017

   Tweet

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

   • random12 26.03.2017 23:05 # −7

     показать все, что скрытоСудя по минусам, прикол получился сложный. Разжую.
     
     

     public class MobileObjectCache : ConcurrentDictionaryCache<int, MobileObject>
     {
         public MobileObjectCache(IdSeries series)
             : base(series)
         {
         }
     
         protected override void OnUpdate(int key, MobileObject oldItem, MobileObject newItem)
         {
             oldItem.Description = newItem.Description;        
         }
     }

     Ответить

     • huesto 27.03.2017 00:04 # −17

       показать все, что скрытоДа ты не переживай за минусы. Тут просто шизик стретор буянит и минусует все подряд. А на говнокоды сейчас уже никто и не смотрит в этом болоте.

       Ответить

       • huesto 27.03.2017 19:09 # −21

         показать все, что скрытоВот и меня шизик заминусовал.

         Ответить

         • inhoguest 27.03.2017 19:57 # −10

           показать все, что скрытоЗа что?

           Ответить

           • guestinh0 27.03.2017 20:22 # −7

             показать все, что скрытоЗа то, что он - уебан, и ты тоже.

             Ответить

             • inhoguest 27.03.2017 20:41 # −14

               показать все, что скрытоУебан - это ты.
               
               Кстати, какой пидор сменил пароль от гуест0?

               Ответить

               • 3_14-pidar 27.03.2017 21:27 # −5

                 показать все, что скрытоЯ.

                 Ответить

                 • MilosTeodosic 28.03.2017 00:01 # −15

                   показать все, что скрытоЖалко 3_14dar забанили

                   Ответить

                   • bormand 28.03.2017 00:05 # −15

                     показать все, что скрытоПричем трижды. Ачивмент анлокед.

                     Ответить

                     • MilosTeodosic 28.03.2017 00:05 # −15

                       показать все, что скрытоЕсли у тебя есть доступ, розбань пожалуйста

                       Ответить

                       • bormand 28.03.2017 00:06 # −15

                         показать все, что скрытоЭто к страйкеру.

                         Ответить

                     • inkanus-gray 28.03.2017 12:29 # −5

                       показать все, что скрытоА можно ачивмент залочить обратно?

                       Ответить

                       • kegdan 28.03.2017 15:31 # −5

                         показать все, что скрытоАчивмент анлок - залочить ачивмент

                         Ответить

           • huesto 28.03.2017 00:45 # −15

             показать все, что скрытоНе защо, а чому. Томущо он шiзiк.

             Ответить

   • ASD_77 27.03.2017 18:52 # −8

     показать все, что скрытоа что будет если в строке 11 oldItem не будет унаследован от MobileObject или в строке 12 newItem не будет унаследован от MobileObject ?

     Ответить

   • roman-kashitsyn 27.03.2017 18:55 # −6

     показать все, что скрытоTItem может быть любым, при условии, что он MobileObject.

     Ответить

     • ASD_77 27.03.2017 19:00 # −7

       показать все, что скрытов какой строке это указанно?
       
       

       where TItem : IIdentified<TKey>, IDataRecordLoadable, new()

       
       в этой строке такого лимитирования нет. а значит на нет и суда нет

       Ответить

       • roman-kashitsyn 27.03.2017 19:07 # −8

         показать все, что скрыто> в этой строке такого лимитирования нет
         
         То, что в той строке его нет, не означает, что его вообще нет. Оно в голове того, кто написал этот бред, что проявляется в строке 11. Я тоже похожий код видел на практике:

         class Clusterizer<K, V> {
           // ...
           void doSomething(K k, V v) {
             Long i = (Long) k; // у меня женерик вместо головного мозга
             // use i
           }
         }

         Ответить

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

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

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

   Где здесь C++, guest?!

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

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

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