finished

Inventory.java

@@ -9,31 +9,50 @@ public class Inventory {
 
     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("Sulfuras, Hand of Ragnaros")) {
+                continue;
             }
 
-            if (!items[i].getName().equals("Sulfuras, Hand of Ragnaros")) {
-                items[i].setSellIn(items[i].getSellIn() - 1);
+
+            if (items[i].getName().equals("Aged Brie")) {
+                updateAgedBrie(i);
+            }else if(items[i].getName().equals("Conjured Item")){
+                updateConjuredItem(i);
+            } else{
+                updateRegularItem(i);
             }
 
-            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);
-                    }
-                }
+           
+            if(items[i].getQuality() < 80){
+                items[i].setSellIn(items[i].getSellIn() -1);
             }
         }
+    }
+
+
+
+        public void updateAgedBrie(int i){
+        if(items[i].getSellIn() > 0 && items[i].getQuality() <= 49){
+            items[i].setQuality(items[i].getQuality() + 1);
+        } else if(items[i].getSellIn() < 0 && items[i].getQuality() <= 48){
+            items[i].setQuality(items[i].getQuality() + 2);
+        }
+    }
+
+    public void updateRegularItem(int i){
+        if(items[i].getSellIn() > 0 && items[i].getQuality() > 0){
+            items[i].setQuality(items[i].getQuality() - 1);
+        } else if(items[i].getSellIn() < 0 && items[i].getQuality() > 1){
+            items[i].setQuality(items[i].getQuality() - 2);
+        }
+    }
+
+    public void updateConjuredItem(int i){
+        if(items[i].getSellIn() > 0 && items[i].getQuality() > 0){
+            items[i].setQuality(items[i].getQuality() - 2);
+        } else if(items[i].getSellIn() < 0 && items[i].getQuality() > 1){
+            items[i].setQuality(items[i].getQuality() - 4);
+        }
+    }
 }
-}
+

package.bluej

@@ -5,18 +5,18 @@ dependency1.type=UsesDependency
 dependency2.from=InventoryTest
 dependency2.to=Inventory
 dependency2.type=UsesDependency
-editor.fx.0.height=722
-editor.fx.0.width=800
-editor.fx.0.x=320
-editor.fx.0.y=50
+editor.fx.0.height=0
+editor.fx.0.width=0
+editor.fx.0.x=0
+editor.fx.0.y=0
 objectbench.height=164
 objectbench.width=776
 package.divider.horizontal=0.6
 package.divider.vertical=0.6845018450184502
 package.editor.height=364
 package.editor.width=674
-package.editor.x=577
-package.editor.y=61
+package.editor.x=258
+package.editor.y=46
 package.frame.height=600
 package.frame.width=800
 package.numDependencies=2