code review

Code Review

Frank Santos facebook.com/aitproeg

Quem faz revisões de código?

O TIME

Tom – “Eu preciso disso agora!!!”

Proprietário.

Harry – “Apenas faça isso!!!”

Gerente

Adam – “O codificador da noite !!!”

Desenvolvedor

Kris – “Hackety Hack!!!” Mestre revisor de código

Bruno – “Ele vai trabalhar!!!”

Revisor aprendiz

Quanto tempo precisamos para começar e ter este projeto feito?

Bem, design, codificação, código, opiniões testes, ...

Será que realmente precisamos de revisão do

código? Você certamente sabe como codificar, como testar e como funciona ... Certo?

Estamos quase prontos, só precisa ser revisto este código.

Hmmm ... todos os desenvolvedores estão ocupados, não temos um sobresselente.

Vamos ignorar essa parte e seguir em frente.

Olá Harry,

Eu preciso de John para rever o meu código.

Johh está ocupado, mas tem o Rob.

Mas Rob é um desenvolvedor júnior, e ele não sabe este sistema.

Você quer ou não quer o seu código revisado?

Rob é tudo o que temos!

Nós fazemos todas essas revisões de código.

Gastamos muito tempo nisso!!

E o código que impulsiona a produção ainda está dando buggy. É uma perda de tempo....

Revisão de Código

Adam -‐ O desenvolvedor Para Kris O Avaliador

Kris,

Eu tenho esse código que eu preciso que você revise ele pra mim!!! Você pode fazer isso por mim, por favor?

O código está no meu repositório. obrigado -‐-‐-‐ Adam

Sistemas de rastreamento de bugs • JIRA • Bugtrak • ManBs

Ferramentas de análise dos

códigos

• Crucible/Fisheye • Gerrit • Github

Revisão de Código


Kris, Eu tenho esse código que eu preciso que você revise ele pra mim!!! Você pode fazer isso por mim, por favor?

O código está no meu repositório. obrigado -‐-‐-‐ Adam

Kris -‐ O Avaliador Para Adam O Desenvolvedor

Adam, Nenhum problema , mas onde você um braço do código? Eu não consigo idenbficar o conjunto de mudanças sem ele. -‐-‐-‐ Kris

O que analisar?

• Ter um controle da versão:

– Especificar o conjunto de mudanças para facilitar o entendimento da proposta de desenvolvimento ou das mudanças.

Revisão de Código

Adam -‐ O desenvolvedor Kris, Eu tenho esse código que eu preciso que você revise ele pra mim!!! (...)

Kris -‐ O Avaliador Para Adam O Desenvolvedor

Adam, Nenhum problema , mas onde você um braço do código? Eu não consigo idenbficar o conjunto de mudanças sem ele. -‐-‐-‐ Kris


Kris, Ah sim. Desculpe. Está no ramo principal. -‐-‐-‐ Adam

O propósito da Revisão

O que ???

Por que?????

Uma forma de fazer as coisas:

Bruno – Ele vai trabalhar!!! Revisor aprendiz

Comentário habitual produzido:

• Isso faz senbdo; • O que foi feito ou alterado; • Sintabcamente correto; • Aprovado ou não;

Uma forma de fazer as coisas:

Kris – Hackety Hack!!! Mestre revisor de código

Ferramentas Usadas: – PHP linter – PHP Code Sniffer – PHPUnit – PHP Documentor – PHP Depend – PHP Mess Detector

Olhando para coisas importantes

Kris – Hackety Hack!!! Mestre revisor de código

Coisas verificadas: – clareza – desempenho – complexidade excessiva – o impacto em outros sistemas – se a solução resolve o problema – duplicações – qualidade de código – potenciais problemas de implantação – falhas de projeto

Os benejcios de uma revisão do código -‐ eles são para você!

Comparblhamento de conhecimento – Encontrar bugs / falhas de projeto cedo – Melhorar a qualidade geral do código – Fomentar a parbcipação colebva do código

Desenvolvedor

Compreender e aceitar que você vai cometer erros.

• Você não é o seu código.

• Não importa o quanto você sabe, alguém vai sempre saber mais. Uhauhauahuahuauha

• Não reescrever o código sem consulta

Code Reviewers

• A única autoridade verdadeira deriva de conhecimento, não de posição

• Cribquem códigos ao invés de pessoas

Como realizar uma boa revisão de código?

• Use ferramentas, não seja uma ferramenta

• Verifique se há duplicações / complexidade

• Veja o impacto em outros sistemas

• Tornar o código claro e auto-‐describvo

Sebas=an Marek, SoAware Architect

code review

Technology