So verwenden Sie Guava Closer richtig

Question

Ich bin schwer zu finden, wie Sie das Closer Dienstprogramm aus der Guava-Bibliothek verwenden. Bitte beachten Sie den folgenden Code.

• Eine Sache ist, dass die IndexWriter ein IOException sowohl Objektinitialisierung wirft und close(). Daher ist der Code in den End- und Wiederholungsblöcken unterstrichen.
• Die andere Frage ist, warum habe ich Throwable statt anderen Ausnahmetypen fangen und kann ich die Fehler erneut auslösen (ich würde es vorziehen, die an der Stelle anzumelden)

`

int getDocumentsCount() { 
    Closer closer = Closer.create(); 
    try { 
     IndexWriter iwriter = closer.register(openIndexWriter()); 
     return iwriter.numDocs(); 
    } catch (Throwable e) { 
     logger.error(e, e); 
     return -1; 
    } finally { 
     closer.close(); 
    } 
} 


IndexWriter openIndexWriter() throws IOException { 
    return new IndexWriter(directory, analyzer, false, 
     IndexWriter.MaxFieldLength.UNLIMITED); 
} 

`

vielen Dank

(mit Java 6 Stück)

Answer 1

Von Guava's own explanation, haben Sie haben zu verwenden Throwable, ja.

Hier ist ihr Beispiel Schnipsel:

public void foo() throws IOException { 
    Closer closer = Closer.create(); 
    try { 
    InputStream in = closer.register(openInputStream()); 
    OutputStream out = closer.register(openOutputStream()); 
    // do stuff with in and out 
    } catch (Throwable e) { // must catch Throwable 
    throw closer.rethrow(e); 
    } finally { 
    closer.close(); 
    } 
} 

anzumerken, dass sie fangen Throwable und rethrow es direkt aus der Closer Instanz.

Wie, warum es ist Throwable und nicht, sagen wir mal IOException oder RuntimeException, es ist, weil die Closermuss wissen, dass ein Fehler aufgetreten ist, so dass es richtig die Ressourcen schließen kann. Alles ist nur eine Frage der richtigen Dinge. Es kann also funktionieren, wenn Sie es nicht richtig machen, aber es ist nicht garantiert.

Nicht, dass, wenn Ihre Methode MyOwnCheckedException zum Beispiel werfen kann, Sie haben sie erklären:

} catch (Throwable t) { 
    throw closer.rethrow(e, MyOwnCheckedException.class); 
} finally { 
    closer.close(); 
} 

Java 7 Beispiel zum Vergleich:

public void foo() throws IOException { 
    try (InputStream in = openInputStream(); 
     OutputStream out = openOutputStream(); 
    // do stuff with in and out 
    } 
} 

Wenn Sie die Closer vergleichen Beispiel mit dem Java 7-Beispiel, Sie sehen, dass ich immer noch die IOException in der Methodensignatur deklarieren muss.

---

Für Ihren Fall ist es das, was Sie tun müssen:

int getDocumentsCount() { 
    try { 
    Closer closer = Closer.create(); 
    try { 
     IndexWriter iwriter = closer.register(openIndexWriter()); 
     return iwriter.numDocs(); 
    } catch (Throwable e) { 
     closer.rethrow(e); 
    } finally { 
     closer.close(); 
    } 
    } catch (IOException e) { 
    logger.error(e, e); 
    return -1; 
    } 
} 

Try-Pyramiden zu vermeiden, würde ich folgendes tun:

int getDocumentsCount() { 
    try { 
    return doGetDocumentsCount(); 
    } catch (IOException e) { 
    logger.error(e, e); 
    return -1; 
    } 
} 

int doGetDocumentsCount() throws IOException { 
    Closer closer = Closer.create(); 
    try { 
    IndexWriter iwriter = closer.register(openIndexWriter()); 
    return iwriter.numDocs(); 
    } catch (Throwable e) { 
    closer.rethrow(e); 
    } finally { 
    closer.close(); 
    } 
}