finished

Inventory.java

@@ -7,33 +7,74 @@ public class Inventory {
         this.items = items;
     }
 
-    public void updateQuality() {
+    public void updateQuality(){
         for (int i = 0; i < items.length; i++) {
-            if (!items[i].getName().equals("Aged Brie")) {
-                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().equals("Aged Brie")){
+                updateBrieWhole(items[i]);
+            } else if (items[i].getName().equals("Sulfuras, Hand of Ragnaros")){
+                updateSulfWhole(items[i]);
+            } else {                
+                updateNormWhole(items[i]);
             }
+        }
+    }
 
-            if (!items[i].getName().equals("Sulfuras, Hand of Ragnaros")) {
-                items[i].setSellIn(items[i].getSellIn() - 1);
-            }
+    public void updateSell(Item item){
+        int updateSellValue = item.getSellIn()-1;
+        item.setSellIn(updateSellValue);
+    }
+    
+    public void updateSulfWhole(Item item){
+        //updateSell(item);
+        updateSulfQual(item);
+    }
+    
+    public void updateSulfQual(Item item){
+      //For future changes to Sulfurus Item         
+    }
+    
+    public void updateBrieWhole(Item item){
+        updateSell(item);
+        updateBrieQual(item);
+    }
 
-            if (items[i].getSellIn() < 0) {
-                if (!items[i].getName().equals("Aged Brie")) {
-                  items[i].setQuality(items[i].getQuality() - 1);
-                } else {
-                    if (items[i].getQuality() < 50) {
-                        items[i].setQuality(items[i].getQuality() + 1);
-                    }
-                }
-            }
+    public void updateBrieQual(Item item){
+        int updateQualValue = item.getQuality()+1;
+        if (item.getQuality() > 50){
+            item.setQuality(50);
         }
-}
+        if (item.getQuality() > 0 && item.getQuality() < 50){
+            item.setQuality(updateQualValue);
+        } else if (item.getQuality() < 0){
+            item.setQuality(0);
+            //Quality cannot be negative
+        } else {
+            item.setQuality(item.getQuality());
+            //keeps it at 50
+        }
+        if (item.getSellIn() < 0){  
+            item.setQuality(updateQualValue + 1);
+        }
+    }
+
+    public void updateNormWhole(Item item){
+        updateSell(item);
+        updateNormQual(item);
+    }
+
+    public void updateNormQual(Item item){
+        int updateQualValue = item.getQuality()-1;
+        if (item.getQuality() > 50){
+            item.setQuality(50);
+        }
+        if (item.getQuality() > 0){
+            item.setQuality(updateQualValue);
+        } else {
+            item.setQuality(0);
+            //Quality cannot be negative
+        }
+        if (item.getSellIn() < 0){  
+            item.setQuality(updateQualValue - 1);
+        }
+    }
 }