Если я ловлю исключения, выданные при закрытии java.sql. Соединение

В очень минимальном, всегда всегда всегда исключения журнала, которые Вы ловите и не действуете на.

Тихо перехваченные исключительные ситуации, которые глотают без самого крошечного взгляда, хуже.

Мне лично нравится Ваша вторая идея, по крайней мере, входа ошибки. Поскольку Вы ловите Исключение, теоретически возможно поймать что-то другое, чем Исключение SQL. Я не уверен, что могло произойти или как редкий (как из исключений памяти, и т.д.), но подавляющий все ошибки не кажется правильным мне.

, Если бы Вы хотите подавить ошибки, я сделал бы это только к очень определенным, которые Вы знаете, должен быть обработан тот путь.

ситуация Hypothecial: что, если бы Ваш sql имел открытую транзакцию и закрытие соединения, вызвало exceptino из-за этого, Вы хотели бы подавить ту ошибку? Даже подавление SQLExceptions могло бы быть немного опасным.

Необходимо обработать исключение. Это не плохая практика. Предположите потерю сети прежде, чем закрыть dabatase соединение. Это, вероятно, выдаст исключение.

это редкий? Да. Я предполагаю, что это - то, что их называют исключениями, и это не причина проигнорировать его. Помните, что, если это могло бы перестать работать, это перестанет работать.

необходимо также думать о том, возможно ли иметь пустое соединение в этой точке (это вызвало бы NullPointerException), или нет.

if (connection != null) {
   try { 
      connection.close(); 
   } catch (SQLException sqle) { 
      logger.log(e.getMessage(), e); 
   }
}

В идеальном мире Вы ничего никогда не должны делать на исключении, конечно, в идеальном мире, Вы никогда не получали бы исключение также 8-)

Так, необходимо исследовать влияние различных вариантов.

Журнал только: операции Базы данных все закончены, ничто, чтобы сделать, но очистить ресурсы. Если исключение происходит в этой точке, оно, скорее всего, не оказывает влияния на выполненную работу, так вход ошибки должен быть достаточным. Конечно, если ошибка происходит во время входа, то в основном необходимо обработать отказавшую операцию базы данных, которая на самом деле не перестала работать.

Пустой обработчик: операции Базы данных все закончены, ничто, чтобы сделать, но очистить ресурсы. Если исключение происходит в этой точке, оно, скорее всего, не оказывает влияния на выполненную работу, таким образом, метод возвращается успешно. Следующий доступ к базе данных может столкнуться с той же проблемой, но должно произойти в начале транзакции, где это справедливо перестанет работать и затем обрабатываться соответственно. Если проблема зафиксировала себя, то не будет никакого признака, что что-либо когда-либо шло не так, как надо.

Это - довольно типичный сценарий для помещения завершения () операция (операции) в наконец блок, чтобы гарантировать, что очистка происходит, так как мы не хотим никакие другие отказы запретить очистку ресурса. Если никакая ошибка не произошла, то Ваш метод не должен перестать работать, когда его операция успешно завершилась. В этом случае пустая обработка исключений довольно нормальна.

, Конечно, мнения будут варьироваться.

Вы могли также бросить RuntimeException:

try {
    connection.close();
 } catch(Exception e) {
     throw new RuntimeException(e); 
 }

Вы не должны будете изменять свою сигнатуру метода и сможете использовать метод Exception.getCause позже для нахождения причины проблемы.

Обратите внимание, что Apache Commons DButils обеспечивает closeQuietly() метод, что можно использовать, чтобы не создавать помехи коду 'избыточными' выгодами. Обратите внимание, что я не рекомендую глотать исключения, но для этого close() сценарий я думаю, что это обычно приемлемо.

На основе моего опыта, игнорирующего исключение, никогда не хорошая идея. Верьте мне, производственные инженеры службы поддержки и аналитики поблагодарят Вас тонна, если Вы зарегистрировали исключение.

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

если это будет "ошибкой, которой никогда не происходит то", случай затем я буду просто повторно бросать Исключение и надеяться, что никто не ловит его.
, если это - какой-либо другой случай, я, вероятно, зарегистрирую его

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

Молчаливое проглатывание исключений означает упущение важной информации для человека, который может исправить код.