Uma situação peculiar
Esse exemplo que você deu é interessante porque uma HibernateException
, que não é uma checked exception (isto é, exceção que deve ser tratada ou repassada para o nível acima) pode ser disparada por uma SQLException
do JDBC, que é checked, que por sua vez pode ser disparada por uma IOException
, que também é checked, e pode acontecer quando a conexão com o banco é perdida (após ter sido estabelecida, naturalmente). Porém o fato de HibernateException
não ser checked desobriga o tratamento da exceção, o que pode ou não ser recomendável nessa situação, a conferir abaixo (e você verá que ela precisa ser tratada).
Quando a conexão cai algum dos métodos beginTransaction()
, save()
ou commit()
irá lançar uma HibernateException
, indicando que a conexão foi perdida e o recurso (socket aberto com o banco) precisa ser liberado. Ou, em outras palavras, sinalizando que sessao.close()
precisa ser chamado. Porém sessao.close()
, que por debaixo dos panos chama Socket.close()
, além de liberar o socket (recurso do sistema) irá também disparar uma exceção ao tentar fechar a conexão que já foi perdida.
Obs.: De acordo com a documentação dos métodos Session.close()
e Session.disconnect()
, eles só precisam ser chamados em aplicações que passam uma conexão JDBC persistente (long-session ou long-conversation) ao Hibernate em vez de deixá-lo criar um pool delas sozinho. Se você usa um pool de conexões não chame sessao.close()
e ignore o que eu falo sobre a segunda exceção.
Segunda exceção: capture e depois ignore
Essa segunda exceção não precisa receber nenhum tratamento especial porque é um acontecimento normal decorrente da perda de conexão de rede. Então basta capturar essa segunda exceção e não fazer nada de mais com ela, nem mesmo logar. O socket será liberado e deixará de ser um recurso ocupado no sistema, o que seria um potencial problema caso o número de conexões abertas seja (ou possa ficar) muito grande. O sistema operacional é limitado no número de sockets que pode manter ocupados, mesmo que desconectados.
O único problema dessa segunda exceção, além de não precisar logar nada, é que o catch (Throwable e)
deveria ser mais específico: catch (HibernateException e)
.
Primeira exceção: passe para o nível acima
Já o catch
para a primeira exceção pode ou não ser necessário. Em geral nessa situação o nível acima irá querer tratar alguma exceção de falha no banco, talvez exibindo uma caixa de diálogo para o usuário ou de alguma outra forma, então em vez de capturar a HibernateException
e simplesmente logar você tem a opção de deixar essa exceção ir para o nível acima diretamente ou então capturá-la e lançar em seu lugar uma exceção de mais "alto nível", que não especifique que se trata de uma exceção do Hibernate. Você pode achar que HibernateException
é uma coisa específica de uma certa solução de persistência que seu sistema deve ser livre para trocar por outras, e que assim deve trocá-la por uma DaoException
(DAOs fazem sentido em Hibernate? Acho que não muito né heheheh), ou PersistenceException
ou algo do tipo (estou dando um exemplo qualquer aqui de exceção, somente para frisar essa ideia):
// Resolvi tornar checked também, mas isso é opcional
public class PersistenceException extends Exception {
public PersistenceException(Throwable e) {
super(e);
}
}
public void salvar(Usuario usuario) throws PersistenceException {
Session sessao = null;
try {
sessao = HibernateUtil.getSessionFactory().openSession();
Transaction transacao = sessao.beginTransaction();
sessao.save(usuario);
transacao.commit();
} catch (HibernateException e) {
throw new PersistenceException(e);
} finally {
try{
if (sessao != null) {
sessao.close();
}
} catch (HibernateException e) {
// Não precisa fazer nada
}
}
}
Conclusão
Nesse caso, você pode observar que mesmo HibernateException
não sendo checked e você não sendo obrigado a tratar as duas exceções que foram geradas, é melhor tratá-las de alguma forma para evitar que um recurso do sistema seja deixado ocupado.
Sobre o try with resources, Session
não implementa a interface Closeable
então nesse caso não dá para usar.