refatoraçao completa, mantendo o comportamento externo

Java/src/main/java/com/gildedrose/GildedRose.java

@@ -1,5 +1,7 @@
 package com.gildedrose;
 
+import com.gildedrose.experts.ItemUpdaterFactory;
+
 class GildedRose {
     Item[] items;
 
@@ -8,55 +10,9 @@ class GildedRose {
     }
 
     public void updateQuality() {
-        for (int i = 0; i < items.length; i++) {
+        for (Item item : items) {
-            if (!items[i].name.equals("Aged Brie")
+            ItemUpdater expert = new ItemUpdaterFactory().createItemUpdater(item);
-                    && !items[i].name.equals("Backstage passes to a TAFKAL80ETC concert")) {
+            expert.update(item);
-                if (items[i].quality > 0) {
-                    if (!items[i].name.equals("Sulfuras, Hand of Ragnaros")) {
-                        items[i].quality = items[i].quality - 1;
-                    }
-                }
-            } else {
-                if (items[i].quality < 50) {
-                    items[i].quality = items[i].quality + 1;
-
-                    if (items[i].name.equals("Backstage passes to a TAFKAL80ETC concert")) {
-                        if (items[i].sellIn < 11) {
-                            if (items[i].quality < 50) {
-                                items[i].quality = items[i].quality + 1;
-                            }
-                        }
-
-                        if (items[i].sellIn < 6) {
-                            if (items[i].quality < 50) {
-                                items[i].quality = items[i].quality + 1;
-                            }
-                        }
-                    }
-                }
-            }
-
-            if (!items[i].name.equals("Sulfuras, Hand of Ragnaros")) {
-                items[i].sellIn = items[i].sellIn - 1;
-            }
-
-            if (items[i].sellIn < 0) {
-                if (!items[i].name.equals("Aged Brie")) {
-                    if (!items[i].name.equals("Backstage passes to a TAFKAL80ETC concert")) {
-                        if (items[i].quality > 0) {
-                            if (!items[i].name.equals("Sulfuras, Hand of Ragnaros")) {
-                                items[i].quality = items[i].quality - 1;
-                            }
-                        }
-                    } else {
-                        items[i].quality = items[i].quality - items[i].quality;
-                    }
-                } else {
-                    if (items[i].quality < 50) {
-                        items[i].quality = items[i].quality + 1;
-                    }
-                }
-            }
         }
     }
 }

Java/src/main/java/com/gildedrose/ItemUpdater.java

@@ -0,0 +1,5 @@
+package com.gildedrose;
+
+public interface ItemUpdater {
+   void update(Item item);
+}

Java/src/main/java/com/gildedrose/experts/AgedBrieUpdater.java

@@ -0,0 +1,17 @@
+package com.gildedrose.experts;
+
+import com.gildedrose.Item;
+import com.gildedrose.ItemUpdater;
+
+public class AgedBrieUpdater implements ItemUpdater {
+    @Override
+    public void update(Item item) {
+        item.sellIn--;
+        if (item.quality < 50) {
+            item.quality++;
+        }
+        if (item.sellIn < 0 && item.quality < 50) {
+            item.quality++;
+        }
+    }
+}

Java/src/main/java/com/gildedrose/experts/BackstagePassUpdater.java

@@ -0,0 +1,25 @@
+package com.gildedrose.experts;
+
+import com.gildedrose.Item;
+import com.gildedrose.ItemUpdater;
+
+public class BackstagePassUpdater implements ItemUpdater {
+    @Override
+    public void update(Item item) {
+        if (item.quality < 50) {
+            item.quality++;
+            if (item.sellIn < 11 && item.quality < 50) {
+                item.quality++;
+
+            }
+            if (item.sellIn < 6 && item.quality < 50) {
+                item.quality++;
+            }
+        }
+        item.sellIn -= 1;
+
+        if (item.sellIn < 0) {
+            item.quality = 0;
+        }
+    }
+}

Java/src/main/java/com/gildedrose/experts/ItemUpdaterFactory.java

@@ -0,0 +1,19 @@
+package com.gildedrose.experts;
+
+import com.gildedrose.Item;
+import com.gildedrose.ItemUpdater;
+
+public class ItemUpdaterFactory {
+    public static ItemUpdater createItemUpdater(Item item) {
+       switch(item.name) {
+           case "Aged Brie":
+               return new AgedBrieUpdater();
+           case "Backstage passes to a TAFKAL80ETC concert":
+               return new BackstagePassUpdater();
+           case "Sulfuras, Hand of Ragnaros":
+               return new SulfurasUpdater();
+           default:
+               return new StandardUpdater();
+       }
+    };
+}

Java/src/main/java/com/gildedrose/experts/StandardUpdater.java

@@ -0,0 +1,18 @@
+package com.gildedrose.experts;
+
+import com.gildedrose.Item;
+import com.gildedrose.ItemUpdater;
+
+public class StandardUpdater implements ItemUpdater {
+    @Override
+    public void update(Item item) {
+        item.sellIn--;
+        if (item.quality > 0) {
+            item.quality--;
+        }
+        if (item.sellIn < 0 && item.quality > 0) {
+            item.quality--;
+        }
+
+    }
+}

Java/src/main/java/com/gildedrose/experts/SulfurasUpdater.java

@@ -0,0 +1,10 @@
+package com.gildedrose.experts;
+
+import com.gildedrose.Item;
+import com.gildedrose.ItemUpdater;
+
+public class SulfurasUpdater implements ItemUpdater {
+    @Override
+    public void update(Item item) {
+    }
+}

Java/src/test/java/com/gildedrose/GildedRoseTest.java

@@ -81,7 +81,7 @@ class GildedRoseTest {
     }
 
     @Test
-    void aumentarQualityEmDoisSellInEntre10A6TAFKAL80ETC() {
+    void aumentarQualityEm2SellInEntre6A10TAFKAL80ETC() {
         Item[] items = new Item[] { new Item("Backstage passes to a TAFKAL80ETC concert",6, 40),
                                     new Item("Backstage passes to a TAFKAL80ETC concert",10, 48)};
         GildedRose app = new GildedRose(items);