Ellie Konuma 28/06 home assignment #21
Conversation
| { | ||
| "name": "Isa", | ||
| "balance": 1000, | ||
| "limit": 1000 |
There was a problem hiding this comment.
gostei que você fez um arquivo diferente em Json para as contas! Eu tinha feito uma constante no próprio arquivo das funções só
| return acc ? { ok: acc } : { error: "account not found" }; | ||
| }; | ||
|
|
||
| const hasError = (res) => (Object.keys(res)[0] === "error" ? true : false); |
There was a problem hiding this comment.
gostei dessa função de erro! não tinha pensado nisso
| @@ -0,0 +1,99 @@ | |||
| const findAccount = (name, accounts) => { | |||
| const acc = accounts.find((acc) => acc.name == name); | |||
There was a problem hiding this comment.
aqui minha única sugestão é usar o === para a busca ser mais restrita, caso tenha alguma questão de tipo de variável e usar um filter pq duas pessoas podem ter o mesmo nome (não no caso do seu json, mas pensando de forma abrangente)
|
|
||
| let { balance } = res.ok; | ||
|
|
||
| return { ok: balance }; |
There was a problem hiding this comment.
eu nem sei se isso é uma boa prática de código, mas vejo aqui a possibilidade de fazer uma única função tipo checkInformation para verificar o balance e o limit na mesma função
| let { limit } = res.ok; | ||
|
|
||
| if (limit == null) return { error: "limit not active" }; | ||
| if (limit === 0) return { error: "limit not available" }; |
There was a problem hiding this comment.
achei bem legal como vc fez tratamento de erro sempre
|
|
||
| describe("check if result has error", () => { | ||
| it("should be true", () => { | ||
| expect(hasError({ error: "bubla" })).toEqual(true); |
| { | ||
| name: "Isa", | ||
| balance: 1000, | ||
| limit: 1000, |
There was a problem hiding this comment.
muito legal que vc usou o beforeEach(), vendo o seu vejo que podia ter usado do mesmo jeito
Marianaliima
left a comment
Marianaliima
left a comment
There was a problem hiding this comment.
Olá tudo bem? seu código ficou muito bem construído, parabéns. Uma dica vi que comentou o beforeEach não seria melhor utilizar para criar um código menos verboso? também vi alguns comentários no código fique atenta a essa questão, mas de novo parabéns pela entrega
3 participants
🤣👌