Fix find command bug

Author: alex-setyawan

src/main/java/seedu/address/logic/parser/AddVisitCommandParser.java

@@ -41,7 +41,7 @@ public AddVisitCommand parse(String args) throws ParseException {
 
         argMultimap.verifyNoDuplicatePrefixesFor(PREFIX_NRIC, PREFIX_DATEOFVISIT,
             PREFIX_SYMPTOM, PREFIX_DIAGNOSIS, PREFIX_STATUS);
-        Nric nric = ParserUtil.parseNric(argMultimap.getValue(PREFIX_NRIC).get().toUpperCase());
+        Nric nric = ParserUtil.parseNric(argMultimap.getValue(PREFIX_NRIC).get());
         DateOfVisit dov = ParserUtil.parseDateOfVisit(argMultimap.getValue(PREFIX_DATEOFVISIT).get());
         Symptom symptom = ParserUtil.parseSymptom(argMultimap.getValue(PREFIX_SYMPTOM).get());
         Diagnosis diagnosis = ParserUtil.parseDiagnosis(argMultimap.getValue(PREFIX_DIAGNOSIS).get());

src/main/java/seedu/address/logic/parser/CreateCommandParser.java

@@ -56,7 +56,7 @@ public CreateCommand parse(String args) throws ParseException {
 
         argMultimap.verifyNoDuplicatePrefixesFor(PREFIX_NRIC, PREFIX_NAME, PREFIX_PHONE, PREFIX_ADDRESS,
                 PREFIX_DATEOFBIRTH, PREFIX_SEX, PREFIX_STATUS);
-        Nric nric = ParserUtil.parseNric(argMultimap.getValue(PREFIX_NRIC).get().toUpperCase());
+        Nric nric = ParserUtil.parseNric(argMultimap.getValue(PREFIX_NRIC).get());
         Name name = ParserUtil.parseName(argMultimap.getValue(PREFIX_NAME).get());
         Phone phone = ParserUtil.parsePhone(argMultimap.getValue(PREFIX_PHONE).get());
         Address address = ParserUtil.parseAddress(argMultimap.getValue(PREFIX_ADDRESS).get());

src/main/java/seedu/address/logic/parser/DeleteCommandParser.java

@@ -18,7 +18,7 @@ public class DeleteCommandParser implements Parser<DeleteCommand> {
      */
     public DeleteCommand parse(String args) throws ParseException {
         try {
-            Nric nric = ParserUtil.parseNric(args.toUpperCase());
+            Nric nric = ParserUtil.parseNric(args);
             return new DeleteCommand(nric);
         } catch (ParseException pe) {
             throw new ParseException(

src/main/java/seedu/address/logic/parser/DeleteInfoCommandParser.java

@@ -57,7 +57,7 @@ public DeleteInfoCommand parse(String args) throws ParseException {
         //Check for NRIC
         Nric nric;
         try {
-            nric = ParserUtil.parseNric(argMultimap.getPreamble().toUpperCase());
+            nric = ParserUtil.parseNric(argMultimap.getPreamble());
         } catch (ParseException pe) {
             throw new ParseException(
                     String.format(MESSAGE_INVALID_COMMAND_FORMAT, DeleteInfoCommand.MESSAGE_USAGE), pe);

src/main/java/seedu/address/logic/parser/FindCommandParser.java

@@ -32,23 +32,41 @@ public FindCommand parse(String args) throws ParseException {
         }
 
         if (trimmedArgs.startsWith(PREFIX_NAME.getPrefix())) {
-            String[] nameKeywords = trimmedArgs.substring(2).trim().split("\\s+");
-            List<String> list = Arrays.asList(nameKeywords);
+            String nameKeywords = trimmedArgs.substring(2);
+            if (nameKeywords.isEmpty()) {
+                throw new ParseException(
+                        String.format(MESSAGE_INVALID_COMMAND_FORMAT, FindCommand.MESSAGE_USAGE));
+            }
+            List<String> list = Arrays.asList(nameKeywords.trim().split("\\s+"));
             return new FindCommand(new NameContainsKeywordsPredicate(list));
         } else if (trimmedArgs.startsWith(PREFIX_ADDRESS.getPrefix())) {
-            String[] addressKeywords = trimmedArgs.substring(2).trim().split(",");
+            String doubleTrimmedArgs = trimmedArgs.substring(2);
+            if (doubleTrimmedArgs.isEmpty()) {
+                throw new ParseException(
+                        String.format(MESSAGE_INVALID_COMMAND_FORMAT, FindCommand.MESSAGE_USAGE));
+            }
+
+            String[] addressKeywords = doubleTrimmedArgs.trim().split(",");
             int len = addressKeywords.length;
             for (int i = 0; i < len; i++) {
                 addressKeywords[i] = addressKeywords[i].trim();
             }
+
             List<String> list = Arrays.asList(addressKeywords);
             return new FindCommand(new AddressContainsKeywordsPredicate(list));
         } else if (trimmedArgs.startsWith(PREFIX_CONDITION.getPrefix())) {
-            String[] conditionKeywords = trimmedArgs.substring(4).trim().split(",");
+            String doubleTrimmedArgs = trimmedArgs.substring(4);
+            if (doubleTrimmedArgs.isEmpty()) {
+                throw new ParseException(
+                        String.format(MESSAGE_INVALID_COMMAND_FORMAT, FindCommand.MESSAGE_USAGE));
+            }
+
+            String[] conditionKeywords = doubleTrimmedArgs.trim().split(",");
             int len = conditionKeywords.length;
             for (int i = 0; i < len; i++) {
                 conditionKeywords[i] = conditionKeywords[i].trim();
             }
+
             List<String> list = Arrays.asList(conditionKeywords);
             return new FindCommand(new ConditionContainsKeywordsPredicate(list));
         } else {