Code review

294 visualizações

Publicada em

Publicada em: Tecnologia
0 comentários
0 gostaram
Estatísticas
Notas
  • Seja o primeiro a comentar

  • Seja a primeira pessoa a gostar disto

Sem downloads
Visualizações
Visualizações totais
294
No SlideShare
0
A partir de incorporações
0
Número de incorporações
13
Ações
Compartilhamentos
0
Downloads
2
Comentários
0
Gostaram
0
Incorporações 0
Nenhuma incorporação

Nenhuma nota no slide

Code review

  1. 1. Code Review Rafael Oliveira <rafaelo@take.net>
  2. 2. Antes de revisar... •Check in early, check in often! –Segurança •Se o código não está no repositório, não existe! –O objetivo não é subir código quebrado •Incompleto != Quebrado –Check in com frequência, permite um feedback mais rápido •A revisão do código é menos trabalhosa 2
  3. 3. •DRY “Every piece of knowledge must have a single, unambiguous, authoritative representation within a system.” •KISS "Simplicity is the ultimate sophistication" •YAGNI “Build what you need as you need it.” 3
  4. 4. •Sem comentários pertinentes •‘Comentar’ o teste em busca do ‘green’ –Assert.IsTrue(true); •Sem tratamento de erros 4
  5. 5. •Código mal formatado •Linhas extensas •Métodos enormes •Erros de digitação 5
  6. 6. •Reduzir erros •Garantir os padrões •Aprender •Ensinar •Melhorar a capacidade de manutenção, segurança, documentação, qualidade... Por que revisar? 6
  7. 7. Então, o que revisar? •Funcionalidades –Está dentro do esperado pela atividade? –A lógica está correta? •Design –Baixo acoplamento e alta coesão? –A classe está muito grande ou muito complexa? 7
  8. 8. Então, o que revisar? •Estilo de código –Atende aos padrões definidos pela empresa? –Tem código duplicado? •Nomenclatura –Nomes coerentes e consistentes? –Erros de digitação, idioma? 8
  9. 9. Então, o que revisar? •Tratamento de erros –Como os erros estão sendo tratados? –O código está tratando todos os erros que podem acontecer? •Segurança –Há algum princípio sendo violado? •Testes de unidade –Foram escritos? Bem escritos? São coerentes? 9
  10. 10. 17

×