Merge Main into branch-A-CodeQuality (#3)

src/main/java/dude/commands/DeadlineCommand.java

@@ -18,11 +18,17 @@ public class DeadlineCommand extends Command {
      * Constructor for the DeadlineCommand class. Returns a command object to add a deadline task to
      * the TaskList object upon execution.
      *
+     *
      * @param input    The input string that resulted in the creation of this command.
      * @param tasklist The TaskList object to which the deadline task is to be added.
      */
     public DeadlineCommand(String input, TaskList tasklist) {
         super(COMMAND_FORMAT, "deadline .* /by .*");
+
+        assert(input != null);
+        assert(tasklist != null);
+        assert(input.contains("deadline"));
+
         this.input = input;
         this.taskList = tasklist;
     }

src/main/java/dude/commands/DeleteCommand.java

@@ -24,6 +24,11 @@ public class DeleteCommand extends Command {
      */
     public DeleteCommand(String input, TaskList tasklist) {
         super(COMMAND_FORMAT, "delete \\d+");
+
+        assert(input != null);
+        assert(tasklist != null);
+        assert(input.contains("delete"));
+
         this.input = input.trim();
         this.taskList = tasklist;
     }

src/main/java/dude/commands/EventCommand.java

@@ -24,6 +24,12 @@ public class EventCommand extends Command {
      */
     public EventCommand(String input, TaskList tasklist) {
         super(COMMAND_FORMAT, "event .* /from .* /to .*");
+
+        assert(input != null);
+        assert(tasklist != null);
+        assert(input.contains("event"));
+
+
         this.input = input;
         this.taskList = tasklist;
     }

src/main/java/dude/commands/FindCommand.java

@@ -25,6 +25,11 @@ public class FindCommand extends Command {
      */
     public FindCommand(String input, TaskList taskList) {
         super(COMMAND_FORMAT, "find .*");
+
+        assert(input != null);
+        assert(taskList != null);
+        assert(input.contains("find"));
+
         this.input = input;
         this.taskList = taskList;
     }

src/main/java/dude/commands/HelpCommand.java

@@ -32,6 +32,10 @@ public class HelpCommand extends Command {
      */
     public HelpCommand(String input) {
         super("help <?command>", "help");
+
+        assert(input != null);
+        assert(input.contains("help"));
+
         this.input = input.trim();
     }
 

src/main/java/dude/commands/InvalidCommand.java

@@ -1,7 +1,7 @@
 package dude.commands;
 
 /**
- * The InvalidCommand class represents a command that is invalid.
+ * The InvalidCommand Class is used to wrap around any invalid commands that are input by the user.
  */
 public class InvalidCommand extends Command {
 
@@ -15,7 +15,7 @@ public InvalidCommand() {
     /**
      * Does nothing
      *
-     * @return The string message for an invalid command.
+     * @return The string message for trying to execute an invalid command.
      */
     public String execute() {
         return "Sorry I don't understand that command. Please try again.";

src/main/java/dude/commands/ListCommand.java

@@ -19,6 +19,9 @@ public class ListCommand extends Command {
      */
     public ListCommand(TaskList tasklist) {
         super("list", "list");
+
+        assert(tasklist != null);
+
         this.tasklist = tasklist;
     }
 

src/main/java/dude/commands/MarkCommand.java

@@ -23,6 +23,11 @@ public class MarkCommand extends Command {
      */
     public MarkCommand(String input, TaskList tasklist) {
         super(COMMAND_FORMAT, "mark \\d+");
+
+        assert(input != null);
+        assert(tasklist != null);
+        assert(input.contains("mark"));
+
         this.input = input.trim();
         this.taskList = tasklist;
     }

src/main/java/dude/commands/Parser.java

@@ -15,6 +15,9 @@ public class Parser {
      * @return The parsed command from the user input.
      */
     public static Command parse(String input, TaskList tasklist) {
+        assert(input != null);
+        assert(tasklist != null);
+
         String[] command = input.split(" ", 2);
         switch (command[0]) {
         case "bye":

src/main/java/dude/commands/TodoCommand.java

@@ -23,6 +23,11 @@ public class TodoCommand extends Command {
      */
     public TodoCommand(String input, TaskList tasklist) {
         super(COMMAND_FORMAT, "todo .*");
+
+        assert(input != null);
+        assert(tasklist != null);
+        assert(input.contains("todo"));
+
         this.input = input;
         this.taskList = tasklist;
     }

src/main/java/dude/commands/UnmarkCommand.java

@@ -23,6 +23,11 @@ public class UnmarkCommand extends Command {
      */
     public UnmarkCommand(String input, TaskList tasklist) {
         super(COMMAND_FORMAT, "unmark \\d+");
+
+        assert(input != null);
+        assert(tasklist != null);
+        assert(input.contains("unmark"));
+
         this.input = input.trim();
         this.taskList = tasklist;
     }

src/main/java/dude/tasks/Deadline.java

@@ -24,6 +24,11 @@ public class Deadline extends Task {
      */
     public Deadline(String description, LocalDateTime by) {
         super(description);
+
+        assert (by != null);
+        assert (description != null);
+        assert (!description.isEmpty());
+
         this.deadlineDate = by;
     }
 
@@ -39,6 +44,7 @@ public Deadline(String description, LocalDateTime by) {
      */
     public static Deadline from(String s) throws InvalidFormatException,
             InvalidDescriptionException, InvalidArgumentException {
+        assert (s != null);
         String rest = Utils.discardFirstWord(s.trim()).trim();
         String[] arr = rest.split(" ");
 

src/main/java/dude/tasks/Event.java

@@ -27,6 +27,12 @@ public class Event extends Task {
      */
     public Event(String description, LocalDateTime fromTime, LocalDateTime toTime) {
         super(description);
+
+        assert (fromTime != null);
+        assert (toTime != null);
+        assert (description != null);
+        assert (!description.isEmpty());
+
         this.fromTime = fromTime;
         this.toTime = toTime;
     }

src/main/java/dude/tasks/Task.java

@@ -21,6 +21,7 @@ public class Task implements Serializable {
      * @param description The description of the task.
      */
     public Task(String description) {
+        assert (description != null);
         this.description = description;
         this.isDone = false;
     }

src/main/java/dude/tasks/TaskList.java

@@ -27,6 +27,8 @@ public TaskList() {
      * @return The TaskList object created from the ArrayList of Task objects.
      */
     public static TaskList from(ArrayList<Task> tasks) {
+        assert (tasks != null);
+
         TaskList taskList = new TaskList();
         taskList.list.addAll(tasks);
         return taskList;
@@ -39,6 +41,8 @@ public static TaskList from(ArrayList<Task> tasks) {
      * @return The TaskList object created from the Task objects.
      */
     public static TaskList from(Task... tasks) {
+        assert (tasks != null);
+
         TaskList taskList = new TaskList();
         for (Task task : tasks) {
             taskList.list.add(task);
@@ -53,6 +57,7 @@ public static TaskList from(Task... tasks) {
      * @throws TaskListFullException if the task list is full.
      */
     public void add_task(Task task) throws TaskListFullException {
+        assert (task != null);
         if (list.size() >= 100) {
             throw new TaskListFullException("Sorry, the task list is full.");
         }
@@ -105,6 +110,7 @@ public void mark_as_undone(int id) throws DudeException {
      * @return An ArrayList of Task objects that contain the keyword.
      */
     public ArrayList<Task> find(String keyword) {
+        assert (keyword != null);
         ArrayList<Task> result = new ArrayList<>();
         String taskString;
         String keywordLowerCase = keyword.toLowerCase();

src/main/java/dude/tasks/Todo.java

@@ -15,6 +15,7 @@ public class Todo extends Task {
      */
     public Todo(String description) {
         super(description);
+        assert (description != null);
     }
 
 

src/main/java/dude/utils/Storage.java

@@ -26,6 +26,8 @@ public class Storage {
      * @param fileLocation The file path to the storage file.
      */
     public Storage(String fileLocation) {
+        assert (fileLocation != null);
+
         this.filepath = extractFilePath(fileLocation);
         this.filename = extractFileName(fileLocation);
     }
@@ -76,6 +78,7 @@ public void deleteStorage() throws SecurityException {
      * @throws SecurityException if a security manager exists and its checkWrite method denies write access to the file.
      */
     public void saveTasks(TaskList taskList) throws IOException, SecurityException {
+        assert (taskList != null);
         try {
             FileOutputStream fos = new FileOutputStream(this.filepath + this.filename);
             ObjectOutputStream oos = new ObjectOutputStream(fos);

src/main/java/dude/utils/Utils.java

@@ -15,6 +15,7 @@ public class Utils {
      * @return The input string with the first word discarded.
      */
     public static String discardFirstWord(String input) {
+        assert (input != null);
         // Split the string by whitespace
         String[] words = input.split(" ", 2);
         // Check if there are at least two words
@@ -49,7 +50,8 @@ public static int countOccurrences(String[] array, String target) {
      *
      * @param array  The array of strings.
      * @param target The target string.
-     * @return The index of the first occurrence of the target string in the array.
+     * @return The index of the first occurrence of the target string in the array. If the target string is not found,
+     * -1 is returned.
      */
     public static int findIndex(String[] array, String target) {
         for (int i = 0; i < array.length; i++) {