Refatorando

Transcrição

1 Refatorando

2 @marcosbrizeno

3 A única maneira de ir rápido é ter qualidade

4

5

6 esforço tempo

7 esforço prazo tempo

8 esforço tempo

9 O Scrum resolve esse problema!

10 esforço... tempo

11 esforço... tempo Iteração 1 Iteração 2... Iteração n

12 You can't ship process

13 A única maneira de ir rápido é ter qualidade

14 Mão na massa: Renomear um método

15 Mão na massa: Renomear um método

16

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 REHACKTORING - Testes passam a cada mudança - Testes talvez passem - Mantém comportamento externo - Talvez mantém o comportamento

27 Refactoring vs Rehacktoring REFACTORING REHACKTORING - Testes passam a cada mudança - Testes talvez passem - Mantém comportamento externo - Talvez mantém o comportamento - Uma mudança por vez - Várias mudanças

28 Refactoring vs Rehacktoring REFACTORING REHACKTORING - Testes passam a cada mudança - Testes talvez passem - Mantém comportamento externo - Talvez mantém o comportamento - Uma mudança por vez - Várias mudanças - Visão clara do objetivo final - 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. 2. Rode os testes 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. 2. Copie o código do método antigo para o novo (adaptando os parâmetros) a. 3. Rode os testes Rode os testes 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. 2. Copie o código do método antigo para o novo (adaptando os parâmetros) a. 3. Rode os testes Faça o método antigo chamar o novo a. 4. Rode os testes Rode os testes Troque o método novo pelo método antigo (um por vez) a. Rode os teste

34 Sério? Sério

35

36 Problemas ao refatorar Falta de feedback

37 Problemas ao refatorar Falta de feedback Ciclo longo

38 Problemas ao refatorar Falta de feedback Ciclo longo Visão técnica

39 Mão na massa: Revisar um Pull Request Julia Carlos

40 Mão na massa: Revisar um Pull Request

44

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.

46

47 Seu código está "ruim". Mas por quê?

48 Code Smells

49 Code Smells Name five Code Smells.

50 Code Smells

51 Code Smells

52 Long Method Large Class Primitive obsession Long Parameter List Data Clump Switch statements Temporary field Refused Bequest Alternative classes with diff interfaces Code Smells Divergent Change Shotgun Surgery Parallel Inheritance Feature envy Inappropriate Intimacy Message Chains Middle Man Lazy Class Data Class Duplicated Code Dead Code Speculative Generality

53 Long Method Large Class Primitive obsession Long Parameter List Data Clump Switch statements Temporary field Refused Bequest Alternative classes with diff interfaces Bloaters Divergent Change Shotgun Surgery Parallel Inheritance Feature envy Inappropriate Intimacy Message Chains Middle Man Lazy Class Data Class Duplicated Code Dead Code Speculative Generality

54 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 Feature envy Inappropriate Intimacy Message Chains Middle Man Lazy Class Data Class Duplicated Code Dead Code Speculative Generality Object Orientation Abusers

55 Divergent Change Shotgun Surgery Parallel Inheritance Long Method Large Class Primitive obsession Long Parameter List Data Clump Switch statements Temporary field Refused Bequest Alternative classes with diff interfaces Feature envy Inappropriate Intimacy Message Chains Middle Man Lazy Class Data Class Duplicated Code Dead Code Speculative Generality Change Preventers

56 Long Method Large Class Primitive obsession Long Parameter List Data Clump Switch statements Temporary field Refused Bequest Alternative classes with diff interfaces The Couplers Divergent Change Shotgun Surgery Parallel Inheritance Feature envy Inappropriate Intimacy Message Chains Middle Man Lazy Class Data Class Duplicated Code Dead Code Speculative Generality

57 Long Method Large Class Primitive obsession Long Parameter List Data Clump Switch statements Temporary field Refused Bequest Alternative classes with diff interfaces The Dispensables Divergent Change Shotgun Surgery Parallel Inheritance Feature envy Inappropriate Intimacy Message Chains Middle Man Lazy Class Data Class Duplicated Code Dead Code Speculative Generality

58 Por onde começar? 99 Bottles of OOP

59 Code Smell: Data Clump

60 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

66

67 Value Class para representar dinheiro lib/series.rb

68 Value Class para representar dinheiro lib/dinheiro.rb

71 Encapsular atributos preco e moeda lib/serie.rb

76 Instanciar Value Object junto com atributos lib/serie.rb

79 Trocar chamada do getter pela value class lib/serie.rb

82 Apagar atributos não utilizados lib/serie.rb

88 Code Smell: Primitive Obsession

89 Code Smell: Primitive Obsession lib/serie.rb lib/cotacao_util.rb

90 Code Smell: Primitive Obsession lib/serie.rb lib/cotacao_util.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

92

93 Criar uma estrutura para agrupar parâmetros lib/dinheiro.rb

94 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

104 Code Smell: Feature Envy lib/serie.rb

108 Mão na massa: mover método 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 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

109

110 Identificar e extrair o método que será movido lib/serie.rb

114 Copiar método para nova classe lib/dinheiro.rb

117 Trocar chamadas pelo método novo lib/serie.rb

121 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 Estados exibir_moeda converter_moeda somar... Mensagens

127 Pensando orientado a objetos obj obj obj obj obj obj

128 Code Smell: Data Class Dinheiro valor: int moeda: String getvalor: setvalor: getmoeda: setmoeda: int void String void DinheiroService Estados Mensagens exibir_moeda converter_moeda somar...

129 Objetos anêmicos Serviços/Util Programação Procedural

130 Proragamação Procedural é ruim?

131 Usar uma linguagem OO para programar procedural é ruim!

132

134 Mão na massa: mover método 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 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

135

145 Como decidir o que depende do que?

146 Quantidade de mudanças e acomplamento Probabilidade de mudanças

147 Quantidade de mudanças e acomplamento 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 Probabilidade de mudanças Cotacao Dinheiro Dependa da direita para a esquerda

149

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

155 Code Smell -> Refactoring industriallogic.com/blog/smells-to-refactorings-cheatsheet/

156 Code Smell é ruim?

157 Simplicidade de código Todos os testes passam Sem duplicac a o Mostra as intenc o es Possui o menor nu mero de classes ou me todos Extreme Programming Explained: embrace change

158 Mais! Refactoring - Martin Fowler