Upgrade to Pro — share decks privately, control downloads, hide ads and more …

Refatoração (PHPSP+Talks)

Refatoração (PHPSP+Talks)

Talk apresentada no PHPSP+Talks.

Davi Marcondes Moreira

April 18, 2017
Tweet

More Decks by Davi Marcondes Moreira

Other Decks in Programming

Transcript

  1. 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.”
  2. 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ã.
  3. 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)
  4. 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?
  5. 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.
  6. 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
  7. 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.
  8. 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 } }
  9. 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.
  10. 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.
  11. 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.
  12. 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.
  13. 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
  14. 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."