demetrm - complete

src/main/java/com/zipcodewilmington/gildedrose/Inventory.java

@@ -9,56 +9,41 @@ public class Inventory {
         this.items = items;
     }
 
-
     public void updateQuality() {
-        /*
-        loop through items.
-
-        1. if name DNE "Aged" && DNE "Backstage
-            a. if quality > 0
-                1. if name is NOT "Sulfuras",  subtract 1 quality | *
-        2. If name is either "Aged" or "Backstage"
-            a. if quality < 50, add 1 quality | **
-                1. if name is "Backstage"
-                    a. if getSellIn < 11
-                        1. if quality < 50, then add 1 quality | *
-                    b. if get sellIn < 6
-                        1. if quality < 50, then add 1 quality | *
-        1. if name DNE "Sulfuras"
-            a. subtract one sellIn | **
-
-        1. If getSellIn < 0
-            a. if name is NOT "Aged"
-                1. if name is NOT "Backstage"
-                    a. if quality > 0
-                        1. if name is NOT "Sulfuras, then subtract 1 quality (useless);
-                2. else set quality to 0;
-            b. else if quality < 50, add 1 quality
-         */
         for (int i = 0; i < items.length; i++) {
-            if (!items[i].getName().equals("Sulfuras, Hand of Ragnaros")) {
-                if (items[i].getQuality() < 50) {
-                    items[i].setQuality(items[i].getQuality() + 1);
-                    if (items[i].getName().equals("Backstage passes to a TAFKAL80ETC concert")) {
-                        if (items[i].getSellIn() < 11) {
+            String name = items[i].getName();
+            int quality = items[i].getQuality();
+            if(!name.equals("Sulfuras, Hand of Ragnaros")) {
+                switch (name) {
+                    case "Conjured":
+                        items[i].setQuality(quality - 2);
+                        break;
+                    case "Backstage passes to a TAFKAL80ETC concert":
+                        items[i].setQuality(items[i].getQuality() + 1);
+                        if (items[i].getSellIn() < 0) {
+                            items[i].setQuality(0);
+                        } else if (items[i].getSellIn() < 6 && quality < 50) {
+                            items[i].setQuality(items[i].getQuality() + 2);
+                        } else if (items[i].getSellIn() < 11 && quality < 50) {
                             items[i].setQuality(items[i].getQuality() + 1);
                         }
-                        if (items[i].getSellIn() < 6) {
+                        break;
+                    case "Aged Brie":
+                        if (quality < 50){
                             items[i].setQuality(items[i].getQuality() + 1);
+                            if (items[i].getSellIn() < 0) {
+                                items[i].setQuality(items[i].getQuality() + 1);
+                            }
                         }
-                    }
-                }
-
-                items[i].setSellIn(items[i].getSellIn() - 1);
-
-                if (items[i].getSellIn() < 0) {
-                    if (items[i].getName().equals("Backstage passes to a TAFKAL80ETC concert")) {
-                        items[i].setQuality(0);
-                    } else if (items[i].getName().equals("Aged Brie") && items[i].getQuality() < 50) {
-                        items[i].setQuality(items[i].getQuality() + 1);
-                    }
+                        break;
+                    default:
+                        if (quality < 50) {
+                            items[i].setQuality(items[i].getQuality() - 1);
+                        }
+                        break;
                 }
             }
+            items[i].setSellIn(items[i].getSellIn() - 1);
         }
     }
 }