Code	  Review	             Frank	  Santos	   facebook.com/aitproeg	  
Quem	  faz	  revisões	  de	          código?	  
O	  TIME	  
 	  	  	  	  	  	  	  	  	  	  	  	  	  	  	  	  	  	  	  	  	  	  	  	  	  	  	  	  	  Tom	  –	  “Eu	  preciso	  	  disso...
 	  	  	  	  	  	  	  	  	  	  	  	  	  	  	  	  	  	  	  	  	  	  	  	  	  Kris	  –	  “Hackety	  Hack!!!”	  	  	  	  	  	...
 	     	   	  	         	   	  Quanto	  tempo	  precisamos	  para	  	         começar	  	   	    	      	   	       e	  te...
 	                	                    	  	  	  	  	  	  	  	  	  	  	  	  	  	  	  	  	  	  	  	  	  	  	  	  	  	  	  Es...
 	  	  	  	  	  	  	  	  	  	  	  	  	  	  	  	  	   	   	   	                        	  	  	  	  	  	  	  Olá	  Harry,	  ...
          	          	      	      Nós	  fazemos	  todas	  essas	  	   	     	  	           	          	      	  	        ...
Revisão	  de	  Código	                 Adam	  -­‐	  	  O	  desenvolvedor	                 Para	  	  Kris	  O	  Avaliador	 ...
Sistemas	  de	  rastreamento	  de	  bugs	  •  JIRA	  •  Bugtrak	  •  ManBs	  Ferramentas	  de	  análise	  dos	  	  códigos...
Revisão	  de	  Código	            Adam	  -­‐	  	  O	  desenvolvedor	            Para	  	  Kris	  O	  Avaliador	           ...
 	  	  	  	  	  	  	  	  	  	  	  	  	  	  	  	  	  	  O	  que	  analisar?	                                         •  Ter...
Revisão	  de	  Código	                Adam	  -­‐	  	  O	  desenvolvedor	                Kris,	  Eu	  tenho	  esse	  código...
O	  propósito	  da	  Revisão	     O	  que	  ???	  Por	  que?????	  
Uma	  forma	  de	  fazer	  as	  coisas:	                                                                   	  	  	  	  	  ...
Uma	  forma	  de	  fazer	  as	  coisas:	  	                                        	                                      ...
Olhando	  para	  coisas	  importantes	  	                                             	                                   ...
Os	  benejcios	  de	  uma	  revisão	  do	        código	  -­‐	  eles	  são	  para	  você!	  Comparblhamento	  de	  conheci...
Desenvolvedor	         Compreender	  e	  aceitar	  que	  você	  vai	                 cometer	  erros.	  •	  Você	  não	  é...
Code	  Reviewers	  •  A	  única	  autoridade	  verdadeira	  deriva	  de	     conhecimento,	  não	  de	  posição	  •  	  Cr...
Como	  realizar	  uma	  boa	  revisão	  de	                     código?	  •  Use	  ferramentas,	  não	  seja	  uma	  ferra...
Sebas=an	  	  Marek,	  SoAware	  Architect	  
Code review
Code review
Code review
Upcoming SlideShare
Loading in...5
×

Code review

362

Published on

Apresentação sobre Code Review

Published in: Technology
0 Comments
0 Likes
Statistics
Notes
  • Be the first to comment

  • Be the first to like this

No Downloads
Views
Total Views
362
On Slideshare
0
From Embeds
0
Number of Embeds
0
Actions
Shares
0
Downloads
3
Comments
0
Likes
0
Embeds 0
No embeds

No notes for 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  
  1. A particular slide catching your eye?

    Clipping is a handy way to collect important slides you want to go back to later.

×