Skip to content

Commit b7f5cf1

Browse files
committed
Fixed some Parser bugs where empty inputs cause a runtime error.
1 parent bbfb097 commit b7f5cf1

File tree

3 files changed

+8
-6
lines changed

3 files changed

+8
-6
lines changed

.gitignore

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -13,3 +13,4 @@ src/main/resources/docs/
1313
*.iml
1414
bin/
1515
data
16+
MANIFEST.MF

src/main/java/duke/command/AddCommand.java

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,8 @@ public AddCommand(String toAdd, Task.TaskType taskType) {
1616

1717
public void execute(TaskList taskList, Storage storage) {
1818
Task newTask = taskList.AddTask(toAdd, taskType);
19-
storage.SaveTaskToFile(newTask);
19+
if (newTask != null)
20+
storage.SaveTaskToFile(newTask);
2021
}
2122

2223
public boolean isExit() {

src/main/java/duke/interaction/Parser.java

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -17,15 +17,15 @@ public static Command parse(String fullCommand) {
1717
case "bye":
1818
return new ExitCommand();
1919
case "done":
20-
return new DoneCommand(in.nextInt() - 1);
20+
return new DoneCommand((in.hasNextInt() ? in.nextInt() - 1 : -1));
2121
case "todo":
22-
return new AddCommand(in.nextLine().trim(), Task.TaskType.TODO);
22+
return new AddCommand((in.hasNextLine() ? in.nextLine().trim() : ""), Task.TaskType.TODO);
2323
case "deadline":
24-
return new AddCommand(in.nextLine(), Task.TaskType.DEADLINE);
24+
return new AddCommand((in.hasNextLine() ? in.nextLine().trim() : ""), Task.TaskType.DEADLINE);
2525
case "event":
26-
return new AddCommand(in.nextLine(), Task.TaskType.EVENT);
26+
return new AddCommand((in.hasNextLine() ? in.nextLine().trim() : ""), Task.TaskType.EVENT);
2727
case "delete":
28-
return new DeleteCommand(in.nextInt() - 1);
28+
return new DeleteCommand((in.hasNextInt() ? in.nextInt() - 1 : -1));
2929
case "date":
3030
return new DateCommand(in.nextLine().trim());
3131
default:

0 commit comments

Comments
 (0)