Histórico da Página

Esse é um checklist rápido e prático para ser usado no dia a dia.

1. Contexto e escopo

 O propósito da mudança está claramente descrito no PR (Pull Request)? (entenda a issue/tarefa relacionada antes de começar)

...

 Métodos não estão excessivamente longos ou complexos?

 Resources foram aplicados para tratamento de traduções?

4. Segurança

 Todas as entradas de usuário são validadas antes do uso?

...

 Logs não incluem informações sensíveis (dados pessoais, senhas, etc.)?

Usar ferramenta de monitoramento de consumo de memória em modo debug para evitar vazamento. (Visual Studio Diagnostic Tools menu: Debug > Windows > Show Diagnostic Tools).
 
5. Performance e boas práticas

 Consultas e loops são eficientes (sem N+1, sem operações desnecessárias)?

 Verificar sempre que possível, a substituição de DataSet, DataRow e DataView para classes ou recursos mais performáticos e atuais.

 Uso correto de async/await (evita .Result ou .Wait() bloqueantes)?

...

 O PR foi aprovado por pelo menos outro desenvolvedor?

Objetivos principais do code review

• Melhorar a qualidade do software — reduzir bugs e dívidas técnicas.

• Garantir padrões de código — manter o estilo e a arquitetura coerentes.

• Compartilhar conhecimento — aumentar o entendimento coletivo do sistema.

• Mentoria e aprendizado — ajudar desenvolvedores menos experientes a crescer.

• Aumentar a segurança — detectar vulnerabilidades cedo.

• Melhorar a colaboração — promover discussões técnicas construtivas.