Este documento resume una presentación sobre técnicas de refactorización y desarrollo guiado por pruebas (TDD). Incluye estadísticas sobre entrenamientos de código limpio realizados, información sobre el orador Rodrigo Branas y su experiencia, y una lista de temas que se abordarán en la presentación como nombres confusos, estructuras de decisión complejas y números mágicos en el código.

1. Rodrigo Branas – @rodrigobranas - http://www.agilecode.com.br/es
Clean Code con
Técnicas de Refactorización y TDD

2. Acerca de la iniciativa...

3. Estatísticas del entrenamiento de Clean Code
hasta la fecha:
Primer grupo en: 05/2012
Clases ya realizadas: 21
Desarrolladores entrenados: 340

4. AgileCode
“Transformar la manera en que los equipos de
desarrollo de software trabajan, aumentando su
nivel de madurez técnica y gerencial”
http://www.agilecode.com.br/es

5. Rodrigo Branas
rodrigo.branas@agilecode.com.br
http://www.agilecode.com.br
•
•
•
•
Desarrollo de Software en Gennera
Entrenamientos en Agile Code
Artículos en Java Magazine
Conferencias sobre desarrollo de software
en eventos, universidades y empresas

6. Formación Académica
Ciencias de la Computación - UFSC
Gestión de Proyectos - FGV
Certificaciones
SCJA, SCJP, SCJD, SCWCD, SCBCD, PMP, MCP e CSM
Experiencia
Desde hace más de 12 años desarrollando
software en la plataforma Java en las empresas:
EDS, HP, NET, Citibank, GM, Dígitro, Softplan,
OnCast, Senai, VALE, RBS, Unimed, Globalcode,
V.Office, Suntech, WPlex e Gennera.

7. El contenido deste entrenamiento
puede afectar su zona de confort

8. ¿Sobre que vamos hablar?

10. Insatisfación con el fracaso!

12. ¿Existe diferencia?

13. Insatisfación con el trabajo

14. ¿Te sientes profesional?

15. ¿Te sientes profesional?

16. ¿Te sientes profesional?

17. ¿Podría ser más productivo?

18. ¿Cuanto tiempo perdemos
intentando entender el código
para arreglar un defecto?

19. ¿Te sientes haciendo un
trabajo brazal y fatigoso?

20. ¿Ya quisiste dejarlo todo?

21. ¿Que motiva a un desarrollador?

22. “Quien trabaja solamente por el
diñero, ganhe lo que for, siempre
será mal pago.”

26. “Cual es tu obra?”
(Mario Cortella)

27. ¿Cuanto custa toda esta
confusión para la empresa?

28. Alta rotatividad

29. Demora excesiva para poner
nuevos productos en marcha

30. ¿Cuantos proyectos ya fueran
desechados?

31. Muchos defectos

32. ¿Cuanto custa la perdida de
confianza de sus clientes?

33. Passamos la major parte del
tiempo leyendo el código!

34. ¿Dos desarrolladores son más
productivos que uno?
(Pair Programming)

35. ¿Que es Clean Code?

36. Echo con cuidado

37. Inteligente

38. Reusable

39. Revela sus intenciones

40. Sencillo

41. Fácil de entender

42. “Cualquier tonto consigue escribir un
código que el ordenador entienda.
Buenos programadores escriben código
que otros humanos comprendan.”
(Martin Fowler)

43. Funcionar es lo mínimo

44. Calidad Externa x Interna

45. ¿Como medir la calidad del
código?

46. ¿Líneas de código?
¿Número de métodos?
¿Número de classes?
¿Tamanho de los método?
¿Complejidad?
???

49. WTF?

50. WTF?
WTF?

51. WTF?
WTF?
WTF?
WTF?

53. Dificuldad para entenderse

56. Decirlo es fácil!

57. Empezando a entrenar

58. Nomes estraños

59. Limitación histórica

60. 1.
2.
3.
4.
5.
6.
7.
8.
9.
READ INPUT TAPE A1, B1, C1;
501 FORMAT A1;
IF (A1) 777, 777, 777
IF (B1) 888, 888, 888
IF (C1) 999, 999, 999
STOP 1
799 S = FLOATF(A1 + B1 + C1) / 2.0
WRITE TO TAPE S
END PROCESS
Las linguajes no ayudavam

61. Los entornos de desarrollo no
ayudavam

62. Los ordenadores no ayudavam

63. Herencia Maldita

64. Aprendiendo con los mestres

65. 1.
2.
3.
4.
5.
6.
7.
8.
// Descripción de la maquina
String mDesc;
// Suma de las parcelas
int s1;
// Color de los ojos
String cOjos;
// Cotización del Dólar
int aux;

66. Revelando su intención

67. 1.
2.
3.
4.
5.
6.
7.
8.
// Descripción de la maquina
String mDesc;
// Suma de las parcelas
int s1;
// Color de los ojos
String cOjos;
// Cotazición del Dólar
int aux;

68. 1.
2.
3.
4.
5.
6.
7.
8.
// Descripción de la maquina
String;
// Suma de las parcelas
int;
// Color de los ojos
String;
// Cotización del Dólar
int;

69. 1.
2.
3.
4.
5.
6.
7.
8.
// Descripción de la maquina
String descriptionDeLaMaquina;
// Suma de las parcelas
int sumaDeLasParcelas;
// Color de los ojos
String colorDeLosOjos;
// Cotización del Dólar
int cotizacionDelDolar;

70. 1.
2.
3.
4.
5.
6.
7.
8.
String descriptionDeLaMaquina;
int sumaDeLasParcelas;
String colorDeLosOjos;
int cotizacionDelDolar;

71. 1.
2.
3.
4.
String descriptionDeLaMaquina;
int sumaDeLasParcelas;
String colorDeLosOjos;
int cotizacionDelDolar;

72. No seas tan genérico

73. 1.
2.
3.
4.
5.
6.
// Ejecutar el cálculo de la nómina
void ejecutar();
// Calcular el impuesto
void calcular();
// Renderizar la imagen
void renderizar();
7.
// Totalizar los costos de las llamadas
8. void totalizar();

74. 1.
2.
3.
4.
5.
6.
// Ejecutar el cálculo de la nómina
void ejecutarCalculoDeLaNomina();
// Calcular el impuesto
void calcularImpuesto();
// Renderizar la imagen
void renderizarImagen();
7.
// Totalizar los costos de las llamadas
8. void totalizarCostosDeLasLlamadas()

75. 1.
2. void ejecutarCalculoDeLaNomina();
3.
4. void calcularImpuesto();
5.
6. void renderizarImagen();
7.
8. void totalizarCostosDeLasLlamadas()

76. 1. void ejecutarCalculoDeLaNomina();
2. void calcularImpuesto();
3. void renderizarImagen();
4. void totalizarCostosDeLasLlamadas()

77. Simplificando estructuras de
decisión

78. 1.
2.
3.
4.
5.
6.
7.
8.
9.
10.
class Taximetro {
...
double calcularCostoDelViaje() {
if (hora > 22:00 || hora < 06:00) {
return distancia * 3.90;
} else {
return distancia * 2.10;
}
}
...

79. 1.
2.
3.
4.
5.
6.
7.
8.
9.
10.
class Taximetro {
...
double calcularCostoDelViaje() {
if () {
return distancia * 3.90;
} else {
return distancia * 2.10;
}
}
...

80. 1.
2.
3.
4.
5.
6.
7.
8.
9.
10.
class Taximetro {
...
double calcularCostoDelViaje() {
if (esBandeiraDos()) {
return distancia * 3.90;
} else {
return distancia * 2.10;
}
}
...

81. 1.
2.
3.
4.
5.
6.
7.
8.
9.
10.
11.
...
double calcularCostoDelViaje() {
if (esBandeiraDos()) {
return distancia * 3.90;
} else {
return distancia * 2.10;
}
Boolean esBandeiraDos() {
return hora > 22:00 || hora < 06:00;
}
...

82. ¿Cuales son las ventajas de
la modificación?

83. ¿Cómo sería una prueba automatizada
antes de la refactorización?

84. 1.
2.
3.
4.
5.
6.
7.
public class TaximetroPrueba {
public void debeSerBanderaDos() {
}
}

85. 1.
2.
3.
4.
5.
6.
7.
8.
public class TaximetroPrueba {
public void debeSerBanderaDos() {
Taximetro taximetro = new Taximetro();
}
}

86. 1.
2.
3.
4.
5.
6.
7.
8.
9.
public class TaximetroPrueba {
public void debeSerBanderaDos() {
Taximetro taximetro = new Taximetro();
taximetro.iniciar(23horas());
}
}

87. 1. public class TaximetroPrueba {
2.
3.
public void debeSerBanderaDos() {
4.
Taximetro taximetro = new Taximetro();
5.
taximetro.iniciar(23horas());
6.
taximetro.distanciaRecorridaEnKM(3.45);
7.
}
8.
9. }
10.

88. 1. public class TaximetroPrueba {
2.
3.
public void debeSerBanderaDos() {
4.
Taximetro taximetro = new Taximetro();
5.
taximetro.iniciar(23horas());
6.
taximetro.distanciaRecorridaEnKM(3.45);
7.
double costo = taximetro.calcularCostoDelViaje();
8.
}
9.
10. }
11.

89. 1. public class TaximetroPrueba {
2.
3.
public void debeSerBanderaDos() {
4.
Taximetro taximetro = new Taximetro();
5.
taximetro.iniciar(23horas());
6.
taximetro.distanciaRecorridaEnKM(3.45);
7.
double costo = taximetro.calcularCostoDelViaje();
8.
assertEquals(23.45, costo);
9.
}
10.
11. }
12.

90. ¿Y depués?

91. 1.
2.
3.
4.
5.
6.
7.
public class TaximetroPrueba {
public void debeSerBanderaDos() {
}
}

92. 1.
2.
3.
4.
5.
6.
7.
8.
public class TaximetroPrueba {
public void debeSerBanderaDos() {
Taximetro taximetro = new Taximetro();
}
}

93. 1.
2.
3.
4.
5.
6.
7.
8.
9.
public class TaximetroPrueba {
public void debeSerBanderaDos() {
Taximetro taximetro = new Taximetro();
taximetro.iniciar(23horas());
}
}

94. 1. public class TaximetroPrueba {
2.
3.
public void debeSerBanderaDos() {
4.
Taximetro taximetro = new Taximetro();
5.
taximetro.iniciar(23horas());
6.
assertTrue(taximetro.esBandeiraDos());
7. }
8.
9. }
10.

95. Magic numbers

96. 1.
2.
3.
4.
5.
6.
7.
8.
9.
10.
11.
...
double calcularCostoDelViaje() {
if (esBandeiraDos()) {
return distancia * 3.90;
} else {
return distancia * 2.10;
}
Boolean esBandeiraDos() {
return hora > 22:00 || hora < 06:00;
}
...

97. 1.
2.
3.
4.
5.
6.
7.
8.
9.
10.
11.
...
double calcularCostoDelViaje() {
if (esBandeiraDos()) {
return distancia * ;
} else {
return distancia * ;
}
Boolean esBandeiraDos() {
return hora > 22:00 || hora < 06:00;
}
...

98. 1.
2.
3.
4.
5.
6.
7.
8.
9.
10.
11.
...
double calcularCostoDelViaje() {
if (esBandeiraDos()) {
return distancia * COSTO_BANDERA_2;
} else {
return distancia * COSTO_BANDERA_1;
}
Boolean esBandeiraDos() {
return hora > 22:00 || hora < 06:00;
}
...

99. Saindo do labirinto

100. 1. double calcularCostoDelViaje() {
2.
if (esBandeiraDos()) {
3.
return distancia * COSTO_BANDERA_2;
4.
} else {
5.
return distancia * COSTO_BANDERA_1;
6.
}

101. 1. double calcularCostoDelViaje() {
2.
if (esBandeiraDos()) {
3.
return distancia * COSTO_BANDERA_2;
4.
} else {
5.
if (esDomingo()) {
6.
return distancia * COSTO_DOMINGO;
7.
} else {
8.
return distancia * COSTO_BANDERA_1;
9.
}
10. }

102. Distancia del borde

103. 1. double calcularCostoDelViaje() {
2.
if (esBandeiraDos()) {
3.
return distancia * COSTO_BANDERA_2;
4.
} else {
5.
if (esDomingo()) {
6.
return distancia * COSTO_DOMINGO;
7.
} else {
8.
return distancia * COSTO_BANDERA_1;
9.
}
10. }
11. }

104. 1. double calcularCostoDelViaje() {
2.
if (esBandeiraDos()) {
3.
return distancia * COSTO_BANDERA_2;
4.
}
5.
if (esDomingo()) {
6.
return distancia * COSTO_DOMINGO;
7.
} else {
8.
return distancia * COSTO_BANDERA_1;
9.
}
10.
11. }

105. 1. double calcularCostoDelViaje() {
2.
if (esBandeiraDos()) {
3.
return distancia * COSTO_BANDERA_2;
4.
}
5.
if (esDomingo()) {
6.
return distancia * COSTO_DOMINGO;
7.
} else {
8.
return distancia * COSTO_BANDERA_1;
9.
}
10.
11. }

106. 1. double calcularCostoDelViaje() {
2.
if (esBandeiraDos()) {
3.
return distancia * COSTO_BANDERA_2;
4.
}
5.
if (esDomingo()) {
6.
return distancia * COSTO_DOMINGO;
7.
} else {
8.
return distancia * COSTO_BANDERA_1;
9.
}
10. }
11.

107. 1. double calcularCostoDelViaje() {
2.
if (esBandeiraDos()) {
3.
return distancia * COSTO_BANDERA_2;
4.
}
5.
if (esDomingo()) {
6.
return distancia * COSTO_DOMINGO;
7.
}
8.
return distancia * COSTO_BANDERA_1;
9.
10. }
11.

108. 1. double calcularCostoDelViaje() {
2.
if (esBandeiraDos()) {
3.
return distancia * COSTO_BANDERA_2;
4.
}
5.
if (esDomingo()) {
6.
return distancia * COSTO_DOMINGO;
7.
}
8.
return distancia * COSTO_BANDERA_1;
9. }
10.
11.

109. 1. double calcularCostoDelViaje() {
2.
if (esBandeiraDos()) {
3.
return distancia * COSTO_BANDERA_2;
4.
}
5.
if (esDomingo()) {
6.
return distancia * COSTO_DOMINGO;
7.
}
8.
return distancia * COSTO_BANDERA_1;
9. }
10.
11.

110. 1. double calcularCostoDelViaje() {
2.
if (esBandeiraDos()) {
3.
return;
4.
}
5.
if (esDomingo()) {
6.
return;
7.
}
8.
return;
9. }
10.
11.

111. 1. double calcularCostoDelViaje() {
2.
if (esBandeiraDos()) {
3.
return calcularBanderaDos(distancia);
4.
}
5.
if (esDomingo()) {
6.
return calcularDomingo(distancia);
7.
}
8.
return calcularBanderaUno(distancia);
9. }
10.
11.

112. 1. double calcularCostoDelViaje() {
2.
3.
4.
5. }
if (esBandeiraDos()) return calcularBanderaDos(distancia);
if (esDomingo()) return calcularDomingo(distancia);
return calcularBanderaUno(distancia);

113. 1. double calcularCostoDelViaje() {
2.
3.
4.
5.
6. }
if (esBandeiraDos()) return calcularBanderaDos(distancia);
if (esDomingo()) return calcularDomingo(distancia);
return calcularBanderaUno(distancia);

114. 1. double calcularCostoDelViaje() {
2.
3.
4.
5.
6. }
if (esBandeiraDos()) return calcularBanderaDos(distancia);
if (esBandeiraTres()) return calcularBanderaTres(distancia);
if (esDomingo()) return calcularDomingo(distancia);
return calcularBanderaUno(distancia);

115. ¿Miedo de estropear algo?

116. Test-Driven Development

117. Ahorre tiempo y diñero, pruebe
más temprano!

118. Herramientas – Hudson + Sonar