Я не буду использовать оператор try-catch в блоке finally. (Ява)

Просто создайте статический служебный метод:

private static void closeQuietly(final Connection conn)
{
    if (conn == null)
        return;
    try {
        conn.close();
    } catch (WhateverException ignored) {
    }
}

потом

conn = whatever(); // create it there, not in the try block
try {
    // work
} catch (WhateverException e) {
    // whatever
} finally {
    closeQuietly(conn);
}

Но со временем вы все равно все равно будете использовать try/catch в блоке finally.

Обратите внимание, что поймать Exception — это плохая идея. Вместо этого перехватывайте исключения, специально вызванные conn.close(). Поймать Exception означает, что вы также поймаете все RuntimeException; и это Плохая Вещь (тм).

Если вы используете Guava, вы также можете использовать Closeables.closeQuietly(), который делает почти то же самое, что и приведенный выше код. Вы также можете ознакомиться с Closer< /a>, что очень интересно.

Наконец (каламбур): если вы используете Java 7, вместо этого используйте оператор try-with-resources (см. AutoCloseable):

try (
    conn = whatever();
) {
    // ...
}

Другим обходным решением для этого может быть полное удаление улова внутри метода:

public void someWork() throws SQLException{
    Connection c;
    try {
        // do something
    } finally {
        if (conn != null){
            c.close();
        }
    }
}

Затем вы выполняете проверку ошибок вне метода в основном методе, где вы вызвали метод someWork, или вы переносите его в другой метод, где вы вызываете someWork():

 public void callSomeWork(){
    try{
     someWork();
    }catch(SQLException ex){
    //handle SQL error here
    }
 }

Таким образом, ошибка, возникающая при попытке или, наконец, будет объявлена ​​​​в одном SQLException.

Вам может не понравиться этот код, но в данной ситуации он необходим. (Хотя вы можете обернуть код в такой метод, как метод closeQuietly @fge.)

Но есть более фундаментальная проблема. Изменение someWork для закрытия соединения в блоке finally недостаточно для предотвращения утечки... в этой ситуации!!

Почему?

Учти это:

SomeClass sc = new SomeClass(createConnection());
// do some stuff
sc.someWork();

Вопрос 1. Что произойдет, если в коде «сделать что-нибудь» возникнет исключение?

Q2: Что произойдет, если возникнет исключение при создании/конструкции SomeClass? (Примечание: это возможно даже с самыми простыми конструкторами. Например, OOME может быть вызван операцией new...)

Ответ: Объект Connection утек... в обеих базах!

Исправление состоит в том, чтобы выделить ресурс (например, объект Connection) внутри или непосредственно перед try/catch/finally; например

Connection c = createConnection();
try {
    // do something
} catch (SomeException e) {
    // some exception code
} finally {
    if (c != null){
        try {c.close();} catch (SomeException e) {}
    }
}

В Java 7 вы также можете написать это так:

try (Connection c = createConnection()) {
    // do something
} catch (SomeException e) {
    // some exception code
}

при условии, что Connection реализует интерфейс AutoCloseable. Версия Java 7 — это «синтаксический сахар» для более сложной реализации, которая заботится о проверке null, вызове close() и разумной обработке любых результирующих исключений. (JLS дает подробную информацию о том, что на самом деле делает «попробовать с ресурсом».)

Для записи:

• Проблема не в сборе/сборке мусора. Проблема в том, что объект Connection должен быть закрыт.

• Присвоение null не приведет к закрытию объекта. Это даже не приведет к сбору мусора. В лучшем случае он может сделать объект подходящим для сборки мусора... в каком-то будущем сборщике мусора.

• Поймать Exception — плохая идея. Прочитайте это: Нужен авторитетный источник почему вы не должны бросать или перехватывать java.lang.Exception

Возможно, вы выходите из зомби-соединения, если оно не закрыто и установлено в NULL.

Начиная с Java 7, это должно быть так

try (Connection conn = DriverManager.getConnection(props)) {
    ...
}

до Явы 7

Connection conn = DriverManager.getConnection();
try {
  ...
} finally {
    conn.close();
}

Если вы аннулируете и не закрываете, убедитесь, что все методы всегда получают новое открытое соединение.

Единственное, что вам нужно знать, это то, что каждый раз, когда вы получаете соединение, вы должны закрыть его (отсюда и блок finally), иначе вы можете оставить это соединение открытым в своем приложении и в конечном итоге исчерпать доступные соединения.

позвольте кому-нибудь другому сделать попытку-поймать и использовать IOUtils Apache Commons

Один из closeQuietly()-методов (например, этот) должен быть именно тем, что вам нужно:

Эквивалентно InputStream.close(), за исключением того, что любые исключения будут игнорироваться. Обычно это используется в блоках finally.