Merge pull request #200 from EugeneChanJiajun/master

Bug fix and userguide update

docs/UserGuide.md

@@ -59,6 +59,7 @@ Expected outcome:
 
 ### Adding a general travel activity : `add`
 Adds a general travel activity into the travel activity list
+* This is for activities that do not fall under the other activity types. EG. visiting a country.
 
 Format: `add DESCRIPTION /date YYYY-MM-DD /duration DURATION [/tag TAG]`
 

src/main/java/seedu/omnitravel/errorhandlers/CheckParameters.java

@@ -33,8 +33,6 @@ public static void addExceptions(String[] input, String commandType, String line
                     "/date YYYY-MM-DD /duration DURATION"
                     + " or add DESCRIPTION /date YYYY-MM-DD /duration DURATION /tag TAG");
         }
-
-
     }
 
     /**
@@ -152,4 +150,19 @@ public static void handleException(Exception exception) {
             Ui.printInterruptedError();
         }
     }
+
+    /**
+     * Checks the input that the users placed into the chatbot and checks if the input contains any ASCII characters.
+     *
+     * @param input Input line that users placed into the chatbot
+     * @throws OmniException if the input contains any non-ASCII characters
+     */
+    public static void asciiCheck(String input) throws OmniException {
+        for (int i = 0; i < input.length(); i++) {
+            char c = input.charAt(i);
+            if (c < 32 || c > 126) {
+                throw new OmniException("Input contains non-ASCII characters.");
+            }
+        }
+    }
 }

src/main/java/seedu/omnitravel/omnitravel/OmniTravel.java

@@ -31,6 +31,7 @@ public static void main(String[] args) throws IOException {
             try {
                 String line = in.nextLine();
                 assert line != null :"Input does not exist!";
+                CheckParameters.asciiCheck(line);
                 String[] command = line.split(" ");
                 logger.log(Level.INFO, command[0]);
                 switch (command[0].toLowerCase()) {