Code review checklist

code-review.md

Code Review

Checklist

• Names (Variables, Functions, Class)
• Loops com complexidade linear O(n) ao invés de O(n²)
• Queries sequenciais sem dependência
• Criação de índice somente com concurrently: true
• Adicionar coluna com defaultValue mesmo que seja null para evitar lock.
• Nomenclatura sucinta sem perder significado
• Não obter todos os atributos de candidates (pois tem hash da senha)
• Não retornar diretamente do repositório sem model.toJSON()
• Não utilizar magic number
• Utilizar enums quando houver múltiplos valores possíveis
• Utilizar constantes ao invés de valores soltos sem contexto
• Evitar utilizar promises e usar async/await
• Utilizar FixtureBuilder ao invés de copiar/colar
• Sempre que um objeto passar por um loop verificar se alguma operação foi realizada nele, caso sim então ele tem que mudar de nome para represenatar semanticamente o que significa
• Funções, Classes devem fazer uma coisa
• Tentar evitar loops imperativos (usar declarativo no lugar) exemplo: map, filter, reduce
• Classes devem ter uma coesão alta..ou seja atributos e variaveis devem estar sendo utilizado nos metodos (classes muito inteligentes não tem como ter uma coesão mt alta)