ADD: resolução de exercicios TDD - Geice #39
Conversation
exercicios/para-casa/entregas/Geice-SousaPinho/resolucaoGeice.js
Outdated
Show resolved
Hide resolved
|
|
||
| sacar(valor){ | ||
| if (valor <= this.saldo || valor <= this.limite){ | ||
| return valor |
There was a problem hiding this comment.
sugestão: retornar uma mensagem com os valores de saldo e limite disponíveis, após o saque
exemplo: Saque no valor de R$${valor} realizado! Saldo disponível: R$${this.saldo}. Limite disponível: R$${this.limite}
| if (valor <= this.saldo || valor <= this.limite){ | ||
| return valor | ||
| } | ||
| else{ |
There was a problem hiding this comment.
sugestão: implementar cenário caso o valor ultrapasse saldo e limite disponíveis, para a mensagem de erro:
Saldo e limite insuficientes para saque. ou, cenário em que o saque acontece, e esses valores ficarão negativados.
| } | ||
| } | ||
|
|
||
| depositar(valor){ |
There was a problem hiding this comment.
sugestão: retornar uma mensagem com o saldo disponível, após depósito.
exemplo:
let saldoAtualizado = valor + this.saldo
return Você depositou R$${valor}. Saldo disponível: R$${saldoAtualizado}
| depositar(valor){ | ||
| if (valor) return valor | ||
| else{ | ||
| throw new Error('houve um erro inesperado, tenve novamente em instantes') |
There was a problem hiding this comment.
dúvida: a mensagem de erro está clara, mas, usando valor como parâmentro, não entendi em que momento esse cenário isso poderia acontecer.
sugestão: implementar lógica usando as informações de agencia ou numero de conta incorretos, já que isso impossibilitaria o sucesso da transação, e são atributos que você criou em seu objeto ContaBancaria :)
typo: corrigir "tenve" para "tente"
There was a problem hiding this comment.
o cenário para o erro é quando não passa um argumento
exercicios/para-casa/entregas/Geice-SousaPinho/resolucaoGeice.js
Outdated
Show resolved
Hide resolved
| // const cliente = 'Geice Sousa'; const agencia = 1230; const conta = 789456.32; const limite = 1500.00; const saldo = 1000.00 | ||
| const valor = 200.00 | ||
| const valorDeAjuste = 500.00 |
There was a problem hiding this comment.
sugestão: já que os valores de testes são fixos, você pode remover essas constantes, e inserir os valores diretamente nas respectivas funções.
exemplo:
em lugar de:
expect(contaCliente.depositar(valor)).toBeTruthy()
implementar:
expect(contaCliente.depositar(200.00)).toBeTruthy()
exercicios/para-casa/entregas/Geice-SousaPinho/resolucaoGeice.test.js
Outdated
Show resolved
Hide resolved
exercicios/para-casa/entregas/Geice-SousaPinho/resolucaoGeice.test.js
Outdated
Show resolved
Hide resolved
faltando 45 para acabar o tempo rs