minor display changes

src/main/java/io/zipcoder/ItemParser.java

@@ -4,8 +4,6 @@ import java.util.ArrayList;
 import java.util.Arrays;
 import java.util.regex.Matcher;
 import java.util.regex.Pattern;
-import static java.util.regex.Pattern.CASE_INSENSITIVE;
-import static java.util.regex.Pattern.LITERAL;
 
 public class ItemParser {
 
@@ -64,7 +62,7 @@ public class ItemParser {
     }
 
     private ArrayList<String> splitStringWithRegexPattern(String stringPattern, String inputString){
-        return new ArrayList<String>(Arrays.asList(inputString.split(stringPattern)));
+        return new ArrayList<>(Arrays.asList(inputString.split(stringPattern)));
     }
 
     public String normalizeRawData(String rawData){
@@ -74,11 +72,11 @@ public class ItemParser {
     }
 
     public String convertWeirdCharactersToSemiColon(String rawData) {
-        ArrayList<String> wierdCharacters = new ArrayList<>(Arrays.asList("^", "!", "*", "%"));
+        ArrayList<String> weirdCharacters = new ArrayList<>(Arrays.asList("^", "!", "*", "%"));
         Pattern pattern = Pattern.compile("@");
         Matcher matcher = pattern.matcher(rawData);
         String unWeird = matcher.replaceAll(";");
-        for (String character : wierdCharacters) {
+        for (String character : weirdCharacters) {
             matcher.reset(unWeird);
             matcher.usePattern(Pattern.compile(character, Pattern.LITERAL));
             unWeird = matcher.replaceAll(";");

src/main/java/io/zipcoder/ListCreation.java

@@ -19,7 +19,7 @@ public class ListCreation {
     }
 
     public String formattedGroceryListString(ArrayList<Item> groceryList) {
-        LinkedHashMap<Double,Integer> priceOccur = new LinkedHashMap<>();
+        TreeMap<Double,Integer> priceOccur = new TreeMap<>();
         String list = "";
         for (String key : keySet(groceryList)) {
             Pattern p = Pattern.compile(key);
@@ -60,8 +60,8 @@ public class ListCreation {
         return keys;
     }
 
-    public LinkedHashMap<Double, Integer> valueCount(ArrayList<Item> groceryList,String key) {
-        LinkedHashMap<Double, Integer> priceOccurrences = new LinkedHashMap<>();
+    public TreeMap<Double, Integer> valueCount(ArrayList<Item> groceryList,String key) {
+        TreeMap<Double, Integer> priceOccurrences = new TreeMap<>();
         for (Item item : groceryList) {
             Double price = item.getPrice();
             if (key.equals(item.getName())) {
@@ -75,16 +75,20 @@ public class ListCreation {
         return priceOccurrences;
     }
 
-    private String printValues(LinkedHashMap<Double, Integer> priceOccurrences) {
+    private String printValues(TreeMap<Double, Integer> priceOccurrences) {
         String valuesAndOccurrences = "";
-        for (Map.Entry<Double, Integer> entry : priceOccurrences.entrySet()) {
+
+        for (Map.Entry<Double, Integer> entry : priceOccurrences.descendingMap().entrySet()) {
             if(entry.getValue().equals(1)) {
                 valuesAndOccurrences += String.format("Price:   %1$.2f       seen: %2$d time \n", entry.getKey(), entry.getValue());
             } else {
                 valuesAndOccurrences += String.format("Price:   %1$.2f       seen: %2$d times\n", entry.getKey(), entry.getValue());
             }
-            valuesAndOccurrences += "-------------       -------------\n";
-
+            if(priceOccurrences.size()>1 && entry.getKey() != priceOccurrences.firstKey()) {
+                valuesAndOccurrences += "-------------       -------------\n";
+            } else if (priceOccurrences.size() == 1) {
+                valuesAndOccurrences += "-------------       -------------\n";
+            }
         }
         valuesAndOccurrences += "\n";
         return valuesAndOccurrences;

src/test/java/io/zipcoder/ListCreationTest.java

@@ -73,7 +73,7 @@ public class ListCreationTest {
         LinkedHashMap<Double,Integer> expected = new LinkedHashMap<>();
         expected.put(3.23,2);
         expected.put(2.00,1);
-        LinkedHashMap<Double,Integer> actual = list.valueCount(itemObjects, "milk");
+        TreeMap<Double,Integer> actual = list.valueCount(itemObjects, "milk");
         Assert.assertEquals(expected, actual);
     }