Atividade Semana 1 - Yane Silva Lobo#35
Conversation
| name: "Jane Doe", | ||
| balance: 1000, | ||
| limit: 5000 | ||
| } |
There was a problem hiding this comment.
Gostei da estruturação do seu código, fazer a conta separada.... acabei esquecendo de fazer isso no meu 😉
| if(account.balance >= value){ | ||
| account.balance -= value; | ||
| return `Your current balance is: ${account.balance}`; | ||
| } else if (account.limit >= value){ |
There was a problem hiding this comment.
Como sugestão só teria feito uma verificação que se ainda tivesse saldo, diminuir do saldo (zerando ele) para daí retirar o restante do limite... mas nem sei se é assim mesmo que funciona na prática 😁
| account.limit -= value; | ||
| return `Your limit was decreased to ${account.limit}`; | ||
| } | ||
|
|
There was a problem hiding this comment.
Acho que poderia ter feito em uma só lógica de reajuste de limite, recebendo o valor, porém pensando em bancos como Nubank em que faz esse incremento e decremento..... teu código faz super sentido.
dessa forma.... só uma verificação se o limite já não está zerado ou uma tratativa de erro.
| expect(limitDeactivate()).toEqual(output4); | ||
| }) | ||
| }); | ||
|
|
There was a problem hiding this comment.
Legal ter colocado as mensagens de saídas separadas, deixando mais fácil de entender e visualizar.
| const output = 'Your current balance is: -4000. Your current limit is: 1000'; | ||
| expect(withdraw(5000)).toEqual(output); | ||
| }) | ||
| }); |
There was a problem hiding this comment.
Poderia fazer um teste para não aceitar valores negativos, percebi que tbm não fiz isso 😜
There was a problem hiding this comment.
parabéns Yane pela entrega, uma dica: é uma boa prática não ter funções que estejam fazendo muitas coisas, uma sugestão para o futuro pensar em como fazer o refactor desse código, sobre os testes seria bom cobrir mais cenários, por exemplo casos infelizes. Mas de novo parabéns pela entrega
3 participants
Adicionados os arquivos de resolução da atividade da semana 1.