Блог

Я не могу понять, почему компилятор предупреждает меня об утечке ресурсов ( Resource leak: 'conn' is not closed at this location ) в следующем коде:

     Connection conn = null;
    try {
        conn = DatabaseConnectionPool.getConnectionFromPool();
        // Some code
        try {
            // Other code
        } catch (final SomeException e) {
            // More code
            throw e; // Resource leak: 'conn' is not closed at this location
        }

    } catch (final SQLException | OtherExceptions e) {
        // Some more code
    } finally {
        try {
            // Another bunch of code
        } finally {
            DatabaseConnectionPool.freeConnection(conn);
        }
    }
 

Обратите внимание, что если я напишу это так

     Connection conn = null;
    try {
        conn = DatabaseConnectionPool.getConnectionFromPool();
        // Some code
        try {
            // Other code
        } catch (final SomeException e) {
            // More code
            throw e;
        } finally {
            DatabaseConnectionPool.freeConnection(conn);
        }

    } catch (final SQLException | OtherExceptions e) {
        // Some more code
    } finally {
        // Another bunch of code
    }
 

предупреждение исчезло.

Компилятор довольно дамп. Вероятно, он не может знать, что DatabaseConnectionPool.freeConnection(conn) вызовет close conn . Я не уверен, почему второй пример не вызывает это предупреждение, но, вероятно, функция просто не совсем идеальна и может давать ложные отрицательные результаты. По сути, любой ресурс должен быть закрыт путем вызова его close метода непосредственно в том месте, где получен ресурс; это единственный способ, которым компилятор может выяснить, что вы хотите его закрыть, это не относится к межпроцедурному анализу, чтобы проверить, вызывает ли вызываемая функция close .

В java7 вам следует рассмотреть возможность использования инструкции try-with-resource; это рекомендуемый способ обработки любых ресурсов, т. Е.:

 try(Connection conn = ...){
   // Do something with conn

   // No close necessary here, done implicitly by the try statement
}