Review
Closed this issue · 1 comments
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 🤓
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;