Código legado é aquele código com um valor imensurável, que não possui testes e que nós temos medo de estender novas funcionalidades. Toda responsabilidade de não cometer erros recai sobre seus ombros, a pressão sobre você aumenta e você deixa de acreditar em certezas. Se você já se sentiu assim e quer evitar ter cabelos brancos cedo, então você precisa assistir essa talk.

1. Refactoring na Prática:
Lidando com código legado

2. Kewerson Hugo
@kewersonhugo

3. Código Legado?

4. Funciona

5. Importante

6. Possui Valor

7. De centavos por
segundo

8. até milhões por
hora

9. Sem Clean Code

10. Sem TESTES

11. Difícil de manter

12. Ou estender
funcionalidade

13. Pressão

14. Medo

15. Dúvidas e
hesitação

16. Eu já sofri

17. Primeiro
emprego

18. Um "simples" PDV

19. Delphi

21. Testes? Kent who?

22. Testes? Kent who?
Test, please!

23. Foi triste

24. E sem querer assustar

25. Mas você
também vai

26. Brincadeira

27. Objetivo da talk

28. Como lidar com o código
legado?

29. O código existente
possui muito valor

30. Nunca reescreva
o código do zero

31. Isso vai dar
trabalho

32. Custa tempo
e dinheiro

33. Cliente:

34. O código diz uma
verdade

35. A regra do negócio

38. Você precisa
de testes

39. Garantir que
nada quebrou

40. Para extrair o
comportamento

41. Só assim
você terá segurança

42. E maximizando a
segurança

43. Coragem
para encarar

44. E utilizar Refactoring

45. Um livro feito pelos
melhores

46. Melhorar a
estrutura do código

47. Sem alterar o
comportamento

48. Que transforma
código feio
(Bad Smells)

49. Em código limpo
(Código Cheiroso)

50. São técnicas
muito simples

51. Acumuladas
melhoram muito

52. Vamos ver um exemplo

53. Gilded Rose Kata

54. function updateQuality() {
for (var i = 0; i < this.items.length; i++) {
if (this.items[i].name != 'Aged Brie' && this.items[i].name != 'Backstage pass') {
if (this.items[i].quality > 0) {
if (this.items[i].name != 'Sulfuras, Hand of Ragnaros') {
this.items[i].quality = this.items[i].quality - 1;
}
}
} else {
if (this.items[i].quality < 50) {
this.items[i].quality = this.items[i].quality + 1;
if (this.items[i].name == 'Backstage pass') {
if (this.items[i].sellIn < 11) {
if (this.items[i].quality < 50) {
this.items[i].quality = this.items[i].quality + 1;
}
}
if (this.items[i].sellIn < 6) {
if (this.items[i].quality < 50) {
this.items[i].quality = this.items[i].quality + 1;
} } } } } // Implementação...

55. function updateQuality() {
for (var i = 0; i < this.items.length; i++) {
if (this.items[i].name != 'Aged Brie' && this.items[i].name != 'Backstage pass') {
if (this.items[i].quality > 0) {
if (this.items[i].name != 'Sulfuras, Hand of Ragnaros') {
this.items[i].quality = this.items[i].quality - 1;
}
}
} else {
if (this.items[i].quality < 50) {
this.items[i].quality = this.items[i].quality + 1;
if (this.items[i].name == 'Backstage pass') {
if (this.items[i].sellIn < 11) {
if (this.items[i].quality < 50) {
this.items[i].quality = this.items[i].quality + 1;
}
}
if (this.items[i].sellIn < 6) {
if (this.items[i].quality < 50) {
this.items[i].quality = this.items[i].quality + 1;
} } } } } // Implementação...

56. Bloco do FOR é
JUST ONE THING

57. function updateQuality() {
for (var i = 0; i < this.items.length; i++) {
if (this.items[i].name != 'Aged Brie' && this.items[i].name != 'Backstage pass') {
if (this.items[i].quality > 0) {
if (this.items[i].name != 'Sulfuras, Hand of Ragnaros') {
this.items[i].quality = this.items[i].quality - 1;
}
}
} else {
if (this.items[i].quality < 50) {
this.items[i].quality = this.items[i].quality + 1;
if (this.items[i].name == 'Backstage pass') {
if (this.items[i].sellIn < 11) {
if (this.items[i].quality < 50) {
this.items[i].quality = this.items[i].quality + 1;
}
}
if (this.items[i].sellIn < 6) {
if (this.items[i].quality < 50) {
this.items[i].quality = this.items[i].quality + 1;
} } } } } // Implementação...

58. function updateQuality() {
for (var i = 0; i < this.items.length; i++) {

59. function updateQuality() {
for (var i = 0; i < this.items.length; i++) {
}
}

60. function updateQuality() {
for (var i = 0; i < this.items.length; i++) {
}
}

61. function updateQuality() {
for (var i = 0; i < this.items.length; i++) {
updateOneItem(items[i]);
}
}

62. function updateQuality() {
for (var i = 0; i < this.items.length; i++) {
updateOneItem(items[i]);
}
}

63. function updateQuality() {
for (var i = 0; i < this.items.length; i++) {
updateOneItem(items[i]);
}
}

64. function updateQuality() {
for (var i = 0; i < this.items.length; i++) {
updateOneItem(items[i]);
}
}

65. function updateQuality() {
for (var i = 0; i < this.items.length; i++) {
updateOneItem(items[i]);
}
}

66. function updateQuality() {
for (var i = 0; i < this.items.length; i++) {
updateOneItem(items[i]);
}
}
function updateOneItem(item) {
if (this.items[i].name != 'Aged Brie' && this.items[i].name != 'Backstage pass') {
if (this.items[i].quality > 0) {
if (this.items[i].name != 'Sulfuras, Hand of Ragnaros') {
this.items[i].quality = this.items[i].quality - 1;
}
}
} else {
if (this.items[i].quality < 50) {
this.items[i].quality = this.items[i].quality + 1;
if (this.items[i].name == 'Backstage pass') {
if (this.items[i].sellIn < 11) {
if (this.items[i].quality < 50) {
this.items[i].quality = this.items[i].quality + 1;
}
} // Implementação...

67. function updateQuality() {
for (var i = 0; i < this.items.length; i++) {
updateOneItem(items[i]);
}
}
function updateOneItem(item) {
if (this.items[i].name != 'Aged Brie' && this.items[i].name != 'Backstage pass') {
if (this.items[i].quality > 0) {
if (this.items[i].name != 'Sulfuras, Hand of Ragnaros') {
this.items[i].quality = this.items[i].quality - 1;
}
}
} else {
if (this.items[i].quality < 50) {
this.items[i].quality = this.items[i].quality + 1;
if (this.items[i].name == 'Backstage pass') {
if (this.items[i].sellIn < 11) {
if (this.items[i].quality < 50) {
this.items[i].quality = this.items[i].quality + 1;
}
} // Implementação...

68. function updateQuality() {
for (var i = 0; i < this.items.length; i++) {
updateOneItem(items[i]);
}
}
function updateOneItem(item) {
if (this.items[i].name != 'Aged Brie' && this.items[i].name != 'Backstage pass') {
if (this.items[i].quality > 0) {
if (this.items[i].name != 'Sulfuras, Hand of Ragnaros') {
this.items[i].quality = this.items[i].quality - 1;
}
}
} else {
if (this.items[i].quality < 50) {
this.items[i].quality = this.items[i].quality + 1;
if (this.items[i].name == 'Backstage pass') {
if (this.items[i].sellIn < 11) {
if (this.items[i].quality < 50) {
this.items[i].quality = this.items[i].quality + 1;
}
} // Implementação...

69. function updateOneItem(item) {
if (this.items[i].name != 'Aged Brie' && this.items[i].name !=
if (this.items[i].quality > 0) {
if (this.items[i].name != 'Sulfuras, Hand of Ragnaros') {
this.items[i].quality = this.items[i].quality - 1;
}
}
}
// Implementação
}

70. function updateOneItem(item) {
if (this.items[i].name != 'Aged Brie' && this.items[i].name !=
if (this.items[i].quality > 0) {
if (this.items[i].name != 'Sulfuras, Hand of Ragnaros') {
this.items[i].quality = this.items[i].quality - 1;
}
}
}
// Implementação
}

71. function updateOneItem(item) {
if (item.name != 'Aged Brie' && item.name != 'Backstage
if (item.quality > 0) {
if (item.name != 'Sulfuras, Hand of Ragnaros') {
item.quality = item.quality - 1;
}
}
}
// Implementação
}

72. function updateOneItem(item) {
if (item.name != 'Aged Brie' && item.name != 'Backstage
if (item.quality > 0) {
if (item.name != 'Sulfuras, Hand of Ragnaros') {
item.quality = item.quality - 1;
}
}
}
// Implementação
}

73. function updateQuality() {
for (var i = 0; i < this.items.length; i++) {
updateOneItem(items[i]);
}
}
function updateOneItem(item) {
if (item.name != 'Aged Brie' && item.name != 'Backstage pass') {
if (item.quality > 0) {
if (item.name != 'Sulfuras, Hand of Ragnaros') {
item.quality = item.quality - 1;
}
}
} // Implementação ...

74. JUST ONE THING!

75. Respeite as
responsabilidade

76. Sempre rode
os testes depois

77. Próximo exemplo

78. function updateOneItem(item) {
if (item.name != 'Aged Brie'
&& item.name != 'Backstage pass')
// Não Aged Brie E Não Backstage Pass ...
}
else {
// É Aged Brie OU É Backstage Pass...
}
}

79. function updateOneItem(item) {
if (item.name != 'Aged Brie'
&& item.name != 'Backstage pass')
// Não Aged Brie E Não Backstage Pass ...
}
else {
// É Aged Brie OU É Backstage Pass...
}
}

80. Condicionais negativas!?

81. Condicionais negativas!?

82. function updateOneItem(item) {
if (item.name != 'Aged Brie'
&& item.name != 'Backstage pass')
// Não Aged Brie E Não Backstage Pass ...
}
else {
// É Aged Brie OU É Backstage Pass...
}
}

83. function updateOneItem(item) {
if (item.name != 'Aged Brie'
&& item.name != 'Backstage pass')
// Não Aged Brie E Não Backstage Pass ...
}
else {
// É Aged Brie OU É Backstage Pass...
}
}

84. function updateOneItem(item) {
if (item.name != 'Aged Brie'
&& item.name != 'Backstage pass')
// Não Aged Brie E Não Backstage Pass ...
}
else {
// É Aged Brie OU É Backstage Pass...
}
}

85. function updateOneItem(item) {
if (item.name != 'Aged Brie'
&& item.name != 'Backstage pass')
// Não Aged Brie E Não Backstage Pass ...
}
else {
// É Aged Brie OU É Backstage Pass...
}
}

86. function updateOneItem(item) {
if (item.name != 'Aged Brie'
&& item.name != 'Backstage pass')
// Não Aged Brie E Não Backstage Pass ...
}
else {
// É Aged Brie OU É Backstage Pass...
}
}

87. function updateOneItem(item) {
if (item.name == 'Aged Brie'
|| item.name == 'Backstage pass')
// Não Aged Brie E Não Backstage Pass ...
}
else {
// É Aged Brie OU É Backstage Pass...
}
}

88. function updateOneItem(item) {
if (item.name == 'Aged Brie'
|| item.name == 'Backstage pass')
// Não Aged Brie E Não Backstage Pass ...
}
else {
// É Aged Brie OU É Backstage Pass...
}
}

89. function updateOneItem(item) {
if (item.name == 'Aged Brie'
|| item.name == 'Backstage pass')
// Não Aged Brie E Não Backstage Pass ...
}
else {
// É Aged Brie OU É Backstage Pass...
}
}

90. function updateOneItem(item) {
if (item.name == 'Aged Brie'
|| item.name == 'Backstage pass')
// É Aged Brie OU É Backstage Pass...
}
else {
// Não Aged Brie E Não Backstage Pass ...
}
}

91. function updateOneItem(item) {
if (item.name == 'Aged Brie'
|| item.name == 'Backstage pass')
// É Aged Brie OU É Backstage Pass...
}
else {
// Não Aged Brie E Não Backstage Pass ...
}
}

92. function updateOneItem(item) {
if (item.name == 'Aged Brie'
|| item.name == 'Backstage pass')
// É Aged Brie OU É Backstage Pass...
}
else {
// Não Aged Brie E Não Backstage Pass ...
}
}

93. function updateOneItem(item) {
if (item.name == 'Aged Brie'
|| item.name == 'Backstage pass')
// É Aged Brie OU É Backstage Pass...
}
else {
// Não Aged Brie E Não Backstage Pass ...
}
}
Condicional
mais simples

94. function updateOneItem(item) {
if (item.name == 'Aged Brie'
|| item.name == 'Backstage pass')
// É Aged Brie OU É Backstage Pass...
}
else {
// Não Aged Brie E Não Backstage Pass ...
}
}
E coerente com o
bloco seguinte

95. Rode os testes

96. Não existe segredo:

97. 1 - Encontre o mau cheiro

98. 2 - Elimine-o

99. 3 - Rode os testes

100. 3 - Rode os testes

101. 3 - Rode os testes

102. 3 - Rode os testes

103. Refactoring é simples

104. Um último exemplo

105. if (item.name == 'Aged Brie' || item.name == 'Backstage pass') {
if (item.quality < 50) {
item.quality = item.quality + 1;
if (item.name == 'Backstage pass') {
if (item.sellIn < 11) {
if (item.quality < 50) {
item.quality = item.quality + 1;
}
}
if (item.sellIn < 6) {
if (item.quality < 50) {
item.quality = item.quality + 1;
}
}
}
}
} else { // Implementação }

106. if (item.name == 'Aged Brie' || item.name == 'Backstage pass') {
if (item.quality < 50) {
item.quality = item.quality + 1;
if (item.name == 'Backstage pass') {
if (item.sellIn < 11) {
if (item.quality < 50) {
item.quality = item.quality + 1;
}
}
if (item.sellIn < 6) {
if (item.quality < 50) {
item.quality = item.quality + 1;
}
}
}
}
} else { // Implementação }

107. Mesmo de antes

108. Bloco do IF também
é JUST ONE THING

109. if (item.name == 'Aged Brie' || item.name == 'Backstage pass') {
if (item.quality < 50) {
item.quality = item.quality + 1;
if (item.name == 'Backstage pass') {
if (item.sellIn < 11) {
if (item.quality < 50) {
item.quality = item.quality + 1;
}
}
if (item.sellIn < 6) {
if (item.quality < 50) {
item.quality = item.quality + 1;
}
}
}
}
} else { // Implementação }

110. if (item.name == 'Aged Brie' || item.name == 'Backstage pass') {
} else { // Implementação }

111. if (item.name == 'Aged Brie' || item.name == 'Backstage pass') {
} else { // Implementação }

112. if (item.name == 'Aged Brie' || item.name == 'Backstage pass') {
????????????(item);
} else { // Implementação }

113. if (item.name == 'Aged Brie' || item.name == 'Backstage pass') {
????????????(item);
} else { // Implementação }

114. if (item.name == 'Aged Brie' || item.name == 'Backstage pass') {
????????????(item);
} else { // Implementação }
Qual nome
escolher?

115. if (item.name == 'Aged Brie' || item.name == 'Backstage pass') {
????????????(item);
} else { // Implementação }

116. if (item.name == 'Aged Brie' || item.name == 'Backstage pass') {
????????????(item);
} else { // Implementação }
Qual a
responsabilidade?

117. if (item.name == 'Aged Brie' || item.name == 'Backstage pass') {
updateAgedBrieOrBackstagePass(item);
} else { // Implementação }

118. if (item.name == 'Aged Brie' || item.name == 'Backstage pass') {
updateAgedBrieOrBackstagePass(item);
} else { // Implementação }

119. if (item.name == 'Aged Brie' || this.item
updateAgedBrieOrBackstagePass(item);
} else { // Implementação }

120. if (item.name == 'Aged Brie' || th
updateAgedBrieOrBackstagePass(item);
} else { // Implementação }

121. if (item.name == 'Aged Brie' || th
updateAgedBrieOrBackstagePass(item);
} else { // Implementação }

122. OU e E no nome
não é ONE THING

123. if (item.name == 'Aged Brie' || item.name == 'Backstage pass') {
if (item.quality < 50) {
item.quality = item.quality + 1;
if (item.name == 'Backstage pass') {
if (item.sellIn < 11) {
if (item.quality < 50) {
item.quality = item.quality + 1;
}
}
if (item.sellIn < 6) {
if (item.quality < 50) {
item.quality = item.quality + 1;
}
}
}
}
} else { // Implementação }

124. if (item.name == 'Aged Brie' || item.name == 'Backstage pass') {
if (item.quality < 50) {
item.quality = item.quality + 1;
if (item.name == 'Backstage pass') {
if (item.sellIn < 11) {
if (item.quality < 50) {
item.quality = item.quality + 1;
}
}
if (item.sellIn < 6) {
if (item.quality < 50) {
item.quality = item.quality + 1;
}
}
}
}
} else { // Implementação }

125. if (item.name == 'Aged Brie' || item.name == 'Backstage pass') {
if (item.quality < 50) {
item.quality = item.quality + 1;
if (item.name == 'Backstage pass') {
if (item.sellIn < 11) {
if (item.quality < 50) {
item.quality = item.quality + 1;
}
}
if (item.sellIn < 6) {
if (item.quality < 50) {
item.quality = item.quality + 1;
}
}
}
}
} else { // Implementação }

126. if (item.name == 'Aged Brie' || item.name == 'Backstage pass') {
if (item.quality < 50) {
item.quality = item.quality + 1;
if (item.name == 'Backstage pass') {
if (item.sellIn < 11) {
if (item.quality < 50) {
item.quality = item.quality + 1;
}
}
if (item.sellIn < 6) {
if (item.quality < 50) {
item.quality = item.quality + 1;
}
}
}
}
} else { // Implementação }

127. if (item.name == 'Aged Brie' || item.name == 'Backstage pass') {
if (item.quality < 50) {
item.quality = item.quality + 1;
if (item.name == 'Backstage pass') {
if (item.sellIn < 11) {
if (item.quality < 50) {
item.quality = item.quality + 1;
}
}
if (item.sellIn < 6) {
if (item.quality < 50) {
item.quality = item.quality + 1;
}
}
}
}
} else { // Implementação }

128. if (item.name == 'Aged Brie' || item.name == 'Backstage pass') {
if (item.quality < 50) {
item.quality = item.quality + 1;
if (item.name == 'Backstage pass') {
if (item.sellIn < 11) {
if (item.quality < 50) {
item.quality = item.quality + 1;
}
}
if (item.sellIn < 6) {
if (item.quality < 50) {
item.quality = item.quality + 1;
}
}
}
}
} else { // Implementação }

129. if (item.name == 'Aged Brie' || item.name == 'Backstage pass') {
if (item.quality < 50) {
item.quality = item.quality + 1;
if (item.name == 'Backstage pass') {
if (item.sellIn < 11) {
if (item.quality < 50) {
item.quality = item.quality + 1;
}
}
if (item.sellIn < 6) {
if (item.quality < 50) {
item.quality = item.quality + 1;
}
}
}
}
} else { // Implementação }

130. if (item.name == 'Aged Brie' || item.name == 'Backstage pass') {
} else { // Implementação }

131. if (item.name == 'Aged Brie' || item.name == 'Backstage pass') {
} else { // Implementação }

132. if (item.name == 'Aged Brie' || item.name == 'Backstage pass') {
} else { // Implementação }

133. if (item.name == 'Aged Brie' ) {
} else { // Implementação }

134. if (item.name == 'Aged Brie') {
} else { // Implementação }

135. if (item.name == 'Aged Brie') {
} else if (item.name == 'Backstage Pass') {
else { // Implementação }

136. if (item.name == 'Aged Brie') {
} else if (item.name == 'Backstage Pass') {
else { // Implementação }

137. if (item.name == 'Aged Brie') {
updateAgedBrie(item);
} else if (item.name == 'Backstage Pass') {
updateBackstagePass(item);
else { // Implementação }

138. if (item.name == 'Aged Brie') {
updateAgedBrie(item);
} else if (item.name == 'Backstage Pass') {
updateBackstagePass(item);
else { // Implementação }

139. if (item.name == 'Aged Brie') {
updateAgedBrie(item);
} else if (item.name == 'Backstage Pass') {
updateBackstagePass(item);
else { // Implementação }

140. function updateAgedBrie(item) {
if (item.quality < 50)
item.quality = item.quality + 1;
}

141. function updateAgedBrie(item) {
if (item.quality < 50)
item.quality = item.quality + 1;
}

142. if (item.name == 'Aged Brie') {
updateAgedBrie(item);
} else if (item.name == 'Backstage Pass') {
updateBackstagePass(item);
else { // Implementação }

143. if (item.name == 'Aged Brie') {
updateAgedBrie(item);
} else if (item.name == 'Backstage Pass') {
updateBackstagePass(item);
else { // Implementação }

144. if (item.name == 'Aged Brie') {
updateAgedBrie(item);
} else if (item.name == 'Backstage Pass') {
updateBackstagePass(item);
else { // Implementação }

145. function updateBackstagePass(item) {
if (item.quality < 50) {
item.quality = item.quality + 1;
if (item.name == 'Backstage pass') {
if (item.sellIn < 11) {
if (item.quality < 50) {
item.quality = item.quality + 1;
}
}
if (item.sellIn < 6) {
if (item.quality < 50) {
item.quality = item.quality + 1;
}
}
}
}

146. function updateBackstagePass(item) {
if (item.quality < 50) {
item.quality = item.quality + 1;
if (item.name == 'Backstage pass') {
if (item.sellIn < 11) {
if (item.quality < 50) {
item.quality = item.quality + 1;
}
}
if (item.sellIn < 6) {
if (item.quality < 50) {
item.quality = item.quality + 1;
}
}
}
}

147. function updateBackstagePass(item) {
if (item.quality < 50) {
item.quality = item.quality + 1;
if (item.sellIn < 11) {
if (item.quality < 50) {
item.quality = item.quality + 1;
}
}
if (item.sellIn < 6) {
if (item.quality < 50) {
item.quality = item.quality + 1;
}
}
}

148. function updateBackstagePass(item) {
if (item.quality < 50) {
item.quality = item.quality + 1;
if (item.sellIn < 11) {
if (item.quality < 50) {
item.quality = item.quality + 1;
}
}
if (item.sellIn < 6) {
if (item.quality < 50) {
item.quality = item.quality + 1;
}
}
}

149. function updateBackstagePass(item) {
if (item.quality < 50) {
item.quality = item.quality + 1;
if (item.sellIn < 11) {
if (item.quality < 50) {
item.quality = item.quality + 1;
}
}
if (item.sellIn < 6) {
if (item.quality < 50) {
item.quality = item.quality + 1;
}
}
}
}

150. function updateBackstagePass(item) {
if (item.quality < 50) {
item.quality = item.quality + 1;
if (item.sellIn < 11) {
if (item.quality < 50) {
item.quality = item.quality + 1;
}
}
if (item.sellIn < 6) {
if (item.quality < 50) {
item.quality = item.quality + 1;
}
}
}
}

151. if (item.name == 'Aged Brie') {
updateAgedBrie(item);
} else if (item.name == 'Backstage Pass') {
updateBackstagePass(item);
else { // Implementação }

152. if (item.name == 'Aged Brie') {
updateAgedBrie(item);
} else if (item.name == 'Backstage Pass') {
updateBackstagePass(item);
else { // Implementação }

153. if (item.name == 'Aged Brie') {
updateAgedBrie(item);
} else if (item.name == 'Backstage Pass') {
updateBackstagePass(item);
else { // Implementação }
O que muda?

154. if (item.name == 'Aged Brie') {
updateAgedBrie(item);
} else if (item.name == 'Backstage Pass') {
updateBackstagePass(item);
} else if (item.name == 'Enchanted Sword') {
updateEnchantedSword(item);
} else if (item.name == 'Potion') {
updatePotion(item);
} else {
updateStandardItem(item);
}

155. if (item.name == 'Aged Brie') {
updateAgedBrie(item);
} else if (item.name == 'Backstage Pass') {
updateBackstagePass(item);
} else if (item.name == 'Enchanted Sword') {
updateEnchantedSword(item);
} else if (item.name == 'Potion') {
updatePotion(item);
} else {
updateStandardItem(item);
}

156. if (item.name == 'Aged Brie') {
updateAgedBrie(item);
} else if (item.name == 'Backstage Pass') {
updateBackstagePass(item);
} else if (item.name == 'Enchanted Sword') {
updateEnchantedSword(item);
} else if (item.name == 'Potion') {
updatePotion(item);
} else {
updateStandardItem(item);
}
Fácil de estender
funcionalidades

157. E repare que incrível

158. if (item.name == 'Aged Brie') {
updateAgedBrie(item);
} else if (item.name == 'Backstage Pass') {
updateBackstagePass(item);
} else if (item.name == 'Enchanted Sword') {
updateEnchantedSword(item);
} else if (item.name == 'Potion') {
updatePotion(item);
} else {
updateStandardItem(item);
}

159. if (item.name == 'Aged Brie') {
updateAgedBrie(item);
} else if (item.name == 'Backstage Pass') {
updateBackstagePass(item);
} else if (item.name == 'Enchanted Sword') {
updateEnchantedSword(item);
} else if (item.name == 'Potion') {
updatePotion(item);
} else {
updateStandardItem(item);
}

160. if (item.name == 'Aged Brie') {
updateAgedBrie(item);
} else if (item.name == 'Backstage Pass') {
updateBackstagePass(item);
} else if (item.name == 'Enchanted Sword') {
updateEnchantedSword(item);
} else if (item.name == 'Potion') {
updatePotion(item);
} else {
updateStandardItem(item);
}
Várias
condicionais

161. if (item.name == 'Aged Brie') {
updateAgedBrie(item);
} else if (item.name == 'Backstage Pass') {
updateBackstagePass(item);
} else if (item.name == 'Enchanted Sword') {
updateEnchantedSword(item);
} else if (item.name == 'Potion') {
updatePotion(item);
} else {
updateStandardItem(item);
}

162. if (item.name == 'Aged Brie') {
updateAgedBrie(item);
} else if (item.name == 'Backstage Pass') {
updateBackstagePass(item);
} else if (item.name == 'Enchanted Sword') {
updateEnchantedSword(item);
} else if (item.name == 'Potion') {
updatePotion(item);
} else {
updateStandardItem(item);
}
Com diferentes
comportamentos

163. Podemos aplicar
polimorfismo

164. Novos refactorings
podem ser notados

165. O Refactoring
ajuda no design

166. E não existe um limite

167. Cuidado

168. Seja Pragmático

169. Tudo possui
um trade-off

170. Refatore até o necessário
para cumprir o requisito

171. Considere os prazos

172. E não se esqueça
de sempre testar!

173. Recomendação
de Leitura:

174. https://goo.gl/wMJeQy

180. Obrigado!

181. E sério,
IMPLEMENTEM TESTES!

182. Melhor forma de evitar
a dor de cabeça

183. Obrigado!!!