silo - code review

Download SILO - Code Review

Post on 09-Apr-2017

113 views

Category:

Documents

1 download

Embed Size (px)

TRANSCRIPT

Folie 1

Code ReviewA favor de um cdigo melhorado e revisado constantemente

de 33www.centralit.com.br | valdemar.junior@centralit.com.br

Code Review

de 33www.centralit.com.br | valdemar.junior@centralit.com.br

Code ReviewCode Review um exame sistemtico, frequentemente conhecido por Peer Review, de cdigos fonte.

de 33www.centralit.com.br | valdemar.junior@centralit.com.br

Por que fazer Code Review?

de 33www.centralit.com.br | valdemar.junior@centralit.com.br

Por que fazer Code Review?

de 33www.centralit.com.br | valdemar.junior@centralit.com.br

Por que fazer Code Review?Encontrar e corrigir erros que passam desapercebidos na fase de desenvolvimento.

de 33www.centralit.com.br | valdemar.junior@centralit.com.br

Por que fazer Code Review? Aumentar a qualidade do cdigo fonte. Aumentar os skills dos desenvolvedores com exemplos de cdigos e corrigindo defeitos nele.

de 33www.centralit.com.br | valdemar.junior@centralit.com.br

Por que fazer Code Review? Identificar defeitos que no implementam os requisitos. Cdigos que podem ser melhorados (Mais legvel, cdigos duplicados, performance)

de 33www.centralit.com.br | valdemar.junior@centralit.com.br

Por que fazer Code Review? Ajudar programadores menos experientes, aos quais o cdigo est sendo examinado, a aprenderem novas tcnicas de programao.

de 33www.centralit.com.br | valdemar.junior@centralit.com.br

Por que fazer Code Review? Compartilhar conhecimento. Melhorar cdigo coletivo. Tentar outras abordagens. Educar. Ajudar a manter o nvel de consistncia de design e implementao.

de 33www.centralit.com.br | valdemar.junior@centralit.com.br

Por que no fazer Code Review? Conflito interpessoal. Efeito Ego. Achar o procedimento muito chato.

de 33www.centralit.com.br | valdemar.junior@centralit.com.br

Como Code Review feito?

de 33www.centralit.com.br | valdemar.junior@centralit.com.br

Como Code Review feito? Inspeo formal (meetings) Lightweight code review

de 33www.centralit.com.br | valdemar.junior@centralit.com.br

Lightweight Code Review Over-the-shoulder Pair Programming Tool-assisted code review (Pastebins e IRC)

de 33www.centralit.com.br | valdemar.junior@centralit.com.br

Como Code Review feito?Num caso de estudo publicado no livro Best Kept Secrets of Peer Code Review, descobriram que a reviso informal de cdigos to eficiente quanto a formal, sendo mais rpida e com custo-benefcio melhor.

de 33www.centralit.com.br | valdemar.junior@centralit.com.br

Ferramentas de Code ReviewSo sistemas que ajudam programadores a encontrarem erros e bugs em cdigos fonte.

de 33www.centralit.com.br | valdemar.junior@centralit.com.br

Ferramentas de Code Review Ferramentas automticas de anlise. Ferramentas manuais de anlise.

de 33www.centralit.com.br | valdemar.junior@centralit.com.br

Ferramentas automticas Analisam cdigo fonte para verificar se est de acordo com regras pr-definidas ou com as boas prticas. (SONAR, PMD).

de 33www.centralit.com.br | valdemar.junior@centralit.com.br

Ferramentas manuais So ferramentas colaborativas para inspecionar e discutir mudanas, armazenando o histrico para futuras referncias.

de 33www.centralit.com.br | valdemar.junior@centralit.com.br

Ferramentas de Code Review

de 33www.centralit.com.br | valdemar.junior@centralit.com.br

Roles e responsabilidades Revisor (Reviewers) Desenvolvedor.

de 33www.centralit.com.br | valdemar.junior@centralit.com.br

Dicas para os desenvolvedores Crie um checklist de coisas que os revisores iro focar mais. Voc no seu cdigo. Entenda que voc pode cometer erros. No importa o quanto sabe, algum sempre saber mais que voc.

de 33www.centralit.com.br | valdemar.junior@centralit.com.br

O trabalho do revisor encontrar defeitos, no leve isso para o pessoal

Dicas para os Revisores Critique cdigo ao invs de pessoas. Seja gentil. Trate pessoas que sabem menos que voc com respeito e pacincia. Voc autoridade pelo conhecimento e no pela cargo ou posio.

de 33www.centralit.com.br | valdemar.junior@centralit.com.br

O trabalho do revisor encontrar defeitos, no leve isso para o pessoal

Dicas para os Revisores Lembre-se de elogiar. Tenha certeza que voc tem conhecimento suficiente para analisar e sugerir melhorias. No tente acelerar/apressar uma reviso, faa de forma eficiente e eficaz.

de 33www.centralit.com.br | valdemar.junior@centralit.com.br

O trabalho do revisor encontrar defeitos, no leve isso para o pessoal

Teddy Bear Technique

de 33www.centralit.com.br | valdemar.junior@centralit.com.br

Teddy Bear TechniqueAparentemente o nome veio de uma universidade onde uma pessoa colocou um urso perto de um computador em um laboratrio de computao.

de 33www.centralit.com.br | valdemar.junior@centralit.com.br

Teddy Bear TechniqueEstudantes tinham que explicar o problema para o Teddy antes que eles pudessem perguntar aos professores.

de 33www.centralit.com.br | valdemar.junior@centralit.com.br

Teddy Bear TechniquePor que? Porque o Teddy frequentemente resolvia os problemas/dvidas deles.

de 33www.centralit.com.br | valdemar.junior@centralit.com.br

Como organizar o Code Review? Usar branches separados no Sistema controle de versionamento (VCS). Usar patches. Ferramentas manuais (Colaborativas).

de 33www.centralit.com.br | valdemar.junior@centralit.com.br

Como comear? Tome decises comuns. Comece devagar. Selecione somente mdulos mais complicados. Inspecione e adapte. Use ferramentas automticas de anlise

de 33www.centralit.com.br | valdemar.junior@centralit.com.br

Livros

de 33www.centralit.com.br | valdemar.junior@centralit.com.br

REGRA DOS ESCOTEIROSDeixe a rea do acampamento mais limpa do que como voca encontrou.

de 33www.centralit.com.br | valdemar.junior@centralit.com.br

Obrigado!E-MailValdemar.junior@centralit.com.br

de 33www.centralit.com.br | valdemar.junior@centralit.com.br