Malos Olores

* Malos olores
* Acoplamiento de control
* ¿Qué buscar?
* Un método o bloque verifica el valor de un parámetro para decidir el camino de ejecución.
* El nombre de un método incluye palabras como: o.
* ¿Por qué es un problema?
* Duplicación: el acoplamiento de control es una forma de duplicación, porque el cliente sabe que camino se debe seguir.
* Flexibilidad: elcliente y el método están acoplados, cualquier cambio en los posibles valores del parámetro de control debe ser reflejado en ambos.
* Simplicidad: el método llamado probablemente también sea un método avaricioso porque incluye por lo menos dos caminos diferentes.
* ¿Cuándo dejarlo?
* No hay razones para dejar este mal olor.
* ¿Cómo llegamos a esto?
* Algunas ocasiones deseamosmodificar ligeramente el comportamiento de un método, pero no deseamos perder el comportamiento original, así que agregamos un parámetro y lo usamos para variar el curso de este.
* ¿Qué hacer?
1. Usa: extrae método para simplificar el código a su parte más básica.
2. Después usa: método en línea para colocar la responsabilidad en el cliente.
3. Repite esto en toda la cadena de llamadoshasta llegar a la fuente del valor de control.
* ¿Qué buscar en seguida?
* Duplicación: si el parámetro de control fue pasado en más de un cliente, el paso para usar la reestructura: método en línea, introducirá duplicación.
* Tamaño: al usar la reestructura: método en línea, se puede generar el mal olor: clase perezosa.
* Abstracción: cuando encuentres la fuente de la variable decontrol, probablemente tengas un caso del mal olor: polimorfismo simulado.
* Malos olores
* Caso especial
* ¿Qué buscar?
* Instrucciones if complejas
* Clausulas de guardia, verificaciones de valores particulares antes de realizar un trabajo, especialmente comparaciones con constantes.
* ¿Por qué es un problema?
* Comunicación: un caso especial incrementa la cantidad deconceptos que el lector debe mantener en la cabeza mientras intenta entender un fragmento de código.
* ¿Cuándo dejarlo?
* En un algoritmo recursivo siempre existe uno o más casos base que terminan la recursión, estos no pueden eliminarse. En ocasiones una clausula if o unless es la forma más sencilla de realizar una tarea.
* ¿Cómo llegamos a esto?
* En ocasiones, introducir uncaso especial es la forma más fácil de pasar las pruebas. En otras ocasiones, una clausula de guardia se introdujo para defenderse contra clientes mal programados, o mientras se simplificaba una expresión booleana complicada.
* ¿Qué hacer?
* Si los condicionales toman el lugar del polimorfismo, usa: reemplazar condicional con polimorfismo. Las cosas serán más sencillas si primero usas:extraer método.
* Si las clausulas if y else son parecidas, puedes reescribirlas para que con el mismo fragmento de código se puedan generar los resultados adecuados para cada caso, después, el condicional puede ser eliminado.
* Si tienes una clausula de guardia defensiva intenta moverla a los clientes del método.
* ¿Qué buscar en seguida?
* Duplicación: remover un caso especial puedevolver el código similar a otros fragmentos o revelar una estructura común.
* Simplicidad: mover los clausulas de guardia a los clientes comúnmente revela una sola causa para múltiples clausulas condicionales. Maneja el caso especial donde ocurra y busca formas de prevenirlo.
* Malos olores
* Código duplicado
* ¿Qué buscar?
* La forma sencilla: dos fragmentos de código parecenidénticos.
* La forma difícil: dos fragmentos de código tienen efectos casi idénticos a cualquier nivel conceptual.
* ¿Por qué es un problema?
* Tamaño: el código es más grande de lo que requiere.
* Flexibilidad: un concepto de diseño expresado más de una vez interfiere con los cambios futuros, además, el cambio debe hacerse en múltiples lugares.
* Comunicación: la repetición...