-
-
Save hvitorino/1670921 to your computer and use it in GitHub Desktop.
Classe C# exemplificando o não uso de método privado quando não é necessário
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
public class LimiteDeSaldo | |
{ | |
private decimal disponivel; | |
private decimal concedido; | |
private decimal margemDeCredito; | |
public LimiteDeSaldo(decimal disponivel, decimal concedido, decimal margemDeCredito) { | |
this.disponivel = disponivel; | |
this.concedido = concedido; | |
this.margemDeCredito = margemDeCredito; | |
} | |
public void Debita(decimal valor) | |
{ | |
if(PossuiDisponibilidade(valor)) | |
disponivel -= valor; | |
else | |
throw new LimiteDeSaldoDisponivelInsuficienteException(disponivel); | |
} | |
public void DebitaSemVerificarDisponibilidade(decimal valorSolicitado) | |
{ | |
disponivel -= valor; | |
} | |
public bool PossuiDisponibilidade(decimal valorSolicitado) | |
{ | |
return ValorDisponivel >= valorSolicitado; | |
} | |
public decimal ValorDisponivel | |
{ | |
get { return Math.Round(disponivel * (margemDeCredito + 1), MidpointRounding.AwayFromZero); } | |
} | |
} |
Condições excepcionais sempre jogo no else.
else para mim normalmente só complica a legibilidade do código, salvo raras exceções. enfim, quanto menos código indentado para direita melhor.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Não curti esse else{} ae, sou adepto do fail-fast :-)
Que pena que o Java não permita ainda trabalhar com decimals desta forma.