m

Inventory.java

@@ -8,32 +8,40 @@ public class Inventory {
     }
 
     public void updateQuality() {
+
         for (int i = 0; i < items.length; i++) {
-            if (!items[i].getName().equals("Aged Brie")) {
+            if (!items[i].getName().equals("Aged Brie") && !items[i].getName().equals("Sulfuras, Hand of Ragnaros")){
                 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);
+                    items[i].setQuality(items[i].getQuality() - 1);
+                    items[i].setSellIn(items[i].getSellIn() - 1);
                 }
+            } else 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 the item name is not Sulfuras and HandOfRagnaros 
+            // the sell in value descrease by 1
+
+            //if (!items[i].getName().equals("Sulfuras, Hand of Ragnaros")) {
+              //  items[i].setSellIn(items[i].getSellIn() - 1);
+            //}
 
+            // if the sell in day of the item is
+            // less than 0
+            // && the name is not Brie
+            //then the quality of the item 
+            // decrease by one
+            // else
             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);
-                    }
+                    items[i].setQuality(items[i].getQuality() - 1);
+
+                } else if (items[i].getQuality() < 50) {
+                    items[i].setQuality(items[i].getQuality() + 1);
+
                 }
             }
         }
+    }
 }
-}
+

package.bluej

@@ -6,17 +6,17 @@ 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.width=1269
+editor.fx.0.x=11
+editor.fx.0.y=23
 objectbench.height=164
-objectbench.width=776
+objectbench.width=461
 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=269
+package.editor.y=23
 package.frame.height=600
 package.frame.width=800
 package.numDependencies=2