/*@jsxRuntime automatic @jsxImportSource react*/ const {Fragment: _Fragment, jsx: _jsx, jsxs: _jsxs} = arguments[0]; const {useMDXComponents: _provideComponents} = arguments[0]; function _createMdxContent(props) { const _components = Object.assign({ p: "p", h2: "h2", blockquote: "blockquote", a: "a", code: "code", pre: "pre", ul: "ul", li: "li", ol: "ol" }, _provideComponents(), props.components); return _jsxs(_Fragment, { children: [_jsx(_components.p, { children: "En ocasiones, me asalta la idea de que debería adoptar una estrategia proactiva\nal enfrentar la refactorización de código legacy. Para mí, el término \"código\nlegacy\" abarca todo código que, desde mi perspectiva, podría ser optimizado. Son\ncontadas las veces que me he topado con código que no requería ninguna\nmodificación o mejora, gracias a una adecuada nominación o estructuración." }), "\n", _jsx(_components.h2, { children: "💨 TL;DR" }), "\n", _jsx(_components.p, { children: "Este año 2023, en Pamplona Software Crafters, tuve la oportunidad de conocer a\nPedro Santos y su charla “Refactoring Priority Premise”, que aborda cómo\nidentificar y remediar code smells siguiendo un orden específico. Dicho orden se\ndeterminaba en función del problema que necesitabas resolver; por ejemplo, la\nlegibilidad ocupaba el primer nivel y podía ser mejorada mediante cambios de\nnombre, comentarios, entre otros. La intención de este artículo, que será el\nprecursor de una serie, es elucidar, mediante ejemplos, por qué no solo es\ncrucial realizar una refactorización, sino cómo llevarla a cabo.No\nprofundizaremos en cómo se realiza la refactorización ni en el tiempo que se le\ndebe dedicar; en cambio, nos enfocaremos en el orden de refactorización." }), "\n", _jsxs(_components.blockquote, { children: ["\n", _jsx(_components.p, { children: "⚠ Aviso de antemano que sera un artículo largo, y soy consciente de técnicas\nque se pueden aplicar para hacerlo mas corto, pero la idea es que se pueda ser\nleído por cualquier persona sin conocimientos previos de refactorización." }), "\n"] }), "\n", _jsx(_components.h2, { children: "📝 Contexto" }), "\n", _jsxs(_components.p, { children: ["Trabajaremos con el código de Gilded Rose, un proyecto que se ha utilizado en\nnumerosas conferencias y talleres de refactorización. El código original fue\nescrito en C#, pero en esta ocasión, trabajaremos con una versión en Kotlin\ntraducida por\n", _jsx(_components.a, { href: "https://github.com/emilybache/GildedRose-Refactoring-Kata/tree/main/Kotlin", children: "Emily Bache" }), "."] }), "\n", _jsx(_components.p, { children: "A menudo me gusta reflexionar sobre la refactorización tal como me la enseñó\nCarlos Blé. Imaginemos un cubo de Rubik, en el cual debemos ejecutar una serie\nde pasos para alcanzar la solución. Estos pasos pueden variar dado que existen\nmúltiples posibilidades, pero el objetivo permanece inalterable. Quiero decir\ncon esto que la refactorización se rige por numerosas heurísticas, y ejecutarla\nadecuadamente dependerá de tu experiencia previa o de la herramienta que estés\nutilizando, como por ejemplo, IntelliJ." }), "\n", _jsx(_components.p, { children: "Para ilustrar esto, trabajaremos con ejemplos detallados, convirtiendo el\nproceso en una experiencia más artesanal, para que no solo comprendamos el\nnombre del refactor, sino cómo se implementa y dónde. En este caso, voy a\nelaborar todas las pruebas (tests) de antemano, ya que sin ellas, resulta muy\ndifícil llevar a cabo una refactorización." }), "\n", _jsx(_components.p, { children: "En esta ocasión, trabajaré con Kotlin, un lenguaje de programación que siempre\nha capturado mi interés, utilizando la versión Community de IntelliJ IDEA, al\nigual que muchos otros desarrolladores. Comenzaremos revisando brevemente el\ncódigo." }), "\n", _jsx(_components.p, { children: _jsx(_components.code, { children: "GildedRose.kt" }) }), "\n", _jsx(_components.pre, { children: _jsx(_components.code, { className: "language-kotlin", children: "package dev.wolfremium.www\n\nclass GildedRose(var items: List) {\n fun updateQuality() {\n for (i in items.indices) {\n if (items[i].name != \"Aged Brie\" && items[i].name != \"Backstage passes to a TAFKAL80ETC concert\") {\n if (items[i].quality > 0) {\n if (items[i].name != \"Sulfuras, Hand of Ragnaros\") {\n items[i].quality = items[i].quality - 1\n }\n }\n } else {\n if (items[i].quality < 50) {\n items[i].quality = items[i].quality + 1\n\n if (items[i].name == \"Backstage passes to a TAFKAL80ETC concert\") {\n if (items[i].sellIn < 11) {\n if (items[i].quality < 50) {\n items[i].quality = items[i].quality + 1\n }\n }\n\n if (items[i].sellIn < 6) {\n if (items[i].quality < 50) {\n items[i].quality = items[i].quality + 1\n }\n }\n }\n }\n }\n\n if (items[i].name != \"Sulfuras, Hand of Ragnaros\") {\n items[i].sellIn = items[i].sellIn - 1\n }\n\n if (items[i].sellIn < 0) {\n if (items[i].name != \"Aged Brie\") {\n if (items[i].name != \"Backstage passes to a TAFKAL80ETC concert\") {\n if (items[i].quality > 0) {\n if (items[i].name != \"Sulfuras, Hand of Ragnaros\") {\n items[i].quality = items[i].quality - 1\n }\n }\n } else {\n items[i].quality = items[i].quality - items[i].quality\n }\n } else {\n if (items[i].quality < 50) {\n items[i].quality = items[i].quality + 1\n }\n }\n }\n }\n }\n}\n" }) }), "\n", _jsx(_components.p, { children: _jsx(_components.code, { children: "Item.kt" }) }), "\n", _jsx(_components.pre, { children: _jsx(_components.code, { className: "language-kotlin", children: "package dev.wolfremium.www\n\nopen class Item(var name: String, var sellIn: Int, var quality: Int) {\n override fun toString(): String {\n return this.name + \", \" + this.sellIn + \", \" + this.quality\n }\n}\n" }) }), "\n", _jsx(_components.p, { children: "Como podrán observar, les invito a examinar nuevamente el código anterior con\ndetenimiento, y destacar los aspectos que les llamen la atención. En mi caso, he\nidentificado los siguientes puntos:" }), "\n", _jsxs(_components.ul, { children: ["\n", _jsx(_components.li, { children: "Complejidad Anidada" }), "\n", _jsx(_components.li, { children: "Código Duplicado" }), "\n", _jsx(_components.li, { children: "Literales Mágicos" }), "\n", _jsx(_components.li, { children: "Violación del Principio de Responsabilidad Única (SRP)" }), "\n", _jsx(_components.li, { children: "Falta de Encapsulación" }), "\n", _jsx(_components.li, { children: "Problemas de Extensibilidad y Mantenibilidad" }), "\n", _jsx(_components.li, { children: "Uso Ineficiente de los Índices" }), "\n", _jsx(_components.li, { children: "Obsesión por los primitivos" }), "\n"] }), "\n", _jsx(_components.p, { children: "Ahora, para maximizar nuestro retorno de inversión, la clave será clasificar\nestos puntos desde el más fácil hasta el más difícil de corregir. Según mi\nanálisis, he establecido el siguiente orden:" }), "\n", _jsxs(_components.ol, { children: ["\n", _jsx(_components.li, { children: "Literales Mágicos" }), "\n", _jsx(_components.li, { children: "Uso Ineficiente de los Índices" }), "\n", _jsx(_components.li, { children: "Complejidad Anidada y Código Duplicado" }), "\n", _jsx(_components.li, { children: "Violación del Principio de Responsabilidad Única (SRP)" }), "\n", _jsx(_components.li, { children: "Falta de Encapsulación" }), "\n", _jsx(_components.li, { children: "Problemas de Extensibilidad y Mantenibilidad" }), "\n"] }), "\n", _jsx(_components.p, { children: "Procederemos a rectificar cada aspecto, explicando dónde se manifiesta en el\ncódigo y cómo corregirlo." }), "\n", _jsx(_components.h2, { children: "🧙‍♂️ Literales Mágicos" }), "\n", _jsxs(_components.p, { children: ["Los literales mágicos son valores que se encuentran en el código sin una\nexplicación o contexto claro. En este caso, es evidente que el valor 50 se\nrepite en múltiples ocasiones, pero no se entiende su significado. Al analizar\nel código, identificamos que ", _jsx(_components.code, { children: "items[i].quality < 50" }), " es una expresión\nrecurrente, y además, el valor 50 representa el límite superior. Por lo tanto,\nes prudente crear una constante llamada ", _jsx(_components.code, { children: "maxQuality" }), " y asignarle el valor 50.\nPara lograr esto, utilizaremos el atajo ", _jsx(_components.code, { children: "Ctrl + Alt + V" }), " en IntelliJ sobre uno\nde los valores 50 para extraer la variable."] }), "\n", _jsx(_components.p, { children: _jsx(_components.code, { children: "GildedRose.kt" }) }), "\n", _jsx(_components.pre, { children: _jsx(_components.code, { className: "language-kotlin", children: "for (i in items.indices) {\n val maxQuality = 50\n if (items[i].name != \"Aged Brie\" && items[i].name != \"Backstage passes to a TAFKAL80ETC concert\") {\n if (items[i].quality > 0) {\n if (items[i].name != \"Sulfuras, Hand of Ragnaros\") {\n items[i].quality = items[i].quality - 1\n }\n }\n } else {\n if (items[i].quality < maxQuality) {\n items[i].quality = items[i].quality + 1\n\n if (items[i].name == \"Backstage passes to a TAFKAL80ETC concert\") {\n if (items[i].sellIn < 11) {\n if (items[i].quality < maxQuality) {\n items[i].quality = items[i].quality + 1\n }\n }\n\n if (items[i].sellIn < 6) {\n if (items[i].quality < maxQuality) {\n items[i].quality = items[i].quality + 1\n }\n }\n }\n }\n }\n// ...\n" }) }), "\n", _jsxs(_components.blockquote, { children: ["\n", _jsx(_components.p, { children: "💡 Nota: Siempre que sea posible, es recomendable lanzar los tests para\nverificar que no hemos introducido ningún error." }), "\n"] }), "\n", _jsx(_components.p, { children: "Es un pequeño cambio, pero ya podemos observar una mejora en la legibilidad del\ncódigo. Vamos a hacer los mismo con:" }), "\n", _jsxs(_components.ul, { children: ["\n", _jsxs(_components.li, { children: [_jsx(_components.code, { children: "0" }), " → ", _jsx(_components.code, { children: "minQuality" })] }), "\n", _jsxs(_components.li, { children: [_jsx(_components.code, { children: "11" }), " → ", _jsx(_components.code, { children: "backstagePassesBigThreshold" })] }), "\n", _jsxs(_components.li, { children: [_jsx(_components.code, { children: "6" }), " → ", _jsx(_components.code, { children: "backstagePassesSmallThreshold" })] }), "\n", _jsxs(_components.li, { children: [_jsx(_components.code, { children: "\"Aged Brie\"" }), " → ", _jsx(_components.code, { children: "agedBrie" })] }), "\n", _jsxs(_components.li, { children: [_jsx(_components.code, { children: "\"Backstage passes to a TAFKAL80ETC concert\"" }), " → ", _jsx(_components.code, { children: "backstagePasses" })] }), "\n", _jsxs(_components.li, { children: [_jsx(_components.code, { children: "\"Sulfuras, Hand of Ragnaros\"" }), " → ", _jsx(_components.code, { children: "sulfuras" })] }), "\n", _jsxs(_components.li, { children: [_jsx(_components.code, { children: "1" }), " → ", _jsx(_components.code, { children: "qualityIncrement" })] }), "\n", _jsxs(_components.li, { children: [_jsx(_components.code, { children: "-1" }), " → ", _jsx(_components.code, { children: "qualityDecrement" })] }), "\n"] }), "\n", _jsx(_components.p, { children: _jsx(_components.code, { children: "GildedRose.kt" }) }), "\n", _jsx(_components.pre, { children: _jsx(_components.code, { className: "language-kotlin", children: "class GildedRose(var items: List) {\n fun updateQuality() {\n for (i in items.indices) {\n val maxQuality = 50\n val minQuality = 0\n val agedBrie = \"Aged Brie\"\n val backstagePasses = \"Backstage passes to a TAFKAL80ETC concert\"\n val sulfuras = \"Sulfuras, Hand of Ragnaros\"\n val qualityIncrease = 1\n val qualityDecrease = 1\n if (items[i].name != agedBrie && items[i].name != backstagePasses) {\n if (items[i].quality > minQuality) {\n if (items[i].name != sulfuras) {\n items[i].quality = items[i].quality - qualityDecrease\n }\n }\n } else {\n if (items[i].quality < maxQuality) {\n items[i].quality = items[i].quality + qualityIncrease\n\n if (items[i].name == backstagePasses) {\n val backstagePassesBigThreshold = 11\n if (items[i].sellIn < backstagePassesBigThreshold) {\n if (items[i].quality < maxQuality) {\n items[i].quality = items[i].quality + qualityIncrease\n }\n }\n\n val backstagePassesSmallThreshold = 6\n if (items[i].sellIn < backstagePassesSmallThreshold) {\n if (items[i].quality < maxQuality) {\n items[i].quality = items[i].quality + qualityIncrease\n }\n }\n }\n }\n }\n\n if (items[i].name != sulfuras) {\n items[i].sellIn = items[i].sellIn - 1\n }\n\n if (items[i].sellIn < minQuality) {\n if (items[i].name != agedBrie) {\n if (items[i].name != backstagePasses) {\n if (items[i].quality > minQuality) {\n if (items[i].name != sulfuras) {\n items[i].quality = items[i].quality - qualityDecrease\n }\n }\n } else {\n items[i].quality = items[i].quality - items[i].quality\n }\n } else {\n if (items[i].quality < maxQuality) {\n items[i].quality = items[i].quality + qualityIncrease\n }\n }\n }\n }\n }\n}\n" }) }), "\n", _jsx(_components.p, { children: "Perfecto 👌. Ahora, podemos observar que el código es más legible, pero aún nos\nqueda un largo camino por recorrer. En este punto, es importante mencionar que\nno debemos preocuparnos por la eficiencia del código, ya que la refactorización\nse centra en la legibilidad y la mantenibilidad. Siempre podemos optimizar el\ncódigo más adelante, pero primero debemos asegurarnos de que sea fácil de\nentender." }), "\n", _jsx(_components.h2, { children: "🤔 Uso Ineficiente de los Índices" }), "\n", _jsxs(_components.p, { children: ["El uso ineficiente de los índices es un problema común en el desarrollo de\nsoftware, y generalmente se debe a la falta de abstracción. En este caso,\npodemos observar que el índice ", _jsx(_components.code, { children: "i" }), " se repite en múltiples ocasiones, y que\nrealmente no necesitamos ese numero para mas nada que para acceder a a posición\nde la lista ", _jsx(_components.code, { children: "items" }), ". Hay veces que IntelliJ no nos ayuda a convertir los bucles\na un ", _jsx(_components.code, { children: "forEach" }), ", por lo que tendremos que realizar un paso intermedio."] }), "\n", _jsxs(_components.p, { children: ["Empezaremos con extraer el código ", _jsx(_components.code, { children: "items[i]" }), " a una variable llamada ", _jsx(_components.code, { children: "item" }), " y\nutilizarla en su lugar. Para lograr esto, utilizaremos el atajo ", _jsx(_components.code, { children: "Ctrl + Alt + V" }), "\nen IntelliJ sobre ", _jsx(_components.code, { children: "items[i]" }), " para extraer la variable."] }), "\n", _jsx(_components.p, { children: _jsx(_components.code, { children: "GildedRose.kt" }) }), "\n", _jsx(_components.pre, { children: _jsx(_components.code, { className: "language-kotlin", children: "fun updateQuality() {\n for (i in items.indices) {\n val maxQuality = 50\n val minQuality = 0\n val agedBrie = \"Aged Brie\"\n val backstagePasses = \"Backstage passes to a TAFKAL80ETC concert\"\n val sulfuras = \"Sulfuras, Hand of Ragnaros\"\n val qualityIncrease = 1\n val qualityDecrease = 1\n val item = items[i]\n if (item.name != agedBrie && item.name != backstagePasses) {\n if (item.quality > minQuality) {\n if (item.name != sulfuras) {\n item.quality = item.quality - qualityDecrease\n }\n }\n// ...\n" }) }), "\n", _jsxs(_components.p, { children: ["Cambiamos el bucle ", _jsx(_components.code, { children: "for" }), " por con lo siguiente manualmente:"] }), "\n", _jsx(_components.pre, { children: _jsx(_components.code, { className: "language-kotlin", children: "for (item in items) {\n// ...\n" }) }), "\n", _jsxs(_components.p, { children: ["Vemos que la variable item que se extrajo anteriormente ya esta asignada por el\nbucle por lo que deberemos borrarla. Ahora si podemos convertir el bucle a un\n", _jsx(_components.code, { children: "forEach" }), " utilizando el atajo ", _jsx(_components.code, { children: "Alt + Enter" }), " en IntelliJ."] }), "\n", _jsx(_components.p, { children: _jsx(_components.code, { children: "GildedRose.kt" }) }), "\n", _jsx(_components.pre, { children: _jsx(_components.code, { className: "language-kotlin", children: "fun updateQuality() {\n items.forEach { item ->\n val maxQuality = 50\n val minQuality = 0\n val agedBrie = \"Aged Brie\"\n val backstagePasses = \"Backstage passes to a TAFKAL80ETC concert\"\n val sulfuras = \"Sulfuras, Hand of Ragnaros\"\n val qualityIncrease = 1\n val qualityDecrease = 1\n if (item.name != agedBrie && item.name != backstagePasses) {\n if (item.quality > minQuality) {\n if (item.name != sulfuras) {\n item.quality = item.quality - qualityDecrease\n }\n }\n// ...\n" }) }), "\n", _jsx(_components.p, { children: "👌 Maravilloso." }), "\n", _jsx(_components.h2, { children: "🤯 Complejidad Anidada y Código Duplicado" }), "\n", _jsxs(_components.p, { children: ["La complejidad anidada es un problema que se manifiesta cuando tenemos múltiples\ncondiciones anidadas, y que generalmente se debe a la falta de abstracción. En\neste caso, podemos observar que el código se vuelve difícil de leer debido a la\ncantidad de condiciones anidadas. Para remediar esto, utilizaremos el atajo\n", _jsx(_components.code, { children: "Alt + Enter" }), " en IntelliJ sobre el ", _jsx(_components.code, { children: "if" }), " para invertir la primera condición."] }), "\n", _jsx(_components.p, { children: _jsx(_components.code, { children: "GildedRose.kt" }) }), "\n", _jsx(_components.pre, { children: _jsx(_components.code, { className: "language-kotlin", children: "fun updateQuality() {\n items.forEach { item ->\n val maxQuality = 50\n val minQuality = 0\n val agedBrie = \"Aged Brie\"\n val backstagePasses = \"Backstage passes to a TAFKAL80ETC concert\"\n val sulfuras = \"Sulfuras, Hand of Ragnaros\"\n val qualityIncrease = 1\n val qualityDecrease = 1\n if (item.name == agedBrie || item.name == backstagePasses) {\n if (item.quality < maxQuality) {\n item.quality = item.quality + qualityIncrease\n\n if (item.name == backstagePasses) {\n val backstagePassesBigThreshold = 11\n if (item.sellIn < backstagePassesBigThreshold) {\n if (item.quality < maxQuality) {\n item.quality = item.quality + qualityIncrease\n }\n }\n\n val backstagePassesSmallThreshold = 6\n if (item.sellIn < backstagePassesSmallThreshold) {\n if (item.quality < maxQuality) {\n item.quality = item.quality + qualityIncrease\n }\n }\n }\n }\n } else {\n if (item.quality > minQuality) {\n if (item.name != sulfuras) {\n item.quality = item.quality - qualityDecrease\n }\n }\n }\n// ...\n" }) }), "\n", _jsxs(_components.p, { children: ["Ahora, vemos que es mas fácil de leer una condición sin negarla pero tenemos que\naplicar el mismo cambio a las otras condiciones. Para lograr esto, utilizaremos\nla misma técnica para aquellos casos que tienen un ", _jsx(_components.code, { children: "else" }), ". Esto solo ha\nsolucionado un poco el problema de la legibilidad."] }), "\n", _jsxs(_components.p, { children: ["La idea es empezar a generar las condiciones idóneas para mover las condiciones\nlo mas a la izquierda posible. Otro tipo de refactor es mergear las condiciones\nde los ", _jsx(_components.code, { children: "if" }), " vamos a hacerlo con este caso:"] }), "\n", _jsx(_components.p, { children: _jsx(_components.code, { children: "GildedRose.kt" }) }), "\n", _jsx(_components.pre, { children: _jsx(_components.code, { className: "language-kotlin", children: "if (item.quality > minQuality) {\n if (item.name != sulfuras) {\n item.quality = item.quality - qualityDecrease\n }\n}\n" }) }), "\n", _jsxs(_components.p, { children: ["Lo convertimos usando el atajo ", _jsx(_components.code, { children: "Alt + Enter" }), " sobre el primer ", _jsx(_components.code, { children: "if" }), " y\nseleccionamos la opción ", _jsx(_components.code, { children: "Merge if's" }), ":"] }), "\n", _jsx(_components.p, { children: _jsx(_components.code, { children: "GildedRose.kt" }) }), "\n", _jsx(_components.pre, { children: _jsx(_components.code, { className: "language-kotlin", children: "if (item.quality > minQuality && item.name != sulfuras) {\n item.quality = item.quality - qualityDecrease\n}\n" }) }), "\n", _jsxs(_components.blockquote, { children: ["\n", _jsxs(_components.p, { children: ["💡 Nota: Esto solo aplica cuando tienes varios ", _jsx(_components.code, { children: "if" }), " seguidos y no tienen un\n", _jsx(_components.code, { children: "else" }), " en medio."] }), "\n"] }), "\n", _jsx(_components.p, { children: "Con esta técnica podemos reducir la complejidad anidada ya que evitamos un\nbloque mas de identación. Vamos a hacer lo mismo con otros bloques de código y\nveamos el resultado:" }), "\n", _jsx(_components.p, { children: _jsx(_components.code, { children: "GildedRose.kt" }) }), "\n", _jsx(_components.pre, { children: _jsx(_components.code, { className: "language-kotlin", children: "class GildedRose(var items: List) {\n fun updateQuality() {\n items.forEach { item ->\n val maxQuality = 50\n val minQuality = 0\n val agedBrie = \"Aged Brie\"\n val backstagePasses = \"Backstage passes to a TAFKAL80ETC concert\"\n val sulfuras = \"Sulfuras, Hand of Ragnaros\"\n val qualityIncrease = 1\n val qualityDecrease = 1\n if (item.name == agedBrie || item.name == backstagePasses) {\n if (item.quality < maxQuality) {\n item.quality = item.quality + qualityIncrease\n if (item.name == backstagePasses) {\n val backstagePassesBigThreshold = 11\n if (item.sellIn < backstagePassesBigThreshold && item.quality < maxQuality) {\n item.quality = item.quality + qualityIncrease\n }\n\n val backstagePassesSmallThreshold = 6\n if (item.sellIn < backstagePassesSmallThreshold && item.quality < maxQuality) {\n item.quality = item.quality + qualityIncrease\n }\n }\n }\n } else {\n if (item.quality > minQuality && item.name != sulfuras) {\n item.quality = item.quality - qualityDecrease\n }\n }\n\n if (item.name != sulfuras) {\n item.sellIn = item.sellIn - 1\n }\n\n if (item.sellIn < minQuality) {\n if (item.name == agedBrie) {\n if (item.quality < maxQuality) {\n item.quality = item.quality + qualityIncrease\n }\n } else {\n if (item.name == backstagePasses) {\n item.quality = item.quality - item.quality\n } else {\n if (item.quality > minQuality && item.name != sulfuras) {\n item.quality = item.quality - qualityDecrease\n }\n }\n }\n }\n }\n }\n}\n" }) }), "\n", _jsx(_components.p, { children: "Ha mejorado un poco pero aún tenemos complejidad anidada. Veo que las\ncondiciones se han complicado un poco, vamos a ver si podemos simplificarlas.\nVamos a empezar por este bloque de código:" }), "\n", _jsx(_components.p, { children: _jsx(_components.code, { children: "GildedRose.kt" }) }), "\n", _jsx(_components.pre, { children: _jsx(_components.code, { className: "language-kotlin", children: "if (item.quality < maxQuality) {\n item.quality = item.quality + qualityIncrease\n if (item.name == backstagePasses) {\n val backstagePassesBigThreshold = 11\n if (item.sellIn < backstagePassesBigThreshold && item.quality < maxQuality) {\n item.quality = item.quality + qualityIncrease\n }\n\n val backstagePassesSmallThreshold = 6\n if (item.sellIn < backstagePassesSmallThreshold && item.quality < maxQuality) {\n item.quality = item.quality + qualityIncrease\n }\n }\n}\n" }) }), "\n", _jsxs(_components.p, { children: ["Vamos a extraer el código ", _jsx(_components.code, { children: "item.quality < maxQuality" }), " a una variable llamada\n", _jsx(_components.code, { children: "isNotTheMaximumQuality" }), " y utilizarla en su lugar. Para lograr esto,\nutilizaremos el atajo ", _jsx(_components.code, { children: "Ctrl + Alt + V" }), " en IntelliJ sobre\n", _jsx(_components.code, { children: "item.quality < maxQuality" }), " para extraer la variable."] }), "\n", _jsx(_components.p, { children: _jsx(_components.code, { children: "GildedRose.kt" }) }), "\n", _jsx(_components.pre, { children: _jsx(_components.code, { className: "language-kotlin", children: "val isNotTheMaximumQuality = item.quality < maxQuality\nif (item.name == agedBrie || item.name == backstagePasses) {\n if (isNotTheMaximumQuality) {\n item.quality = item.quality + qualityIncrease\n if (item.name == backstagePasses) {\n val backstagePassesBigThreshold = 11\n if (!(item.sellIn >= backstagePassesBigThreshold) && isNotTheMaximumQuality) {\n item.quality = item.quality + qualityIncrease\n }\n\n val backstagePassesSmallThreshold = 6\n if (item.sellIn < backstagePassesSmallThreshold && isNotTheMaximumQuality) {\n item.quality = item.quality + qualityIncrease\n }\n }\n }\n// ...\n" }) }), "\n", _jsx(_components.p, { children: "Vamos a hacer lo mismo con el resto de condiciones y veamos el resultado:" }), "\n", _jsx(_components.p, { children: _jsx(_components.code, { children: "GildedRose.kt" }) }), "\n", _jsx(_components.pre, { children: _jsx(_components.code, { className: "language-kotlin", children: "class GildedRose(var items: List) {\n fun updateQuality() {\n items.forEach { item ->\n val maxQuality = 50\n val minQuality = 0\n val agedBrie = \"Aged Brie\"\n val backstagePasses = \"Backstage passes to a TAFKAL80ETC concert\"\n val sulfuras = \"Sulfuras, Hand of Ragnaros\"\n val qualityIncrease = 1\n val qualityDecrease = 1\n val isNotTheMaximumQuality = item.quality < maxQuality\n val isAgedBrie = item.name == agedBrie\n val isABackStagePass = item.name == backstagePasses\n val areAgeBrieOrPasses = isAgedBrie || isABackStagePass\n val hasQuality = item.quality > minQuality\n val isNotSulfuras = item.name != sulfuras\n if (areAgeBrieOrPasses) {\n if (isNotTheMaximumQuality) {\n item.quality = item.quality + qualityIncrease\n if (isABackStagePass) {\n val backstagePassesBigThreshold = 11\n val areInDateToBeSold = item.sellIn < backstagePassesBigThreshold\n if (areInDateToBeSold && isNotTheMaximumQuality) {\n item.quality = item.quality + qualityIncrease\n }\n\n val backstagePassesSmallThreshold = 6\n val areOnTheLastDays = item.sellIn < backstagePassesSmallThreshold\n if (areOnTheLastDays && isNotTheMaximumQuality) {\n item.quality = item.quality + qualityIncrease\n }\n }\n }\n } else {\n if (hasQuality && isNotSulfuras) {\n item.quality = item.quality - qualityDecrease\n }\n }\n\n if (isNotSulfuras) {\n item.sellIn = item.sellIn - 1\n }\n\n val notExpiredNumber = 0\n val isExpired = item.sellIn < notExpiredNumber\n if (isExpired) {\n if (isAgedBrie) {\n if (isNotTheMaximumQuality) {\n item.quality = item.quality + qualityIncrease\n }\n } else {\n if (isABackStagePass) {\n item.quality = item.quality - item.quality\n } else {\n if (hasQuality && isNotSulfuras) {\n item.quality = item.quality - qualityDecrease\n }\n }\n }\n }\n }\n }\n}\n" }) }), "\n", _jsx(_components.p, { children: "Ahora, podemos observar que las variables le dan legibilidad al código, pero aún\ntenemos complejidad anidada para desgracia de nosotros. Esta vez vamos a\nreordenar y extraer en métodos las condiciones para que sea mas fácil de leer.\nVamos a empezar por este bloque de código:" }), "\n", _jsx(_components.p, { children: _jsx(_components.code, { children: "GildedRose.kt" }) }), "\n", _jsx(_components.pre, { children: _jsx(_components.code, { className: "language-kotlin", children: "if (isNotSulfuras) {\n item.sellIn = item.sellIn - 1\n}\n" }) }), "\n", _jsx(_components.p, { children: "Podemos antes de empezar realizar una pequeña refactorización para que sea mas\nfácil de leer:" }), "\n", _jsx(_components.p, { children: _jsx(_components.code, { children: "GildedRose.kt" }) }), "\n", _jsx(_components.pre, { children: _jsx(_components.code, { className: "language-kotlin", children: "if (isNotSulfuras) {\n val dayDecrease = 1\n item.sellIn -= dayDecrease\n}\n" }) }), "\n", _jsx(_components.p, { children: "Os preguntareis porque no lo hice antes, la respuesta es que no era tan fácil de\nleer el código por lo que aunque estemos resolviendo un problema de complejidad\nhace falta poner foco en lo sencillo y asequible. Volviendo a resolver\nlegibilidad antes que complejidad siguiendo el orden de lo que es mas fácil de\nresolver." }), "\n", _jsxs(_components.p, { children: ["Ahora si podemos extraer el código a un método llamado ", _jsx(_components.code, { children: "decreaseSellIn" }), " y\nutilizarlo en su lugar. Para lograr esto, utilizaremos el atajo ", _jsx(_components.code, { children: "Ctrl + Alt + M" }), "\nen IntelliJ sobre para extraer el método."] }), "\n", _jsx(_components.p, { children: _jsx(_components.code, { children: "GildedRose.kt" }) }), "\n", _jsx(_components.pre, { children: _jsx(_components.code, { className: "language-kotlin", children: "private fun decreaseSellIn(isNotSulfuras: Boolean, item: Item) {\n if (isNotSulfuras) {\n val dayDecrease = 1\n item.sellIn -= dayDecrease\n }\n}\n" }) }), "\n", _jsxs(_components.p, { children: ["Sin darnos cuenta ya hemos encapsulado en un método la complejidad anidada de\nreducir ", _jsx(_components.code, { children: "SellIn" }), ". Vemos que el método anterior tiene un parámetro que no es\nutilizado en el método, esto es un indicador de que el método no esta bien\nencapsulado. Vamos a extraer el código ", _jsx(_components.code, { children: "isNotSulfuras" }), " a un método llamado\n", _jsx(_components.code, { children: "isNotSulfuras" }), " y utilizarlo en su lugar. Para lograr esto, utilizaremos el\natajo ", _jsx(_components.code, { children: "Ctrl + Alt + M" }), " en IntelliJ sobre para extraer el método."] }), "\n", _jsx(_components.p, { children: _jsx(_components.code, { children: "GildedRose.kt" }) }), "\n", _jsx(_components.pre, { children: _jsx(_components.code, { className: "language-kotlin", children: "private fun isNotSulfuras(item: Item): Boolean {\n val sulfuras = \"Sulfuras, Hand of Ragnaros\"\n return item.name != sulfuras\n}\n" }) }), "\n", _jsxs(_components.p, { children: ["Se va a quedar la variable ", _jsx(_components.code, { children: "val isNotSulfuras = isNotSulfuras(item)" }), " que\nrepresenta lo mismo que el método pero no es necesario ya que el método es mas\nlegible. Vamos a eliminar la variable y utilizar el método en su lugar, usando\nel atajo ", _jsx(_components.code, { children: "Ctrl + Alt + N" }), " o ", _jsx(_components.code, { children: "Inline property" }), " sobre la variable."] }), "\n", _jsx(_components.p, { children: _jsx(_components.code, { children: "GildedRose.kt" }) }), "\n", _jsx(_components.pre, { children: _jsx(_components.code, { className: "language-kotlin", children: "//...\nelse {\n if (hasQuality && isNotSulfuras(item)) {\n item.quality = item.quality - qualityDecrease\n }\n}\n\ndecreaseSellIn(isNotSulfuras(item), item)\n//...\n" }) }), "\n", _jsxs(_components.p, { children: ["Queda feo el método ", _jsx(_components.code, { children: "decreaseSellIn" }), " con el parámetro ", _jsx(_components.code, { children: "isNotSulfuras(item)" }), " por\nlo que vamos a quitar el parámetro y utilizar el método ", _jsx(_components.code, { children: "isNotSulfuras" }), " dentro\ndel método ", _jsx(_components.code, { children: "decreaseSellIn" }), "."] }), "\n", _jsx(_components.p, { children: _jsx(_components.code, { children: "GildedRose.kt" }) }), "\n", _jsx(_components.pre, { children: _jsx(_components.code, { className: "language-kotlin", children: "private fun decreaseSellIn(isNotSulfuras: Boolean, item: Item) {\n if (isNotSulfuras(item)) {\n val dayDecrease = 1\n item.sellIn -= dayDecrease\n }\n}\n" }) }), "\n", _jsxs(_components.p, { children: ["Seguidamente borramos el parámetro ", _jsx(_components.code, { children: "isNotSulfuras" }), " que no esta siendo utilizado\ncon ", _jsx(_components.code, { children: "Alt + Enter" }), " y seleccionamos la opción ", _jsx(_components.code, { children: "Remove parameter 'isNotSulfuras'" }), ":"] }), "\n", _jsx(_components.p, { children: _jsx(_components.code, { children: "GildedRose.kt" }) }), "\n", _jsx(_components.pre, { children: _jsx(_components.code, { className: "language-kotlin", children: "private fun decreaseSellIn(item: Item) {\n if (isNotSulfuras(item)) {\n val dayDecrease = 1\n item.sellIn -= dayDecrease\n }\n}\n" }) }), "\n", _jsxs(_components.p, { children: ["Ahora que hemos separado toda la lógica del ", _jsx(_components.code, { children: "SellIn" }), " nos quedaría la mejora de\nla calidad y el empeoramiento de la misma. Vamos a mirar detenidamente este\ncaso:"] }), "\n", _jsx(_components.p, { children: _jsx(_components.code, { children: "GildedRose.kt" }) }), "\n", _jsx(_components.pre, { children: _jsx(_components.code, { className: "language-kotlin", children: "//...\nif (areAgeBrieOrPasses) {\n if (isNotTheMaximumQuality) {\n item.quality += qualityIncrease\n if (isABackStagePass) {\n val backstagePassesBigThreshold = 11\n val areInDateToBeSold = item.sellIn < backstagePassesBigThreshold\n if (areInDateToBeSold && isNotTheMaximumQuality) {\n item.quality = item.quality + qualityIncrease\n }\n\n val backstagePassesSmallThreshold = 6\n val areOnTheLastDays = item.sellIn < backstagePassesSmallThreshold\n if (areOnTheLastDays && isNotTheMaximumQuality) {\n item.quality = item.quality + qualityIncrease\n }\n }\n }\n} else {\n if (hasQuality && isNotSulfuras(item)) {\n item.quality = item.quality - qualityDecrease\n }\n}\ndecreaseSellIn(item)\n\nval notExpiredNumber = 0\n val isExpired = item.sellIn < notExpiredNumber\n if (isExpired) {\n if (isAgedBrie) {\n if (isNotTheMaximumQuality) {\n item.quality = item.quality + qualityIncrease\n }\n } else {\n if (isABackStagePass) {\n item.quality = item.quality - item.quality\n } else {\n if (hasQuality && isNotSulfuras(item)) {\n item.quality = item.quality - qualityDecrease\n }\n }\n }\n }\n}\n//...\n" }) }), "\n", _jsxs(_components.p, { children: ["Vemos que se ha mezclado totalmente la lógica de la calidad y el empeoramiento.\nVamos a separarlos agregando las condiciones y moveremos el código a la\nizquierda para que sea mas fácil luego extraer un método. En el bloque de arriba\ntodo aumenta menos el primer ", _jsx(_components.code, { children: "else" }), "."] }), "\n", _jsx(_components.p, { children: _jsx(_components.code, { children: "GildedRose.kt" }) }), "\n", _jsx(_components.pre, { children: _jsx(_components.code, { className: "language-kotlin", children: "else {\n if (hasQuality && isNotSulfuras(item)) {\n item.quality = item.quality - qualityDecrease\n }\n}\n" }) }), "\n", _jsx(_components.p, { children: "Para convertirlo en un if simple vamos a añadir la negación del if mas sus\ncondiciones:" }), "\n", _jsx(_components.p, { children: _jsx(_components.code, { children: "GildedRose.kt" }) }), "\n", _jsx(_components.pre, { children: _jsx(_components.code, { className: "language-kotlin", children: "if (!areAgeBrieOrPasses && hasQuality && isNotSulfuras(item)) {\n item.quality -= qualityDecrease\n}\n" }) }), "\n", _jsx(_components.p, { children: "Bien ahora vamos a hacer lo mismo con el bloque de abajo:" }), "\n", _jsx(_components.p, { children: _jsx(_components.code, { children: "GildedRose.kt" }) }), "\n", _jsx(_components.pre, { children: _jsx(_components.code, { className: "language-kotlin", children: "//...\nval notExpiredNumber = 0\nval isExpired = item.sellIn < notExpiredNumber\nif (isExpired) {\n if (isAgedBrie) {\n if (isNotTheMaximumQuality) {\n item.quality = item.quality + qualityIncrease\n }\n } else {\n if (isABackStagePass) {\n item.quality = item.quality - item.quality\n } else {\n if (hasQuality && isNotSulfuras(item)) {\n item.quality = item.quality - qualityDecrease\n }\n }\n }\n}\n//...\n" }) }), "\n", _jsxs(_components.p, { children: ["Aquí, la linea que nos molesta es\n", _jsx(_components.code, { children: "item.quality = item.quality + qualityIncrease" }), " porque todo lo demás resta\ncalidad al producto. Hacemos lo mismo que antes, añadimos toda la ruta de\ncondiciones."] }), "\n", _jsx(_components.p, { children: _jsx(_components.code, { children: "GildedRose.kt" }) }), "\n", _jsx(_components.pre, { children: _jsx(_components.code, { className: "language-kotlin", children: "if (isExpired && isAgedBrie && isNotTheMaximumQuality) {\n item.quality = item.quality + qualityIncrease\n}\n" }) }), "\n", _jsx(_components.p, { children: "Ahora tenemos la libertad de mover todo aquello que incremente o decremente\njunto para poder extraerlo a un método. He reordenado el código para que sea mas\nfácil de leer y extraer el método." }), "\n", _jsx(_components.p, { children: _jsx(_components.code, { children: "GildedRose.kt" }) }), "\n", _jsx(_components.pre, { children: _jsx(_components.code, { className: "language-kotlin", children: "fun updateQuality() {\n items.forEach { item ->\n // Increase quality part\n val maxQuality = 50\n val agedBrie = \"Aged Brie\"\n val backstagePasses = \"Backstage passes to a TAFKAL80ETC concert\"\n val qualityIncrease = 1\n val isNotTheMaximumQuality = item.quality < maxQuality\n val isAgedBrie = item.name == agedBrie\n val isABackStagePass = item.name == backstagePasses\n val areAgeBrieOrPasses = isAgedBrie || isABackStagePass\n if (areAgeBrieOrPasses) {\n if (isNotTheMaximumQuality) {\n item.quality += qualityIncrease\n if (isABackStagePass) {\n val backstagePassesBigThreshold = 11\n val areInDateToBeSold = item.sellIn < backstagePassesBigThreshold\n if (areInDateToBeSold && isNotTheMaximumQuality) {\n item.quality += qualityIncrease\n }\n\n val backstagePassesSmallThreshold = 6\n val areOnTheLastDays = item.sellIn < backstagePassesSmallThreshold\n if (areOnTheLastDays && isNotTheMaximumQuality) {\n item.quality += qualityIncrease\n }\n }\n }\n }\n decreaseSellIn(item)\n val notExpiredNumber = 0\n val isExpired = item.sellIn < notExpiredNumber\n if(isExpired && isAgedBrie && isNotTheMaximumQuality){\n item.quality += qualityIncrease\n }\n\n // Decrease quality part\n val qualityDecrease = 1\n val minQuality = 0\n val hasQuality = item.quality > minQuality\n if (!areAgeBrieOrPasses && hasQuality && isNotSulfuras(item)) {\n item.quality -= qualityDecrease\n }\n if (isExpired) {\n if (!isAgedBrie) {\n if (isABackStagePass) {\n item.quality -= item.quality\n } else {\n if (hasQuality && isNotSulfuras(item)) {\n item.quality -= qualityDecrease\n }\n }\n }\n }\n }\n}\n" }) }), "\n", _jsxs(_components.p, { children: ["Vamos a extraer el código a un método llamado ", _jsx(_components.code, { children: "increaseQuality" }), " y\n", _jsx(_components.code, { children: "decreaseQuality" }), " utilizarlos en su lugar. Para lograr esto, utilizaremos el\natajo ", _jsx(_components.code, { children: "Ctrl + Alt + M" }), " en IntelliJ sobre para extraer el método."] }), "\n", _jsx(_components.p, { children: _jsx(_components.code, { children: "GildedRose.kt" }) }), "\n", _jsx(_components.pre, { children: _jsx(_components.code, { className: "language-kotlin", children: "fun updateQuality() {\n items.forEach { item ->\n increaseQuality(item)\n decreaseQuality(item)\n }\n}\n\nprivate fun decreaseQuality(item: Item) {\n val agedBrie = \"Aged Brie\"\n val backstagePasses = \"Backstage passes to a TAFKAL80ETC concert\"\n val isAgedBrie = item.name == agedBrie\n val isABackStagePass = item.name == backstagePasses\n val areAgeBrieOrPasses = isAgedBrie || isABackStagePass\n val notExpiredNumber = 0\n val isExpired = item.sellIn < notExpiredNumber\n val qualityDecrease = 1\n val minQuality = 0\n val hasQuality = item.quality > minQuality\n if (!areAgeBrieOrPasses && hasQuality && isNotSulfuras(item)) {\n item.quality -= qualityDecrease\n }\n if (isExpired) {\n if (!isAgedBrie) {\n if (isABackStagePass) {\n item.quality -= item.quality\n } else {\n if (hasQuality && isNotSulfuras(item)) {\n item.quality -= qualityDecrease\n }\n }\n }\n }\n}\n\nprivate fun increaseQuality(item: Item) {\n val maxQuality = 50\n val agedBrie = \"Aged Brie\"\n val backstagePasses = \"Backstage passes to a TAFKAL80ETC concert\"\n val qualityIncrease = 1\n val isNotTheMaximumQuality = item.quality < maxQuality\n val isAgedBrie = item.name == agedBrie\n val isABackStagePass = item.name == backstagePasses\n val areAgeBrieOrPasses = isAgedBrie || isABackStagePass\n if (areAgeBrieOrPasses) {\n if (isNotTheMaximumQuality) {\n item.quality += qualityIncrease\n if (isABackStagePass) {\n val backstagePassesBigThreshold = 11\n val areInDateToBeSold = item.sellIn < backstagePassesBigThreshold\n if (areInDateToBeSold && isNotTheMaximumQuality) {\n item.quality += qualityIncrease\n }\n\n val backstagePassesSmallThreshold = 6\n val areOnTheLastDays = item.sellIn < backstagePassesSmallThreshold\n if (areOnTheLastDays && isNotTheMaximumQuality) {\n item.quality += qualityIncrease\n }\n }\n }\n }\n decreaseSellIn(item)\n val notExpiredNumber = 0\n val isExpired = item.sellIn < notExpiredNumber\n if (isExpired && isAgedBrie && isNotTheMaximumQuality) {\n item.quality += qualityIncrease\n }\n}\n" }) }), "\n", _jsxs(_components.p, { children: ["Si vemos detenidamente el método ", _jsx(_components.code, { children: "updateQuality" }), " parece que se ha simplificado\nbastante el nivel de abstracción del método. Per los métodos aun tienen bastante\nlógica de condiciones compartida, recordemos que es mejor tener código duplicado\nque tener una mala abstracción. Como ya hemos visto antes con ", _jsx(_components.code, { children: "isNotSulfuras" }), "\nvamos a extraer aquellas condiciones que se repiten en los métodos a un método."] }), "\n", _jsx(_components.p, { children: _jsx(_components.code, { children: "GildedRose.kt" }) }), "\n", _jsx(_components.pre, { children: _jsx(_components.code, { className: "language-kotlin", children: "private fun decreaseQuality(item: Item) {\n val isAgedBrie = isAgedBrie(item)\n val isABackStagePass = isABackstagePass(item)\n val isExpired = isExpired(item)\n val qualityDecrease = 1\n val minQuality = 0\n val hasQuality = item.quality > minQuality\n val areAgeBrieOrPasses = isAgedBrie || isABackStagePass\n if (!areAgeBrieOrPasses && hasQuality && isNotSulfuras(item)) {\n item.quality -= qualityDecrease\n }\n if (isExpired) {\n if (!isAgedBrie) {\n if (isABackStagePass) {\n item.quality -= item.quality\n } else {\n if (hasQuality && isNotSulfuras(item)) {\n item.quality -= qualityDecrease\n }\n }\n }\n }\n}\n\nprivate fun increaseQuality(item: Item) {\n val maxQuality = 50\n val isNotTheMaximumQuality = item.quality < maxQuality\n val isAgedBrie = isAgedBrie(item)\n val isABackStagePass = isABackstagePass(item)\n val qualityIncrease = 1\n val areAgeBrieOrPasses = isAgedBrie || isABackStagePass\n if (areAgeBrieOrPasses) {\n if (isNotTheMaximumQuality) {\n item.quality += qualityIncrease\n if (isABackStagePass) {\n val backstagePassesBigThreshold = 11\n val areInDateToBeSold = item.sellIn < backstagePassesBigThreshold\n if (areInDateToBeSold && isNotTheMaximumQuality) {\n item.quality += qualityIncrease\n }\n\n val backstagePassesSmallThreshold = 6\n val areOnTheLastDays = item.sellIn < backstagePassesSmallThreshold\n if (areOnTheLastDays && isNotTheMaximumQuality) {\n item.quality += qualityIncrease\n }\n }\n }\n }\n decreaseSellIn(item)\n val isExpired = isExpired(item)\n if (isExpired && isAgedBrie && isNotTheMaximumQuality) {\n item.quality += qualityIncrease\n }\n}\n" }) }), "\n", _jsxs(_components.p, { children: ["Parece que va mejorando pero la realidad es que aun queda por delante bastante.\nAhora vamos a seguir atacando al método ", _jsx(_components.code, { children: "decreaseQuality" }), ", vamos a reducir la\ncomplejidad anidada de este método. Hay una constante que se repite en el método\n", _jsx(_components.code, { children: "hasQuality" }), " por lo que vamos a crear una clausula guarda para que sea mas fácil\nde leer."] }), "\n", _jsx(_components.p, { children: _jsx(_components.code, { children: "GildedRose.kt" }) }), "\n", _jsx(_components.pre, { children: _jsx(_components.code, { className: "language-kotlin", children: "private fun decreaseQuality(item: Item) {\n val minQuality = 0\n val hasQuality = item.quality > minQuality\n if(!hasQuality) return\n val isAgedBrie = isAgedBrie(item)\n val isABackStagePass = isABackstagePass(item)\n val isExpired = isExpired(item)\n val qualityDecrease = 1\n val areAgeBrieOrPasses = isAgedBrie || isABackStagePass\n if (!areAgeBrieOrPasses && isNotSulfuras(item)) {\n item.quality -= qualityDecrease\n }\n if (isExpired) {\n if (!isAgedBrie) {\n if (isABackStagePass) {\n item.quality -= item.quality\n } else {\n if (isNotSulfuras(item)) {\n item.quality -= qualityDecrease\n }\n }\n }\n }\n}\n" }) }), "\n", _jsxs(_components.p, { children: ["Con este cambio podemos eliminar la variable ", _jsx(_components.code, { children: "hasQuality" }), " ya que no es necesario\nvolver a comprobarlo. vamos a hacer la misma técnica con el resto de\ncondiciones."] }), "\n", _jsx(_components.p, { children: _jsx(_components.code, { children: "GildedRose.kt" }) }), "\n", _jsx(_components.pre, { children: _jsx(_components.code, { className: "language-kotlin", children: "private fun decreaseQuality(item: Item) {\n val minQuality = 0\n val hasQuality = item.quality > minQuality\n if (!hasQuality) return\n val isExpired = isExpired(item)\n val qualityDecrease = 1\n if (isExpired && isABackstagePass(item)) {\n item.quality = minQuality\n return\n }\n val hasAnExpirationDate = !(isAgedBrie(item) || isABackstagePass(item)) && isNotSulfuras(item)\n if (isExpired && hasAnExpirationDate) {\n val expirationMultiplier = 2\n item.quality -= qualityDecrease * expirationMultiplier\n return\n }\n if (hasAnExpirationDate) {\n item.quality -= qualityDecrease\n }\n}\n" }) }), "\n", _jsxs(_components.p, { children: ["Ahora podemos ver que el método ", _jsx(_components.code, { children: "decreaseQuality" }), " ha quedado bastante mas\nlimpio. He ordenado los métodos de mas concreto a mas abstracto para salir del\nmétodo lo antes posible, evitando errores. Vamos a hacer lo mismo con el método\n", _jsx(_components.code, { children: "increaseQuality" }), "."] }), "\n", _jsx(_components.p, { children: _jsx(_components.code, { children: "GildedRose.kt" }) }), "\n", _jsx(_components.pre, { children: _jsx(_components.code, { className: "language-kotlin", children: "private fun increaseQuality(item: Item) {\n val maxQuality = 50\n val isNotTheMaximumQuality = item.quality < maxQuality\n val itHasTheMaximumQuality = item.quality == maxQuality\n if (itHasTheMaximumQuality) return\n val qualityIncrease = 1\n\n if (isABackstagePass(item)) {\n item.quality += qualityIncrease\n val backstagePassesBigThreshold = 11\n val areInDateToBeSold = item.sellIn < backstagePassesBigThreshold\n if (areInDateToBeSold && isNotTheMaximumQuality) {\n item.quality += qualityIncrease\n }\n val backstagePassesSmallThreshold = 6\n val areOnTheLastDays = item.sellIn < backstagePassesSmallThreshold\n if (areOnTheLastDays && isNotTheMaximumQuality) {\n item.quality += qualityIncrease\n }\n decreaseSellIn(item)\n return\n }\n\n if (isAgedBrie(item)) {\n item.quality += qualityIncrease\n decreaseSellIn(item)\n val isExpired = isExpired(item)\n if (isExpired && isNotTheMaximumQuality) {\n item.quality += qualityIncrease\n }\n return\n }\n decreaseSellIn(item)\n}\n" }) }), "\n", _jsxs(_components.p, { children: ["Esta mejor pero el código nos esta diciendo dos cosas, que arreglaremos mas\nadelante, la primera es que el método ", _jsx(_components.code, { children: "increaseQuality" }), " tiene demasiadas\nresponsabilidades y la segunda es que el método ", _jsx(_components.code, { children: "increaseQuality" }), " tiene\ndemasiadas condiciones hacia items específicos."] }), "\n", _jsx(_components.h2, { children: "🐍 Violación del Principio de Responsabilidad Única (SRP)" }), "\n", _jsxs(_components.p, { children: ["El principio de responsabilidad única nos dice que cada clase debe tener una\núnica responsabilidad y que esta debe estar encapsulada en la clase. En este\ncaso, podemos observar que la clase ", _jsx(_components.code, { children: "GildedRose" }), " tiene varias responsabilidades\ncomo son la actualización de la calidad y la actualización de la fecha de\ncaducidad. Resulta que antes de extraer nada tenemos que separar las reglas de\nnegocio:"] }), "\n", _jsxs(_components.ul, { children: ["\n", _jsx(_components.li, { children: "Para \"Aged Brie\", la calidad aumenta en 1 cada día, y si ha expirado, la\ncalidad aumenta en 2." }), "\n", _jsx(_components.li, { children: "Para \"Backstage passes\", la calidad aumenta en 1 si faltan más de 10 días para\nel concierto, en 2 si faltan 10 días o menos, y en 3 si faltan 5 días o menos.\nSin embargo, la calidad cae a 0 después del concierto." }), "\n", _jsx(_components.li, { children: "Para los items regulares, la calidad disminuye en 1 cada día, y si ha\nexpirado, la calidad disminuye en 2." }), "\n", _jsx(_components.li, { children: "Para \"Sulfuras, Hand of Ragnaros\", la calidad y sellIn permanecen sin cambios." }), "\n", _jsx(_components.li, { children: "La calidad de un item nunca es negativa y nunca es mayor que 50." }), "\n"] }), "\n", _jsxs(_components.p, { children: ["Para remediar esto, vamos a crear una clase abstracta llamada ", _jsx(_components.code, { children: "ItemUpdater" }), " que\nse encargue de generar una interfaz para actualizar los items."] }), "\n", _jsx(_components.p, { children: _jsx(_components.code, { children: "ItemUpdater.kt" }) }), "\n", _jsx(_components.pre, { children: _jsx(_components.code, { className: "language-kotlin", children: "abstract class ItemUpdater {\n protected val minQuality = 0\n protected val maxQuality = 50\n protected val qualityIncrease = 1\n protected val qualityDecrease = 1\n protected val dayDecrease = 1\n\n abstract fun updateQuality(item: Item)\n abstract fun decreaseSellIn(item: Item)\n\n}\n" }) }), "\n", _jsxs(_components.p, { children: ["Ahora, vamos a crear una clase llamada ", _jsx(_components.code, { children: "DefaultItemUpdater" }), " que implemente la\ninterfaz ", _jsx(_components.code, { children: "ItemUpdater" }), " y que se encargue de actualizar los items."] }), "\n", _jsx(_components.p, { children: _jsx(_components.code, { children: "DefaultItemUpdater.kt" }) }), "\n", _jsx(_components.pre, { children: _jsx(_components.code, { className: "language-kotlin", children: "class DefaultItemUpdater : ItemUpdater() {\n override fun updateQuality(item: Item) {\n if (hadQuality(item)) {\n decreaseQuality(item)\n }\n if (isExpired(item) && hadQuality(item)) {\n decreaseQuality(item)\n }\n }\n\n override fun decreaseSellIn(item: Item) {\n item.sellIn -= dayDecrease\n }\n\n private fun isExpired(item: Item): Boolean {\n return item.sellIn <= 0\n }\n\n private fun hadQuality(item: Item): Boolean {\n return item.quality > minQuality\n }\n\n private fun decreaseQuality(item: Item) {\n item.quality -= this.qualityDecrease\n }\n\n}\n" }) }), "\n", _jsxs(_components.p, { children: ["Lo mismo para el resto de items. No importa mucho si de momento dejamos clases\nprivadas duplicadas. La clase ", _jsx(_components.code, { children: "GildedRose" }), " quedaría así:"] }), "\n", _jsx(_components.p, { children: _jsx(_components.code, { children: "GildedRose.kt" }) }), "\n", _jsx(_components.pre, { children: _jsx(_components.code, { className: "language-kotlin", children: "class GildedRose(var items: List) {\n fun updateQuality() {\n items.forEach { item ->\n val updater = when (item.name) {\n \"Aged Brie\" -> AgedBrieUpdater()\n \"Backstage passes to a TAFKAL80ETC concert\" -> BackstagePassUpdater()\n \"Sulfuras, Hand of Ragnaros\" -> SulfurasUpdater()\n else -> DefaultItemUpdater()\n }\n updater.updateQuality(item)\n updater.decreaseSellIn(item)\n }\n }\n}\n" }) }), "\n", _jsx(_components.h2, { children: "📥 Falta de Encapsulación" }), "\n", _jsxs(_components.p, { children: ["La solución propuesta puede enfrentar varios problemas de falta de\nencapsulación. La creación directa de instancias de ", _jsx(_components.code, { children: "ItemUpdater" }), " dentro del\nmétodo ", _jsx(_components.code, { children: "updateQuality" }), " de la clase ", _jsx(_components.code, { children: "GildedRose" }), " podría violar el principio de\nencapsulación, ya que idealmente, la creación y gestión de estos objetos debería\nestar encapsulada en una fábrica o en un proveedor de servicios. Además, las\nclases ", _jsx(_components.code, { children: "ItemUpdater" }), " específicas están expuestas y pueden ser accedidas desde\nfuera de la clase ", _jsx(_components.code, { children: "GildedRose" }), ", lo que puede dar lugar a un acoplamiento no\ndeseado."] }), "\n", _jsx(_components.p, { children: _jsx(_components.code, { children: "ItemUpdaterFactory.kt" }) }), "\n", _jsx(_components.pre, { children: _jsx(_components.code, { className: "language-kotlin", children: "class ItemUpdaterFactory {\n fun getUpdater(item: Item): ItemUpdater {\n return when(item.name) {\n \"Aged Brie\" -> AgedBrieUpdater()\n \"Backstage passes to a TAFKAL80ETC concert\" -> BackstagePassUpdater()\n \"Sulfuras, Hand of Ragnaros\" -> SulfurasUpdater()\n else -> DefaultItemUpdater()\n }\n }\n}\n" }) }), "\n", _jsx(_components.p, { children: _jsx(_components.code, { children: "GildedRose.kt" }) }), "\n", _jsx(_components.pre, { children: _jsx(_components.code, { className: "language-kotlin", children: "class GildedRose(var items: List) {\n private val itemUpdaterFactory = ItemUpdaterFactory()\n\n fun updateQuality() {\n items.forEach { item ->\n val updater = itemUpdaterFactory.getUpdater(item)\n updater.updateQuality(item)\n updater.decreaseSellIn(item)\n }\n }\n}\n" }) }), "\n", _jsxs(_components.p, { children: ["También, las clases ", _jsx(_components.code, { children: "ItemUpdater" }), " modifican directamente las propiedades de\n", _jsx(_components.code, { children: "Item" }), ", violando la encapsulación ya que cualquier modificación a los estados de\nun objeto debería ser manejada a través de métodos en la clase del objeto. La\nfalta de abstracción para diferentes tipos de items y la modificación directa de\nlas propiedades de ", _jsx(_components.code, { children: "Item" }), " sin métodos setter que permitan controlar o validar\nlas modificaciones, podrían llevar a estados inválidos o inconsistentes. Cada\ntipo específico de item tendría su propia subclase de Item, y cada subclase\nimplementaría los métodos de actualización de calidad y decrecimiento de sellIn\ndirectamente. La falta de extensibilidad y flexibilidad se evidencia si se\nnecesitan agregar más tipos de items en el futuro, ya que se tendría que\nmodificar la expresión ", _jsx(_components.code, { children: "when" }), " en ", _jsx(_components.code, { children: "GildedRose" }), ", violando el principio de\nabierto/cerrado. Hare una implementación basada en el código de los\nItemUpdaters:"] }), "\n", _jsx(_components.p, { children: _jsx(_components.code, { children: "Item.kt" }) }), "\n", _jsx(_components.pre, { children: _jsx(_components.code, { className: "language-kotlin", children: "abstract class Item(protected val name: String, protected var sellIn: Int, protected var quality: Int) {\n protected val minQuality = 0\n protected val maxQuality = 50\n protected val qualityIncrease = 1\n protected val qualityDecrease = 1\n protected val dayDecrease = 1\n abstract fun updateQuality()\n abstract fun decreaseSellIn()\n abstract fun currentQuality(): Int\n}\n" }) }), "\n", _jsx(_components.p, { children: _jsx(_components.code, { children: "DefaultItem.kt" }) }), "\n", _jsx(_components.pre, { children: _jsx(_components.code, { className: "language-kotlin", children: "class DefaultItem(name: String, sellIn: Int, quality: Int) : Item(name, sellIn, quality) {\n override fun updateQuality() {\n if (hadQuality()) {\n decreaseQuality()\n }\n if (isExpired() && hadQuality()) {\n decreaseQuality()\n }\n }\n override fun decreaseSellIn() {\n sellIn -= dayDecrease\n }\n\n override fun currentQuality(): Int {\n return quality\n }\n\n private fun isExpired(): Boolean {\n return sellIn <= 0\n }\n\n private fun hadQuality(): Boolean {\n return quality > minQuality\n }\n\n private fun decreaseQuality() {\n quality -= this.qualityDecrease\n }\n}\n" }) }), "\n", _jsxs(_components.p, { children: ["Ya no hace falta la dependencia de ", _jsx(_components.code, { children: "Item" }), " como en el caso de ", _jsx(_components.code, { children: "ItemUpdater" }), " ya\nque la clase ", _jsx(_components.code, { children: "GildedRose" }), " no necesita conocer los detalles de implementación de\n", _jsx(_components.code, { children: "Item" }), ". La clase ", _jsx(_components.code, { children: "GildedRose" }), " solo necesita conocer la interfaz de ", _jsx(_components.code, { children: "Item" }), " para\npoder actualizar los items. La clase ", _jsx(_components.code, { children: "GildedRose" }), " quedaría así:"] }), "\n", _jsx(_components.p, { children: _jsx(_components.code, { children: "GildedRose.kt" }) }), "\n", _jsx(_components.pre, { children: _jsx(_components.code, { className: "language-kotlin", children: "class GildedRose(var items: List) {\n fun updateQuality() {\n items.forEach { item ->\n item.updateQuality()\n item.decreaseSellIn()\n }\n }\n}\n" }) }), "\n", _jsx(_components.p, { children: "Una solución más encapsulada y extensible podría ser implementar una fábrica o\nun registro de actualizadores de items que pueda ser extendido sin modificar el\ncódigo existente. El problema es que implementar esta solución ha costado mucho\nmas que los otros cambios. Por suerte ya teníamos la implementación en el\nupdater." }), "\n", _jsx(_components.h2, { children: "🍃 Problemas de Extensibilidad y Mantenibilidad" }), "\n", _jsxs(_components.p, { children: ["Para concluir este gran artículo, voy a comentar los problemas de\nextensibilidad. La solución actual presenta desafíos de extensibilidad y\nmantenibilidad, como la dificultad en la adición de nuevos tipos de items y la\nmodificación de la lógica de actualización, lo que podría resultar en código\nduplicado. También hay una dispersión en la validación de la calidad del item,\nlo que puede conducir a cambios en múltiples lugares si las reglas de validación\nevolucionan. Aunque se han creado subclases para diferentes tipos de items, la\nfalta de polimorfismo y la violación de la separación de responsabilidades,\ndonde las subclases de ", _jsx(_components.code, { children: "Item" }), " manejan tanto la representación de datos como la\nlógica de actualización, pueden afectar negativamente la mantenibilidad. Además,\nla solución muestra una \"obsesión primitiva\", al depender en gran medida de\ntipos de datos primitivos y estructuras de control en lugar de encapsular la\nlógica en clases y métodos bien definidos, lo que podría complicar futuras\nextensiones y refactorizaciones del sistema."] }), "\n", _jsxs(_components.p, { children: ["Os imagináis si lo primero que hago es crear una clase ", _jsx(_components.code, { children: "Item" }), " con su lógica? Es\nmuy probable que sin un poco de limpieza y refactorización no hubiera sido\nposible."] }), "\n", _jsx(_components.p, { children: "Siempre se puede ir mejorando pero hay que saber cuando parar, es probable que\nla solución cuando la visiten este cambiada o ni siquiera se haya acabado la\nKata 😹. Espero que os haya gustado este artículo y que os haya servido para\naprender algo nuevo." }), "\n", _jsx(_components.h2, { children: "📚 Referencias" }), "\n", _jsxs(_components.ul, { children: ["\n", _jsx(_components.li, { children: _jsx(_components.a, { href: "https://github.com/Wolfremium13/kata-guilded-rose", children: "Repositorio" }) }), "\n", _jsx(_components.li, { children: _jsx(_components.a, { href: "https://www.youtube.com/watch?v=-M0KZHNBgG8", children: "Pedro Santos" }) }), "\n", _jsx(_components.li, { children: _jsx(_components.a, { href: "https://youtu.be/K7xSsNpeM8I?si=_mvk53iliLsut_ut", children: "Emily Bache" }) }), "\n", _jsx(_components.li, { children: _jsx(_components.a, { href: "https://www.youtube.com/@carlosble/videos", children: "Carlos Ble" }) }), "\n", _jsx(_components.li, { children: _jsx(_components.a, { href: "https://martinfowler.com/articles/refactoring-2nd-ed.html", children: "Martin Fowler" }) }), "\n"] })] }); } function MDXContent(props = {}) { const {wrapper: MDXLayout} = Object.assign({}, _provideComponents(), props.components); return MDXLayout ? _jsx(MDXLayout, Object.assign({}, props, { children: _jsx(_createMdxContent, props) })) : _createMdxContent(props); } return { default: MDXContent };

🌹 Premisa de prioridad de refactorización