Polêmica
Isso gera um pouco de polêmica porque há quem defenda uma coisa e outros que acham o oposto. Ninguém provou qual é o certo, então todo mundo só tem opiniões. Alguns fundamentam mais que outros. Dependendo de quem lê e do conhecimento prévio de cada um, quem lê vai acreditar em uma coisa ou outra. Eu mesmo já falei disto em diversas perguntas sobre o assunto:
Basicamente a pergunta já foi respondida.
O resumo de tudo isso é que tem uma escola que diz que exceção é a solução para tudo que não era bem o que você queria, ou em outras palavras "exceção cura câncer" e você deve fazer exatamente o que fez. De fato em Java há uma tendência de fazer isso, a API toda adota isso.
Java se modernizou
Porém mais recentemente Java parou de ser tão dogmática, de ficar repetindo o mantra de que a linguagem é orientada a objeto, e para o bem de todos evoluiu adotando um estilo mais funcional. Acho fantástico que boa parte da comunidade passou adotar esse estilo. E ele prega menos o uso de exceções. Por exemplo criaram o Optional
para usar um código de erro em vez de lançar uma exceção quando o erro é esperado.
Então código Java moderno não faria o que foi feito e adotaria um có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.
Seu caso
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.
O mais correto quando cria-se uma exceção nova é tentar fazer algo que só ela tem. A exceção já indica qual foi o problema e em cada contexto que capturar pode ser que a mensagem seja outra. Talvez o mais adequado seja só agregar o saldo (não como mensagem, ter um campo para isso) na exceção para que o consumidor da exceção o use em uma mensagem personalizada. Mas não estou dizendo que está errado, depende do requisito, pode ser que nem o saldo deveria colocar, por segurança e privacidade. Talvez o valor do saque seja o caso de ser incluído.
Veja mais em Criar exceções personalizadas mais específicas ou mais genéricas?.
Mas tem outro problema. 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.
Eu não gosto de exceções checadas, na forma como o Java implementou, em sistemas complexos muita coisa errada pode acontecer, mas se vai seguir essa escola de abuso das exceções, então sua exceção deveria ser checada já que você não pode ignorar a indicação de falha (pelo menos essa é minha interpretação para o problema).
E isso dá um indício que exceção é um mecanismo difícil de usar. Quase todo mundo que posta algo aqui está cometendo erros com ela e em geral cometem o mesmo erro sempre.
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.
Na minha escola, que é seguida por alguns dos melhores programadores do mundo, não ter saldo não é uma falha, é uma situação esperada que acontece e faz parte da regra de negócio, e hoje Java tem o jeito certo de tratar isso.
Solução alternativa
Eu não costumo usar Java e principalmente os recursos novos não tenho domínio, mas seria mais ou menos isto:
import java.util.*;
class Conta {
public double saldo;
}
class ContaCorrente extends Conta {
public void deposita(double valor) {
super.saldo += (valor);
}
public Optional<Double> saca(double valor) {
if (super.saldo < valor) return Optional.empty();
super.saldo += (valor - 0.10);
return Optional.of(valor);
}
}
public class Main {
public static void main(String[] args) {
ContaCorrente conta = new ContaCorrente();
conta.deposita(200);
var sacado = conta.saca(300);
if (sacado.isEmpty()) System.out.println("O saldo nao eh suficiente para saque");
else System.out.println("A operação foi concluída e foi sacado " + sacado.get());
}
}
Veja funcionando no ideone. E no repl.it. Também coloquei no GitHub para referência futura.
Dá para fazer de forma mais funcional ainda, eu acho até bom não mudar demais o jeito que aprendeu. Não tome como o código perfeito, esta implementação pode ser melhorada.
Pode ser que o requisito indique que pode sacar o saldo disponível em vez de negar o saque todo. Isso funciona com esse mecanismo, com exceção tem como, mas fica esquisito porque terá dado exceção, mas deu certo e teria que transportar o valor que deu certo dentro da exceção. Mesmo que não precise fazer isso nesse caso, em outro pode precisar. Usando o mecanismo certo fica consistente.
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 sempre teve, ficaria algo assim:
import java.util.*;
class Conta {
public double saldo;
}
class ContaCorrente extends Conta {
public void deposita(double valor) {
super.saldo += (valor);
}
public boolean saca(double valor) {
if (super.saldo < valor) return false;
super.saldo += (valor - 0.10);
return true;
}
}
public class Main {
public static void main(String[] args) {
ContaCorrente conta = new ContaCorrente();
conta.deposita(200);
if (!conta.saca(300)) System.out.println("O saldo nao eh suficiente para saque");
}
}
Veja funcionando no ideone. E no repl.it. Também coloquei no GitHub para referência futura.
Atenção que ainda há outro erro no código. E tem outros erros conceituais nele.