Resolve o que foi pedido?
Antes de qualquer detalhe técnico: a alteração atende à regra combinada, inclusive nas exceções levantadas na descoberta.
Nenhuma alteração entra sem passar pelos olhos de outra pessoa. Não é desconfiança do time — é o mecanismo mais barato que a engenharia conhece para achar um erro antes de ele custar dinheiro.
Depois de duas horas dentro de um problema, o autor lê o que quis escrever. Outra pessoa lê o que está escrito.
Revisão de código é uma pessoa da equipe lendo a alteração de outra antes de ela ser aceita: o que mudou, por quê, se a solução resolve o caso combinado e se ela não abre um buraco em outro lugar. Leva minutos e evita horas — a maior parte dos defeitos caros é achada aqui, não em produção.
Há um segundo efeito, tão importante quanto: o conhecimento se espalha. Quando duas pessoas passaram por cada parte do sistema, ninguém é insubstituível. Isso protege a sua empresa de um risco concreto e comum — o sistema que só uma pessoa entende, e que vira refém de uma agenda, de umas férias ou de um pedido de demissão.
A revisão também é onde o débito técnico deixa de ser invisível. Quando a solução rápida é a certa para o prazo, ela é aceita — mas registrada, com o motivo e o que precisará ser feito depois. É a diferença entre uma dívida que você conhece e uma que só aparece quando já não dá para pagar.
Revisão não é caça a culpado. O alvo é o código, nunca a pessoa — e a regra vale para todo mundo, inclusive para quem tem mais tempo de casa.
Seis frentes, na ordem em que importam.
Antes de qualquer detalhe técnico: a alteração atende à regra combinada, inclusive nas exceções levantadas na descoberta.
O efeito colateral em outra parte do sistema — a integração antiga, o relatório que ninguém abre há meses.
Dado sensível exposto, consulta sem filtro de acesso, entrada de usuário sem validação.
Consulta que funciona com cem registros e trava com cem mil. É mais barato ver agora.
Nome que explica, função curta, ausência de truque esperto. Código é lido muitas vezes mais do que escrito.
Alteração relevante chega acompanhada do teste que a protege. Sem isso, a próxima pessoa quebra sem saber.
Cada linha alterada fica visível, comentável e ligada ao motivo da mudança.
Um ciclo curto, medido em horas e não em dias.
Quem escreveu explica o que mudou e por quê, com link para o item do backlog que originou a mudança.
Outra pessoa lê e comenta na linha exata. Dúvida vira pergunta, não suposição.
Quem escreveu ajusta ou explica a escolha. Divergência que persiste sobe para uma decisão técnica registrada.
Com a revisão aprovada e os testes verdes, a alteração é aceita e segue para a esteira.
Riscos que não aparecem na tela, mas aparecem na fatura.
Regras de convivência que fazem a prática durar.
Casos que aparecem com frequência — todos baratos de corrigir antes, caros depois.
| O que aparece | Por que passa despercebido | O que aconteceria em produção |
|---|---|---|
| Consulta sem filtro de empresa | funciona bem no teste, com uma empresa só | um cliente enxergando dado de outro |
| Cálculo sem arredondamento definido | a diferença é de centavos | divergência acumulada entre o sistema e a contabilidade |
| Consulta dentro de laço | rápido com poucos registros | a tela travando quando a base cresce |
| Erro engolido em silêncio | a tela não mostra nada de errado | falha que ninguém percebe até o dado sumir |
| Regra fixa no código | atende exatamente o pedido de hoje | mudança de alíquota ou de prazo exigindo nova publicação |
O que combinamos antes de começar — e cobramos de nós mesmos durante o projeto.
As dúvidas que mais aparecem nesta etapa. Sobre prazo, forma de cobrança e contrato, veja as perguntas frequentes da YveTech.
A revisão custa minutos por alteração; um defeito que chega em produção custa horas de correção, mais o efeito na sua operação. Na conta do projeto inteiro, revisar é o que sai mais barato — e é por isso que a prática se tornou padrão na indústria.
Outra pessoa da equipe, inclusive quem tem menos tempo de casa. Sênior comete menos erros de sintaxe e mais erros de suposição — que são exatamente os que alguém de fora do problema percebe.
Da revisão de código, não — ela é técnica. O seu papel equivalente é a homologação: usar a entrega do ciclo e dizer se resolve o problema. As duas coisas são conferências, mas de coisas diferentes.
Discutir é parte do trabalho. Quando não há acordo, a decisão sobe para a arquitetura e é registrada com o motivo — assim ela não precisa ser rediscutida a cada mês, e quem chegar depois entende por que o sistema é como é.
Uma auditoria de código mostra o estado real, os riscos e o que dá para aproveitar antes de qualquer decisão.
As quatro etapas do método e os três recortes da operação — cada uma com o seu próprio detalhamento.