Finished Part 1, will finish part 2 after the other Labs are complete if I have time

Inventory.java

@@ -1,3 +1,5 @@
+import java.util.List;
+import java.util.Arrays;
 
 public class Inventory {
     private Item[] items;
@@ -10,44 +12,40 @@ public class Inventory {
     public void updateQuality() {
 
         for (Item item: items) {
-           
-            //This loop runs at the end of each day
-            //SellIn is the value which denotes the number of days we have to sell the item
-            //Quality value denotes how values for every item
-            
             if (item.getName().equals("Aged Brie")) {
-               
-                if (item.getQuality() < 50) {
-                    item.setQuality(item.getQuality() + 1);
-                }     
-                
-                if (item.getSellIn() < 0 && item.getQuality() < 50) {
-                        item.setQuality(item.getQuality() + 1);
-                }
-                
-            } else {
-                
-                if (item.getQuality() > 0) {
-                    if (!item.getName().equals("Sulfuras, Hand of Ragnaros")) {
-                        item.setQuality(item.getQuality() - 1);
-                    }
-                }
-            
-                if (item.getSellIn() < 0) {
-                  item.setQuality(item.getQuality() - 1);
-                }
-
-            }
-            
-            if (!item.getName().equals("Sulfuras, Hand of Ragnaros")) {
-                item.setSellIn(item.getSellIn() - 1);
+                setAgedBrie(item);
+            } else if(isNotLegenday(item)) {
+                setNoneLegendary(item);
             }
         }
         
     }
     
     private boolean isNotLegenday(Item item){
-        return item.getName().equals("Sulfuras, Hand of Ragnaros") ? true : false;
+    List<String> legendayItem = Arrays.asList
+    ("Sulfuras, Hand of Ragnaros");
+        return legendayItem.contains(item.getName()) ? false : true;
+    }
+    
+    private void setAgedBrie(Item item){
+        if (item.getQuality() < 50) {
+            item.setQuality(item.getQuality() + 1);
+        }     
+        if (item.getSellIn() < 0 && item.getQuality() < 50) {
+                item.setQuality(item.getQuality() + 1);
+        }        
+    }
+    
+    private void setNoneLegendary(Item item){
+        if (item.getQuality() > 0) {
+            item.setQuality(item.getQuality() - 1);
+        }
+        if (item.getSellIn() < 0) {
+          item.setQuality(item.getQuality() - 1);
+        }
+        
+        item.setSellIn(item.getSellIn() - 1);
     }
+    
    
 }

package.bluej

@@ -1,56 +1,56 @@
-#BlueJ package file
-dependency1.from=InventoryTest
-dependency1.to=Item
-dependency1.type=UsesDependency
-dependency2.from=InventoryTest
-dependency2.to=Inventory
-dependency2.type=UsesDependency
-dependency3.from=Inventory
-dependency3.to=Item
-dependency3.type=UsesDependency
-editor.fx.0.height=28
-editor.fx.0.width=160
-editor.fx.0.x=-32000
-editor.fx.0.y=-32000
-objectbench.height=151
-objectbench.width=934
-package.divider.horizontal=0.6
-package.divider.vertical=0.8331573389651531
-package.editor.height=782
-package.editor.width=844
-package.editor.x=953
-package.editor.y=0
-package.frame.height=1047
-package.frame.width=974
-package.numDependencies=3
-package.numTargets=3
-package.showExtends=true
-package.showUses=true
-project.charset=UTF-8
-readme.height=58
-readme.name=@README
-readme.width=47
-readme.x=10
-readme.y=10
-target1.height=50
-target1.name=Item
-target1.showInterface=false
-target1.type=ClassTarget
-target1.width=80
-target1.x=570
-target1.y=180
-target2.height=50
-target2.name=InventoryTest
-target2.showInterface=false
-target2.type=UnitTestTargetJunit4
-target2.width=80
-target2.x=570
-target2.y=240
-target3.association=InventoryTest
-target3.height=50
-target3.name=Inventory
-target3.showInterface=false
-target3.type=ClassTarget
-target3.width=80
-target3.x=540
-target3.y=270
+#BlueJ package file
+dependency1.from=InventoryTest
+dependency1.to=Item
+dependency1.type=UsesDependency
+dependency2.from=InventoryTest
+dependency2.to=Inventory
+dependency2.type=UsesDependency
+dependency3.from=Inventory
+dependency3.to=Item
+dependency3.type=UsesDependency
+editor.fx.0.height=1047
+editor.fx.0.width=989
+editor.fx.0.x=938
+editor.fx.0.y=0
+objectbench.height=151
+objectbench.width=375
+package.divider.horizontal=0.6643478260869565
+package.divider.vertical=0.7193605683836589
+package.editor.height=385
+package.editor.width=481
+package.editor.x=285
+package.editor.y=165
+package.frame.height=663
+package.frame.width=611
+package.numDependencies=3
+package.numTargets=3
+package.showExtends=true
+package.showUses=true
+project.charset=UTF-8
+readme.height=58
+readme.name=@README
+readme.width=47
+readme.x=10
+readme.y=10
+target1.height=50
+target1.name=Item
+target1.showInterface=false
+target1.type=ClassTarget
+target1.width=80
+target1.x=570
+target1.y=180
+target2.height=50
+target2.name=InventoryTest
+target2.showInterface=false
+target2.type=UnitTestTargetJunit4
+target2.width=80
+target2.x=570
+target2.y=240
+target3.association=InventoryTest
+target3.height=50
+target3.name=Inventory
+target3.showInterface=false
+target3.type=ClassTarget
+target3.width=80
+target3.x=540
+target3.y=270