Entrega exercício da semana 1 - Teste Drive Developmente (TDD)#29
Entrega exercício da semana 1 - Teste Drive Developmente (TDD)#29carolmarquezini wants to merge 5 commits into
Conversation
…mente atividade para casa.
debbsgomes
left a comment
There was a problem hiding this comment.
Parabéns em finalizar o exercício :)
| (para mais e para menos) ou desativado. Use a abordagem Red - Green - Refactor para desenvolver | ||
| essa aplicação. | ||
| */ | ||
| function ContaBancaria(nome, saldo, limite) { |
There was a problem hiding this comment.
Eu teria feito uma const ContaBancaria ao invés de uma função, e sem passar os parâmetros na função. Mas curti a lógica usada aqui também.
| this.limite = limite; | ||
| } | ||
|
|
||
| const limiteMax = 300; |
There was a problem hiding this comment.
Acho que os limites como valores eram pra ser definidos no teste em si, aqui seria somente um código de como funciona uma conta bancária, mas sem os valores.
There was a problem hiding this comment.
boa, não tinha pensado por esse lado, acabei atribuindo o valor na variável para conseguir trabalhar com as condições. Vou repensar como fazer a partir daqui ...
obrigada pelo toque!!!
| const limiteMax = 300; | ||
| const limiteMin = 0; | ||
|
|
||
| function depositar(conta, valorDeposito) { |
There was a problem hiding this comment.
Curti que a lógica que você usou para o deposito é a mesma que usei para o saque e meio que as duas coisas funcionam, rs.
|
|
||
| function sacar(conta, valorSaque) { | ||
|
|
||
| if (conta.saldo >= valorSaque) { |
There was a problem hiding this comment.
Acredito que faltou o else também de caso usasse o limite e o saldo juntos, tipo, limite é 50, saldo é 50, mas se faz uma compra de 70. Não sei se não vi, mas acho q teria q ter isso.
There was a problem hiding this comment.
esse é um ponto que fiquei presa porque não estou conseguindo pensar como fazer o else, as formas como eu estava tentando não estavam dando certo.
| function consultarSaldo(conta) { | ||
| return conta.saldo + conta.limite | ||
| } | ||
|
|
There was a problem hiding this comment.
Não tinha que ter uma função de desabilitar o limite também? Fora isso boa lógica no código e sério, adorei a lógica que fiz pro saque sendo usada no deposito.
There was a problem hiding this comment.
tinha sim, eu esqueci desse item rss mandei mal ...
muito obrigada pelos comentários, gostei bastante.
|
|
||
| const { ContaBancaria, consultarSaldo, sacar, depositar } = require('./contaBancaria') | ||
|
|
||
| describe("Verificar saldo", () => { |
There was a problem hiding this comment.
Mas saldo não é diferente de limite? Mas se a intenção era testar os 2 juntos ok.
| }) | ||
| }) | ||
|
|
||
| describe("Verificar saque", () => { |
There was a problem hiding this comment.
Curti o jeito que fez a lógica no código, o que você escreveu em 3 linhas acho que fiz 5.
| }) | ||
| }) | ||
|
|
||
| describe("Verificar deposito", () => { |
There was a problem hiding this comment.
Nossa, código mega limpo de novo, congrats...
Marianaliima
left a comment
There was a problem hiding this comment.
olá, primeiramente parabéns pela entrega, sua colega de turma comentou alguns pontos que concordo sobre seu código. Também seria interessante o uso de beforeEach porque ai não iria precisar instanciar sempre a função em cada teste.
Exercício da semana 1 - Teste Drive Developmente (TDD)
Conta bancária - implementando TDD
Como executar teste
Resultado: