fixed file save bug

config/checkstyle/suppressions.xml

@@ -0,0 +1,10 @@
+<?xml version="1.0"?>
+
+<!DOCTYPE suppressions PUBLIC
+        "-//Checkstyle//DTD SuppressionFilter Configuration 1.2//EN"
+        "https://checkstyle.org/dtds/suppressions_1_2.dtd">
+
+<suppressions>
+    <suppress checks="JavadocType" files=".*Test\.java"/>
+    <suppress checks="MissingJavadocMethodCheck" files=".*Test\.java"/>
+</suppressions>

data.txt

@@ -1,2 +1,3 @@
 T;0;readbook
 D;0;return book ;2/12/2019 1800
+T;0;week 4 ip

src/main/java/Duke.java

@@ -1,10 +1,11 @@
+import java.io.FileNotFoundException;
+
 import duke.commands.Command;
 import duke.utils.Parser;
 import duke.utils.Storage;
 import duke.utils.TaskList;
 import duke.utils.Ui;
 
-import java.io.FileNotFoundException;
 
 /**
  * Command Line Task Manager called Duke
@@ -22,32 +23,35 @@ public class Duke {
     /**
      * Constructor that instantiates Duke
      */
-    public Duke(){
+    public Duke() {
         ui = new Ui();
         parser = new Parser();
         this.storage = new Storage();
-        try{
+        try {
             taskList = storage.loadTaskList();
-        } catch(FileNotFoundException e){
+        } catch (FileNotFoundException e) {
             System.out.println(e.getMessage());
         }
 
     }
 
-    public void run(){
+    /**
+     * Method to start running the application
+     */
+    public void run() {
         ui.start();
         boolean isExit = false;
 
-        while(!isExit){
-            try{
+        while (!isExit) {
+            try {
                 String fullCommand = ui.readCommand();
                 ui.showLine(); // show the divider line ("_______")
                 Command c = parser.parseInput(fullCommand);
                 c.execute(taskList, ui, storage);
                 isExit = c.isExit();
 
 
-            } catch (Exception e){
+            } catch (Exception e) {
                 ui.showError(e);
             } finally {
                 ui.showLine();
@@ -61,7 +65,7 @@ public void run(){
      * Commences Duke
      * @param args CLI arguments
      */
-    public static void main(String[] args){
+    public static void main(String[] args) {
 
         new Duke().run();
 

src/main/java/duke/commands/Command.java

@@ -3,6 +3,8 @@
 import duke.exceptions.DukeException;
 import duke.utils.*;
 
+import java.io.IOException;
+
 /**
  * Abstract class that represents a Command in the application
  */
@@ -15,7 +17,7 @@ public abstract class Command {
      * @param storage the Storage object that is being used in the app
      * @throws DukeException
      */
-    public abstract void execute(TaskList taskList, Ui ui, Storage storage) throws DukeException;
+    public abstract void execute(TaskList taskList, Ui ui, Storage storage) throws DukeException, IOException;
 
     /**
      * Returns whether or not this command is an exit command

src/main/java/duke/commands/ExitCommand.java

@@ -2,14 +2,17 @@
 
 import duke.utils.*;
 
+import java.io.IOException;
+
 /**
  * Class that is a subclass of Command class
  * and handles the behaviour of the Command for
  * when exiting the program
  */
-public class ExitCommand extends Command{
+public class ExitCommand extends Command {
     @Override
-    public void execute(TaskList task, Ui ui, Storage storage){
+    public void execute(TaskList tasks, Ui ui, Storage storage) throws IOException {
+        storage.saveTaskListToDisk(tasks);
         ui.end();
     }
 

src/main/java/duke/utils/Storage.java

@@ -37,22 +37,22 @@ public TaskList loadTaskList() throws FileNotFoundException {
         File f = new File(dataFilePath);
         ArrayList<Task> taskList = new ArrayList<Task>();
 
-        if(!f.exists()){
+        if (!f.exists()) {
             return new TaskList(new ArrayList<Task>());
         }
 
-        try{
+        try {
             Scanner dataReader = new Scanner(f);
 
-            while(dataReader.hasNextLine()){
+            while (dataReader.hasNextLine()) {
                 // convert saved info to Task
                 String currLine = dataReader.nextLine();
                 Task currTask = convertStringToTask(currLine);
                 taskList.add(currTask);
             }
             dataReader.close();
 
-        } catch (FileNotFoundException e){
+        } catch (FileNotFoundException e) {
             System.out.println("File unable to load");
         }
 
@@ -67,34 +67,36 @@ public TaskList loadTaskList() throws FileNotFoundException {
      * @param taskList tasklist
      * @throws IOException
      */
-    public void saveTaskListToDisk(TaskList taskList) throws IOException{
-
+    public void saveTaskListToDisk(TaskList taskList) throws IOException {
         File f = new File(dataFilePath);
-        if(!f.exists()) {
-            try{
+        if (!f.exists()) {
+            try {
                 boolean created = f.createNewFile();
             } catch (IOException e) {
                 System.out.println("Error creating file");
             }
         }
 
-        try{
+        try {
             FileWriter writeTasks = new FileWriter(dataFilePath);
             String allTasksString = "";
 
-            for(int i = 0; i < taskList.numberOfTasks(); i++){
+            for (int i = 0; i < taskList.numberOfTasks(); i++) {
                 Task currTask = taskList.getTask(i);
                 String taskString = convertTaskToString(currTask);
                allTasksString += taskString + "\n";
+                System.out.println("written" + taskString);
 
             }
 
             writeTasks.write(allTasksString);
             writeTasks.flush();
             writeTasks.close();
+            System.out.println("I have saved your tasks.");
+
 
 
-        } catch (IOException e){
+        } catch (IOException e) {
             System.out.println("Error creating file");
         }
 
@@ -107,7 +109,7 @@ public void saveTaskListToDisk(TaskList taskList) throws IOException{
      * @param task
      * @return String representation of the task
      */
-    public String convertTaskToString(Task task){
+    public String convertTaskToString(Task task) {
 
 
         // Example storage in file:
@@ -160,7 +162,7 @@ public Task convertStringToTask(String line){
 
         Task currTask;
 
-        switch (taskType){
+        switch (taskType) {
             case TODO:
                 currTask = new TodoTask(lineSplit[2]);
                 break;