Code review

599 visualizações

Publicada em

Apresentação sobre Code Review

Publicada em: Tecnologia
0 comentários
0 gostaram
Estatísticas
Notas
  • Seja o primeiro a comentar

  • Seja a primeira pessoa a gostar disto

Sem downloads
Visualizações
Visualizações totais
599
No SlideShare
0
A partir de incorporações
0
Número de incorporações
2
Ações
Compartilhamentos
0
Downloads
6
Comentários
0
Gostaram
0
Incorporações 0
Nenhuma incorporação

Nenhuma nota no slide

Code review

  1. 1. Code  Review   Frank  Santos   facebook.com/aitproeg  
  2. 2. Quem  faz  revisões  de   código?  
  3. 3. O  TIME  
  4. 4.                                                            Tom  –  “Eu  preciso    disso  agora!!!”                                                                                                                                          Proprietário.                                                                        Harry  –  “Apenas  faça  isso!!!”                                                                                                                                                                      Gerente                                                              Adam  –  “O  codificador  da  noite  !!!”                                                                                                                                Desenvolvedor  
  5. 5.                                                    Kris  –  “Hackety  Hack!!!”                                                                                              Mestre  revisor    de  código                                                      Bruno  –  “Ele  vai  trabalhar!!!”                                                                                                  Revisor  aprendiz  
  6. 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. 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. 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. 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. 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. 11. Sistemas  de  rastreamento  de  bugs  •  JIRA  •  Bugtrak  •  ManBs  Ferramentas  de  análise  dos    códigos  •  Crucible/Fisheye  •  Gerrit  •  Github  
  12. 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. 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. 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. 15. O  propósito  da  Revisão   O  que  ???  Por  que?????  
  16. 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. 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. 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. 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. 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. 21. Code  Reviewers  •  A  única  autoridade  verdadeira  deriva  de   conhecimento,  não  de  posição  •   Cribquem  códigos  ao  invés  de  pessoas  
  22. 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. 23. Sebas=an    Marek,  SoAware  Architect  

×