cleand code

Inventory.java

@@ -9,31 +9,33 @@ 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);
-                    }
+            if (items[i].getName().equals("Aged Brie")) {
+                if(items[i].getQuality() < 50){
+                    items[i].setQuality(items[i].getQuality() + 1);
                 }
-            } else {
-                if (items[i].getQuality() < 50) {
+                if(items[i].getSellIn() < 0 && items[i].getQuality() < 50){
                     items[i].setQuality(items[i].getQuality() + 1);
                 }
             }
+            else if(items[i].getName().equals("Sulfuras, Hand of Ragnaros")){
+                items[i].setQuality(items[i].getQuality());
 
-            if (!items[i].getName().equals("Sulfuras, Hand of Ragnaros")) {
-                items[i].setSellIn(items[i].getSellIn() - 1);
             }
+            else {
+                if(items[i].getSellIn() > 0 && items[i].getQuality() > 0 ){
+                    items[i].setQuality(items[i].getQuality() -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);
-                    }
                 }
+                else if(items[i].getSellIn() <= 0 && items[i].getQuality() > 0 ) {
+                    items[i].setQuality(items[i].getQuality() - 2);
+                }
+            }
+            if(items[i].getQuality() < 80){
+                items[i].setSellIn(items[i].getSellIn() - 1);
+
             }
+
         }
-}
+
+    }
 }

package.bluej

@@ -41,13 +41,13 @@ target2.name=InventoryTest
 target2.showInterface=false
 target2.type=UnitTestTargetJunit4
 target2.width=80
-target2.x=40
-target2.y=160
+target2.x=340
+target2.y=140
 target3.association=InventoryTest
 target3.height=50
 target3.name=Inventory
 target3.showInterface=false
 target3.type=ClassTarget
 target3.width=80
-target3.x=10
-target3.y=190
+target3.x=310
+target3.y=170