Code review
Upcoming SlideShare
Loading in...5
×

Like this? Share it with your network

Share
  • Full Name Full Name Comment goes here.
    Are you sure you want to
    Your message goes here
    Be the first to comment
    Be the first to like this
No Downloads

Views

Total Views
644
On Slideshare
644
From Embeds
0
Number of Embeds
0

Actions

Shares
Downloads
2
Comments
0
Likes
0

Embeds 0

No embeds

Report content

Flagged as inappropriate Flag as inappropriate
Flag as inappropriate

Select your reason for flagging this presentation as inappropriate.

Cancel
    No notes for slide

Transcript

  • 1. Code  Review   Frank  Santos   facebook.com/aitproeg  
  • 2. Quem  faz  revisões  de   código?  
  • 3. O  TIME  
  • 4.                                                            Tom  –  “Eu  preciso    disso  agora!!!”                                                                                                                                          Proprietário.                                                                        Harry  –  “Apenas  faça  isso!!!”                                                                                                                                                                      Gerente                                                              Adam  –  “O  codificador  da  noite  !!!”                                                                                                                                Desenvolvedor  
  • 5.                                                    Kris  –  “Hackety  Hack!!!”                                                                                              Mestre  revisor    de  código                                                      Bruno  –  “Ele  vai  trabalhar!!!”                                                                                                  Revisor  aprendiz  
  • 6.              Quanto  tempo  precisamos  para     começar             e  ter  este  projeto  fei       Bem,  design,  codificação,  código,                           Será  que  realmente  precisamos  de  revisão  do         código?  Você  certamente  sabe  como       codificar,  como  testar  e  como  funciona  ...             Certo?  
  • 7.                                                            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.  
  • 8.                                                        Olá  Harry,                    Eu  preciso  de  John  para  rever                            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!  
  • 9.         Nós  fazemos  todas  essas                       Gastamos  muito  tempo  nisso!!                 E  o  código  que  impulsiona  a  produção  ainda  está  dando  buggy.             É  uma  perda  de  tempo....  
  • 10. 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  
  • 11. Sistemas  de  rastreamento  de  bugs  •  JIRA  •  Bugtrak  •  ManBs  Ferramentas  de  análise  dos    códigos  •  Crucible/Fisheye  •  Gerrit  •  Github  
  • 12. 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   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  
  • 13.                                      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.  
  • 14. 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   Adam  -­‐    O  desenvolvedor   Para    Kris  O  Avaliador   Kris,   Ah  sim.  Desculpe.  Está  no  ramo  principal.   -­‐-­‐-­‐   Adam  
  • 15. O  propósito  da  Revisão   O  que  ???  Por  que?????  
  • 16. 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;  
  • 17. 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  
  • 18. 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  
  • 19. 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    
  • 20. 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  
  • 21. Code  Reviewers  •  A  única  autoridade  verdadeira  deriva  de   conhecimento,  não  de  posição  •   Cribquem  códigos  ao  invés  de  pessoas  
  • 22. 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  
  • 23. Sebas=an    Marek,  SoAware  Architect