Globalcode – Open4education
Décio Heinzelmann Luckow
Desenvolvedor - Neogrid
Design de Código e XP
Existem informações valiosas escondidas
nos code reviews do seu time
Globalcode – Open4education
O Décio!
Desde 2015 na Neogrid (desde 2000 programando)
HTML é programação? Então desde 1998
Produto Fiscal (emissão de NFe, CTe, etc...)
Foco em Java Back-end
(Desenvolvimento, atendimento em último nível, investigação de incidentes, análise de performance)
Incentivador de code review, clean code, design patterns e
compartilhamento de conhecimento
Globalcode – Open4education
Revisando o código alheio
(since 2014)
-1
-1
-1
-20
-1
-1 -1
-1
-1
-1
-10
-1000
-1
-1
-1
-1
-1 Já existe método
fazendo isso, veja na
classe XPTOService
O nome do método não
está condizente com o
código. Sugiro usar....
Este método está
muito grande
Crie um teste unitário, é um caso
bem interessante e necessário
Extraia este método
para uma classe
própria
Globalcode – Open4education
O que se encontra num
code review
Bibliotecas
utilitárias
Conhecimentos das classes
e estrutura do projeto
Dicas de programação
e desempenho
Reutilização de
código
Clean code
Design patterns
Globalcode – Open4education
Como também poderia ser
Esta classe torna um monte de
IFs por ai desnecessários
Boa mudança, o
garbagecollector agradece
Que interessante, nunca
tinha usado este método
Meu primeiro design pattern
Ficou muito melhor este nome
do método
Já existe um método no guava
que faz exatamente isso.
Globalcode – Open4education
Como também poderia ser
Esta classe torna um monte de
IFs por ai desnecessários
#divulgar
Boa mudança, o
#garbagecollector agradece
Que #interessante, nunca
tinha usado este método
Meu primeiro #design-pattern
Ficou muito melhor este nome
do método #cleancode
Já existe um método no
#guava que faz exatamente
isso. #reaproveitamento
Globalcode – Open4education
Como buscar todos os
#destaques?
Globalcode – Open4education
Os primórdios
+ = TXT
+ API = TXT
Globalcode – Open4education
O Divulgit
https://github.com/decioluckow/divulgit
Globalcode – Open4education
Globalcode – Open4education
Globalcode – Open4education
Globalcode – Open4education
Globalcode – Open4education
Reunião Técnica
#
#
#
#
#
#
#
#
#
#
#
#
#
#
#
#
#
#
Coffee Tech
Tech Talk
Reunião
Pizza Meeting
Coffee Coffee
Pizza Tech
Globalcode – Open4education
Reunião Técnica
 Grupo pequeno: somente os devs que trabalham juntos
 Não inibe a participação de ninguém
 Recorrência fixa
+ - O que está estudando em casa
- Material interessante que descobriu
- Qual livro está lendo
- Projeto/estudo pessoal
- Sempre tem assunto relevante e
contextualizado para o time
Pequenos
Assuntos
Aleatórios
Globalcode – Open4education
Casos reais de
#destaques
Globalcode – Open4education
Globalcode – Open4education
Globalcode – Open4education
Globalcode – Open4education
Globalcode – Open4education
Globalcode – Open4education
Boas Práticas
Aquele que marcou com #, é o
responsável por puxar o assunto.
Ou se destacou, mas não tem o domínio
do assunto, combinar com quem tem.
Globalcode – Open4education
Boas Práticas
É importante acesso o DivulGit antes do encontro
só pra lembrar o contexto do comentário
Globalcode – Open4education
Boas Práticas
Incentivar no dia a dia a apresentação de
assuntos externos.
Com o tempo os desenvolvedores passam a ter
assuntos preparados e acumulados
previamente.
Globalcode – Open4education
Cuidados
Exposição de erros é um assunto delicado.
 Foco na solução e não no problema.
 A discussão ocorre dentro de um grupo
pequeno (que é o circulo diário e de
confiança).
 No momento que o erro for discutido, a
solução já foi realizada.
Globalcode – Open4education
O que esperar (ou não)
Não espere 100% de adesão
Sempre existirão as pessoas
que participarão ativamente do
processo, as que só ouvem e
as que não participam.
Globalcode – Open4education
O que esperar (ou não)
Não espere que seja fácil
Temos muito mais facilidade
para enxergar defeitos do
que enxergar qualidades.
Globalcode – Open4education
Ganhos para o time
Compartilhamento de conhecimento
Mostrar seu trabalho sem ter que esperar por
grandes entregas
Exposição até dos tímidos e iniciantes
Valorização do profissional
Globalcode – Open4education
Existem informações valiosas
escondidas nos code-reviews do
seu time
pessoas
Globalcode – Open4education
Às vezes, quando considero as tremendas consequências advindas das
pequenas coisas ... sou tentado a pensar ... não existem pequenas coisas.
Bruce Barton
https://www.linkedin.com/in/decioluckow/
https://medium.com/@decioluckow
decioluckow@gmail.com
Obrigado!
https://github.com/decioluckow/divulgit
Existem informações valiosas escondidas nos code-reviews do seu time

Existem informações valiosas escondidas nos code-reviews do seu time

Notas do Editor

  • #4 Basicamente temos duas formas de fazer um code view, comentando -1 no código no lugar onde está o problema e o desenvolvedor tem que se virar pra descobrir o que exatamente está errado, ou comentar de forma amigável qual o problema sugerir uma melhoria. As suas formas apontam problema, o que é necessário. Quanto eu trouxe a pratica de code review pra neogrid, eu nem contei pra ninguém que poderia simplesmente escrever -1, sempre incentivei uma linguagem mais amigável.
  • #13 # incluir imagem adiciona com o merge request sendo aberto
  • #25 Neste caso tivemos devs juniors falando sobre design patterns, testadores exploratórios e experientes mostrando os caminhos que estão tomando para aprender a programa