- Gostaria de ver um exemplo de um tratamento adequado para essa exceção nesse caso.
Depende muito do tipo de sistema.
Se for desktop, você pode mostrar uma caixa de diálogo com a mensagem de erro. Se for web, você pode devolver a página para o usuário com a mensagem de erro em alguma parte do HTML.
O importante ao tratar uma exceção é que o comportamento do sistema seja coerente, que ele não caia num erro pior ainda e não esconda os erros do usuário.
Como exibir mensagens de erro não deve ser a responsabilidade das classes de acesso a banco de dados (Data Access Objects, por exemplo), então uma forma de realizar um tratamento adequado é encapsular as exceções do banco em uma exceção genérica de banco de dados e tratar esta outra exceção em classes de controle (Controllers).
Por exemplo:
public void update(Pessoa pessoa) throws DataBaseException {
Connection conexao = getConexao();
try {
PreparedStatement preparedStatement = conexao.prepareStatement(sqlUpdate);
preparedStatement.setString(1, pessoa.getNome());
preparedStatement.setString(2, pessoa.getCpf());
preparedStatement.setInt(3, pessoa.getIdPessoa());
preparedStatement.execute();
preparedStatement.close();
conexao.close();
} catch (Exception e) {
logger.error(e);
throw new DataBaseException("Erro ao atualizar a pessoa!", e);
} finally {
if (conexao != null) {
try {
conexao.close();
} catch (SQLException e) {
logger.error(e);
}
}
}
}
O método acima salva o erro original num arquivo de log usando uma API próprio, pois "printar" erros para a saída não é uma boa prática.
Além disso, ele encapsula qualquer exceção numa exceção customizada DataBaseException
. Esta exceção seria usada em todas as rotinas de acesso a banco de dados.
Outro detalhe é que é necessário um bloco finally
para fechar a conexão, caso contrário um erro resultaria em vazamento de conexões, isto é, vários conexões abertas e não fechadas. Isso é uma grande dor de cabeça em sistemas mal planejados. O servidor começa a travar após algum tempo de uso e vai demorar até alguém descobrir o problema.
O segundo catch
não precisa de tratamento além do log, pois ele só tenta fechar a conexão. Se isso falhar o problema é maios embaixo e não há muito o que fazer.
Em geral, é "feio" capturar uma exceção genérica como Exception
. Mas há um motivo para isso. Existem casos, como o desenvolvedor colocar um parâmetro a mais no PreparedStatement
, onde ocorreria um erro diferente de SQLException
. E há uma regra do mundo dos negócios: não mostre o erro técnico para o usuário.
Deixar escapar um erro como NullPointerException
ou StringIndexOutOfBroundsException
para a tela é pior do que mostrar um erro genérico. A mensagem "ocorreu um erro inesperado, entre em contato com o suporte" é às vezes o que há de mais útil para o usuário que não conhece nada sobre tecnologia.
Os casos de erro em que realmente o usuário pode atuar podem ser tratados especificamente. Por exemplo, caso o usuário tente excluir um registro que é chave estrangeira de outra tabela, então é possível mostrar uma mensagem específica de que é necessário excluir os dados da outra tabela antes de excluir o registro atual.
Dá pra fazer um mecanismo mais sofisticado do que apenas colocar catch
com Exception
, mas vou deixar assim nesta resposta para efeito de exemplo. Esse mecanismo poderia consistir numa rotina de tratamento global que mostrasse para o usuário mensagens diferentes para cada tipo de exceção.
Vamos imaginar agora um método que trata a ação do usuário para atualizar os dados da Pessoa
.
Se fosse um sistema desktop:
public void doCliqueBotarAtualizarAction() {
Pessoa pessoa = recuperarPessoaDosCamposDaTela();
try {
pessoaDAO.update(pessoa);
mostrarDialogoSucesso("Dados alterados com sucesso!");
} catch (DataBaseException e) {
mostrarDialogoErro(e.getMessage());
}
}
O método acima simplesmente mostra uma caixa de sucesso ou fracasso dependendo do caso.
Se fosse um sistema web com Spring MVC, por exemplo:
@RequestMapping(value="/atualizaPessoa", method=RequestMethod.POST)
public ModelAndView atualizarPessoa(Pessoa pessoa) {
try {
pessoaDAO.update(pessoa);
return new ModelAndView("tela-pesquisa")
.addObject("mensagem", "Dados alterados com sucesso!");
} catch (DataBaseException e) {
return new ModelAndView("tela-edicao")
.addObject("pessoa", pessoa)
.addObject("mensagemErro", e.getMessage());
}
}
O método acima recebe uma requisição POST
para atualizar a pessoa. Se ocorrer sucesso o usuário é direcionado para a tela de pesquisa com a mensagem de sucesso. Se ocorrer falha, o usuário é direcionado para a tela de edição mostrando a pessoa e também a mensagem de erro.
Outra abordagem seria tratar separadamente SQLException
de outro tipo de Exception
, porém deve-se analisar se na prática vai dar alguma diferença, pois replicar o tratamento de erro para fazer a mesma coisa simplesmente não faria sentido.
- Ao manipular essa checked exception, guardando o StrackTrace em um Log e mostrando um aviso ao usuário que algo de que não foi possível realizar o Update dos dados, seria considerado que a exceção foi tratada?
Sim, desde que garante que o sistema não vai ficar num estado inconsistente.
Mostrar a mensagem é suficiente se não houver efeitos colaterais indesejados.
Para casos, por exemplo, houve são alteradas várias tabelas, é importante colocar as alterações numa transação e fazer o rollback em caso de falha.
Veja um exemplo:
public void transferencia(Conta fonte, Conta destino, BigDecimal valor) throws BusinessException {
boolean sucesso = false;
iniciarTransacao();
try {
contaDao.debitar(fonte, valor);
contaDao.creditar(destino, valor);
sucesso = true;
} catch (Exception e) {
logger.error(e);
throw new BusinessException("Não foi possível efetuar a transferência!", e);
} finally {
if (sucesso) efetuarTransacao();
else desfazerTransacao();
}
}
- E se ao tópico acima fosse acrescentado que fossem limpos os campos de preenchimento dos dados pessoais e voltasse a tela inicial de alteração de dados para que o usuário tente fazer a edição novamente, seria considerado que a exceção foi tratada?
Limpar os dados geralmente não é uma boa ideia, afinal é trabalho a mais e dor de cabeça para o usuário.
Mostrar os campos com problema é uma tarefa melhor se for o caso. Para isso é importante validar os campos antes de tentar executar a consulta no banco de dados.
Em geral, as constraints do banco de dados como unique ou not null devem ser usadas para garantir a integridade do banco, mas para validação de entrada do usuário elas devem ser o último recurso.
E se o erro for de SQL, como uma sintaxe incorreta, não vai adiantar o usuário tentar de novo, provavelmente é um bug do sistema.
Considerações
O nível de detalhe do tratamento de erros varia muito de caso para caso.
O melhor é codificar de forma "segura" validando bem todas as entradas, criando testes unitários para validar suas consultas e, assim, minimizar qualquer possibilidade de um erro no SQL.