Então código Java moderno não faria o que foi feito e adotaria um código de errocódigo de erro que poderia ser melhor que uma exceção checada. Você informa a inadequação do que se tentou fazer de forma sem fazer uma bagunça no código, sem um peso enorme da exceção.
Java começou adotar a escola que prega que exceções não devem ser usadas para controle de fluxo, que elas só devem ser lançadas quando aconteceu algo que não deveria acontecer, ou seja, erros de programação (que não devem ser tratadas), e falhas de ambiente.
O seu código é o jeito antigo de fazer algo porque é esperado que não tenha saldo em algumas situações, não é um erro que o programador cometeu e não é algo que o ambiente causou, é uma regra de negócio esperada, não é excepcional.
Parabéns por ter criado uma exceção específica para isso, em geral as pessoas não criam. Mas ainda tem um outro erro que quase todos cometem. Ele basicamente usa a mesma estrutura de Exception
e manda uma mensagem, que pode não ser o adequado para uso em determinada situação.
Mas tem outro problema, na. Na hora de capturar a exceção pegou a mais genérica possível e isso não faz sentido. Não vou entrar em detalhes porque já respondi em: Há algum inconveniente em sempre capturar Exception e não algo mais específico?. Se der um erro de memória está fazendo algo que nem imagina.
Há quem diga que seu código está perfeito (raro), há quem diga que só errou em capturar Exception
em vez de SaldoInsuficienteException
, e também há os que dirão que também errou em fazer a exceção não checada, e finalmente os que dirão que nem deveria ter usado exceção.
Mas eu tenho minhas dúvidas se deve fazer assim. Talvez seja só uma questão de retornar verdadeiro ou falso se a operação não deu certo, e isso Java semrpesempre teve, ficaria algo assim: