Done

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

@@ -9,58 +9,150 @@ public class Inventory {
         this.items = items;
     }
 
-
-    public void updateQuality() {
+    public void qualityUpdate() {
         for (int i = 0; i < items.length; i++) {
-            if (!items[i].getName().equals("Aged Brie")
-                    && !items[i].getName().equals("Backstage passes to a TAFKAL80ETC concert")) {
-                if (items[i].getQuality() > 0) {
-                    if (!items[i].getName().equals("Sulfuras, Hand of Ragnaros")) {
-                        items[i].setQuality(items[i].getQuality() - 1);
-                    }
-                }
-            } else {
-                if (items[i].getQuality() < 50) {
-                    items[i].setQuality(items[i].getQuality() + 1);
-
-                    if (items[i].getName() == "Backstage passes to a TAFKAL80ETC concert") {
-                        if (items[i].getSellIn() < 11) {
-                            if (items[i].getQuality() < 50) {
-                                items[i].setQuality(items[i].getQuality() + 1);
-                            }
-                        }
-
-                        if (items[i].getSellIn() < 6) {
-                            if (items[i].getQuality() < 50) {
-                                items[i].setQuality(items[i].getQuality() + 1);
-                            }
-                        }
+
+            switch (items[i].getName()) {
+
+                case "A book":
+                    if (items[i].getQuality() > 0) {
+                        items[i].setQuality(items[i].getQuality() - 2);
+                        break;
                     }
-                }
+
+                case "Aged Brie":
+                    increaseQuality(items[i]);
+                    increaseQuality(items[i]);
+                    break;
+
+                case "Backstage passes to a TAFKAL80ETC concert":
+                    increaseQuality(items[i]);
+                    checkSellIn(items[i]);
+                    overSellIn(items[i]);
+                    break;
+
+                case "Sulfuras, Hand of Ragnaros":
+                    break;
+
+                case "Conjuring":
+                    decreaseQuality(items[i]);
+                    items[i].setQuality(items[i].getQuality() - 3);
+                    overSellIn(items[i]);
+                    break;
+
+                default:
+                    decreaseQuality(items[i]);
+                    checkOverSellIn(items[i]);
+
             }
 
-            if (!items[i].getName().equals("Sulfuras, Hand of Ragnaros")) {
-                items[i].setSellIn(items[i].getSellIn() - 1);
+        }
+    }
+
+
+        public static void increaseQuality(Item item){
+            if (item.getQuality() < 50) {
+                item.setQuality(item.getQuality() + 1);
+
             }
+        }
+            public static void decreaseQuality(Item item){
+            if (item.getQuality() > 0) {
+                item.setQuality(item.getQuality() - 1);
 
-            if (items[i].getSellIn() < 0) {
-                if (!items[i].getName().equals("Aged Brie")) {
-                    if (!items[i].getName().equals("Backstage passes to a TAFKAL80ETC concert")) {
-                        if (items[i].getQuality() > 0) {
-                            if (!items[i].getName().equals("Sulfuras, Hand of Ragnaros")) {
-                                items[i].setQuality(items[i].getQuality() - 1);
-                            }
-                        }
-                    } else {
-                        items[i].setQuality(items[i].getQuality()
-                                - items[i].getQuality());
-                    }
-                } else {
-                    if (items[i].getQuality() < 50) {
-                        items[i].setQuality(items[i].getQuality() + 1);
-                    }
-                }
             }
         }
+
+    public static void checkSellIn(Item item) {
+        if (item.getSellIn() < 11) {
+            increaseQuality(item);
+        }
+        if (item.getSellIn() < 6) {
+            increaseQuality(item);
+        }
+    }
+
+    public static void checkOverSellIn(Item item){
+        if (item.getQuality() < 0) {
+            decreaseQuality(item);
+        }
+    }
+
+    public static void overSellIn(Item item){
+        switch (item.getName()) {
+            case "Aged Brie":
+                increaseQuality(item);
+                break;
+
+            case "Backstage passes to a TAFKAL80ETC concert":
+                if(item.getQuality() < 0) {
+                    item.setQuality(0);
+                }
+                break;
+
+            case "Conjuring":
+                checkOverSellIn(item);
+                break;
+
+            default:
+                checkOverSellIn(item);
+                break;
+        }
+
+
     }
+
+//    public void updateQuality() {
+//        for (int i = 0; i < items.length; i++) {
+//            if (!items[i].getName().equals("Aged Brie")
+//                    && !items[i].getName().equals("Backstage passes to a TAFKAL80ETC concert")) {
+//                if (items[i].getQuality() > 0) {
+//                    if (!items[i].getName().equals("Sulfuras, Hand of Ragnaros")) {
+//                        items[i].setQuality(items[i].getQuality() - 1);
+//                    }
+//                }
+//            } else {
+//                if (items[i].getQuality() < 50) {
+//                    items[i].setQuality(items[i].getQuality() + 1);
+//
+//                    if (items[i].getName() == "Backstage passes to a TAFKAL80ETC concert") {
+//                        if (items[i].getSellIn() < 11) {
+//                            if (items[i].getQuality() < 50) {
+//                                items[i].setQuality(items[i].getQuality() + 1);
+//                            }
+//                        }
+//
+//                        if (items[i].getSellIn() < 6) {
+//                            if (items[i].getQuality() < 50) {
+//                                items[i].setQuality(items[i].getQuality() + 1);
+//                            }
+//                        }
+//                    }
+//                }
+//            }
+//
+//            if (!items[i].getName().equals("Sulfuras, Hand of Ragnaros")) {
+//                items[i].setSellIn(items[i].getSellIn() - 1);
+//            }
+//
+//            if (items[i].getSellIn() < 0) {
+//                if (!items[i].getName().equals("Aged Brie")) {
+//                    if (!items[i].getName().equals("Backstage passes to a TAFKAL80ETC concert")) {
+//                        if (items[i].getQuality() > 0) {
+//                            if (!items[i].getName().equals("Sulfuras, Hand of Ragnaros")) {
+//                                items[i].setQuality(items[i].getQuality() - 1);
+//                            }
+//                        }
+//                    } else {
+//                        items[i].setQuality(items[i].getQuality()
+//                                - items[i].getQuality());
+//                    }
+//                } else {
+//                    if (items[i].getQuality() < 50) {
+//                        items[i].setQuality(items[i].getQuality() + 1);
+//                    }
+//                }
+//            }
+//        }
+//    }
 }

src/test/java/com/zipcodewilmington/gildedrose/InventoryTest.java

@@ -5,8 +5,73 @@ import org.junit.Test;
 
 public class InventoryTest {
     @Test
-    public void updateQuantityTest(){
+    public void updateQuantityTest() {
+
+
         Assert.assertEquals(1, 1);
+
+        Item item = new Item("A book", 12, 20);
+        Item[] items = {item};
+
+        Inventory inv = new Inventory(items);
+        inv.qualityUpdate();
+        System.out.println(item.getQuality());
+        Assert.assertEquals(18, item.getQuality());
+
+    }
+
+    @Test
+    public void updateQuantityTest2() {
+
+        Item item = new Item("Aged Brie", -1, 15);
+        Item[] items = {item};
+
+        Inventory inv = new Inventory(items);
+        inv.qualityUpdate();
+        System.out.println(item.getQuality());
+        Assert.assertEquals(17, item.getQuality());
+
     }
+
+    @Test
+    public void updateQuantityTest3() {
+
+        Item item = new Item("Backstage passes to a TAFKAL80ETC concert", 5, 15);
+        Item[] items = {item};
+
+        Inventory inv = new Inventory(items);
+        inv.qualityUpdate();
+        System.out.println(item.getQuality());
+        Assert.assertEquals(18, item.getQuality());
+
+    }
+
+    @Test
+    public void updateQuantityTest4() {
+
+        Item item = new Item("Sulfuras, Hand of Ragnaros", 0, 15);
+        Item[] items = {item};
+
+        Inventory inv = new Inventory(items);
+        inv.qualityUpdate();
+
+        System.out.println(item.getQuality());
+        Assert.assertEquals(15, item.getQuality());
+
+    }
+
+    @Test
+    public void updateQuantityTest5() {
+
+        Item item = new Item("Conjuring", -1, 15);
+        Item[] items = {item};
+
+        Inventory inv = new Inventory(items);
+        inv.qualityUpdate();
+        System.out.println(item.getQuality());
+        Assert.assertEquals(11, item.getQuality());
+
+    }
+
 }