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

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

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

1. PHP (5703)
2. C++ (2738)
3. Куча (2335)
4. JavaScript (2030)
5. C# (1929)
6. Java (1503)
7. Си (1102)
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 (31)
21. Swift (29)
22. Haskell (28)
23. Kotlin (15)

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

   +111

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

   public void close() throws IOException {
       try {
         flush();
       } catch (IOException ignored) {
       }
       out.close();
   }

   java.io.FilterOutputStream

   Запостил: someone, 01 Августа 2013

   Tweet

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

   • nazymko 01.08.2013 12:23 # 0

     Экстренный выход ?
     Все равно по рукам надавать

     Ответить

   • 3.14159265 01.08.2013 17:11 # +1

     А что такого? autoflush. Хуже когда его нету.

     Ответить

     • someone 01.08.2013 21:23 # 0

       Игнорируется исключение при flush. Никуда не сообщается, даже в лог. И это при том, что close вообще-то throws IOException, казалось бы, пусть себе бросает.

       Ответить

       • blackhearted 02.08.2013 09:58 # 0

         out.close() и бросит.

         Ответить

       • 3.14159265 02.08.2013 13:46 # 0

         try{
         flush();
         }finally{
         out.close();
         }
         ?
         Ну в принципе получше, да.
         
         > И это при том, что close вообще-то throws IOException, казалось бы
         В 99% cлучаев этот IOException от close глушат при вызове, никуда не сообщая.

         Ответить

         • bormand 02.08.2013 18:08 # 0

           Кстати, а чем это лучше, чем просто close()? close() же, по идее, сам сделает flush перед тем как закрывать? Или все-таки не сделает?

           Ответить

           • 3.14159265 12.08.2013 13:49 # 0

             >Или все-таки не сделает?
             Так в посте и приведен close!
             Этот, например делает.

             Ответить

             • bormand 12.08.2013 14:58 # +1

               Все, понял в чем фишка. Невнимательно прочел код, и подумал, что он флашит и следом закрывает низлежащий поток. Но на самом деле флаш он вызвал у себя (или у потомка), чтобы сбросить свой буфер.

               Ответить

               • myzone 13.08.2013 23:43 # 0

                 Я, черт возьми, точно так же его прочитал :)
                 Но мне, все таки, кажется, что вариант с finally получше будет

                 Ответить

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

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

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

   Помни, guest, за тобой могут следить!

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

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

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