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

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

    • ava nazymko 01.08.2013 12:23 # 0

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

    • ava 3.14159265 01.08.2013 17:11 # +1

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

      • ava someone 01.08.2013 21:23 # 0

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

        • ava blackhearted 02.08.2013 09:58 # 0

          out.close() и бросит.
          Ответить

        • ava 3.14159265 02.08.2013 13:46 # 0

          try{
          flush();
          }finally{
          out.close();
          }
          ?
          Ну в принципе получше, да.

          > И это при том, что close вообще-то throws IOException, казалось бы
          В 99% cлучаев этот IOException от close глушат при вызове, никуда не сообщая.
          Ответить

          • ava bormand 02.08.2013 18:08 # 0

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

            • ava 3.14159265 12.08.2013 13:49 # 0

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

              • ava bormand 12.08.2013 14:58 # +1

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

                • ava myzone 13.08.2013 23:43 # 0

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

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