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

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

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

1. PHP (5703)
2. C++ (2737)
3. Куча (2343)
4. JavaScript (2028)
5. C# (1928)
6. Java (1502)
7. Си (1108)
8. Pascal (647)
9. Python (583)
10. 1C (535)
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 (30)
21. Swift (27)
22. Haskell (26)
23. Kotlin (14)

1. Java / Говнокод #6683

   +79

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

   Object sync;
   
   try {
       synchronized (sync){
           sync.wait();
       }
   } catch (NullPointerException npe){
       sync = new Object();
       synchronized (sync){
           sync.wait();
       }
   }

   Вот так чел пытался избежать "ненужного" создания объекта-монитора. Память экономил...

   Запостил: Skipy, 17 Мая 2011

   Tweet

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

   • guest 17.05.2011 12:13 # +4

     Как писать надежный код
     
     try {
     or.setField1(ReportHelper.EMPTY_STRING);
     } catch (NullPointerException e2) {
     or.setField1(ReportHelper.EMPTY_STRING);
     }

     Ответить

     • Lure Of Chaos 17.05.2011 14:30 # 0

       надежнее

       while(true)
       try {
        or.setField1(ReportHelper.EMPTY_STRING);
        break;
        } catch (NullPointerException e2) {
        }

       Ответить

       • guest 17.05.2011 15:06 # 0

         А зачем break? Функция может коварно выбросить исключение через некоторое время.

         Ответить

         • Lure Of Chaos 17.05.2011 15:41 # 0

           гарантированно выбросит?

           Ответить

           • guest 17.05.2011 16:16 # 0

             > может

             Ответить

             • Lure Of Chaos 17.05.2011 16:28 # +1

               если не выбросит, то break; поможет прекратить попытки

               Ответить

   • Lure Of Chaos 17.05.2011 14:29 # 0

     ааа, я понял замысел: это ленивое создание обьекта!
     кстати, чел не понимает, что sync вполне может замениться на this -- объяснять пробовали?

     Ответить

     • sogekoder 17.05.2011 15:29 # 0

       >>вполне может замениться на this
       это вы имеете ввиду эквивалентную замену? Если да, то вы не правы, сэр.

       Ответить

       • Lure Of Chaos 17.05.2011 15:43 # 0

         как я понимаю, извне обьект монитора не трогается (иначе бы мы могли работать на разных мониторах), а раз монитор один, то сгодится и текущий обьект.
         О, контекст тут наверное статический - тогда да, this не будет.

         Ответить

         • Skipy 17.05.2011 17:36 # 0

           > а раз монитор один, то сгодится и текущий обьект.
           
           На нем кто-то может сдуру вызвать notify. Не, чем глубже спрятан монитор, тем лучше. Так что -
           
           private final Object sync = new Object();

           Ответить

           • Lure Of Chaos 17.05.2011 18:00 # 0

             я это и имел ввиду

             Ответить

   • guest8 08.04.2019 21:07 # −999

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

     Ответить

   • guest8 09.04.2019 10:46 # −999

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

     Ответить

   • guest8 09.04.2019 17:57 # −999

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

     Ответить

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

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

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

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

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

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

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