Implement more error handling

Author: desmondwong1215

src/main/java/helperbot/storage/data/HelperBot.txt

@@ -1,5 +1,5 @@
 T,0,read book
-D,0,return book,2025-10-30,21:30
+D,0,return book,2025-10-21,
 D,0,ca1,2025-09-12,
 E,0,ca7,2025-10-11,21:30,2025-10-17,21:30
 E,0,ca2,2025-09-12,,2025-10-12,

src/main/java/helperbot/task/Deadline.java

@@ -22,7 +22,7 @@ public class Deadline extends Task {
      * @param byDate the date of the deadline
      * @param byTime the time of the deadline
      */
-    public Deadline(String description, LocalDate byDate, LocalTime byTime) {
+    public Deadline(String description, LocalDate byDate, LocalTime byTime) throws HelperBotArgumentException {
         super(description);
         this.byDate = byDate;
         this.byTime = byTime;
@@ -51,7 +51,8 @@ public static Deadline fromUserInput(String message) throws HelperBotArgumentExc
         } catch (IndexOutOfBoundsException e) {
             throw new HelperBotArgumentException("Wrong format for Deadline");
         } catch (DateTimeParseException e) {
-            throw new HelperBotArgumentException("Please enter date and time in YYYY-MM-DD hh:mm after /by");
+            throw new HelperBotArgumentException("Invalid date or time. "
+                    + "Please enter date and time in YYYY-MM-DD hh:mm after /by");
         }
     }
 
@@ -128,12 +129,16 @@ private static Deadline getDeadline(String[] message) throws HelperBotFileExcept
         if (message.length == 5) {
             byTime = LocalTime.parse(message[4]);
         }
-        Deadline deadline = new Deadline(message[2], byDate, byTime);
-        if (message[1].equals("1")) {
-            deadline.markAsDone();
-        } else if (!message[1].equals("0")) {
-            throw new HelperBotFileException("Invalid status " + message[0] + " for Task");
+        try {
+            Deadline deadline = new Deadline(message[2], byDate, byTime);
+            if (message[1].equals("1")) {
+                deadline.markAsDone();
+            } else if (!message[1].equals("0")) {
+                throw new HelperBotFileException("Invalid status " + message[0] + " for Task");
+            }
+            return deadline;
+        } catch (HelperBotArgumentException e) {
+            throw new HelperBotFileException("Empty description.");
         }
-        return deadline;
     }
 }

src/main/java/helperbot/task/Event.java

@@ -26,7 +26,8 @@ public class Event extends Task {
      * @param toDate The end date of the event.
      * @param toTime The end time of the event.
      */
-    public Event(String description, LocalDate fromDate, LocalTime fromTime, LocalDate toDate, LocalTime toTime) {
+    public Event(String description, LocalDate fromDate, LocalTime fromTime, LocalDate toDate,
+                 LocalTime toTime) throws HelperBotArgumentException {
         super(description);
         this.fromDate = fromDate;
         this.fromTime = fromTime;
@@ -67,7 +68,8 @@ public static Event fromUserInput(String message) throws HelperBotArgumentExcept
         } catch (IndexOutOfBoundsException e) {
             throw new HelperBotArgumentException("Wrong format for Event");
         } catch (DateTimeParseException e) {
-            throw new HelperBotArgumentException("Please enter date and time in YYYY-MM-DD hh:mm after /from and /to");
+            throw new HelperBotArgumentException("Invalid date or time. "
+                    + "Please enter date and time in YYYY-MM-DD hh:mm after /from and /to");
         }
     }
 
@@ -165,13 +167,17 @@ private static Event getEvent(String[] message) throws HelperBotFileException {
         if (Event.isToBeforeFrom(fromDate, fromTime, toDate, toTime)) {
             throw new HelperBotFileException("From datetime must be before to datetime");
         }
-        Event event = new Event(message[2], fromDate, fromTime, toDate, toTime);
-        if (message[1].equals("1")) {
-            event.markAsDone();
-        } else if (!message[1].equals("0")) {
-            throw new HelperBotFileException("Invalid status " + message[0] + " for Task");
+        try {
+            Event event = new Event(message[2], fromDate, fromTime, toDate, toTime);
+            if (message[1].equals("1")) {
+                event.markAsDone();
+            } else if (!message[1].equals("0")) {
+                throw new HelperBotFileException("Invalid status " + message[0] + " for Task");
+            }
+            return event;
+        } catch (HelperBotArgumentException e) {
+            throw new HelperBotFileException("Empty description.");
         }
-        return event;
     }
 
     private static boolean isToBeforeFrom(LocalDate fromDate, LocalTime fromTime,

src/main/java/helperbot/task/Task.java

@@ -16,7 +16,10 @@ public class Task {
      * Generates a <code>Task</code>.
      * @param description The description of the task.
      */
-    public Task(String description) {
+    public Task(String description) throws HelperBotArgumentException {
+        if (description.isEmpty()) {
+            throw new HelperBotArgumentException("Empty description.");
+        }
         this.description = description;
         this.isDone = false;
     }

src/main/java/helperbot/task/ToDo.java

@@ -12,7 +12,7 @@ public class ToDo extends Task {
      * Generates a <code>ToDo</code>
      * @param description the name of the task.
      */
-    public ToDo(String description) {
+    public ToDo(String description) throws HelperBotArgumentException {
         super(description);
     }
 
@@ -25,9 +25,6 @@ public ToDo(String description) {
     public static ToDo fromUserInput(String message) throws HelperBotArgumentException {
         try {
             String detail = message.substring(5).trim();
-            if (detail.isEmpty()) {
-                throw new HelperBotArgumentException("Empty description.");
-            }
             return new ToDo(detail);
         } catch (IndexOutOfBoundsException e) {
             throw new HelperBotArgumentException("Wrong format for ToDo.");
@@ -51,6 +48,8 @@ public static ToDo of(String[] message) throws HelperBotFileException {
             return toDo;
         } catch (IndexOutOfBoundsException e) {
             throw new HelperBotFileException("Incomplete data for Task.");
+        } catch (HelperBotArgumentException e) {
+            throw new HelperBotFileException("Empty description.");
         }
     }
 

src/test/java/helperbot/task/DeadlineTest.java

@@ -80,13 +80,13 @@ void of_validDataWithoutTime_success() throws HelperBotFileException {
     }
 
     @Test
-    void toSaveFormat_withoutByTime_success() {
+    void toSaveFormat_withoutByTime_success() throws HelperBotArgumentException {
         Deadline deadline = new Deadline("deadline1", LocalDate.parse("2025-09-12"), null);
         assertEquals("D,0,deadline1,2025-09-12,", deadline.toSavaFormat());
     }
 
     @Test
-    void toSaveFormat_havaByTime_success() {
+    void toSaveFormat_havaByTime_success() throws HelperBotArgumentException {
         Deadline deadline = new Deadline("deadline1", LocalDate.parse("2025-09-12"),
                 LocalTime.parse("21:30"));
         assertEquals("D,0,deadline1,2025-09-12,21:30", deadline.toSavaFormat());

src/test/java/helperbot/task/EventTest.java

@@ -117,15 +117,15 @@ void of_fromAfterTo_exceptionThrown() {
     }
 
     @Test
-    void toSaveFormat_eventWithTime_correctFormat() {
+    void toSaveFormat_eventWithTime_correctFormat() throws HelperBotArgumentException {
         Event event = new Event("go to the gym", LocalDate.of(2025, 11, 10),
                 LocalTime.of(17, 30),
                 LocalDate.of(2025, 11, 10), LocalTime.of(19, 0));
         assertEquals("E,0,go to the gym,2025-11-10,17:30,2025-11-10,19:00", event.toSavaFormat());
     }
 
     @Test
-    void toSaveFormat_eventWithoutTime_correctFormat() {
+    void toSaveFormat_eventWithoutTime_correctFormat() throws HelperBotArgumentException {
         Event event = new Event("take a trip", LocalDate.of(2025, 12, 20),
                     null,
                 LocalDate.of(2025, 12, 25), null);

src/test/java/helperbot/task/ToDoTest.java

@@ -62,7 +62,7 @@ void of_incompleteData_exceptionThrown() {
     }
 
     @Test
-    void toSaveFormat_validTask_correctFormat() {
+    void toSaveFormat_validTask_correctFormat() throws HelperBotArgumentException {
         ToDo toDo = new ToDo("take out the trash");
         assertEquals("T,0,take out the trash", toDo.toSavaFormat());
     }

src/test/java/helperbot/ui/ResponseTest.java

@@ -32,7 +32,7 @@ void getGreetMessage_validOutput_success() {
     }
 
     @Test
-    void getMarkCommandResponse_validTask_correctOutput() {
+    void getMarkCommandResponse_validTask_correctOutput() throws HelperBotArgumentException {
         Task mockTask = new Task("test task") {
             @Override
             public String toString() {
@@ -46,7 +46,7 @@ public String toString() {
     }
 
     @Test
-    void getUnmarkCommandResponse_validTask_correctOutput() {
+    void getUnmarkCommandResponse_validTask_correctOutput() throws HelperBotArgumentException {
         Task mockTask = new Task("another test task") {
             @Override
             public String toString() {