Merge pull request #194 from soons1/fix-OutofBoundsException

Fix AOOB exception when parsing pair, unpair and logadd inputs
AY2324S2-CS2103T-T09-3 · Apr 14, 2024 · 3917e0d · 3917e0d
2 parents 2c2bc86 + 0e179d3
commit 3917e0d
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 6 deletions.
diff --git a/src/main/java/scrolls/elder/logic/parser/LogAddCommandParser.java b/src/main/java/scrolls/elder/logic/parser/LogAddCommandParser.java
@@ -51,8 +51,8 @@ public LogAddCommand parse(String args) throws ParseException {
         try {
             index1 = ParserUtil.parseIndex(pairIndexes[0]);
             index2 = ParserUtil.parseIndex(pairIndexes[1]);
-        } catch (ParseException pe) {
-            throw new ParseException(String.format(MESSAGE_INVALID_COMMAND_FORMAT, LogAddCommand.MESSAGE_USAGE), pe);
+        } catch (ParseException | ArrayIndexOutOfBoundsException e) {
+            throw new ParseException(String.format(MESSAGE_INVALID_COMMAND_FORMAT, LogAddCommand.MESSAGE_USAGE), e);
         }
 
         String title = argMultimap.getValue(CliSyntax.PREFIX_TITLE).get().trim();

diff --git a/src/main/java/scrolls/elder/logic/parser/PairCommandParser.java b/src/main/java/scrolls/elder/logic/parser/PairCommandParser.java
@@ -28,8 +28,8 @@ public PairCommand parse(String args) throws ParseException {
         try {
             index1 = ParserUtil.parseIndex(pairIndexes[0]);
             index2 = ParserUtil.parseIndex(pairIndexes[1]);
-        } catch (ParseException pe) {
-            throw new ParseException(String.format(MESSAGE_INVALID_COMMAND_FORMAT, PairCommand.MESSAGE_USAGE), pe);
+        } catch (ParseException | ArrayIndexOutOfBoundsException e) {
+            throw new ParseException(String.format(MESSAGE_INVALID_COMMAND_FORMAT, PairCommand.MESSAGE_USAGE), e);
         }
 
         return new PairCommand(index1, index2);

diff --git a/src/main/java/scrolls/elder/logic/parser/UnpairCommandParser.java b/src/main/java/scrolls/elder/logic/parser/UnpairCommandParser.java
@@ -28,8 +28,8 @@ public UnpairCommand parse(String args) throws ParseException {
         try {
             index1 = ParserUtil.parseIndex(pairIndexes[0]);
             index2 = ParserUtil.parseIndex(pairIndexes[1]);
-        } catch (ParseException pe) {
-            throw new ParseException(String.format(MESSAGE_INVALID_COMMAND_FORMAT, UnpairCommand.MESSAGE_USAGE), pe);
+        } catch (ParseException | ArrayIndexOutOfBoundsException e) {
+            throw new ParseException(String.format(MESSAGE_INVALID_COMMAND_FORMAT, UnpairCommand.MESSAGE_USAGE), e);
         }
 
         return new UnpairCommand(index1, index2);