A-Packages

Divide classes into packages. Put all classes into duke package, organize commands in duke.command

src/main/java/Duke.java

@@ -1,7 +1,8 @@
-import java.util.*;
+import duke.*;
+import duke.command.Command;
+
 import java.io.*;
-import java.time.LocalDateTime;
-import java.time.format.DateTimeFormatter;
+
 public class Duke {
     private Ui ui;
     private Storage storage;

src/main/java/Deadline.java → src/main/java/duke/Deadline.java

@@ -1,3 +1,5 @@
+package duke;
+
 import java.time.LocalDateTime;
 public class Deadline extends Task {
 

src/main/java/Event.java → src/main/java/duke/Event.java

@@ -1,7 +1,9 @@
+package duke;
+
 import java.time.LocalDateTime;
 import java.util.concurrent.atomic.AtomicLong;
 
-public class Event extends Task{
+public class Event extends Task {
 
     protected LocalDateTime from;
     protected LocalDateTime to;

src/main/java/Parser.java → src/main/java/duke/Parser.java

@@ -1,3 +1,7 @@
+package duke;
+
+import duke.command.*;
+
 import java.time.LocalDateTime;
 import java.time.format.DateTimeFormatter;
 import java.util.NoSuchElementException;

src/main/java/Storage.java → src/main/java/duke/Storage.java

@@ -1,3 +1,8 @@
+package duke;
+
+import duke.Deadline;
+import duke.Event;
+
 import java.io.*;
 import java.time.LocalDateTime;
 import java.util.ArrayList;

src/main/java/Task.java → src/main/java/duke/Task.java

@@ -1,3 +1,5 @@
+package duke;
+
 public class Task {
     private boolean mark;
     private String value;

src/main/java/TaskList.java → src/main/java/duke/TaskList.java

@@ -1,3 +1,7 @@
+package duke;
+
+import duke.Task;
+
 import java.util.ArrayList;
 public class TaskList {
     private ArrayList<Task> list;

src/main/java/Todo.java → src/main/java/duke/Todo.java

@@ -1,3 +1,7 @@
+package duke;
+
+import duke.Task;
+
 public class Todo extends Task {
 
     public Todo(String value,boolean mark) {

src/main/java/Ui.java → src/main/java/duke/Ui.java

@@ -1,3 +1,7 @@
+package duke;
+
+import duke.Task;
+
 import java.io.BufferedReader;
 import java.io.IOException;
 import java.io.InputStreamReader;

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

@@ -1,6 +1,11 @@
-import java.io.FileWriter;
+package duke.command;
 
-public class AddCommand extends Command{
+import duke.Storage;
+import duke.TaskList;
+import duke.Ui;
+import duke.Task;
+
+public class AddCommand extends Command {
 
     private Task task;
 
@@ -9,7 +14,7 @@ public AddCommand(Task t) {
     }
 
     @Override
-    public void execute(TaskList tasks,Ui ui,Storage storage) {
+    public void execute(TaskList tasks, Ui ui, Storage storage) {
         tasks.add(this.task);
         ui.customMessage("Added : " + task.getValue());
         ui.showNumberOfListings(tasks.size());

src/main/java/duke/command/Command.java

@@ -0,0 +1,14 @@
+package duke.command;
+
+import duke.Storage;
+import duke.TaskList;
+import duke.Ui;
+
+public abstract class Command {
+
+    public abstract void execute(TaskList tasks, Ui ui, Storage storage);
+
+    public boolean isExit() {
+        return false;
+    }
+}

src/main/java/duke/command/CommandNotFound.java

@@ -0,0 +1,17 @@
+package duke.command;
+
+import duke.Storage;
+import duke.TaskList;
+import duke.Ui;
+
+public class CommandNotFound extends Command {
+
+    public CommandNotFound() {
+
+    }
+
+    @Override
+    public void execute(TaskList tasks, Ui ui, Storage storage) {
+        ui.showCommandNotFound();
+    }
+}

src/main/java/DeleteCommand.java → src/main/java/duke/command/DeleteCommand.java

@@ -1,4 +1,11 @@
-public class DeleteCommand extends Command  {
+package duke.command;
+
+import duke.Storage;
+import duke.TaskList;
+import duke.Ui;
+
+
+public class DeleteCommand extends Command {
 
     private int index;
 
@@ -7,7 +14,7 @@ public DeleteCommand(int index) {
     }
 
     @Override
-    public void execute(TaskList tasks,Ui ui,Storage storage) {
+    public void execute(TaskList tasks, Ui ui, Storage storage) {
         ui.customMessage("Noted. I've removed this task: \n");
         ui.customMessage(tasks.get(index));
         tasks.remove(index);

src/main/java/ExitCommand.java → src/main/java/duke/command/ExitCommand.java

@@ -1,3 +1,9 @@
+package duke.command;
+
+import duke.Storage;
+import duke.TaskList;
+import duke.Ui;
+
 public class ExitCommand extends Command {
 
     boolean exit;
@@ -9,7 +15,7 @@ public boolean isExit() {
         return this.exit;
     }
     @Override
-    public void execute(TaskList tasks,Ui ui,Storage storage) {
+    public void execute(TaskList tasks, Ui ui, Storage storage) {
         ui.byeMessage();
     }
 

src/main/java/duke/command/ListCommand.java

@@ -0,0 +1,16 @@
+package duke.command;
+
+import duke.Storage;
+import duke.TaskList;
+import duke.Ui;
+
+public class ListCommand extends Command {
+
+    public ListCommand() {
+
+    }
+    @Override
+    public void execute(TaskList tasks, Ui ui, Storage storage) {
+        tasks.print();
+    }
+}

src/main/java/MarkCommand.java → src/main/java/duke/command/MarkCommand.java

@@ -1,3 +1,9 @@
+package duke.command;
+
+import duke.Storage;
+import duke.TaskList;
+import duke.Ui;
+
 public class MarkCommand extends Command {
 
     private int index;
@@ -6,7 +12,7 @@ public MarkCommand(int index) {
         this.index = index - 1;
     }
     @Override
-    public void execute(TaskList tasks,Ui ui,Storage storage) {
+    public void execute(TaskList tasks, Ui ui, Storage storage) {
         tasks.mark(index);
         ui.showMarkSucess(tasks.get(index));
     }