completed

Inventory.java

@@ -9,31 +9,36 @@ 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);
-                }
-            }
+            //handles Sulfuras
+            if(items[i].getName().equals("Sulfuras, Hand of Ragnaros")){
+                continue;}
 
-            if (!items[i].getName().equals("Sulfuras, Hand of Ragnaros")) {
+            if(items[i].getName().equals("Aged Brie")){
+                agedBrieUpdate(i);
+            } else{regularItemUpdate(i);}
+         
+            if (items[i].getQuality() < 80) {
                 items[i].setSellIn(items[i].getSellIn() - 1);
             }
 
-            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 agedBrieUpdate(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 regularItemUpdate(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);
+        } 
+    }
 }
-}
+
+

package.bluej

@@ -5,17 +5,17 @@ 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=1080
+editor.fx.0.width=1920
+editor.fx.0.x=-335
+editor.fx.0.y=-1080
 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.x=480
 package.editor.y=61
 package.frame.height=600
 package.frame.width=800