correcting raw types for the item Iterator.

iterator/src/main/java/com/iluwatar/iterator/App.java

@@ -25,6 +25,7 @@ import static com.iluwatar.iterator.list.ItemType.WEAPON;
 
 import com.iluwatar.iterator.bst.BstIterator;
 import com.iluwatar.iterator.bst.TreeNode;
+import com.iluwatar.iterator.list.Item;
 import com.iluwatar.iterator.list.ItemType;
 import com.iluwatar.iterator.list.TreasureChest;
 import org.slf4j.Logger;
@@ -47,7 +48,7 @@ public class App {
   private static void demonstrateTreasureChestIteratorForType(ItemType itemType) {
     LOGGER.info("------------------------");
     LOGGER.info("Item Iterator for ItemType " + itemType + ": ");
-    Iterator itemIterator = TREASURE_CHEST.iterator(itemType);
+    Iterator<Item> itemIterator = TREASURE_CHEST.iterator(itemType);
     while (itemIterator.hasNext()) {
       LOGGER.info(itemIterator.next().toString());
     }

iterator/src/main/java/com/iluwatar/iterator/list/TreasureChest.java

@@ -48,7 +48,7 @@ public class TreasureChest {
     items.add(new Item(ItemType.WEAPON, "Dagger of poison"));
   }
 
-  public Iterator iterator(ItemType itemType) {
+  public Iterator<Item> iterator(ItemType itemType) {
     return new TreasureChestItemIterator(this, itemType);
   }
 
@@ -56,9 +56,7 @@ public class TreasureChest {
    * Get all items
    */
   public List<Item> getItems() {
-    List<Item> list = new ArrayList<>();
+    return new ArrayList<>(items);
-    list.addAll(items);
-    return list;
   }
 
 }

iterator/src/main/java/com/iluwatar/iterator/list/TreasureChestItemIterator.java

@@ -26,7 +26,7 @@ import java.util.List;
  * TreasureChestItemIterator
  *
  */
-public class TreasureChestItemIterator implements Iterator {
+public class TreasureChestItemIterator implements Iterator<Item> {
 
   private TreasureChest chest;
   private int idx;

iterator/src/test/java/com/iluwatar/iterator/list/TreasureChestTest.java

@@ -63,11 +63,11 @@ public class TreasureChestTest {
   @MethodSource("dataProvider")
   public void testIterator(Item expectedItem) {
     final TreasureChest chest = new TreasureChest();
-    final Iterator iterator = chest.iterator(expectedItem.getType());
+    final Iterator<Item> iterator = chest.iterator(expectedItem.getType());
     assertNotNull(iterator);
 
     while (iterator.hasNext()) {
-      final Item item = (Item) iterator.next();
+      final Item item = iterator.next();
       assertNotNull(item);
       assertEquals(expectedItem.getType(), item.getType());