Czy poprawnie używam Java 7 try-with-resources

Oczekuję, że buforowany czytnik i czytnik plików zostaną zamknięte, a zasoby zwolnione, jeśli wyjątek zostanie zgłoszony.

public static Object[] fromFile(String filePath) throws FileNotFoundException, IOException
{
    try (BufferedReader br = new BufferedReader(new FileReader(filePath)))
    {
        return read(br);
    } 
}

Czy jednak istnieje wymóg posiadania catchklauzuli umożliwiającej skuteczne zamknięcie?

EDYTOWAĆ:

Zasadniczo powyższy kod w Javie 7 jest odpowiednikiem poniższego kodu w Javie 6:

public static Object[] fromFile(String filePath) throws FileNotFoundException, IOException
{

    BufferedReader br = null;

    try
    {
        br = new BufferedReader(new FileReader(filePath));

        return read(br);
    }
    catch (Exception ex)
    {
        throw ex;
    }
    finally
    {
        try
        {
            if (br != null) br.close();
        }
        catch(Exception ex)
        {
        }
    }

    return null;
}

Jest poprawna i nie ma wymogu catchklauzuli. Dokument Oracle java 7 mówi, że zasób zostanie zamknięty niezależnie od tego, czy wyjątek zostanie faktycznie zgłoszony, czy nie.

catchKlauzuli należy używać tylko wtedy, gdy chcesz zareagować na wyjątek. catchKlauzula zostanie wykonana po zasób jest zamknięta.

Oto fragment z samouczka Oracle :

Poniższy przykład odczytuje pierwszy wiersz z pliku. Używa wystąpienia BufferedReader do odczytu danych z pliku. BufferedReader to zasób, który należy zamknąć po zakończeniu programu z nim:

static String readFirstLineFromFile(String path) throws IOException {
    try (BufferedReader br =
                   new BufferedReader(new FileReader(path))) {
        return br.readLine();
    }
} // In this example, the resource declared in the try-with-resources statement is a BufferedReader.

... Ponieważ instancja BufferedReader jest zadeklarowana w instrukcji try-with-resource, zostanie zamknięta niezależnie od tego, czy instrukcja try zakończy się normalnie czy nagle (w wyniku wyrzucenia IOException przez metodę BufferedReader.readLine).

EDYTOWAĆ

Odnośnie nowego zredagowanego pytania:

Kod w Javie 6 wykonuje blok, catcha następnie finally. Powoduje to, że zasoby są nadal potencjalnie otwarte w catchbloku.

W 7 składni Javy, zasoby są zamknięte przed tym catchblokiem, więc zasoby są już zamknięte podczas catchwykonywania bloku. Jest to udokumentowane w powyższym linku:

W instrukcji try-with-resources każdy blok catch lub final jest uruchamiany po zamknięciu zadeklarowanych zasobów.

Korzystanie z zasobów typu „try-with-resources” będzie działać dobrze w tym konkretnym przypadku, ale ogólnie nie jest całkiem poprawne. Nie powinieneś łączyć zasobów w taki sposób, ponieważ może to prowadzić do niemiłych niespodzianek. Załóżmy, że masz zmienny rozmiar bufora:

public static Object[] fromFile(String filePath) throws FileNotFoundException, IOException
{
    int sz = /* get buffer size somehow */
    try (BufferedReader br = new BufferedReader(new FileReader(filePath), sz))
    {
        return read(br);
    } 
}

Załóżmy, że coś poszło nie tak i skończyłeś z sznegatywnym wynikiem . W takim przypadku Twój zasób plikowy (utworzony przez new FileReader(filePath)) NIE zostanie zamknięty.

Aby uniknąć tego problemu, należy określić każdy zasób osobno w następujący sposób:

public static Object[] fromFile(String filePath) throws FileNotFoundException, IOException
{
    int sz = /* get buffer size somehow */
    try (FileReader file = new FileReader(filePath);
         BufferedReader br = new BufferedReader(file, sz))
    {
        return read(br);
    } 
}

W tym przypadku, nawet jeśli inicjalizacja brawarii filenadal zostanie zamknięta. Więcej szczegółów znajdziesz tutaj i tutaj .