Slide 1

Slide 1 text

Code Smells, Refatoração e Elixir uma conversa sobre qualidade de código

Slide 2

Slide 2 text

Elaine Naomi Watanabe twitter.com/elaine_nw speakerdeck.com/elainenaomi Engenheira de Software (The RealReal) Mestrado em Ciência da Computação (USP) slides disponíveis

Slide 3

Slide 3 text

therealreal.com/careers Referral Source Elaine Naomi Watanabe

Slide 4

Slide 4 text

The RealReal - Elixir adoption story youtube.com/watch?v=sTs_4T1ufLY

Slide 5

Slide 5 text

Sobre a Plataformatec...

Slide 6

Slide 6 text

No content

Slide 7

Slide 7 text

o que aprontei por lá =) além de trabalhar como dev Sobre a Plataformatec... twitter.com/elaine_nw/status/1214748684593840128

Slide 8

Slide 8 text

No content

Slide 9

Slide 9 text

REFATORAÇÃO

Slide 10

Slide 10 text

REFATORAÇÃO + ELIXIR

Slide 11

Slide 11 text

DEFINIÇÃO Code SMells Refatoração Na PRÁTICA Resumo e referências

Slide 12

Slide 12 text

Code SMells DEFINIÇÃO Refatoração Na PRÁTICA Resumo e referências

Slide 13

Slide 13 text

REFATORAÇÃO

Slide 14

Slide 14 text

REFATORAÇÃO

Slide 15

Slide 15 text

REFATORAÇÃO

Slide 16

Slide 16 text

No content

Slide 17

Slide 17 text

No content

Slide 18

Slide 18 text

No content

Slide 19

Slide 19 text

No content

Slide 20

Slide 20 text

No content

Slide 21

Slide 21 text

código sendo refatorado falhando por dias?

Slide 22

Slide 22 text

código sendo refatorado falhando por dias?

Slide 23

Slide 23 text

código sendo refatorado falhando por dias? REFATORAÇÃO

Slide 24

Slide 24 text

código sendo refatorado falhando por dias? REFATORAÇÃO

Slide 25

Slide 25 text

Alteração da estrutura interna do código Tornar + fácil de ser compreendida Tornar + fácil de ser alterada Manter comportamento externo Refatoração Refactoring

Slide 26

Slide 26 text

Refatoração Alteração da estrutura interna do código Tornar + fácil de ser compreendida Tornar + fácil de ser alterada Manter comportamento externo Refactoring

Slide 27

Slide 27 text

Refatoração Alteração da estrutura interna do código Tornar + fácil de ser compreendida Tornar + fácil de ser alterada Manter comportamento externo Refactoring

Slide 28

Slide 28 text

Refatoração Alteração da estrutura interna do código Tornar + fácil de ser compreendida Tornar + fácil de ser alterada Manter comportamento externo Refactoring

Slide 29

Slide 29 text

Refatoração Alteração da estrutura interna do código Tornar + fácil de ser compreendida Tornar + fácil de ser alterada Manter comportamento externo Refactoring

Slide 30

Slide 30 text

Refatoração Alteração da estrutura interna do código Tornar + fácil de ser compreendida Tornar + fácil de ser alterada Manter comportamento externo Refactoring

Slide 31

Slide 31 text

RefatorAR Reestruturar um software Série de pequenas refatorações Manter o comportamento externo Refactoring

Slide 32

Slide 32 text

RefatorAR Refactoring Reestruturar um software Série de pequenas refatorações Manter o comportamento externo

Slide 33

Slide 33 text

RefatorAR Refactoring Reestruturar um software Série de pequenas refatorações Manter o comportamento externo

Slide 34

Slide 34 text

RefatorAR Refactoring Reestruturar um software Série de pequenas refatorações Manter o comportamento externo

Slide 35

Slide 35 text

RefatorAR Refactoring Reestruturar um software Série de pequenas refatorações Manter o comportamento externo

Slide 36

Slide 36 text

Reestruturar um software Série de pequenas refatorações Manter o comportamento externo RefatorAR Refactoring não é qualquer limpeza de código

Slide 37

Slide 37 text

Martin Fowler 1999

Slide 38

Slide 38 text

Martin Fowler 1999

Slide 39

Slide 39 text

22 code smells 72 técnicas de refatoração 1999

Slide 40

Slide 40 text

Martin Fowler 22 code smells 72 técnicas de refatoração 1999

Slide 41

Slide 41 text

Martin Fowler 22 code smells 72 técnicas de refatoração 1999

Slide 42

Slide 42 text

a partir de um sistema existente

Slide 43

Slide 43 text

design/arquitetura + implementação a partir de um sistema existente

Slide 44

Slide 44 text

design/arquitetura + implementação a partir de um sistema existente como alterar a estrutura do sistema

Slide 45

Slide 45 text

design/arquitetura + implementação a partir de um sistema existente como alterar a estrutura do sistema detecção de code smells

Slide 46

Slide 46 text

design/arquitetura + implementação detecção de code smells a partir de um sistema existente como alterar a estrutura do sistema + refatoração

Slide 47

Slide 47 text

design/arquitetura + implementação detecção de code smells + refatoração a partir de um sistema existente como alterar a estrutura do sistema

Slide 48

Slide 48 text

detecção de code smells + refatoração design/arquitetura + implementação a partir de um sistema existente como alterar a estrutura do sistema promover o redesign

Slide 49

Slide 49 text

aperfeiçoar o design depois de escrito

Slide 50

Slide 50 text

aperfeiçoar o design depois de escrito

Slide 51

Slide 51 text

evolução contínua do design

Slide 52

Slide 52 text

evolução contínua do design

Slide 53

Slide 53 text

2018 +18 anos depois 24 code smells 61 técnicas de refatoração

Slide 54

Slide 54 text

2018 +18 anos depois 24 code smells 61 técnicas de refatoração

Slide 55

Slide 55 text

2018 +18 anos depois 24 code smells 61 técnicas de refatoração

Slide 56

Slide 56 text

2018 +18 anos depois 24 code smells 61 técnicas de refatoração

Slide 57

Slide 57 text

2018 visão sobre mundo OO + funcional +18 anos depois 24 code smells 61 técnicas de refatoração

Slide 58

Slide 58 text

visão sobre mundo OO + funcional 2018 +18 anos depois 24 code smells 61 técnicas de refatoração

Slide 59

Slide 59 text

encapsulamento coesão composição polimorfismo 2018

Slide 60

Slide 60 text

encapsulamento coesão composição polimorfismo 2018 em ambos paradigmas

Slide 61

Slide 61 text

2018 classes? objetos?

Slide 62

Slide 62 text

modules maps e structs protocols behaviours metaprogramming 2018 classes? objetos?

Slide 63

Slide 63 text

por que o design é importante?

Slide 64

Slide 64 text

por que o design é importante?

Slide 65

Slide 65 text

Do livro Refactoring DESIGN RUIM Funcionalidades acumuladas Tempo

Slide 66

Slide 66 text

Do livro Refactoring Funcionalidades acumuladas Tempo DESIGN RUIM

Slide 67

Slide 67 text

Entregas começam a atrasar Mais código para fazer as mesmas tarefas Pouco reúso de código Código difícil de entender Código difícil de alterar

Slide 68

Slide 68 text

Entregas começam a atrasar Mais código para fazer as mesmas tarefas Pouco reúso de código Código difícil de entender Código difícil de alterar

Slide 69

Slide 69 text

Entregas começam a atrasar Mais código para fazer as mesmas tarefas Pouco reúso de código Código difícil de entender Código difícil de alterar

Slide 70

Slide 70 text

Entregas começam a atrasar Mais código para fazer as mesmas tarefas Pouco reúso de código Código difícil de entender Código difícil de alterar

Slide 71

Slide 71 text

Entregas começam a atrasar Mais código para fazer as mesmas tarefas Pouco reúso de código Código difícil de entender Código difícil de alterar

Slide 72

Slide 72 text

Entregas começam a atrasar Mais código para fazer as mesmas tarefas Pouco reúso de código Código difícil de entender Código difícil de alterar

Slide 73

Slide 73 text

No content

Slide 74

Slide 74 text

pensando na evolutibilidade do software

Slide 75

Slide 75 text

pensando na evolutibilidade do software

Slide 76

Slide 76 text

pensando na evolutibilidade do software capacidade de adaptação ao longo do tempo

Slide 77

Slide 77 text

o código será mais lido e modificado do que será escrito

Slide 78

Slide 78 text

o código será mais lido e modificado do que será escrito

Slide 79

Slide 79 text

o código será mais lido e modificado do que será escrito

Slide 80

Slide 80 text

Do livro Refactoring DESIGN RUIM Hipótese da Estamina do Design (Design Stamina Hypothesis) DESIGN BOM Funcionalidades acumuladas Tempo

Slide 81

Slide 81 text

Do livro Refactoring Hipótese da Estamina do Design (Design Stamina Hypothesis) DESIGN RUIM DESIGN BOM Funcionalidades acumuladas Tempo

Slide 82

Slide 82 text

Código bem estruturado Fácil de entender Fácil de alterar Fácil de apagar

Slide 83

Slide 83 text

Código bem estruturado Fácil de entender Fácil de alterar Fácil de apagar

Slide 84

Slide 84 text

Código bem estruturado Fácil de entender Fácil de alterar Fácil de apagar

Slide 85

Slide 85 text

Código bem estruturado Fácil de entender Fácil de alterar Fácil de apagar

Slide 86

Slide 86 text

Código bem estruturado Fácil de entender Fácil de alterar Fácil de apagar

Slide 87

Slide 87 text

Código bem estruturado Fácil de entender Fácil de alterar Fácil de apagar

Slide 88

Slide 88 text

Código bem estruturado Fácil de entender Fácil de alterar Fácil de apagar spark joy

Slide 89

Slide 89 text

No content

Slide 90

Slide 90 text

como identificar o que precisamos refatorar?

Slide 91

Slide 91 text

como identificar o que precisamos refatorar?

Slide 92

Slide 92 text

DEFINIÇÃO Code SMells Refatoração Na PRÁTICA Resumo e referências

Slide 93

Slide 93 text

Code smells Características do código que normalmente indicam problemas de design mais profundos no sistema https://martinfowler.com/bliki/CodeSmell.html

Slide 94

Slide 94 text

Code smells Características do código que normalmente indicam problemas de design mais profundos no sistema https://martinfowler.com/bliki/CodeSmell.html Também chamados de Bad Smells (maus cheiros)

Slide 95

Slide 95 text

Code smells Características do código que normalmente indicam problemas de design mais profundos no sistema https://martinfowler.com/bliki/CodeSmell.html

Slide 96

Slide 96 text

Code smells Características do código que normalmente indicam problemas de design mais profundos no sistema https://martinfowler.com/bliki/CodeSmell.html

Slide 97

Slide 97 text

No content

Slide 98

Slide 98 text

No content

Slide 99

Slide 99 text

Código duplicado Duplicated Code

Slide 100

Slide 100 text

Regra de negócio duplicada Texto duplicado

Slide 101

Slide 101 text

Regra de negócio duplicada Texto duplicado

Slide 102

Slide 102 text

defmodule CarrinhoCompra do # ... def calcular_frete(cep, assinatura) do if (Enum.member?([3, 4], assinatura.id)) do 0 else 10.0 * Localizacao.calcular(cep) end end def aplicar_desconto(total, assinatura) do if (Enum.member?([3, 4], assinatura.id)) do total * 0.9 else total end end end

Slide 103

Slide 103 text

defmodule CarrinhoCompra do # ... def calcular_frete(cep, assinatura) do if (Enum.member?([3, 4], assinatura.id)) do 0 else 10.0 * Localizacao.calcular(cep) end end def aplicar_desconto(total, assinatura) do if (Enum.member?([3, 4], assinatura.id)) do total * 0.9 else total end end end

Slide 104

Slide 104 text

defmodule CarrinhoCompra do # ... def calcular_frete(cep, assinatura) do if (Enum.member?([3, 4], assinatura.id)) do 0 else 10.0 * Localizacao.calcular(cep) end end def aplicar_desconto(total, assinatura) do if (Enum.member?([3, 4], assinatura.id)) do total * 0.9 else total end end end frete

Slide 105

Slide 105 text

defmodule CarrinhoCompra do # ... def calcular_frete(cep, assinatura) do if (Enum.member?([3, 4], assinatura.id)) do 0 else 10.0 * Localizacao.calcular(cep) end end def aplicar_desconto(total, assinatura) do if (Enum.member?([3, 4], assinatura.id)) do total * 0.9 else total end end end desconto no valor total

Slide 106

Slide 106 text

defmodule CarrinhoCompra do # ... def calcular_frete(cep, assinatura) do if (Enum.member?([3, 4], assinatura.id)) do 0 else 10.0 * Localizacao.calcular(cep) end end def aplicar_desconto(total, assinatura) do if (Enum.member?([3, 4], assinatura.id)) do total * 0.9 else total end end end contextos diferentes

Slide 107

Slide 107 text

defmodule CarrinhoCompra do # ... def calcular_frete(cep, assinatura) do if (Enum.member?([3, 4], assinatura.id)) do 0 else 10.0 * Localizacao.calcular(cep) end end def aplicar_desconto(total, assinatura) do if (Enum.member?([3, 4], assinatura.id)) do total * 0.9 else total end end end que compartilham a mesma condicional

Slide 108

Slide 108 text

mas cadê o pattern matching? =P

Slide 109

Slide 109 text

defmodule CarrinhoCompra do # ... def calcular_frete(cep, %{id: 3}), do: 0.0 def calcular_frete(cep, %{id: 4}), do: 0.0 def calcular_frete(cep, _), do: 10.0 * Localizacao.calcular(cep) def aplicar_desconto(total, %{id: 3}), do: total * 0.9 def aplicar_desconto(total, %{id: 4}), do: total * 0.9 def aplicar_desconto(total, _), do: total end

Slide 110

Slide 110 text

defmodule CarrinhoCompra do # ... def calcular_frete(cep, %{id: 3}), do: 0.0 def calcular_frete(cep, %{id: 4}), do: 0.0 def calcular_frete(cep, _), do: 10.0 * Localizacao.calcular(cep) def aplicar_desconto(total, %{id: 3}), do: total * 0.9 def aplicar_desconto(total, %{id: 4}), do: total * 0.9 def aplicar_desconto(total, _), do: total end pattern matching

Slide 111

Slide 111 text

defmodule CarrinhoCompra do # ... def calcular_frete(cep, %{id: 3}), do: 0.0 def calcular_frete(cep, %{id: 4}), do: 0.0 def calcular_frete(cep, _), do: 10.0 * Localizacao.calcular(cep) def aplicar_desconto(total, %{id: 3}), do: total * 0.9 def aplicar_desconto(total, %{id: 4}), do: total * 0.9 def aplicar_desconto(total, _), do: total end frete

Slide 112

Slide 112 text

defmodule CarrinhoCompra do # ... def calcular_frete(cep, %{id: 3}), do: 0.0 def calcular_frete(cep, %{id: 4}), do: 0.0 def calcular_frete(cep, _), do: 10.0 * Localizacao.calcular(cep) def aplicar_desconto(total, %{id: 3}), do: total * 0.9 def aplicar_desconto(total, %{id: 4}), do: total * 0.9 def aplicar_desconto(total, _), do: total end desconto no valor total

Slide 113

Slide 113 text

defmodule CarrinhoCompra do # ... def calcular_frete(cep, %{id: 3}), do: 0.0 def calcular_frete(cep, %{id: 4}), do: 0.0 def calcular_frete(cep, _), do: 10.0 * Localizacao.calcular(cep) def aplicar_desconto(total, %{id: 3}), do: total * 0.9 def aplicar_desconto(total, %{id: 4}), do: total * 0.9 def aplicar_desconto(total, _), do: total end

Slide 114

Slide 114 text

defmodule CarrinhoCompra do # ... def calcular_frete(cep, %{id: 3}), do: 0.0 def calcular_frete(cep, %{id: 4}), do: 0.0 def calcular_frete(cep, _), do: 10.0 * Localizacao.calcular(cep) def aplicar_desconto(total, %{id: 3}), do: total * 0.9 def aplicar_desconto(total, %{id: 4}), do: total * 0.9 def aplicar_desconto(total, _), do: total end código duplicado

Slide 115

Slide 115 text

defmodule CarrinhoCompra do # ... def calcular_frete(cep, %{id: 3}), do: 0.0 def calcular_frete(cep, %{id: 4}), do: 0.0 def calcular_frete(cep, _), do: 10.0 * Localizacao.calcular(cep) def aplicar_desconto(total, %{id: 3}), do: total * 0.9 def aplicar_desconto(total, %{id: 4}), do: total * 0.9 def aplicar_desconto(total, _), do: total end regra de negócio misteriosa

Slide 116

Slide 116 text

No content

Slide 117

Slide 117 text

cuidado com código que parece duplicado

Slide 118

Slide 118 text

cuidado com código que parece duplicado

Slide 119

Slide 119 text

precisamos entender o domínio para entender se as abstrações fazem sentido

Slide 120

Slide 120 text

precisamos entender o domínio para entender se as abstrações fazem sentido

Slide 121

Slide 121 text

Nome misterioso Mysterious Name

Slide 122

Slide 122 text

defmodule Ordem do # ... def exec(i, d) do # ... end end

Slide 123

Slide 123 text

defmodule Ordem do # ... def exec(i, d) do # ... end end

Slide 124

Slide 124 text

defmodule Ordem do # ... def exec(i, d) do # ... end end o que são esses argumentos? o que eles representam?

Slide 125

Slide 125 text

defmodule Ordem do # ... def exec(i, d) do # ... end end

Slide 126

Slide 126 text

defmodule Ordem do # ... def exec(i, d) do # ... end end

Slide 127

Slide 127 text

defmodule Ordem do # ... def exec(i, d) do # ... end end o que essa função faz?

Slide 128

Slide 128 text

defmodule Ordem do # ... def exec(i, d) do i |> Enum.map(& &1.pco * &1.qtd) |> Enum.sum() |> Kernel.-(d) end end

Slide 129

Slide 129 text

defmodule Ordem do # ... def exec(i, d) do i |> Enum.map(& &1.pco * &1.qtd) |> Enum.sum() |> Kernel.-(d) end end

Slide 130

Slide 130 text

defmodule Ordem do # ... def exec(i, d) do i |> Enum.map(& &1.pco * &1.qtd) |> Enum.sum() |> Kernel.-(d) end end o que significam essas siglas?

Slide 131

Slide 131 text

defmodule Ordem do # ... def exec(i, d) do i |> Enum.map(& &1.pco * &1.qtd) |> Enum.sum() |> Kernel.-(d) end end o que significam essas siglas?

Slide 132

Slide 132 text

DAR NOMES é uma das duas coisas mais difíceis em computação https://martinfowler.com/bliki/TwoHardThings.html

Slide 133

Slide 133 text

DAR NOMES é uma das duas coisas mais difíceis em computação https://martinfowler.com/bliki/TwoHardThings.html

Slide 134

Slide 134 text

DAR NOMES é uma das duas coisas mais difíceis em computação https://martinfowler.com/bliki/TwoHardThings.html

Slide 135

Slide 135 text

DAR NOMES é uma das duas coisas mais difíceis em computação https://martinfowler.com/bliki/TwoHardThings.html variáveis, funções, módulos devem representar o domínio de negócio

Slide 136

Slide 136 text

DAR NOMES é uma das duas coisas mais difíceis em computação https://martinfowler.com/bliki/TwoHardThings.html variáveis, funções, módulos devem representar o domínio de negócio

Slide 137

Slide 137 text

FunÇão Longa Long Function

Slide 138

Slide 138 text

é uma função longa

Slide 139

Slide 139 text

é uma função longa

Slide 140

Slide 140 text

é uma função longa mas quão longa?

Slide 141

Slide 141 text

defmodule Relatorio do def imprimir(usuario, pedido) do imprimir_dados_empresa() IO.puts("Nome: #{usuario.nome} #{usuario.sobrenome}") pedido.items |> Enum.filter(&(&1.status == 3)) |> Enum.map(fn item -> IO.puts("Item: #{item.nome}") IO.puts("Preço: #{item.preco}") IO.puts("Quantidade: #{item.quantidade}") total = item.preco * item.quantidade IO.puts("Total: #{total}") end) end end

Slide 142

Slide 142 text

defmodule Relatorio do def imprimir(usuario, pedido) do imprimir_dados_empresa() IO.puts("Nome: #{usuario.nome} #{usuario.sobrenome}") pedido.items |> Enum.filter(&(&1.status == 3)) |> Enum.map(fn item -> IO.puts("Item: #{item.nome}") IO.puts("Preço: #{item.preco}") IO.puts("Quantidade: #{item.quantidade}") total = item.preco * item.quantidade IO.puts("Total: #{total}") end) end end

Slide 143

Slide 143 text

defmodule Relatorio do def imprimir(usuario, pedido) do imprimir_dados_empresa() IO.puts("Nome: #{usuario.nome} #{usuario.sobrenome}") pedido.items |> Enum.filter(&(&1.status == 3)) |> Enum.map(fn item -> IO.puts("Item: #{item.nome}") IO.puts("Preço: #{item.preco}") IO.puts("Quantidade: #{item.quantidade}") total = item.preco * item.quantidade IO.puts("Total: #{total}") end) end end dados da empresa

Slide 144

Slide 144 text

defmodule Relatorio do def imprimir(usuario, pedido) do imprimir_dados_empresa() IO.puts("Nome: #{usuario.nome} #{usuario.sobrenome}") pedido.items |> Enum.filter(&(&1.status == 3)) |> Enum.map(fn item -> IO.puts("Item: #{item.nome}") IO.puts("Preço: #{item.preco}") IO.puts("Quantidade: #{item.quantidade}") total = item.preco * item.quantidade IO.puts("Total: #{total}") end) end end dados do usuário

Slide 145

Slide 145 text

defmodule Relatorio do def imprimir(usuario, pedido) do imprimir_dados_empresa() IO.puts("Nome: #{usuario.nome} #{usuario.sobrenome}") pedido.items |> Enum.filter(&(&1.status == 3)) |> Enum.map(fn item -> IO.puts("Item: #{item.nome}") IO.puts("Preço: #{item.preco}") IO.puts("Quantidade: #{item.quantidade}") total = item.preco * item.quantidade IO.puts("Total: #{total}") end) end end dados do pedido (com filtros e cálculos)

Slide 146

Slide 146 text

Necessidade de separar blocos para entender Vontade de adicionar comentários

Slide 147

Slide 147 text

Necessidade de separar blocos para entender Vontade de adicionar comentários

Slide 148

Slide 148 text

Necessidade de separar blocos para entender Vontade de adicionar comentários

Slide 149

Slide 149 text

Necessidade de separar blocos para entender Vontade de adicionar comentários que tal criar uma nova função?

Slide 150

Slide 150 text

Classe Grande Long Class

Slide 151

Slide 151 text

Classe Grande Long Class módulo

Slide 152

Slide 152 text

Classe Grande Long Class módulo

Slide 153

Slide 153 text

um módulo que faz muitas coisas, que cuida de várias regras de negócio

Slide 154

Slide 154 text

um módulo que faz muitas coisas, que cuida de várias regras de negócio

Slide 155

Slide 155 text

um módulo que faz muitas coisas, que cuida de várias regras de negócio

Slide 156

Slide 156 text

defmodule CarrinhoCompra do def calcular_total(items, assinatura) do # ... end def calcular_frete(cep, %{id: 3}), do: 0.0 def calcular_frete(cep, %{id: 4}), do: 0.0 def calcular_frete(cep, _), do: 10.0 * Localizacao.calcular(cep) def aplicar_desconto(total, %{id: 3}), do: total * 0.9 def aplicar_desconto(total, %{id: 4}), do: total * 0.9 def aplicar_desconto(total, _), do: total def mandar_mensagem_assinatura(%{id: 3}, _), do: nil def mandar_mensagem_assinatura(%{id: 4}, _), do: nil def mandar_mensagem_assinatura(assinatura, usuario), do: Assinatura.enviar_email_upgrade(assinatura, usuario) def imprimir(usuario, pedido) do # ... end end

Slide 157

Slide 157 text

defmodule CarrinhoCompra do def calcular_total(items, assinatura) do # ... end def calcular_frete(cep, %{id: 3}), do: 0.0 def calcular_frete(cep, %{id: 4}), do: 0.0 def calcular_frete(cep, _), do: 10.0 * Localizacao.calcular(cep) def aplicar_desconto(total, %{id: 3}), do: total * 0.9 def aplicar_desconto(total, %{id: 4}), do: total * 0.9 def aplicar_desconto(total, _), do: total def mandar_mensagem_assinatura(%{id: 3}, _), do: nil def mandar_mensagem_assinatura(%{id: 4}, _), do: nil def mandar_mensagem_assinatura(assinatura, usuario), do: Assinatura.enviar_email_upgrade(assinatura, usuario) def imprimir(usuario, pedido) do # ... end end versão com pattern matching

Slide 158

Slide 158 text

defmodule CarrinhoCompra do def calcular_total(items, assinatura) do # ... end def calcular_frete(cep, %{id: 3}), do: 0.0 def calcular_frete(cep, %{id: 4}), do: 0.0 def calcular_frete(cep, _), do: 10.0 * Localizacao.calcular(cep) def aplicar_desconto(total, %{id: 3}), do: total * 0.9 def aplicar_desconto(total, %{id: 4}), do: total * 0.9 def aplicar_desconto(total, _), do: total def mandar_mensagem_assinatura(%{id: 3}, _), do: nil def mandar_mensagem_assinatura(%{id: 4}, _), do: nil def mandar_mensagem_assinatura(assinatura, usuario), do: Assinatura.enviar_email_upgrade(assinatura, usuario) def imprimir(usuario, pedido) do # ... end end cálculo de total de items

Slide 159

Slide 159 text

defmodule CarrinhoCompra do def calcular_total(items, assinatura) do # ... end def calcular_frete(cep, %{id: 3}), do: 0.0 def calcular_frete(cep, %{id: 4}), do: 0.0 def calcular_frete(cep, _), do: 10.0 * Localizacao.calcular(cep) def aplicar_desconto(total, %{id: 3}), do: total * 0.9 def aplicar_desconto(total, %{id: 4}), do: total * 0.9 def aplicar_desconto(total, _), do: total def mandar_mensagem_assinatura(%{id: 3}, _), do: nil def mandar_mensagem_assinatura(%{id: 4}, _), do: nil def mandar_mensagem_assinatura(assinatura, usuario), do: Assinatura.enviar_email_upgrade(assinatura, usuario) def imprimir(usuario, pedido) do # ... end end cálculo de frete (com regras para assinaturas)

Slide 160

Slide 160 text

defmodule CarrinhoCompra do def calcular_total(items, assinatura) do # ... end def calcular_frete(cep, %{id: 3}), do: 0.0 def calcular_frete(cep, %{id: 4}), do: 0.0 def calcular_frete(cep, _), do: 10.0 * Localizacao.calcular(cep) def aplicar_desconto(total, %{id: 3}), do: total * 0.9 def aplicar_desconto(total, %{id: 4}), do: total * 0.9 def aplicar_desconto(total, _), do: total def mandar_mensagem_assinatura(%{id: 3}, _), do: nil def mandar_mensagem_assinatura(%{id: 4}, _), do: nil def mandar_mensagem_assinatura(assinatura, usuario), do: Assinatura.enviar_email_upgrade(assinatura, usuario) def imprimir(usuario, pedido) do # ... end end cálculo de desconto (com regras para assinaturas)

Slide 161

Slide 161 text

defmodule CarrinhoCompra do def calcular_total(items, assinatura) do # ... end def calcular_frete(cep, %{id: 3}), do: 0.0 def calcular_frete(cep, %{id: 4}), do: 0.0 def calcular_frete(cep, _), do: 10.0 * Localizacao.calcular(cep) def aplicar_desconto(total, %{id: 3}), do: total * 0.9 def aplicar_desconto(total, %{id: 4}), do: total * 0.9 def aplicar_desconto(total, _), do: total def mandar_mensagem_assinatura(%{id: 3}, _), do: nil def mandar_mensagem_assinatura(%{id: 4}, _), do: nil def mandar_mensagem_assinatura(assinatura, usuario), do: Assinatura.enviar_email_upgrade(assinatura, usuario) def imprimir(usuario, pedido) do # ... end end envio de email para upgrade de assinatura

Slide 162

Slide 162 text

defmodule CarrinhoCompra do def calcular_total(items, assinatura) do # ... end def calcular_frete(cep, %{id: 3}), do: 0.0 def calcular_frete(cep, %{id: 4}), do: 0.0 def calcular_frete(cep, _), do: 10.0 * Localizacao.calcular(cep) def aplicar_desconto(total, %{id: 3}), do: total * 0.9 def aplicar_desconto(total, %{id: 4}), do: total * 0.9 def aplicar_desconto(total, _), do: total def mandar_mensagem_assinatura(%{id: 3}, _), do: nil def mandar_mensagem_assinatura(%{id: 4}, _), do: nil def mandar_mensagem_assinatura(assinatura, usuario), do: Assinatura.enviar_email_upgrade(assinatura, usuario) def imprimir(usuario, pedido) do # ... end end impressão dos dados do pedido

Slide 163

Slide 163 text

defmodule CarrinhoCompra do def calcular_total(items, assinatura) do # ... end def calcular_frete(cep, %{id: 3}), do: 0.0 def calcular_frete(cep, %{id: 4}), do: 0.0 def calcular_frete(cep, _), do: 10.0 * Localizacao.calcular(cep) def aplicar_desconto(total, %{id: 3}), do: total * 0.9 def aplicar_desconto(total, %{id: 4}), do: total * 0.9 def aplicar_desconto(total, _), do: total def mandar_mensagem_assinatura(%{id: 3}, _), do: nil def mandar_mensagem_assinatura(%{id: 4}, _), do: nil def mandar_mensagem_assinatura(assinatura, usuario), do: Assinatura.enviar_email_upgrade(assinatura, usuario) def imprimir(usuario, pedido) do # ... end end o que esse módulo faz? -> total do pedido -> desconto para assinaturas -> frete de acordo com assinaturas -> email de upgrade de assinatura -> impressão

Slide 164

Slide 164 text

defmodule CarrinhoCompra do def calcular_total(items, assinatura) do # ... end def calcular_frete(cep, %{id: 3}), do: 0.0 def calcular_frete(cep, %{id: 4}), do: 0.0 def calcular_frete(cep, _), do: 10.0 * Localizacao.calcular(cep) def aplicar_desconto(total, %{id: 3}), do: total * 0.9 def aplicar_desconto(total, %{id: 4}), do: total * 0.9 def aplicar_desconto(total, _), do: total def mandar_mensagem_assinatura(%{id: 3}, _), do: nil def mandar_mensagem_assinatura(%{id: 4}, _), do: nil def mandar_mensagem_assinatura(assinatura, usuario), do: Assinatura.enviar_email_upgrade(assinatura, usuario) def imprimir(usuario, pedido) do # ... end end o que esse módulo faz? -> total do pedido -> desconto para assinaturas -> frete de acordo com assinaturas -> email de upgrade de assinatura -> impressão

Slide 165

Slide 165 text

defmodule CarrinhoCompra do def calcular_total(items, assinatura) do # ... end def calcular_frete(cep, %{id: 3}), do: 0.0 def calcular_frete(cep, %{id: 4}), do: 0.0 def calcular_frete(cep, _), do: 10.0 * Localizacao.calcular(cep) def aplicar_desconto(total, %{id: 3}), do: total * 0.9 def aplicar_desconto(total, %{id: 4}), do: total * 0.9 def aplicar_desconto(total, _), do: total def mandar_mensagem_assinatura(%{id: 3}, _), do: nil def mandar_mensagem_assinatura(%{id: 4}, _), do: nil def mandar_mensagem_assinatura(assinatura, usuario), do: Assinatura.enviar_email_upgrade(assinatura, usuario) def imprimir(usuario, pedido) do # ... end end o que esse módulo faz? -> total do pedido -> desconto para assinaturas -> frete de acordo com assinaturas -> email de upgrade de assinatura -> impressão

Slide 166

Slide 166 text

defmodule CarrinhoCompra do def calcular_total(items, assinatura) do # ... end def calcular_frete(cep, %{id: 3}), do: 0.0 def calcular_frete(cep, %{id: 4}), do: 0.0 def calcular_frete(cep, _), do: 10.0 * Localizacao.calcular(cep) def aplicar_desconto(total, %{id: 3}), do: total * 0.9 def aplicar_desconto(total, %{id: 4}), do: total * 0.9 def aplicar_desconto(total, _), do: total def mandar_mensagem_assinatura(%{id: 3}, _), do: nil def mandar_mensagem_assinatura(%{id: 4}, _), do: nil def mandar_mensagem_assinatura(assinatura, usuario), do: Assinatura.enviar_email_upgrade(assinatura, usuario) def imprimir(usuario, pedido) do # ... end end o que esse módulo faz? -> total do pedido -> desconto para assinaturas -> frete de acordo com assinaturas -> email de upgrade de assinatura -> impressão

Slide 167

Slide 167 text

defmodule CarrinhoCompra do def calcular_total(items, assinatura) do # ... end def calcular_frete(cep, %{id: 3}), do: 0.0 def calcular_frete(cep, %{id: 4}), do: 0.0 def calcular_frete(cep, _), do: 10.0 * Localizacao.calcular(cep) def aplicar_desconto(total, %{id: 3}), do: total * 0.9 def aplicar_desconto(total, %{id: 4}), do: total * 0.9 def aplicar_desconto(total, _), do: total def mandar_mensagem_assinatura(%{id: 3}, _), do: nil def mandar_mensagem_assinatura(%{id: 4}, _), do: nil def mandar_mensagem_assinatura(assinatura, usuario), do: Assinatura.enviar_email_upgrade(assinatura, usuario) def imprimir(usuario, pedido) do # ... end end o que esse módulo faz? -> total do pedido -> desconto para assinaturas -> frete de acordo com assinaturas -> email de upgrade de assinatura -> impressão

Slide 168

Slide 168 text

defmodule CarrinhoCompra do def calcular_total(items, assinatura) do # ... end def calcular_frete(cep, %{id: 3}), do: 0.0 def calcular_frete(cep, %{id: 4}), do: 0.0 def calcular_frete(cep, _), do: 10.0 * Localizacao.calcular(cep) def aplicar_desconto(total, %{id: 3}), do: total * 0.9 def aplicar_desconto(total, %{id: 4}), do: total * 0.9 def aplicar_desconto(total, _), do: total def mandar_mensagem_assinatura(%{id: 3}, _), do: nil def mandar_mensagem_assinatura(%{id: 4}, _), do: nil def mandar_mensagem_assinatura(assinatura, usuario), do: Assinatura.enviar_email_upgrade(assinatura, usuario) def imprimir(usuario, pedido) do # ... end end o que esse módulo faz? -> total do pedido -> desconto para assinaturas -> frete de acordo com assinaturas -> email de upgrade de assinatura -> impressão

Slide 169

Slide 169 text

defmodule CarrinhoCompra do def calcular_total(items, assinatura) do # ... end def calcular_frete(cep, %{id: 3}), do: 0.0 def calcular_frete(cep, %{id: 4}), do: 0.0 def calcular_frete(cep, _), do: 10.0 * Localizacao.calcular(cep) def aplicar_desconto(total, %{id: 3}), do: total * 0.9 def aplicar_desconto(total, %{id: 4}), do: total * 0.9 def aplicar_desconto(total, _), do: total def mandar_mensagem_assinatura(%{id: 3}, _), do: nil def mandar_mensagem_assinatura(%{id: 4}, _), do: nil def mandar_mensagem_assinatura(assinatura, usuario), do: Assinatura.enviar_email_upgrade(assinatura, usuario) def imprimir(usuario, pedido) do # ... end end o que esse módulo faz? -> total do pedido -> desconto para assinaturas -> frete de acordo com assinaturas -> email de upgrade de assinatura -> impressão

Slide 170

Slide 170 text

esses contextos poderiam estar em outros módulos

Slide 171

Slide 171 text

Lista Longa de PArâmetros Long Parameter List

Slide 172

Slide 172 text

defmodule Usuario do def criar_usuario(nome, email, senha, apelido, status, ativo, admin) do # ... end end

Slide 173

Slide 173 text

defmodule Usuario do def criar_usuario(nome, email, senha, apelido, status, ativo, admin) do # ... end end

Slide 174

Slide 174 text

defmodule Usuario do def criar_usuario( nome, email, senha, apelido, status, ativo, admin ) do # ... end end Usuario.criar_usuario/7

Slide 175

Slide 175 text

Usuario.criar_usuario(

Slide 176

Slide 176 text

Usuario.criar_usuario(

Slide 177

Slide 177 text

defmodule Usuario do def criar_usuario( nome, email, senha, apelido, status, ativo, admin ) do # ... end end

Slide 178

Slide 178 text

defmodule Usuario do def criar_usuario( nome, email, senha, apelido, status, ativo, admin ) do # ... end end

Slide 179

Slide 179 text

defmodule Usuario do def criar_usuario( nome, email, senha, apelido, status, ativo, admin ) do # ... end end

Slide 180

Slide 180 text

defmodule Usuario do def criar_usuario( nome, email, senha, apelido, status, ativo, admin ) do # ... end end

Slide 181

Slide 181 text

defmodule Usuario do def criar_usuario( nome, email, senha, apelido, status, ativo, admin ) do # ... end end

Slide 182

Slide 182 text

defmodule Usuario do def criar_usuario( nome, email, senha, apelido, status, ativo, admin ) do # ... end end

Slide 183

Slide 183 text

defmodule Usuario do def criar_usuario( nome, email, senha, apelido, status, ativo, admin ) do # ... end end

Slide 184

Slide 184 text

Usuario.criar_usuario("Elaine", nome

Slide 185

Slide 185 text

Usuario.criar_usuario("Elaine", "[email protected]", nome email

Slide 186

Slide 186 text

Usuario.criar_usuario("Elaine", "[email protected]", "senha", nome email senha

Slide 187

Slide 187 text

Usuario.criar_usuario("Elaine", "[email protected]", "senha", "elainenaomi", nome email senha apelido

Slide 188

Slide 188 text

Usuario.criar_usuario("Elaine", "[email protected]", "senha", "elainenaomi", 1, nome email senha apelido status

Slide 189

Slide 189 text

Usuario.criar_usuario("Elaine", "[email protected]", "senha", "elainenaomi", 1, true, nome email senha apelido status ativo

Slide 190

Slide 190 text

Usuario.criar_usuario("Elaine", "[email protected]", "senha", "elainenaomi", 1, true, false) nome email senha apelido status ativo admin

Slide 191

Slide 191 text

status 1 - gratuito 2 - basico 3 - plus Usuario.criar_usuario("Elaine", "[email protected]", "senha", "elainenaomi", 1, true, false) nome email senha apelido status ativo admin

Slide 192

Slide 192 text

defmodule Usuario do def criar_usuario( nome, email, senha, apelido, status, ativo, admin ) do # ... end end

Slide 193

Slide 193 text

defmodule Usuario do def criar_usuario( nome, email, senha, apelido, status, ativo, admin, pais ) do # ... end end

Slide 194

Slide 194 text

defmodule Usuario do def criar_usuario( nome, email, senha, apelido, status, ativo, admin, pais ) do # ... end end

Slide 195

Slide 195 text

mais de três ou quatro parâmetros? provavelmente existe uma abstração que poderia ser criada

Slide 196

Slide 196 text

mais de três ou quatro parâmetros? provavelmente existe uma abstração que poderia ser criada

Slide 197

Slide 197 text

Inveja de recursos Feature Envy

Slide 198

Slide 198 text

Acessa mais a dados de outros contextos do que de seu próprio contexto/módulo Violação do princípio Tell, don't ask Chamadas repetidas a outro módulo

Slide 199

Slide 199 text

Acessa mais a dados de outros contextos do que de seu próprio contexto/módulo Violação do princípio Tell, don't ask Chamadas repetidas a outro módulo

Slide 200

Slide 200 text

Acessa mais a dados de outros contextos do que de seu próprio contexto/módulo Violação do princípio Tell, don't ask Chamadas repetidas a outro módulo

Slide 201

Slide 201 text

Acessa mais a dados de outros contextos do que de seu próprio contexto/módulo Violação do princípio Tell, don't ask Chamadas repetidas a outro módulo

Slide 202

Slide 202 text

defmodule Pedido do # ... def calcula_total_item(id) do item = ItemPedido.buscar_item(id) total = (item.preco + item.impostos) * item.quantidade desconto = ItemPedido.buscar_desconto(item) unless is_nil(desconto) do total - total * desconto else total end end end

Slide 203

Slide 203 text

defmodule Pedido do # ... def calcula_total_item(id) do item = ItemPedido.buscar_item(id) total = (item.preco + item.impostos) * item.quantidade desconto = ItemPedido.buscar_desconto(item) unless is_nil(desconto) do total - total * desconto else total end end end

Slide 204

Slide 204 text

defmodule Pedido do # ... def calcula_total_item(id) do item = ItemPedido.buscar_item(id) total = (item.preco + item.impostos) * item.quantidade desconto = ItemPedido.buscar_desconto(item) unless is_nil(desconto) do total - total * desconto else total end end end

Slide 205

Slide 205 text

defmodule Pedido do # ... def calcula_total_item(id) do item = ItemPedido.buscar_item(id) total = (item.preco + item.impostos) * item.quantidade desconto = ItemPedido.buscar_desconto(item) unless is_nil(desconto) do total - total * desconto else total end end end

Slide 206

Slide 206 text

defmodule Pedido do # ... def calcula_total_item(id) do item = ItemPedido.buscar_item(id) total = (item.preco + item.impostos) * item.quantidade desconto = ItemPedido.buscar_desconto(item) unless is_nil(desconto) do total - total * desconto else total end end end

Slide 207

Slide 207 text

defmodule Pedido do # ... def calcula_total_item(id) do item = ItemPedido.buscar_item(id) total = (item.preco + item.impostos) * item.quantidade desconto = ItemPedido.buscar_desconto(item) unless is_nil(desconto) do total - total * desconto else total end end end

Slide 208

Slide 208 text

defmodule Pedido do # ... def calcula_total_item(id) do item = ItemPedido.buscar_item(id) total = (item.preco + item.impostos) * item.quantidade desconto = ItemPedido.buscar_desconto(item) unless is_nil(desconto) do total - total * desconto else total end end end nesse exemplo, todos os dados vêm de ItemPedido ItemPedido poderia encapsular essa regra

Slide 209

Slide 209 text

Intimidade inapropriada Inappropriate Intimacy

Slide 210

Slide 210 text

módulo ou função que conhece e pode alterar detalhes internos de outros módulos ou funções

Slide 211

Slide 211 text

módulo ou função que conhece e pode alterar detalhes internos de outros módulos ou funções

Slide 212

Slide 212 text

módulo ou função que conhece e pode alterar detalhes internos de outros módulos ou funções

Slide 213

Slide 213 text

módulo ou função que conhece e pode alterar detalhes internos de outros módulos ou funções dentro do contexto de funções impuras

Slide 214

Slide 214 text

defmodule Pedido do def cobrar(total, usuario) do cartao = Usuario.buscar_cartao_de_credito(usuario) if (cartao.status == 3) do gateway = Cartao.buscar_gateway_pagamento(cartao) Gateway.cobrar(gateway, cartao, total) end end end

Slide 215

Slide 215 text

defmodule Pedido do def cobrar(total, usuario) do cartao = Usuario.buscar_cartao_de_credito(usuario) if (cartao.status == 3) do gateway = Cartao.buscar_gateway_pagamento(cartao) Gateway.cobrar(gateway, cartao, total) end end end

Slide 216

Slide 216 text

defmodule Pedido do def cobrar(total, usuario) do cartao = Usuario.buscar_cartao_de_credito(usuario) if (cartao.status == 3) do gateway = Cartao.buscar_gateway_pagamento(cartao) Gateway.cobrar(gateway, cartao, total) end end end detalhes internos

Slide 217

Slide 217 text

defmodule Pedido do def cobrar(total, usuario) do cartao = Usuario.buscar_cartao_de_credito(usuario) if (cartao.status == 3) do gateway = Cartao.buscar_gateway_pagamento(cartao) Gateway.cobrar(gateway, cartao, total) end end end

Slide 218

Slide 218 text

defmodule Pedido do def cobrar(total, usuario) do cartao = Usuario.buscar_cartao_de_credito(usuario) if (cartao.status == 3) do gateway = Cartao.buscar_gateway_pagamento(cartao) Gateway.cobrar(gateway, cartao, total) end end end provavelmente essa regra deveria estar encapsulada

Slide 219

Slide 219 text

Cirurgia com Rifle Shotgun Surgery

Slide 220

Slide 220 text

No content

Slide 221

Slide 221 text

No content

Slide 222

Slide 222 text

No content

Slide 223

Slide 223 text

No content

Slide 224

Slide 224 text

No content

Slide 225

Slide 225 text

No content

Slide 226

Slide 226 text

é só um exemplo

Slide 227

Slide 227 text

mas baseados em fatos reais

Slide 228

Slide 228 text

mas baseados em fatos reais

Slide 229

Slide 229 text

Mesma alteração em vários arquivos Buscas globais para toda alteração

Slide 230

Slide 230 text

Mesma alteração em vários arquivos Buscas globais para toda alteração

Slide 231

Slide 231 text

Mesma alteração em vários arquivos Buscas globais para toda alteração

Slide 232

Slide 232 text

defmodule CarrinhoCompra do # ... def calcular_frete(cep, %{id: 3}), do: 0.0 def calcular_frete(cep, %{id: 4}), do: 0.0 def calcular_frete(cep, _), do: 10.0 * Localizacao.calcular(cep) def aplicar_desconto(total, %{id: 3}), do: total * 0.9 def aplicar_desconto(total, %{id: 4}), do: total * 0.9 def aplicar_desconto(total, _), # sem desconto do: total end

Slide 233

Slide 233 text

defmodule CarrinhoCompra do # ... def calcular_frete(cep, %{id: 3}), do: 0.0 def calcular_frete(cep, %{id: 4}), do: 0.0 def calcular_frete(cep, _), do: 10.0 * Localizacao.calcular(cep) def aplicar_desconto(total, %{id: 3}), do: total * 0.9 def aplicar_desconto(total, %{id: 4}), do: total * 0.9 def aplicar_desconto(total, _), # sem desconto do: total end

Slide 234

Slide 234 text

defmodule CarrinhoCompra do # ... def calcular_frete(cep, %{id: 3}), do: 0.0 def calcular_frete(cep, %{id: 4}), do: 0.0 def calcular_frete(cep, _), do: 10.0 * Localizacao.calcular(cep) def aplicar_desconto(total, %{id: 3}), do: total * 0.9 def aplicar_desconto(total, %{id: 4}), do: total * 0.9 def aplicar_desconto(total, _), # sem desconto do: total end

Slide 235

Slide 235 text

defmodule CarrinhoCompra do # ... def calcular_frete(cep, %{id: 3}), do: 0.0 def calcular_frete(cep, %{id: 4}), do: 0.0 def calcular_frete(cep, _), do: 10.0 * Localizacao.calcular(cep) def aplicar_desconto(total, %{id: 3}), do: total * 0.9 def aplicar_desconto(total, %{id: 4}), do: total * 0.9 def aplicar_desconto(total, _), # sem desconto do: total end Enum.member?([3, 4], assinatura.id)

Slide 236

Slide 236 text

defmodule CarrinhoCompra do # ... def calcular_frete(cep, %{id: 3}), do: 0.0 def calcular_frete(cep, %{id: 4}), do: 0.0 def calcular_frete(cep, _), do: 10.0 * Localizacao.calcular(cep) def aplicar_desconto(total, %{id: 3}), do: total * 0.9 def aplicar_desconto(total, %{id: 4}), do: total * 0.9 def aplicar_desconto(total, _), # sem desconto do: total end Enum.member?([3, 4], assinatura.id)

Slide 237

Slide 237 text

defmodule CarrinhoCompra do # ... def calcular_frete(cep, %{id: 3}), do: 0.0 def calcular_frete(cep, %{id: 4}), do: 0.0 def calcular_frete(cep, _), do: 10.0 * Localizacao.calcular(cep) def aplicar_desconto(total, %{id: 3}), do: total * 0.9 def aplicar_desconto(total, %{id: 4}), do: total * 0.9 def aplicar_desconto(total, _), # sem desconto do: total end Enum.member?([3, 4, 8], assinatura.id)

Slide 238

Slide 238 text

defmodule CarrinhoCompra do # ... def calcular_frete(cep, %{id: 3}), do: 0.0 def calcular_frete(cep, %{id: 4}), do: 0.0 def calcular_frete(cep, _), do: 10.0 * Localizacao.calcular(cep) def aplicar_desconto(total, %{id: 3}), do: total * 0.9 def aplicar_desconto(total, %{id: 4}), do: total * 0.9 def aplicar_desconto(total, _), # sem desconto do: total end Enum.member?([3, 4, 8], assinatura.id)

Slide 239

Slide 239 text

defmodule CarrinhoCompra do # ... def calcular_frete(cep, %{id: 3}), do: 0.0 def calcular_frete(cep, %{id: 4}), do: 0.0 def calcular_frete(cep, _), do: 10.0 * Localizacao.calcular(cep) def aplicar_desconto(total, %{id: 3}), do: total * 0.9 def aplicar_desconto(total, %{id: 4}), do: total * 0.9 def aplicar_desconto(total, _), # sem desconto do: total end Enum.member?([3, 4, 8], assinatura.id) apenas seguindo o design...

Slide 240

Slide 240 text

defmodule CarrinhoCompra do # ... def calcular_frete(cep, %{id: 3}), do: 0.0 def calcular_frete(cep, %{id: 4}), do: 0.0 def calcular_frete(cep, %{id: 8}), do: 0.0 def calcular_frete(cep, _), do: 10.0 * Localizacao.calcular(cep) def aplicar_desconto(total, %{id: 3}), do: total * 0.9 def aplicar_desconto(total, %{id: 4}), do: total * 0.9 def aplicar_desconto(total, %{id: 8}), do: total * 0.9 # ... end Enum.member?([3, 4, 8], assinatura.id)

Slide 241

Slide 241 text

defmodule CarrinhoCompra do # ... def calcular_frete(cep, %{id: 3}), do: 0.0 def calcular_frete(cep, %{id: 4}), do: 0.0 def calcular_frete(cep, %{id: 8}), do: 0.0 def calcular_frete(cep, _), do: 10.0 * Localizacao.calcular(cep) def aplicar_desconto(total, %{id: 3}), do: total * 0.9 def aplicar_desconto(total, %{id: 4}), do: total * 0.9 def aplicar_desconto(total, %{id: 8}), do: total * 0.9 # ... end Enum.member?([3, 4, 8], assinatura.id) e buscando outros lugares para alterar...

Slide 242

Slide 242 text

defmodule Renovacao do def executar(%{id: id}, user) do case id do 3 -> Cobranca.cobrar(user, id, 100.0) 4 -> Cobranca.cobrar(user, id, 95.0) 2 -> Cobranca.cobrar(user, id, 50.0) end end end

Slide 243

Slide 243 text

defmodule Renovacao do def executar(%{id: id}, user) do case id do 3 -> Cobranca.cobrar(user, id, 100.0) 4 -> Cobranca.cobrar(user, id, 95.0) 2 -> Cobranca.cobrar(user, id, 50.0) end end end

Slide 244

Slide 244 text

defmodule Renovacao do def executar(%{id: id}, user) do case id do 3 -> Cobranca.cobrar(user, id, 100.0) 4 -> Cobranca.cobrar(user, id, 95.0) 8 -> Cobranca.cobrar(user, id, 90.0) 2 -> Cobranca.cobrar(user, id, 50.0) end end end

Slide 245

Slide 245 text

defmodule Renovacao do def executar(%{id: id}, user) do case id do 3 -> Cobranca.cobrar(user, id, 100.0) 4 -> Cobranca.cobrar(user, id, 95.0) 8 -> Cobranca.cobrar(user, id, 90.0) 2 -> Cobranca.cobrar(user, id, 50.0) end end end

Slide 246

Slide 246 text

Alteração Divergente Divergent Change

Slide 247

Slide 247 text

defmodule Conta do # ... def logar_via_google(dados_google) do # ... end def logar_via_apple(dados_apple) do # ... end def cobrar_assinatura(assinatura) do # ... end def calcular_desconto_assinatura(assinatura) do # ... end end

Slide 248

Slide 248 text

defmodule Conta do # ... def logar_via_google(dados_google) do # ... end def logar_via_apple(dados_apple) do # ... end def cobrar_assinatura(assinatura) do # ... end def calcular_desconto_assinatura(assinatura) do # ... end end

Slide 249

Slide 249 text

defmodule Conta do # ... def logar_via_google(dados_google) do # ... end def logar_via_apple(dados_apple) do # ... end def cobrar_assinatura(assinatura) do # ... end def calcular_desconto_assinatura(assinatura) do # ... end end

Slide 250

Slide 250 text

defmodule Conta do # ... def logar_via_google(dados_google) do # ... end def logar_via_apple(dados_apple) do # ... end def cobrar_assinatura(assinatura) do # ... end def calcular_desconto_assinatura(assinatura) do # ... end end mais de uma razão para mudar

Slide 251

Slide 251 text

Módulo alterado constantemente Mudanças não relacionadas

Slide 252

Slide 252 text

Módulo alterado constantemente Mudanças não relacionadas

Slide 253

Slide 253 text

Módulo alterado constantemente Mudanças não relacionadas Provavelmente precisamos de novas abstrações

Slide 254

Slide 254 text

Generalidade Especulativa Speculative Generalization

Slide 255

Slide 255 text

campo, parâmetro, função ou módulo não utilizado

Slide 256

Slide 256 text

campo, parâmetro, função ou módulo não utilizado

Slide 257

Slide 257 text

campo, parâmetro, função ou módulo não utilizado adicionado para o futuro...

Slide 258

Slide 258 text

defmodule Pessoa do def formatar_cpf(%{cpf: cpf}) do # ... end end

Slide 259

Slide 259 text

defmodule PessoaFisica do def formatar_documento(%{cpf: cpf}) do # ... end end

Slide 260

Slide 260 text

defmodule PessoaFisica do def formatar_documento(%{cpf: cpf}) do # ... end end defmodule PessoaJuridica do def formatar_documento(%{cnpj: cnpj}) do # ... end end

Slide 261

Slide 261 text

defmodule PessoaFisica do def formatar_documento(%{cpf: cpf}) do # ... end end defmodule PessoaJuridica do def formatar_documento(%{cnpj: cnpj}) do # ... end end

Slide 262

Slide 262 text

defmodule PessoaFisica do def formatar_documento(%{cpf: cpf}) do # ... end end defmodule PessoaJuridica do def formatar_documento(%{cnpj: cnpj}) do # ... end end utilizado em lugar nenhum nesse exemplo

Slide 263

Slide 263 text

defmodule Assinatura do # ... def enviar_email(assinatura, true), do: Email.enviar_email_assinatura_ativa(assinatura) def enviar_email(assinatura, false), do: nil end

Slide 264

Slide 264 text

defmodule Assinatura do # ... def enviar_email(assinatura, true), do: Email.enviar_email_assinatura_ativa(assinatura) def enviar_email(assinatura, false), do: nil end flag

Slide 265

Slide 265 text

defmodule Assinatura do # ... def enviar_email(assinatura, true), do: Email.enviar_email_assinatura_ativa(assinatura) def enviar_email(assinatura, false), do: nil end sempre true...

Slide 266

Slide 266 text

defmodule CarrinhoCompra do # ... def desconto_item(%{categoria: categoria} = produto) do case categoria do _ -> produto.desconto end end end

Slide 267

Slide 267 text

defmodule CarrinhoCompra do # ... def desconto_item(%{categoria: categoria} = produto) do case categoria do _ -> produto.desconto end end end forçando um padrão

Slide 268

Slide 268 text

defmodule CarrinhoCompra do # ... def desconto_item(%{categoria: categoria} = produto) do case categoria do _ -> produto.desconto end end end que não será utilizado

Slide 269

Slide 269 text

módulos, funções, argumentos utilizados somente pelos casos de teste

Slide 270

Slide 270 text

módulos, funções, argumentos utilizados somente pelos casos de teste

Slide 271

Slide 271 text

Comentários Comments

Slide 272

Slide 272 text

esse é polêmico

Slide 273

Slide 273 text

qualquer comentário?

Slide 274

Slide 274 text

é sobre comentários que explicam o que a função faz

Slide 275

Slide 275 text

é sobre comentários que explicam o que a função faz

Slide 276

Slide 276 text

defmodule Pedido do def cobrar(total, usuario) do cartao = Usuario.buscar_cartao_de_credito(usuario) # verifica se o cartão está ativo if (cartao.status == 3) do # ... end end end

Slide 277

Slide 277 text

defmodule Pedido do def cobrar(total, usuario) do cartao = Usuario.buscar_cartao_de_credito(usuario) # verifica se o cartão está ativo if (cartao.status == 3) do # ... end end end

Slide 278

Slide 278 text

e vale sempre lembrar

Slide 279

Slide 279 text

comentário != documentação

Slide 280

Slide 280 text

comentário != documentação

Slide 281

Slide 281 text

doctests

Slide 282

Slide 282 text

@doc """ Updates the `key` in `map` with the given function. If `key` is present in `map` then the existing value is passed to `fun` and its result is used as the updated value of `key`. If `key` is not present in `map`, `default` is inserted as the value of `key`. The default value will not be passed through the update function. ## Examples iex> Map.update(%{a: 1}, :a, 13, fn existing_value -> existing_value * 2 end) %{a: 2} iex> Map.update(%{a: 1}, :b, 11, fn existing_value -> existing_value * 2 end) %{a: 1, b: 11} """ @spec update(map, key, default :: value, (existing_value :: value -> updated_value :: value)) :: map def update(map, key, default, fun) when is_function(fun, 1) do # ...

Slide 283

Slide 283 text

Agrupamento de dados (Data Clumps) Obsessão por primitivos (Primitive Obsession) Switches Repetidos (Repeated Switches) Campo temporário (Temporary Field) Cadeias de mensagens (Message Chains) Intermediário (Middle man) Outros code smells

Slide 284

Slide 284 text

Trocas escusas (Insider Trading) Classes alternativas com interfaces diferentes (Alternative Classes with Different Interfaces) Classe de dados (Data Class) Herança recusada (Refused Bequest) Elemento ocioso (Lazy Element)

Slide 285

Slide 285 text

Refatoração Na PRÁTICA DEFINIÇÃO Code SMells Resumo e referências

Slide 286

Slide 286 text

No content

Slide 287

Slide 287 text

refatorar? detecção de Code Smell executar a refatoração fazer commit se sim compilar executar testes

Slide 288

Slide 288 text

refatorar? detecção de Code Smell executar a refatoração fazer commit se sim compilar executar testes

Slide 289

Slide 289 text

refatorar? detecção de Code Smell executar a refatoração fazer commit se sim compilar executar testes

Slide 290

Slide 290 text

refatorar? detecção de Code Smell executar a refatoração fazer commit se sim compilar executar testes

Slide 291

Slide 291 text

refatorar? detecção de Code Smell executar a refatoração fazer commit se sim compilar executar testes

Slide 292

Slide 292 text

refatorar? detecção de Code Smell executar a refatoração fazer commit se sim compilar executar testes

Slide 293

Slide 293 text

refatorar? detecção de Code Smell executar a refatoração fazer commit se sim compilar executar testes

Slide 294

Slide 294 text

refatorar? detecção de Code Smell executar a refatoração fazer commit se sim compilar executar testes

Slide 295

Slide 295 text

refatorar? detecção de Code Smell executar a refatoração fazer commit se sim compilar executar testes

Slide 296

Slide 296 text

alguns exemplos de refatorações

Slide 297

Slide 297 text

Renomear Variável Rename variable

Slide 298

Slide 298 text

defmodule Ordem do # ... def exec(i, d) do # ... end end

Slide 299

Slide 299 text

defmodule Ordem do # ... def exec(itens, desconto_total) do # ... end end

Slide 300

Slide 300 text

defmodule Ordem do # ... def exec(itens, desconto_total) do # ... end end representações do contexto de negócio

Slide 301

Slide 301 text

MUDAR DEClaraçÃo de Função Change Function Declaration

Slide 302

Slide 302 text

defmodule Ordem do # ... def exec(itens, desconto_total) do # ... end end o que essa função faz?

Slide 303

Slide 303 text

defmodule Ordem do # ... def calcular_total(itens, desconto_total) do # ... end end

Slide 304

Slide 304 text

defmodule Assinatura do # ... def enviar_email(assinatura, true), do: Email.enviar_email_assinatura_ativa(assinatura) def enviar_email(assinatura, false), do: nil end

Slide 305

Slide 305 text

defmodule Assinatura do # ... def enviar_email(assinatura), do: Email.enviar_email_assinatura_ativa(assinatura) end remover argumentos

Slide 306

Slide 306 text

defmodule Assinatura do # ... def enviar_email(%Assinatura{} = assinatura), do: Email.enviar_email_assinatura_ativa(assinatura) end alterar assinatura da função

Slide 307

Slide 307 text

Introduzir objeto de parâmetros Introduce Parameter Object

Slide 308

Slide 308 text

abstração? módulo? struct? estrutura de dados? Introduzir objeto de parâmetros Introduce Parameter Object

Slide 309

Slide 309 text

defmodule Relatorio do # ... def extrato_por_intervalo(inicio,fim) do # ... end end extrato_por_intervalo/2

Slide 310

Slide 310 text

defmodule Relatorio do # ... def extrato_por_intervalo(intervalo) do # ... end end %{inicio: "2020-01-10", fim: "2020-02-10"} extrato_por_intervalo/1

Slide 311

Slide 311 text

defmodule Relatorio do # ... def extrato_por_intervalo(%{ inicio: inicio, fim: fim }) do # ... end end extrato_por_intervalo/1 pattern matching

Slide 312

Slide 312 text

defmodule Usuario do def criar_usuario( nome, email, senha, apelido, status, ativo, admin ) do # ... end end

Slide 313

Slide 313 text

defmodule Usuario do def criar_usuario(user) do # ... end end

Slide 314

Slide 314 text

defmodule Usuario do def criar_usuario(%Usuario{} = usuario) do # ... end end novo_usuario = %Usuario{ nome: "Elaine", email: "[email protected]", senha: "123", apelido: "elainenaomi", status: 1, ativo: true, admin: false } Usuario.criar_usuario(novo_usuario)

Slide 315

Slide 315 text

REnomear campo Rename Field

Slide 316

Slide 316 text

defmodule Ordem do # ... def calcular_total(itens, desconto_total) do items |> Enum.map(& &1.preco * &1.qtd) |> Enum.sum() |> Kernel.-(desconto) end end %{preco: 99.99, qtd: 1}

Slide 317

Slide 317 text

defmodule Ordem do # ... def calcular_total(itens, desconto_total) do items |> Enum.map(& &1.preco * &1.quantidade) |> Enum.sum() |> Kernel.-(desconto) end end %{preco: 99.99, quantidade: 1}

Slide 318

Slide 318 text

defmodule Ordem do # ... def calcular_total(itens, desconto_total) do items |> Enum.map(&calcular_total_item/1) |> Enum.sum() |> Kernel.-(desconto) end end Extrair função

Slide 319

Slide 319 text

defmodule Ordem do # ... def calcular_total_item(%{qtd: _} = item), do: item.preco * item.qtd def calcular_total_item(item), do: item.preco * item.quantidade end campo legado (em migração)

Slide 320

Slide 320 text

defmodule Ordem do # ... def calcular_total_item(%{qtd: _} = item), do: item.preco * item.qtd def calcular_total_item(item), do: item.preco * item.quantidade end novo campo

Slide 321

Slide 321 text

defmodule Ordem do # ... def calcular_total(itens, desconto_total) do items |> Enum.map(& &1.preco * &1.qtd) |> Enum.sum() |> Kernel.-(desconto) end end %Ordem.ItemPedido {preco: 99.99, qtd: 1}

Slide 322

Slide 322 text

defmodule Ordem.ItemPedido do @type t :: %__MODULE__{ id: String.t(), nome: String.t(), preco: Decimal.t(), qtd: integer() } defstruct id: nil, nome: nil, preco: nil, qtd: nil end

Slide 323

Slide 323 text

defmodule Ordem.ItemPedido do @type t :: %__MODULE__{ id: String.t(), nome: String.t(), preco: Decimal.t(), quantidade: integer() } defstruct id: nil, nome: nil, preco: nil, quantidade: nil end

Slide 324

Slide 324 text

defmodule Ordem.ItemPedido do @type t :: %__MODULE__{ id: String.t(), nome: String.t(), preco: Decimal.t(), quantidade: integer() } defstruct id: nil, nome: nil, preco: nil, quantidade: nil end compilador <3

Slide 325

Slide 325 text

Extrair FUnção Extract Function

Slide 326

Slide 326 text

defmodule Relatorio do def imprimir(usuario, pedido) do imprimir_dados_empresa() IO.puts("Nome: #{usuario.nome}") pedido.items |> Enum.filter(&(&1.status == 3)) |> Enum.map(fn item -> IO.puts("Item: #{item.nome}") IO.puts("Preço: #{item.preco}") IO.puts("Quantidade: #{item.quantidade}") total = item.preco * item.quantidade IO.puts("Total: #{total}") end) end end

Slide 327

Slide 327 text

defmodule Relatorio do def imprimir(usuario, pedido) do imprimir_dados_empresa() IO.puts("Nome: #{usuario.nome}") pedido.items |> Enum.filter(&(&1.status == 3)) |> Enum.map(fn item -> IO.puts("Item: #{item.nome}") IO.puts("Preço: #{item.preco}") IO.puts("Quantidade: #{item.quantidade}") total = item.preco * item.quantidade IO.puts("Total: #{total}") end) end end

Slide 328

Slide 328 text

defmodule Relatorio do def imprimir(usuario, pedido) do imprimir_dados_empresa() imprimir_dados_usuário(usuario) pedido.items |> Enum.filter(&(&1.status == 3)) |> Enum.map(fn item -> IO.puts("Item: #{item.nome}") IO.puts("Preço: #{item.preco}") IO.puts("Quantidade: #{item.quantidade}") total = item.preco * item.quantidade IO.puts("Total: #{total}") end) end end

Slide 329

Slide 329 text

defmodule Relatorio do def imprimir(usuario, pedido) do imprimir_dados_empresa() imprimir_dados_usuário(usuario) pedido.items |> Enum.filter(&(&1.status == 3)) |> Enum.map(fn item -> IO.puts("Item: #{item.nome}") IO.puts("Preço: #{item.preco}") IO.puts("Quantidade: #{item.quantidade}") total = item.preco * item.quantidade IO.puts("Total: #{total}") end) end end

Slide 330

Slide 330 text

defmodule Relatorio do def imprimir(usuario, pedido) do imprimir_dados_empresa() imprimir_dados_usuário(usuario) imprimir_dados_pedido(pedido) end # ... end

Slide 331

Slide 331 text

Mover FunçÃO Move Function

Slide 332

Slide 332 text

defmodule Pedido do # ... def calcula_total_item(id) do item = ItemPedido.buscar_item(id) total = (item.preco + item.impostos) * item.quantidade desconto = ItemPedido.buscar_desconto(item) unless is_nil(desconto) do total - total * desconto else total end end end

Slide 333

Slide 333 text

defmodule Pedido do # ... def calcula_total_item(id) do item = ItemPedido.buscar_item(id) total = (item.preco + item.impostos) * item.quantidade desconto = ItemPedido.buscar_desconto(item) unless is_nil(desconto) do total - total * desconto else total end end end

Slide 334

Slide 334 text

defmodule Pedido do # ... def calcula_total_item(id) do ItemPedido.calcular_valor_item(id) end end

Slide 335

Slide 335 text

defmodule Pedido do # ... def calcula_total_item(id) do ItemPedido.calcular_valor_item(id) end end podem aparecer outros code smells nesse processo

Slide 336

Slide 336 text

defmodule Pedido do # ... def calcula_total_item(id) do ItemPedido.calcular_valor_item(id) end end podem aparecer outros code smells nesse processo mas é só continuar o ciclo: refatorar, executar testes, fazer commit e reiniciar o processo

Slide 337

Slide 337 text

Remover código morto Remove Dead Code

Slide 338

Slide 338 text

defmodule CarrinhoCompras do def calcular_preco_total(item, _flag) do total = item.preco * item.quantidade if false do # desconto novo usuario total * 0.9 end total end end

Slide 339

Slide 339 text

defmodule CarrinhoCompras do def calcular_preco_total(item, _flag) do total = item.preco * item.quantidade if false do # desconto novo usuario total * 0.9 end total end end

Slide 340

Slide 340 text

defmodule CarrinhoCompras do def calcular_preco_total(item, _flag) do total = item.preco * item.quantidade if false do # desconto novo usuario total * 0.9 end total end end

Slide 341

Slide 341 text

defmodule CarrinhoCompras do def calcular_preco_total(item, _flag) do total = item.preco * item.quantidade if false do # desconto novo usuario total * 0.9 end total end end

Slide 342

Slide 342 text

defmodule CarrinhoCompras do def calcular_preco_total(item) do item.preco * item.quantidade end end

Slide 343

Slide 343 text

Combinar Funções em Classe Combine Functions into Class

Slide 344

Slide 344 text

Combinar Funções em Classe Combine Functions into Class módulo

Slide 345

Slide 345 text

defmodule Conta do # ... def logar_via_google(dados_google) do # ... end def logar_via_apple(dados_apple) do # ... end def cobrar_assinatura(assinatura) do # ... end def calcular_desconto_assinatura(assinatura) do # ... end end

Slide 346

Slide 346 text

defmodule Conta do # ... def logar_via_google(dados_google) do # ... end def logar_via_apple(dados_apple) do # ... end def cobrar_assinatura(assinatura) do # ... end def calcular_desconto_assinatura(assinatura) do # ... end end defmodule Login do # ... def logar_via_google(dados_google) do # ... end def logar_via_apple(dados_apple) do # ... end end novo módulo

Slide 347

Slide 347 text

defmodule Conta do # ... def logar_via_google(dados_google) do # ... end def logar_via_apple(dados_apple) do # ... end def cobrar_assinatura(assinatura) do # ... end def calcular_desconto_assinatura(assinatura) do # ... end end

Slide 348

Slide 348 text

defmodule Conta do # ... def logar_via_google(dados_google) do # ... end def logar_via_apple(dados_apple) do # ... end def cobrar_assinatura(assinatura) do # ... end def calcular_desconto_assinatura(assinatura) do # ... end end defmodule Assinatura do def cobrar_assinatura(assinatura) do # ... end def calcular_desconto_assinatura(assinatura) do # ... end end novo módulo

Slide 349

Slide 349 text

defmodule Conta do # ... end defmodule Login do # ... end defmodule Assinatura do # ... end melhor isolamento dos domínios e regras de negócio

Slide 350

Slide 350 text

Remover subclasse Remove subclass

Slide 351

Slide 351 text

Remover subclasse Remove subclass subtipo

Slide 352

Slide 352 text

Remover subclasse Remove subclass "submódulo"

Slide 353

Slide 353 text

defmodule PessoaFisica do def formatar_documento(%{cpf: cpf}) do # ... end end defmodule PessoaJuridica do def formatar_documento(%{cnpj: cnpj}) do # ... end end

Slide 354

Slide 354 text

defmodule PessoaFisica do def formatar_documento(%{cpf: cpf}) do # ... end end defmodule PessoaJuridica do def formatar_documento(%{cnpj: cnpj}) do # ... end end PessoaJuridica não é utilizado nesse exemplo

Slide 355

Slide 355 text

defmodule PessoaFisica do def formatar_documento(%{cpf: cpf}) do # ... end end

Slide 356

Slide 356 text

defmodule Pessoa do def formatar_documento(%{cpf: cpf}) do # ... end end módulo representando a linguagem do domínio se não existe pessoa jurídica, não faz sentido diferenciar pessoa física

Slide 357

Slide 357 text

Substituir condicional por polimorfismo Replace Conditional With Polymorphism

Slide 358

Slide 358 text

defmodule Relatorio do def enviar_relatorio(usuario, conteudo, tipo) do case tipo do :pdf -> PDF.criar(conteudo) :texto -> Texto.gerar(conteudo, margin: false) _ -> nil end |> Email.enviar(usuario.email, conteudo) end end

Slide 359

Slide 359 text

defmodule Relatorio do def enviar_relatorio(usuario, conteudo, tipo) do case tipo do :pdf -> PDF.criar(conteudo) :texto -> Texto.gerar(conteudo, margin: false) _ -> nil end |> Email.enviar(usuario.email, conteudo) end end

Slide 360

Slide 360 text

defmodule Relatorio do def enviar_relatorio(usuario, conteudo, tipo) do case tipo do :pdf -> PDF.criar(conteudo) :texto -> Texto.gerar(conteudo, margin: false) _ -> nil end |> Email.enviar(usuario.email, conteudo) end end podemos definir uma interface

Slide 361

Slide 361 text

usando protocols

Slide 362

Slide 362 text

defprotocol Relatorio do def criar_arquivo(conteudo) end defimpl Relatorio, for: PDF do def criar_arquivo(linhas), do: PDF.criar(linhas) end defimpl Relatorio, for: Texto do def criar_arquivo(conteudo), do: Texto.exibir(conteudo) end

Slide 363

Slide 363 text

defprotocol Relatorio do def criar_arquivo(conteudo) end defimpl Relatorio, for: PDF do def criar_arquivo(linhas), do: PDF.criar(linhas) end defimpl Relatorio, for: Texto do def criar_arquivo(conteudo), do: Texto.exibir(conteudo) end

Slide 364

Slide 364 text

defprotocol Relatorio do def criar_arquivo(conteudo) end defimpl Relatorio, for: PDF do def criar_arquivo(linhas), do: PDF.criar(linhas) end defimpl Relatorio, for: Texto do def criar_arquivo(conteudo), do: Texto.exibir(conteudo) end

Slide 365

Slide 365 text

defprotocol Relatorio do def criar_arquivo(conteudo) end defimpl Relatorio, for: PDF do def criar_arquivo(linhas), do: PDF.criar(linhas) end defimpl Relatorio, for: Texto do def criar_arquivo(conteudo), do: Texto.exibir(conteudo) end

Slide 366

Slide 366 text

defmodule Relatorio do def enviar_relatorio(usuario, conteudo, tipo) do case tipo do :pdf -> PDF.criar(conteudo) :texto -> Texto.gerar(conteudo, margin: false) _ -> nil end |> Email.enviar(usuario.email, conteudo) end end

Slide 367

Slide 367 text

defmodule Relatorio do def enviar_relatorio(usuario, conteudo) do conteudo |> Relatorio.criar_arquivo() |> Email.enviar(usuario.email, conteudo) end end

Slide 368

Slide 368 text

Internalizar função (Inline Function) Extrair variável (Extract Variable) Internalizar variável (Inline Variable) Encapsular variável (Encapsulate Variable) Combinar funções em transformação (Combine Functions into Transform) Separar em fases (Split Phase) Encapsular registro (Encapsulate Record) Encapsular coleção (Encapsulate Collection) Substituir primitivo por objeto (Replace Primitive with Object) Substituir variável temporária por consulta (Replace Temp with Query) Internalizar classe (Inline Class) Ocultar delegação (Hide Delegate) Remover intermediário (Remove Middle Man) Outras refatorações

Slide 369

Slide 369 text

Substituir algoritmo (Substitute Algorithm) Mover campo (Move Field) Mover instruções para uma função (Move statements into Function) Mover instruções para os pontos de chamada (Move Statements to Callers) Substituir código internalizado por chamada de função (Replace Inline Code with Function Call) Deslocar instruções (Slide Statements) Dividir laço (Split Loop) Substituir laço por pipeline (Replace Loop with Pipeline) Separar variável (Split Variable) Substituir variável derivada por consulta (Replace Derived Variable with Query) Mudar referência para valor (Change Reference to Value) Mudar valor para referência (Change Value to Reference) Decompor condicional (Decompose Conditional) Consolidar expressão condicional (Consolidate Conditional Expression) Substituir condicional aninhada por cláusulas de guarda (Replace Nested Conditional with Guard Clauses)

Slide 370

Slide 370 text

Introduzir caso especial (Introduce Special Case) Introduzir asserção (Introduce Assertion) Separar consulta de modificador (Separate Query from Modifier) Parametrizar função (Parameterize Function) Remover argumento de flag (Remove Flag Argument) Preservar objeto inteiro (Preserve Whole Object) Substituir parâmetro por consulta (Replace Parameter with Query) Substituir consulta por parâmetro (Replace Query with Parameter) Remover método de escrita (Remove Setting Method) Substituir construtor por função de factory (Replace Constructor with Factory Function) Substituir função por comando (Replace Function with Command) Substituir comando por função (Replace Command with Function) Subir método (Pull Up Method) Subir campo (Pull Up Field) Subir corpo do construtor (Pull Up Constructor Body)

Slide 371

Slide 371 text

Extrair superclasse (Extract Superclass) Substituir subclasse por delegação (Replace Subclass with Delegate) Substituir superclasse por delegação (Replace Superclass with Delegate) Descer método (Push Down Method) Descer campo (Push Down Field) Substituir código de tipos por subclasses (Replace Type Code with Subclasses)

Slide 372

Slide 372 text

No content

Slide 373

Slide 373 text

Coesão Encapsulamento Acoplamento Abstração de Tipos

Slide 374

Slide 374 text

Coesão Encapsulamento Acoplamento Abstração de Tipos uma lista dos principais objetivos no processo de refatoração

Slide 375

Slide 375 text

não esqueçam da importância dos testes

Slide 376

Slide 376 text

não esqueçam da importância dos testes

Slide 377

Slide 377 text

ciclo contínuo de feedback

Slide 378

Slide 378 text

ciclo contínuo de feedback

Slide 379

Slide 379 text

e ferramentas podem auxiliar nesse processo

Slide 380

Slide 380 text

hexdocs.pm/credo

Slide 381

Slide 381 text

refatoração, identificação de bugs, consistência de código, etc

Slide 382

Slide 382 text

DEFINIÇÃO Code SMells Refatoração Na PRÁTICA Resumo e referências

Slide 383

Slide 383 text

No content

Slide 384

Slide 384 text

DESIGN DE CÓDIGO sobre a estrutura do código sobre abstrações do domínio e regras de negócio sobre código fácil de ser compreendido e alterado sobre como as abstrações interagem entre si

Slide 385

Slide 385 text

DESIGN DE CÓDIGO sobre a estrutura do código sobre abstrações do domínio e regras de negócio sobre código fácil de ser compreendido e alterado sobre como as abstrações interagem entre si

Slide 386

Slide 386 text

DESIGN DE CÓDIGO sobre a estrutura do código sobre abstrações do domínio e regras de negócio sobre código fácil de ser compreendido e alterado sobre como as abstrações interagem entre si

Slide 387

Slide 387 text

DESIGN DE CÓDIGO sobre a estrutura do código sobre abstrações do domínio e regras de negócio sobre código fácil de ser compreendido e alterado sobre como as abstrações interagem entre si

Slide 388

Slide 388 text

DESIGN DE CÓDIGO sobre a estrutura do código sobre abstrações do domínio e regras de negócio sobre código fácil de ser compreendido e alterado sobre como as abstrações interagem entre si

Slide 389

Slide 389 text

DESIGN DE CÓDIGO sobre a estrutura do código sobre abstrações do domínio e regras de negócio sobre código fácil de ser compreendido e alterado sobre como as abstrações interagem entre si

Slide 390

Slide 390 text

apontam possíveis problemas de design alertam sobre redução da qualidade indicam possível impacto no negócio CODE SMELLS

Slide 391

Slide 391 text

apontam possíveis problemas de design alertam sobre redução da qualidade indicam possível impacto no negócio CODE SMELLS

Slide 392

Slide 392 text

apontam possíveis problemas de design alertam sobre redução da qualidade indicam possível impacto no negócio CODE SMELLS

Slide 393

Slide 393 text

apontam possíveis problemas de design alertam sobre redução da qualidade indicam possível impacto no negócio CODE SMELLS

Slide 394

Slide 394 text

apontam possíveis problemas de design alertam sobre redução da qualidade indicam possível impacto no negócio CODE SMELLS

Slide 395

Slide 395 text

REFATORAÇÃO

Slide 396

Slide 396 text

reorganização contínua do código

Slide 397

Slide 397 text

reorganização contínua do código

Slide 398

Slide 398 text

é sobre identificação de code smells e possíveis refatorações

Slide 399

Slide 399 text

é sobre identificação de code smells e possíveis refatorações

Slide 400

Slide 400 text

é sobre pequenas mudanças

Slide 401

Slide 401 text

é sobre pequenas mudanças

Slide 402

Slide 402 text

é sobre evoluir o design do código de maneira saudável

Slide 403

Slide 403 text

é sobre evoluir o design do código de maneira saudável

Slide 404

Slide 404 text

revisão de código por pares pode ser uma parte importante desse processo

Slide 405

Slide 405 text

revisão de código por pares pode ser uma parte importante desse processo

Slide 406

Slide 406 text

https://sourcelevel.io/code-review-ebook

Slide 407

Slide 407 text

No content

Slide 408

Slide 408 text

refatoração != revolução

Slide 409

Slide 409 text

refatoração != revolução

Slide 410

Slide 410 text

não é para refatorar só porque existe um code smell

Slide 411

Slide 411 text

ou tratar todo code smell como um falso positivo

Slide 412

Slide 412 text

precisamos olhar para o código que deixamos como legado

Slide 413

Slide 413 text

precisamos olhar para o código que deixamos como legado

Slide 414

Slide 414 text

converse com seu time sobre a importância da qualidade do código

Slide 415

Slide 415 text

converse com seu time sobre a importância da qualidade do código

Slide 416

Slide 416 text

analisem juntos os trade-offs

Slide 417

Slide 417 text

analisem juntos os trade-offs

Slide 418

Slide 418 text

negociar quando refatorar faz parte do processo

Slide 419

Slide 419 text

negociar quando refatorar faz parte do processo

Slide 420

Slide 420 text

cuidado com big design up front e overengineering

Slide 421

Slide 421 text

cuidado com big design up front e overengineering

Slide 422

Slide 422 text

e não esqueçam:

Slide 423

Slide 423 text

codar é um processo de comunicação

Slide 424

Slide 424 text

codar é um processo de comunicação

Slide 425

Slide 425 text

escrevemos código para pessoas, não para máquinas

Slide 426

Slide 426 text

escrevemos código para pessoas, não para máquinas

Slide 427

Slide 427 text

Mais sobre qualidade de código? Design de código, SOLID Padrões de projeto TDD, DDD, Agile

Slide 428

Slide 428 text

Mais sobre qualidade de software? Design de código, SOLID Padrões de projeto TDD, DDD, Agile

Slide 429

Slide 429 text

minhas referências

Slide 430

Slide 430 text

No content

Slide 431

Slide 431 text

Functional Design Patterns - Scott Wlaschin https://youtu.be/srQt1NAHYC0 Refactoring Elixir - Lessons Learned from a Year on Exercism.Io - Devon Estes https://youtu.be/tJJMrtJEK1A Refactoring Techniques for Elixir, Ecto, and Phoenix - Gary Rennie https://youtu.be/V21DAKtY31Q Using Credo to improve your Elixir code - Vitali Tatarintev https://whatdidilearn.info/2018/05/27/using-credo-to-improve-your-elixir-code.html Protocols Are Powerful but Beware - Timmon Verlaan https://timmo.immo/blog/protocols-beware Polymorphism in Elixir - Mustafa Turan https://medium.com/elixirlabs/polymorphism-in-elixir-cd0c765b6929

Slide 432

Slide 432 text

No content

Slide 433

Slide 433 text

No content

Slide 434

Slide 434 text

No content

Slide 435

Slide 435 text

No content

Slide 436

Slide 436 text

All the Little Things - Sandi Metz https://youtu.be/8bZh5LMaSmE Evitando o Jenga Driven Development - João Britto https://pt-br.eventials.com/locaweb/evitando-o-jenga-driven-development-com-joao-britto/ How Elixir made me a better Java programmer - Juliana Helena https://youtu.be/O86LQ58PvKU Arquitetura Hexagonal: mantendo sua aplicação saudável por Camila Campos https://youtu.be/mYWG8Qv8W5k Encontrando equilíbrio do DDD enquanto sua aplicação cresce - Carol Karklis https://pt.slideshare.net/CarolinaKarklisPacke/encontrando-equilbrio-do-ddd-enquanto-sua -aplicao-cresce

Slide 437

Slide 437 text

No content

Slide 438

Slide 438 text

No content

Slide 439

Slide 439 text

No content

Slide 440

Slide 440 text

No content

Slide 441

Slide 441 text

No content

Slide 442

Slide 442 text

thoughtbot.com/upcase/clean-code refactoring.guru elixirschool.com refactoring.com/catalog Mais referências

Slide 443

Slide 443 text

Exercism.io become a mentor https://exercism.io/become-a-mentor

Slide 444

Slide 444 text

No content

Slide 445

Slide 445 text

No content

Slide 446

Slide 446 text

No content

Slide 447

Slide 447 text

No content

Slide 448

Slide 448 text

espero conhecer vocês pessoalmente ano que vem

Slide 449

Slide 449 text

speakerdeck.com/elainenaomi elainenaomi.dev Inveja de Recursos (Feature Envy) Muito obrigada

Slide 450

Slide 450 text

ilustrações do undraw.co