SILO - Code Review
-
Upload
valdemar-junior -
Category
Documents
-
view
155 -
download
1
Transcript of SILO - Code Review
![Page 1: SILO - Code Review](https://reader035.fdocumentos.com/reader035/viewer/2022081422/58e94aa61a28ab262c8b51ab/html5/thumbnails/1.jpg)
1 de 33www.centralit.com.br | [email protected]
Code ReviewA favor de um código melhorado e revisado constantemente
![Page 3: SILO - Code Review](https://reader035.fdocumentos.com/reader035/viewer/2022081422/58e94aa61a28ab262c8b51ab/html5/thumbnails/3.jpg)
3 de 33www.centralit.com.br | [email protected]
Code ReviewCode Review é um exame sistemático, frequentemente conhecido por Peer Review, de códigos fonte.
![Page 6: SILO - Code Review](https://reader035.fdocumentos.com/reader035/viewer/2022081422/58e94aa61a28ab262c8b51ab/html5/thumbnails/6.jpg)
6 de 33www.centralit.com.br | [email protected]
Por que fazer Code Review?Encontrar e corrigir erros que passam desapercebidos na fase de desenvolvimento.
![Page 7: SILO - Code Review](https://reader035.fdocumentos.com/reader035/viewer/2022081422/58e94aa61a28ab262c8b51ab/html5/thumbnails/7.jpg)
7 de 33www.centralit.com.br | [email protected]
Por que fazer Code Review?l Aumentar a qualidade do código fonte.l Aumentar os skills dos desenvolvedores
com exemplos de códigos e corrigindo defeitos nele.
![Page 8: SILO - Code Review](https://reader035.fdocumentos.com/reader035/viewer/2022081422/58e94aa61a28ab262c8b51ab/html5/thumbnails/8.jpg)
8 de 33www.centralit.com.br | [email protected]
Por que fazer Code Review?l Identificar defeitos que não
implementam os requisitos.l Códigos que podem ser melhorados
(Mais legível, códigos duplicados, performance)
![Page 9: SILO - Code Review](https://reader035.fdocumentos.com/reader035/viewer/2022081422/58e94aa61a28ab262c8b51ab/html5/thumbnails/9.jpg)
9 de 33www.centralit.com.br | [email protected]
Por que fazer Code Review?l Ajudar programadores menos
experientes, aos quais o código está sendo examinado, a aprenderem novas técnicas de programação.
![Page 10: SILO - Code Review](https://reader035.fdocumentos.com/reader035/viewer/2022081422/58e94aa61a28ab262c8b51ab/html5/thumbnails/10.jpg)
10 de 33www.centralit.com.br | [email protected]
Por que fazer Code Review?l Compartilhar conhecimento.l Melhorar código coletivo.l Tentar outras abordagens.l Educar.l Ajudar a manter o nível de consistência
de design e implementação.
![Page 11: SILO - Code Review](https://reader035.fdocumentos.com/reader035/viewer/2022081422/58e94aa61a28ab262c8b51ab/html5/thumbnails/11.jpg)
11 de 33www.centralit.com.br | [email protected]
Por que não fazer Code Review?l Conflito interpessoal.l Efeito “Ego”.l Achar o procedimento muito chato.
![Page 13: SILO - Code Review](https://reader035.fdocumentos.com/reader035/viewer/2022081422/58e94aa61a28ab262c8b51ab/html5/thumbnails/13.jpg)
13 de 33www.centralit.com.br | [email protected]
Como Code Review é feito?l Inspeção formal (meetings)l Lightweight code review
![Page 14: SILO - Code Review](https://reader035.fdocumentos.com/reader035/viewer/2022081422/58e94aa61a28ab262c8b51ab/html5/thumbnails/14.jpg)
14 de 33www.centralit.com.br | [email protected]
Lightweight Code Reviewl Over-the-shoulderl Pair Programmingl Tool-assisted code review (Pastebins e
IRC)
![Page 15: SILO - Code Review](https://reader035.fdocumentos.com/reader035/viewer/2022081422/58e94aa61a28ab262c8b51ab/html5/thumbnails/15.jpg)
15 de 33www.centralit.com.br | [email protected]
Como Code Review é feito?l Num caso de estudo publicado no livro
Best Kept Secrets of Peer Code Review, descobriram que a revisão informal de códigos é tão eficiente quanto a formal, sendo mais rápida e com custo-benefício melhor.
![Page 16: SILO - Code Review](https://reader035.fdocumentos.com/reader035/viewer/2022081422/58e94aa61a28ab262c8b51ab/html5/thumbnails/16.jpg)
16 de 33www.centralit.com.br | [email protected]
Ferramentas de Code Reviewl São sistemas que ajudam programadores
a encontrarem erros e bugs em códigos fonte.
![Page 17: SILO - Code Review](https://reader035.fdocumentos.com/reader035/viewer/2022081422/58e94aa61a28ab262c8b51ab/html5/thumbnails/17.jpg)
17 de 33www.centralit.com.br | [email protected]
Ferramentas de Code Reviewl Ferramentas automáticas de análise.l Ferramentas manuais de análise.
![Page 18: SILO - Code Review](https://reader035.fdocumentos.com/reader035/viewer/2022081422/58e94aa61a28ab262c8b51ab/html5/thumbnails/18.jpg)
18 de 33www.centralit.com.br | [email protected]
Ferramentas automáticasl Analisam código fonte para verificar se
está de acordo com regras pré-definidas ou com as boas práticas. (SONAR, PMD).
![Page 19: SILO - Code Review](https://reader035.fdocumentos.com/reader035/viewer/2022081422/58e94aa61a28ab262c8b51ab/html5/thumbnails/19.jpg)
19 de 33www.centralit.com.br | [email protected]
Ferramentas manuaisl São ferramentas colaborativas para
inspecionar e discutir mudanças, armazenando o histórico para futuras referências.
![Page 21: SILO - Code Review](https://reader035.fdocumentos.com/reader035/viewer/2022081422/58e94aa61a28ab262c8b51ab/html5/thumbnails/21.jpg)
21 de 33www.centralit.com.br | [email protected]
Roles e responsabilidadesl Revisor (Reviewers)l Desenvolvedor.
![Page 22: SILO - Code Review](https://reader035.fdocumentos.com/reader035/viewer/2022081422/58e94aa61a28ab262c8b51ab/html5/thumbnails/22.jpg)
22 de 33www.centralit.com.br | [email protected]
Dicas para os desenvolvedoresl Crie um checklist de coisas que os
revisores irão focar mais.l Você não é seu código.l Entenda que você pode cometer erros.l Não importa o quanto sabe, alguém
sempre saberá mais que você.
![Page 23: SILO - Code Review](https://reader035.fdocumentos.com/reader035/viewer/2022081422/58e94aa61a28ab262c8b51ab/html5/thumbnails/23.jpg)
23 de 33www.centralit.com.br | [email protected]
Dicas para os Revisoresl Critique código ao invés de pessoas. Seja
gentil. l Trate pessoas que sabem menos que
você com respeito e paciência.l Você é “autoridade” pelo conhecimento
e não pela cargo ou posição.
![Page 24: SILO - Code Review](https://reader035.fdocumentos.com/reader035/viewer/2022081422/58e94aa61a28ab262c8b51ab/html5/thumbnails/24.jpg)
24 de 33www.centralit.com.br | [email protected]
Dicas para os Revisoresl Lembre-se de elogiar.l Tenha certeza que você tem
conhecimento suficiente para analisar e sugerir melhorias.l Não tente acelerar/apressar uma
revisão, faça de forma eficiente e eficaz.
![Page 26: SILO - Code Review](https://reader035.fdocumentos.com/reader035/viewer/2022081422/58e94aa61a28ab262c8b51ab/html5/thumbnails/26.jpg)
26 de 33www.centralit.com.br | [email protected]
Teddy Bear Techniquel Aparentemente o nome veio de uma
universidade onde uma pessoa colocou um urso perto de um computador em um laboratório de computação.
![Page 27: SILO - Code Review](https://reader035.fdocumentos.com/reader035/viewer/2022081422/58e94aa61a28ab262c8b51ab/html5/thumbnails/27.jpg)
27 de 33www.centralit.com.br | [email protected]
Teddy Bear Techniquel Estudantes tinham que explicar o
problema para o Teddy antes que eles pudessem perguntar aos professores.
![Page 28: SILO - Code Review](https://reader035.fdocumentos.com/reader035/viewer/2022081422/58e94aa61a28ab262c8b51ab/html5/thumbnails/28.jpg)
28 de 33www.centralit.com.br | [email protected]
Teddy Bear Techniquel Por que? Porque o Teddy
frequentemente resolvia os problemas/dúvidas deles.
![Page 29: SILO - Code Review](https://reader035.fdocumentos.com/reader035/viewer/2022081422/58e94aa61a28ab262c8b51ab/html5/thumbnails/29.jpg)
29 de 33www.centralit.com.br | [email protected]
Como organizar o Code Review?l Usar branches separados no Sistema
controle de versionamento (VCS).l Usar patches.l Ferramentas manuais (Colaborativas).
![Page 30: SILO - Code Review](https://reader035.fdocumentos.com/reader035/viewer/2022081422/58e94aa61a28ab262c8b51ab/html5/thumbnails/30.jpg)
30 de 33www.centralit.com.br | [email protected]
Como começar?l Tome decisões comuns.l Comece devagar.l Selecione somente módulos mais
complicados.l Inspecione e adapte.l Use ferramentas automáticas de análise
![Page 32: SILO - Code Review](https://reader035.fdocumentos.com/reader035/viewer/2022081422/58e94aa61a28ab262c8b51ab/html5/thumbnails/32.jpg)
32 de 33www.centralit.com.br | [email protected]
REGRA DOS ESCOTEIROSDeixe a área do acampamento mais limpa do que como vocêa encontrou.