Czy można w końcu użyć funkcji catch catch inside?

Question

Używam buforowanego programu piszącego i mój kod, zamyka pisarz w bloku finally. Mój kod jest taki.

........... 
    BufferedWriter theBufferedWriter = null; 
    try{ 
    theBufferedWriter =..... 
    .... 
    ...... 
    ..... 
    } catch (IOException anException) { 
    .... 
    } finally { 
     try { 
      theBufferedWriter.close();    
     } catch (IOException anException) { 
      anException.printStackTrace(); 
      } 
    } 

muszę użyć try catch up wewnątrz czystego kodu w końcu jako theBufferedWriter może również rzucić IOException. Nie chcę odrzucać tego wyjątku do meto wzywającego. Czy jest to dobra praktyka, aby w końcu użyć funkcji catch catch? Jeśli nie, jaka jest alternatywa? Proszę zasugeruj.

Pozdrawiam, Hiral

Answer 1

nieco ładniejszy sposób to zrobić jest użycie IOUtils.closeQuiety z Apache commons-io. Utrzymuje porządek w twoim kodzie i eliminuje niektóre elementy, które są nieodłączne w Javie.

kod staje się wówczas:

BufferedWriter theBufferedWriter = null; 
try{ 
    theBufferedWriter = ... 
    ... 
} catch (IOException anException) { 
    ... 
} finally { 
    IOUtils.closeQuietly(theBufferedWriter); 
} 

wiele ładniejszy i bardziej wyraziste.

Answer 2

W pre Java 7, powiedziałbym tego, co zostało napisane jest najlepszym rozwiązaniem.

W języku Java 7 i nowszych wersjach masz Automatic Resource Management przeznaczone do uproszczenia tych rzeczy. Dzięki tej funkcji można zrobić

BufferedWriter theBufferedWriter = null; 
try (BufferedWriter theBufferedWriter = ...) { 
.... 
...... 
..... 
} catch (IOException anException) { 
.... 
} 

Answer 3

To jest to, z czym będziemy musieli żyć do momentu wydania Java 7 i ARM Blocks.

Answer 4

Możesz też użyć adnotacji Lombok i @Cleanup, a nigdy więcej nie napiszesz wpadki próbnej do środka.

ten sposób normalnie można napisać to (uwaga throws IOException):

//Vanilly Java 

import java.io.*; 

public class CleanupExample { 
    public static void main(String[] args) throws IOException { 
    InputStream in = new FileInputStream(args[0]); 
    try { 
     OutputStream out = new FileOutputStream(args[1]); 
     try { 
     byte[] b = new byte[10000]; 
     while (true) { 
      int r = in.read(b); 
      if (r == -1) break; 
      out.write(b, 0, r); 
     } 
     } finally { 
     out.close(); 
     } 
    } finally { 
     in.close(); 
    } 
    } 
} 

Teraz z Lombok po prostu napisać @Cleanup na strumieniach

import lombok.Cleanup; 
import java.io.*; 

public class CleanupExample { 
    public static void main(String[] args) throws IOException { 
    @Cleanup InputStream in = new FileInputStream(args[0]); 
    @Cleanup OutputStream out = new FileOutputStream(args[1]); 
    byte[] b = new byte[10000]; 
    while (true) { 
     int r = in.read(b); 
     if (r == -1) break; 
     out.write(b, 0, r); 
    } 
    } 
} 

Answer 5

Jest OK, ale należy sprawdzić, czy theBufferedWriter nie jest pusty przed zamknięciem.
Można również zrobić:

BufferedWriter theBufferedWriter; 
try { 
    theBufferedWriter = new ... 
    try { 
     ... 
    } finally { 
     try { 
      theBufferedWriter.close(); 
     } catch (IOException closeException) { 
      closeException.printStackTrace(); 
     } 
    } 
} catch (IOException anException) { 
    ... 
} 

czyli

BufferedWriter theBufferedWriter; 
try { 
    theBufferedWriter = new ... 
} catch (IOException createException) { 
    // do something with createException 
    return; // assuming we are in a method returning void 
} 

try { 
    ... 
} catch (IOException anException) { 
    ... 
    // assuming we don't return here 
} 

try { 
    theBufferedWriter.close(); 
} catch (IOException closeException) { 
    closeException.printStackTrace(); 
} 

ale przeważnie robię takie operacje (np napisanie pliku) w specjalnej metody i wolą rzucać/wyjątek tak rozmówcy

void someMethod(...) throws IOException { 
    BufferedWriter theBufferedWriter = new ... 

    try { 
     ... 
    } catch (IOExcepption anException) { 
     try { 
      theBufferedWriter.close(); 
     } catch (IOException closeException) { 
      closeException.printStackTrace(); 
      // closeException is not thrown, anException represents the main/first problem 
     } 
     throw anException; 
    } 

    theBufferedWriter.close(); // throws the Exception, if any 
} 

_{Uwaga: angielski nie jest moim jodły można go (np prośbą o innym pliku, zatrzymanie aplikacji ...) uchwyt t ani mój drugi język, każda pomoc zostanie doceniona:}

Answer 6

Jest w porządku, aby umieścić próbny połów w końcu. Jest to narzędzie, które robi to, co chcesz robić. Jednak czuję, że rzucony na siebie wyjątek IOException jest na tyle rzadki, że pozwoliłbym mu tłumić każdy wyjątek w ciele.

try { 
    BufferedWriter writer = ..... 
    try { 
     ..... 
    } finally { 
     writer.close(); 
    } 
} catch (IOException e) { 
    .... 
}