Refatorando tudo! [Agile brazil 2017]
•
1 gostou•329 visualizações
O documento discute técnicas de refatoração de código, enfatizando a importância de manter os testes automatizados passando durante o processo de refatoração. A refatoração é apresentada como uma forma de melhorar a qualidade e manutenibilidade do código de forma incremental, sem alterar seu comportamento externo. Vários exemplos de code smells e refatorações são apresentados, como a introdução de value objects e parameter objects.
Recomendados
Mais conteúdo relacionado
Semelhante a Refatorando tudo! [Agile brazil 2017]
Semelhante a Refatorando tudo! [Agile brazil 2017] (11)
Mais de Marcos Brizeno
Mais de Marcos Brizeno (18)
Refatorando tudo! [Agile brazil 2017]
- 3. A única maneira de ir rápido é ter qualidade
- 9. O Scrum resolve esse problema!
- 11. Iteração 1 Iteração 2 Iteração n... ... tempo esforço
- 12. You can't ship process
- 13. A única maneira de ir rápido é ter qualidade https://www.youtube.com/watch?v=hG4LH6P8Syk
- 14. Mão na massa: Renomear um método
- 15. Mão na massa: Renomear um método
- 17. Renomear um método é simples!
- 18. Renomear um método é simples! - Muda o nome do método
- 19. Renomear um método é simples! - Muda o nome do método - Procura onde é chamado e atualiza
- 20. Renomear um método é simples! - Muda o nome do método - Procura onde é chamado e atualiza - Roda os testes
- 21. Renomear um método é simples!
- 22. Renomear um método é simples? - Quem chama o método vai quebrar? (conflitos de nomes)
- 23. Renomear um método é simples? - Quem chama o método vai quebrar? (conflitos de nomes) - Mudar o nome do método não vai quebrar outro ponto? (metaprogramação)
- 24. Renomear um método é simples? - Quem chama o método vai quebrar? (conflitos de nomes) - Mudar o nome do método não vai quebrar outro ponto? (metaprogramação) - Tudo ou nada!
- 25. Refactoring vs Rehacktoring REFACTORING - Testes passam a cada mudança REHACKTORING - Testes talvez passem
- 26. Refactoring vs Rehacktoring REFACTORING - Testes passam a cada mudança - Mantém comportamento externo REHACKTORING - Testes talvez passem - Talvez mantém o comportamento
- 27. Refactoring vs Rehacktoring REFACTORING - Testes passam a cada mudança - Mantém comportamento externo - Uma mudança por vez REHACKTORING - Testes talvez passem - Talvez mantém o comportamento - Várias mudanças
- 28. Refactoring vs Rehacktoring REFACTORING - Testes passam a cada mudança - Mantém comportamento externo - Uma mudança por vez - Visão clara do objetivo final REHACKTORING - Testes talvez passem - Talvez mantém o comportamento - Várias mudanças - Objetivo final incerto
- 29. Refatoração: Renomear método 1. Declare um método com o novo nome
- 30. Refatoração: Renomear método 1. Declare um método com o novo nome a. Rode os testes
- 31. Refatoração: Renomear método 1. Declare um método com o novo nome a. Rode os testes 2. Copie o código do método antigo para o novo (adaptando os parâmetros) a. Rode os testes
- 32. Refatoração: Renomear método 1. Declare um método com o novo nome a. Rode os testes 2. Copie o código do método antigo para o novo (adaptando os parâmetros) a. Rode os testes 3. Faça o método antigo chamar o novo a. Rode os testes
- 33. Refatoração: Renomear método 1. Declare um método com o novo nome a. Rode os testes 2. Copie o código do método antigo para o novo (adaptando os parâmetros) a. Rode os testes 3. Faça o método antigo chamar o novo a. Rode os testes 4. Troque o método novo pelo método antigo (um por vez) a. Rode os teste
- 34. Sério?Sério
- 36. Falta de feedback Problemas ao refatorar
- 37. Falta de feedback Ciclo longo Problemas ao refatorar
- 38. Falta de feedback Ciclo longo Visão técnica Problemas ao refatorar
- 39. Mão na massa: Revisar um Pull Request Julia Carlos
- 40. Mão na massa: Revisar um Pull Request
- 41. Mão na massa: Revisar um Pull Request
- 42. Mão na massa: Revisar um Pull Request
- 43. Mão na massa: Revisar um Pull Request
- 45. Mão na massa: Revisar um Pull Request - Outras classes vão ter preços - Também vão precisar retornar uma string com o preço cotado - Será necessário ter mais operações envolvendo dinheiro - Somar, subtrair, etc.
- 47. Seu código está "ruim". Mas por quê?
- 48. Code Smells
- 49. Code Smells Name five Code Smells.
- 51. Code Smells
- 52. Code Smells http://mikamantyla.eu/BadCodeSmellsTaxonomy.html Long Method Large Class Primitive obsession Long Parameter List Data Clump Switch statements Temporary field Refused Bequest Alternative classes with diff interfaces Divergent Change Shotgun Surgery Parallel Inheritance Lazy Class Data Class Duplicated Code Dead Code Speculative Generality Feature envy Inappropriate Intimacy Message Chains Middle Man
- 53. Bloaters http://mikamantyla.eu/BadCodeSmellsTaxonomy.html Long Method Large Class Primitive obsession Long Parameter List Data Clump Switch statements Temporary field Refused Bequest Alternative classes with diff interfaces Divergent Change Shotgun Surgery Parallel Inheritance Lazy Class Data Class Duplicated Code Dead Code Speculative Generality Feature envy Inappropriate Intimacy Message Chains Middle Man
- 54. Object Orientation Abusers http://mikamantyla.eu/BadCodeSmellsTaxonomy.html Long Method Large Class Primitive obsession Long Parameter List Data Clump Switch statements Temporary field Refused Bequest Alternative classes with diff interfaces Divergent Change Shotgun Surgery Parallel Inheritance Lazy Class Data Class Duplicated Code Dead Code Speculative Generality Feature envy Inappropriate Intimacy Message Chains Middle Man
- 55. Change Preventers http://mikamantyla.eu/BadCodeSmellsTaxonomy.html Long Method Large Class Primitive obsession Long Parameter List Data Clump Switch statements Temporary field Refused Bequest Alternative classes with diff interfaces Divergent Change Shotgun Surgery Parallel Inheritance Lazy Class Data Class Duplicated Code Dead Code Speculative Generality Feature envy Inappropriate Intimacy Message Chains Middle Man
- 56. The Couplers http://mikamantyla.eu/BadCodeSmellsTaxonomy.html Long Method Large Class Primitive obsession Long Parameter List Data Clump Switch statements Temporary field Refused Bequest Alternative classes with diff interfaces Divergent Change Shotgun Surgery Parallel Inheritance Lazy Class Data Class Duplicated Code Dead Code Speculative Generality Feature envy Inappropriate Intimacy Message Chains Middle Man
- 57. The Dispensables http://mikamantyla.eu/BadCodeSmellsTaxonomy.html Long Method Large Class Primitive obsession Long Parameter List Data Clump Switch statements Temporary field Refused Bequest Alternative classes with diff interfaces Divergent Change Shotgun Surgery Parallel Inheritance Lazy Class Data Class Duplicated Code Dead Code Speculative Generality Feature envy Inappropriate Intimacy Message Chains Middle Man
- 58. Por onde começar? 99 Bottles of OOP https://www.sandimetz.com/99bottles/
- 59. Code Smell: Data Clump https://martinfowler.com/bliki/DataClump.html
- 60. Code Smell: Data Clump lib/series.rb
- 61. Code Smell: Data Clump lib/series.rb
- 62. Code Smell: Data Clump lib/series.rb
- 63. Code Smell: Data Clump lib/series.rb
- 64. Code Smell: Data Clump lib/series.rb
- 65. Mão na massa: introduzir Value Class 1. Criar uma Value Class com construtores e getters a. Deve incluir o valor e a moeda 2. Encapsular atributos ao invés de chamá-las diretamente 3. Instanciar Value Object junto com os atributos 4. Trocar chamada getter por getter da Value Class 5. Apagar atributos não utilizados
- 67. Value Class para representar dinheiro lib/series.rb
- 68. Value Class para representar dinheiro lib/dinheiro.rb
- 69. Value Class para representar dinheiro lib/dinheiro.rb
- 70. Value Class para representar dinheiro lib/dinheiro.rb
- 71. Encapsular atributos preco e moeda lib/serie.rb
- 72. Encapsular atributos preco e moeda lib/serie.rb
- 73. Encapsular atributos preco e moeda lib/serie.rb
- 74. Encapsular atributos preco e moeda lib/serie.rb
- 75. Encapsular atributos preco e moeda lib/serie.rb
- 76. Instanciar Value Object junto com atributos lib/serie.rb
- 77. Instanciar Value Object junto com atributos lib/serie.rb
- 78. Instanciar Value Object junto com atributos lib/serie.rb
- 79. Trocar chamada do getter pela value class lib/serie.rb
- 80. Trocar chamada do getter pela value class lib/serie.rb
- 81. Trocar chamada do getter pela value class lib/serie.rb
- 82. Apagar atributos não utilizados lib/serie.rb
- 83. Apagar atributos não utilizados lib/serie.rb
- 84. Apagar atributos não utilizados lib/serie.rb
- 85. Apagar atributos não utilizados lib/serie.rb
- 86. Apagar atributos não utilizados lib/serie.rb
- 87. Apagar atributos não utilizados lib/serie.rb
- 88. Code Smell: Primitive Obsession http://wiki.c2.com/?PrimitiveObsession
- 89. Code Smell: Primitive Obsession lib/cotacao_util.rb lib/serie.rb
- 90. Code Smell: Primitive Obsession lib/cotacao_util.rb lib/serie.rb
- 91. Mão na massa: introduzir Parameter Object 1. Criar uma estrutura para agrupar parâmetros (se não existir) 2. Mudar a assinatura do método a. Criar um método com mesmo corpo e nova assinatura b. Atualizar referências ao novo parâmetro 3. Chamar o método novo no método antigo 4. Atualizar as referências do métodos antigos 5. Apagar método antigo
- 93. Criar uma estrutura para agrupar parâmetros lib/dinheiro.rb
- 94. Mudar a assinatura do método lib/cotacao_util.rb
- 95. Mudar a assinatura do método lib/cotacao_util.rb
- 96. Mudar a assinatura do método lib/cotacao_util.rb
- 97. Mudar a assinatura do método lib/cotacao_util.rb
- 98. Mudar a assinatura do método lib/cotacao_util.rb
- 99. Mudar a assinatura do método lib/cotacao_util.rb
- 100. Atualizar as referências do método antigo lib/serie.rb
- 101. Atualizar as referências do método antigo lib/serie.rb
- 102. Apagar método antigo
- 103. Code Smell: Feature Envy http://wiki.c2.com/?FeatureEnvySmell
- 104. Code Smell: Feature Envy lib/serie.rb
- 105. Code Smell: Feature Envy lib/serie.rb
- 106. Code Smell: Feature Envy lib/serie.rb
- 107. Code Smell: Feature Envy lib/serie.rb
- 108. Mão na massa: mover método 1. Identificar e extrair o método que será movido a. Copiar conteúdo para novo método b. Converter variáveis locais em parâmetro 2. Copiar método para nova classe a. trocar parâmetros por atributos (quando possível) 3. Trocar chamadas do método antigo pelo novo método 4. Apagar métodos antigos
- 110. Identificar e extrair o método que será movido lib/serie.rb
- 111. Identificar e extrair o método que será movido lib/serie.rb
- 112. Identificar e extrair o método que será movido lib/serie.rb
- 113. Identificar e extrair o método que será movido lib/serie.rb
- 114. Copiar método para nova classe lib/dinheiro.rb
- 115. Copiar método para nova classe lib/dinheiro.rb
- 116. Copiar método para nova classe lib/dinheiro.rb
- 117. Trocar chamadas pelo método novo lib/serie.rb
- 118. Trocar chamadas pelo método novo lib/serie.rb
- 119. Trocar chamadas pelo método novo lib/serie.rb
- 120. Apagar método antigo
- 121. Code Smell: Feature Envy lib/serie.rb lib/cotacao_util.rb
- 122. Code Smell: Feature Envy lib/serie.rb lib/cotacao_util.rb
- 123. Code Smell: Feature Envy lib/serie.rb lib/cotacao_util.rb
- 124. Classes Util/Service e Orientação a Objetos
- 125. Objetos se comunicam através de mensagens, visando proteger e esconder seu estado interno
- 126. Pensando orientado a objetos Dinheiro valor: int moeda: String exibir_moeda converter_moeda somar ... Estados Mensagens
- 127. Pensando orientado a objetos obj obj obj obj obj obj
- 128. Code Smell: Data Class Dinheiro valor: int moeda: String getValor: int setValor: void getMoeda: String setMoeda: void Estados Mensagens DinheiroService exibir_moeda converter_moeda somar ...
- 130. Proragamação Procedural é ruim?
- 131. Usar uma linguagem OO para programar procedural é ruim!
- 133. Code Smell: Feature Envy lib/serie.rb lib/cotacao_util.rb
- 134. Mão na massa: mover método 1. Identificar e extrair o método que será movido a. Copiar conteúdo para novo método b. Converter variáveis locais em parâmetro 2. Copiar método para nova classe a. trocar parâmetros por atributos (quando possível) 3. Trocar chamadas do método antigo pelo novo método 4. Apagar métodos antigos
- 136. Identificar e extrair o método que será movido lib/serie.rb
- 137. Identificar e extrair o método que será movido lib/serie.rb
- 138. Identificar e extrair o método que será movido lib/serie.rb
- 139. Copiar método para nova classe lib/dinheiro.rb
- 140. Copiar método para nova classe lib/dinheiro.rb
- 141. Trocar chamadas pelo método novo lib/serie.rb
- 142. Trocar chamadas pelo método novo lib/serie.rb
- 143. Trocar chamadas pelo método novo lib/serie.rb
- 144. Apagar método antigo
- 145. Como decidir o que depende do que?
- 146. Quantidade de mudanças e acomplamento https://www.youtube.com/watch?v=v-2yFMzxqwU Probabilidade de mudanças
- 147. Quantidade de mudanças e acomplamento https://www.youtube.com/watch?v=v-2yFMzxqwU Probabilidade de mudanças Cotacao Dinheiro Extrair os valores da cotação para um arquivo de configuração
- 148. Quantidade de mudanças e acomplamento https://www.youtube.com/watch?v=v-2yFMzxqwU Probabilidade de mudanças Dependa da direita para a esquerda Cotacao Dinheiro
- 150. Como melhorar seu código efetivamente 1. Entenda como seu programa vai evoluir (Keep it Simple, Stupid) 2. Identifique os Code Smells mais fáceis de resolver 3. Aplique refatorações 4. Repita até que o código esteja aberto a extensão 5. Aplique a mudança
- 151. Garantir que os testes sempre passem Feedback rápido
- 152. Garantir que os testes sempre passem Feedback rápido Fácil de reverter
- 153. Garantir que os testes sempre passem Feedback rápido Fácil de reverter Reduz ansiedade
- 154. Code Smell -> Refactoring https://refactoring.com/catalog/
- 155. Code Smell -> Refactoring industriallogic.com/blog/smells-to-refactorings-cheatsheet/
- 156. Code Smell é ruim?
- 157. Simplicidade de código Extreme Programming Explained: embrace change Todos os testes passam Sem duplicação Mostra as intenções Possui o menor número de classes ou métodos
- 159. Mais! 99 Bottles of Object Oriented Programming - Sandi Metz & Katrina Owen
- 160. Mais! Refatorando com Padrões de Projeto - Marcos Brizeno
- 161. Mais! Refatorando com Padrões de Projeto - Marcos Brizeno
- 164. Valeu!@marcosbrizeno