Merge pull request #251 from CS2103AUG2016-W11-C3/Refactoring_and_add…

…ing_authors_Hui_Qi Refactoring and adding authors hui qi
CS2103AUG2016-W11-C3 · Nov 7, 2016 · 295e334 · 295e334
2 parents 788b6ce + 7905e78
commit 295e334
Show file tree

Hide file tree

Showing 3 changed files with 2 additions and 24 deletions.
diff --git a/src/main/java/harmony/mastermind/logic/commands/ListCommand.java b/src/main/java/harmony/mastermind/logic/commands/ListCommand.java
@@ -118,7 +118,6 @@ private static String formatList(ArrayList<GenericMemory> list) {
         String result = listName + "\n" + LINE;
         for (int i=0; i < list.size(); i++) {
             String line = i + 1 + BRACKET_OPEN;
-            line = markSymbol(list, i, line);
             line += BRACKET_CLOSE + list.get(i).getName();
 
             result = result + "\n" + line;
@@ -128,20 +127,6 @@ private static String formatList(ArrayList<GenericMemory> list) {
         return result;
     }
 
-    //@author A0143378Y
-    // Selects and prints the appropriate marking symbol
-    private static String markSymbol(ArrayList<GenericMemory> list, int i,
-            String line) {
-        if (list.get(i).getState() == 1) { // show marking for overdue/over items
-            line += MARK;
-        } else if (list.get(i).getState() == 2) { // show alert marking for overdue/ongoing items
-            line += ALERT;
-        } else { // show blank space for upcoming/incomplete items
-            line+= BLANK;
-        }
-        return line;
-    }
-
     //@author A0143378Y
     // Displays list of item with the corresponding type
     public static void displayType(ArrayList<GenericMemory> list, String type) {
@@ -153,13 +138,4 @@ public static void displayType(ArrayList<GenericMemory> list, String type) {
             displayDetailed(list.get(0));
         }
     }
-
-    //@author A0143378Y
-    // Redisplays item view or list view to reflect changes such as updates, marks, or deletes
-    public static void refreshDisplay() {
-        if(detailedView != null) { 
-            displayDetailed(detailedView);
-        }
-    }
-
 }
diff --git a/src/test/java/harmony/mastermind/logic/TestParserMemory.java b/src/test/java/harmony/mastermind/logic/TestParserMemory.java
@@ -23,6 +23,7 @@ public void test() {
         test_isUselessCommand();
     }
 
+    //@@author A0143378Y
     private void test_set() { 
         ParserMemoryMain.setCommand("test1");
         assertEquals("Test set command", ParserMemoryMain.getCommand(), "test1");

diff --git a/src/test/java/harmony/mastermind/logic/TestParserSearch.java b/src/test/java/harmony/mastermind/logic/TestParserSearch.java
@@ -23,6 +23,7 @@ public void test() {
         testInit();
     }
 
+    //@@author A0143378Y
     private void testInit() { 
         ParserSearch search = null;
         search.initVar();