Code Review
E um pouco de boas práticas.
Code Review
- Importância:
- Manter um padrão de qualidade nos códigos fontes;
- Evitar códigos repetidos (implementação de algo que já
existe);
- Evitar implementações desnecessárias;
- Melhorar eficiência dos algoritmos;
- Gantir que o código está suficientemente legível;
- Aprender e ensinar;
- etc.
Code Review
Para se fazer o code review, não necessariamente o revisor
deverá estar em dia do contexto da aplicação.
[OPINIÃO] O revisor, a princípio, deve ter apenas conhecimento
das tecnologias utilizadas e boas práticas sobre ela.
Nunca levar as correções para o lado pessoal.
Code Review
Por onde começar ?
- Legibilidade do código desenvolvido:
- Nomes de variáveis;
- Complexidade.
- Verificar bom uso dos frameworks/bibliotecas utilizados na
aplicação.
- Verificar Styleguide.
Code Review - Apontamentos
Os apontamentos nos códigos dos colegas devem ser, na grande
maioria das vezes, descritíveis.
Sempre que necessário, apontar soluções.
Boas práticas
Boas práticas #1
Evitar condicionais
extensas e complexas
dentro do IF.
Sempre que possível, criar
uma variável booleana
para melhorar a leitura.
Boas práticas #2
Diminuir complexidade ciclomática (quantidade de caminhos que
seu método/classe pode tomar até o seu fim).
Boas práticas #3
Métodos devem ser pequenos.
Tamanho máximo recomendado:
● Método <= 20 linhas
● Linha <= 100 caracteres (evitar o scroll horizontal)
● Classe <= 500 linhas
Boas práticas
Ler sobre refatoração e código limpo.
Resumo Clean Code:
http://eduardokurita.com/?p=34
Fim

Code Review

  • 1.
    Code Review E umpouco de boas práticas.
  • 2.
    Code Review - Importância: -Manter um padrão de qualidade nos códigos fontes; - Evitar códigos repetidos (implementação de algo que já existe); - Evitar implementações desnecessárias; - Melhorar eficiência dos algoritmos; - Gantir que o código está suficientemente legível; - Aprender e ensinar; - etc.
  • 3.
    Code Review Para sefazer o code review, não necessariamente o revisor deverá estar em dia do contexto da aplicação. [OPINIÃO] O revisor, a princípio, deve ter apenas conhecimento das tecnologias utilizadas e boas práticas sobre ela. Nunca levar as correções para o lado pessoal.
  • 4.
    Code Review Por ondecomeçar ? - Legibilidade do código desenvolvido: - Nomes de variáveis; - Complexidade. - Verificar bom uso dos frameworks/bibliotecas utilizados na aplicação. - Verificar Styleguide.
  • 5.
    Code Review -Apontamentos Os apontamentos nos códigos dos colegas devem ser, na grande maioria das vezes, descritíveis. Sempre que necessário, apontar soluções.
  • 6.
  • 7.
    Boas práticas #1 Evitarcondicionais extensas e complexas dentro do IF. Sempre que possível, criar uma variável booleana para melhorar a leitura.
  • 8.
    Boas práticas #2 Diminuircomplexidade ciclomática (quantidade de caminhos que seu método/classe pode tomar até o seu fim).
  • 9.
    Boas práticas #3 Métodosdevem ser pequenos. Tamanho máximo recomendado: ● Método <= 20 linhas ● Linha <= 100 caracteres (evitar o scroll horizontal) ● Classe <= 500 linhas
  • 10.
    Boas práticas Ler sobrerefatoração e código limpo. Resumo Clean Code: http://eduardokurita.com/?p=34
  • 11.