Better code quality

tahsinhasem · Mar 3, 2024 · 1066bae · 1066bae
1 parent c6fbb9b
commit 1066bae
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 4 deletions.
diff --git a/src/main/java/dude/Dude.java b/src/main/java/dude/Dude.java
@@ -48,7 +48,7 @@ public String getResponse(String input) {
         Command c = Parser.parse(input, taskList);
         String response = executeCommand(c);
         try {
-            saveToDisk();
+            saveTaskListToDisk();
         } catch (IOException e) {
             return STORAGE_SAVING_ERROR_MESSAGE;
         }
@@ -63,7 +63,7 @@ private static String executeCommand(Command command) {
         }
     }
 
-    private void saveToDisk() throws IOException, SecurityException {
+    private void saveTaskListToDisk() throws IOException, SecurityException {
         this.storage.saveTasks(taskList);
     }
 

diff --git a/src/main/java/dude/gui/DialogBox.java b/src/main/java/dude/gui/DialogBox.java
@@ -6,14 +6,14 @@
 
 import javafx.collections.FXCollections;
 import javafx.collections.ObservableList;
-import javafx.scene.control.Label;
-import javafx.scene.layout.HBox;
 import javafx.fxml.FXML;
 import javafx.fxml.FXMLLoader;
 import javafx.geometry.Pos;
 import javafx.scene.Node;
+import javafx.scene.control.Label;
 import javafx.scene.image.Image;
 import javafx.scene.image.ImageView;
+import javafx.scene.layout.HBox;
 
 // @@author Jeffry Lum
 // Solution below is reused from https://se-education.org/guides/tutorials/javaFxPart4.html

diff --git a/src/main/java/dude/gui/MainView.java b/src/main/java/dude/gui/MainView.java
@@ -31,6 +31,9 @@ public class MainView extends AnchorPane {
     private Image dudeImage;
     private Dude dude;
 
+    /**
+     * Initializes the MainView, setting up the images and the Dude object.
+     */
     @FXML
     public void initialize() {
         this.dude = new Dude("data/tasks.ser");