Review

[rafaelbcerri]

Poxa vida 😢 as const estavam tão bem no outro, o que aconteceu com elas aqui?

Todas essas aqui abaixo, poderiam ser const:

https://github.com/solangealmn/validCardLib/blob/787a1f6ae626fff7fd5d146ae5422310f8dcd331/index.js#L8
https://github.com/solangealmn/validCardLib/blob/787a1f6ae626fff7fd5d146ae5422310f8dcd331/index.js#L12
https://github.com/solangealmn/validCardLib/blob/787a1f6ae626fff7fd5d146ae5422310f8dcd331/index.js#L15
https://github.com/solangealmn/validCardLib/blob/787a1f6ae626fff7fd5d146ae5422310f8dcd331/index.js#L16
https://github.com/solangealmn/validCardLib/blob/787a1f6ae626fff7fd5d146ae5422310f8dcd331/index.js#L19

---

Achei show o uso do reduce, mas preciso te contar uma coisa... O reduce não é muito amigo do for não, então vamos mudar esse for para um map. Para todos ficarem felizes 🤓

https://github.com/solangealmn/validCardLib/blob/787a1f6ae626fff7fd5d146ae5422310f8dcd331/index.js#L14-L18

---

O sum % 10 === 0 já me retorna true e false, então a gente não precisa do ternário aqui:

return sum % 10 === 0;

https://github.com/solangealmn/validCardLib/blob/787a1f6ae626fff7fd5d146ae5422310f8dcd331/index.js#L20

[solangenalao]

Nossa que legal esse review! Como eu só fui ver isso agora???😮 Muito obrigada Rafa! Vou trocar pode deixar, só vou terminar essa SPA e aí eu arrumo, sim!🤓 Muito obrigada mesmo pelo feedback! Em seg, 17 de set de 2018 às 23:50, Rafael Cerri <notifications@github.com> escreveu:

…

Poxa vida 😢 as const estavam tão bem no outro, o que aconteceu com elas aqui? Todas essas aqui abaixo, poderiam ser const: https://github.com/solangealmn/validCardLib/blob/787a1f6ae626fff7fd5d146ae5422310f8dcd331/index.js#L8 https://github.com/solangealmn/validCardLib/blob/787a1f6ae626fff7fd5d146ae5422310f8dcd331/index.js#L12 https://github.com/solangealmn/validCardLib/blob/787a1f6ae626fff7fd5d146ae5422310f8dcd331/index.js#L15 https://github.com/solangealmn/validCardLib/blob/787a1f6ae626fff7fd5d146ae5422310f8dcd331/index.js#L16 https://github.com/solangealmn/validCardLib/blob/787a1f6ae626fff7fd5d146ae5422310f8dcd331/index.js#L19 ------------------------------ Achei show o uso do reduce, mas preciso te contar uma coisa... O reduce não é muito amigo do for não, então vamos mudar esse for para um map. Para todos ficarem felizes 🤓 https://github.com/solangealmn/validCardLib/blob/787a1f6ae626fff7fd5d146ae5422310f8dcd331/index.js#L14-L18 ------------------------------ O sum % 10 === 0 já me retorna true e false, então a gente não precisa do ternário aqui: return sum % 10 === 0; https://github.com/solangealmn/validCardLib/blob/787a1f6ae626fff7fd5d146ae5422310f8dcd331/index.js#L20 — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub <#1>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AP9UuvKeBub5rmK8lawnZ1jrM9CTyR3mks5ucF97gaJpZM4WtEbL> .

-- *Solange NalãoDesigner Digital11 7662.7350*