Atividade semana 1 - Brena O'Dwyer#14
Conversation
| return typeof account === 'object'; | ||
| } | ||
|
|
||
| function isAcountComplete(account) { //verifica se a conta tem as propriedades name, balance, limit |
There was a problem hiding this comment.
essa função é para verificar se a conta é existente, correto? não conhecia esse método hasOwnProperty que retorna um valor booleano, achei interessante essa verificação a mais, não havia pensado por essa perspectiva!
| ; | ||
| } | ||
| } | ||
|
|
There was a problem hiding this comment.
Essas suas funções withdramFromBalance e withdrawmfromLimit, pensei que poderiam ser a mesma withdraw: saque & saldo disponiveis; saldo indisponivel & saque com limite disponível, mas de qualquer forma, acho que a lógica que você usou foi para mostrar o limite restante e atual, o que faz sentido e também não havia sido pedido no exercício. Bem bacana
There was a problem hiding this comment.
eu pensei a mesma coisa! mas não tive tempo de fazer sendo a mesma função e também fiquei meio na dúvida se seria a melhor prática, mas obrigada pela observação
| describe ("function bank", () => { | ||
| //teste para ver se a conta é um objeto | ||
| it("expects the account to be an object", () => { | ||
| expect(isObject(contaBancaria)).toBe(true) |
There was a problem hiding this comment.
achei interessante a função e o testepara a verificação da conta ser um objeto, po que você pensou em fazer isso? fiquei curiosa sobre! 🦝
There was a problem hiding this comment.
no começo do exercício dizia que a conta tinha que ser um objeto, aí eu achei que era uma necessidade, pensei que podia ser um banco de dados com dados mal colocados
| }) | ||
| // teste para ver se o if que verifica se existe saldo suficiente para o saque funciona | ||
| it("should show a message and denial the withdrawal if the balance is insufficient", () => { | ||
| expect(withDrawalFromBalance(contaBancaria, 1200)).toEqual("O saque de 1200 não é possível, seu saldo é de apenas 1000") // pode melhorar pra passar as variáveis e não a frase inteira |
There was a problem hiding this comment.
Eu ia comentar exatamente o que você deixou aí, que poderia ter sido colocado um output mesmo, ou então ter usado os valores como template strings, não?
Marianaliima
left a comment
Marianaliima
left a comment
There was a problem hiding this comment.
olá tudo bem? parabéns pela entrega do desafio. Alguns pontos de melhoria, será que seria preciso tantas funções ? Outra coisa, cuidado com comentários no código lembra do que foi dito em sala o código deve ser escrito de forma a ser entendido sem ser necessários comentários.
3 participants
Resolução do exercício.
Criei os testes em um arquivo de testes e um objeto de conta e funções em um arquivo js.
Meu pull não pode automatically merge porque fui fazendo comentários no código para minha compreensão ao longo da aula.