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.
12. Sistemas
de
rastreamento
de
bugs
• JIRA
• Bugtrak
• ManBs
Ferramentas
de
análise
dos
códigos
• Crucible/Fisheye
• Gerrit
• Github
13. 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
14.
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.
15. 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
17. 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;
18. 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
19.
20.
21. 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
22. 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
23. 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
24. Code
Reviewers
• A
única
autoridade
verdadeira
deriva
de
conhecimento,
não
de
posição
•
Cribquem
códigos
ao
invés
de
pessoas
25. 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