Refatoração (PHPSP+Talks)

Refatoração (PHPSP+Talks)

Talk apresentada no PHPSP+Talks.

049fbbe4e5fb94c45d6ccd656290d6fb?s=128

Davi Marcondes Moreira

April 18, 2017
Tweet

Transcript

  1. 3.
  2. 4.

    Broken Window Theory Por James Q Wilson e George L

    Kelling, Março 1982 https://en.wikipedia.org/wiki/Broken_windows_theory “Desordem gera desordem.”
  3. 6.

    Refatoração: O que é? • Processo de reestruturar o código

    sem alterar seu comportamento. Mas IMHO também é: • Sempre deixar a casa melhor do que você encontrou. • Tornar o software mais confiável para você e para quem vai mexer nele amanhã.
  4. 9.

    Correções • Débito técnico (o famoso TODO) • Vulnerabilidades e

    bugs • Acúmulo de más práticas Melhorias • Atualização de dependências • Melhorar performance • Diminuir complexidade (o código deve ser simples)
  5. 10.

    Correções • Débito técnico (o famoso TODO) • Vulnerabilidades e

    bugs • Acúmulo de más práticas Melhorias • Atualização de dependências • Melhorar performance • Diminuir complexidade (o código deve ser simples) Criação • TDD: red, green, e, qual era o terceiro mesmo?
  6. 11.

    Correções • Débito técnico (o famoso TODO) • Vulnerabilidades e

    bugs • Acúmulo de más práticas Melhorias • Atualização de dependências • Melhorar performance • Diminuir complexidade (o código deve ser simples) Criação • TDD: red, green, e, qual era o terceiro mesmo? Refactor.
  7. 13.

    Colaboração • Refatorar não é uma atividade isolada. ◦ Pedir

    ajuda aos colegas ◦ Discutir em grupo/equipe ◦ Alinhar métricas e regras ▪ Mínimo de % na cobertura de testes, padrões de estilo ◦ Não ser negligente e assumir responsabilidade
  8. 15.
  9. 16.
  10. 17.
  11. 18.
  12. 19.
  13. 20.
  14. 21.
  15. 22.
  16. 23.
  17. 25.

    1. Renomeando métodos • Usando forma mais claras, curtas e

    concisas, ou mesmo verbosas se necessário. • Ser o mais explícito possível. • Em casos críticos, reescreva a nova chamada dentro do método antigo, depreciando o método anterior e evitando quebrar compatibilidades enquanto a adoção do novo método é feita por toda a equipe.
  18. 26.

    1. Renomeando métodos // Classe usada por toda a aplicação

    class Foo { public function oldScaryMethod( $foo, $bar, $baz ) { // Complexidade no contexto } } // Chamada encapsulada em outro método class Foo { public function newMethod( $foo ) { // Podemos agir aqui $this->oldScaryMethod(...); } public function oldScaryMethod(...) { // Complexidade isolada } }
  19. 27.

    2. Extrair responsabilidades de um método em um novo(a) método/classe

    • Identificar que parte de um código pode ser reescrita em outro lugar, reduzindo a complexidade. • Abre espaço para refletir sobre princípios como 'o que de fato isso deveria estar fazendo?', 'qual a verdadeira responsabilidade desse método?' e encaixar boas práticas como SRP de orientação à objetos.
  20. 28.

    3. Eliminar código morto • Código que não tem mais

    uso, ou é fruto de copiar de uma parte específica do projeto para outra, que ficou para trás. • Não ter medo de excluir código. Partes antigas podem ser recuperadas pelo SCV. • Requer uma boa estratégia de teste.
  21. 31.

    Entregas pontuais O código fica mais compreensível, organizado e flexível

    à mudanças. Confiança ao editar o código Maior visibilidade dos problemas.
  22. 32.

    Entregas pontuais O código fica mais compreensível, organizado e flexível

    à mudanças. Confiança ao editar o código Maior visibilidade dos problemas. Garantia de qualidade Fim da síndrome da casa de vidro.
  23. 33.

    Entregas pontuais O código fica mais compreensível, organizado e flexível

    à mudanças. Confiança ao editar o código Maior visibilidade dos problemas. Garantia de qualidade Fim da síndrome da casa de vidro. Uma bela noite de sono =P
  24. 35.

    No episódio de hoje… • Tudo o que se precisa

    é intenção. • Refatoração acontece em vários momentos. E todos podem e devem ajudar. • Quanto mais cedo e pontual, mais fácil de fazer. • Nem toda refatoração é simples, e cada caso precisa ser muito bem analisado para verificar todo impacto e esforço.. • "A única certeza sobre software é que ele muda."