Refactor usage of level file

Author: Arun Prasaad

src/main/java/ihm/Editor.java

@@ -1,27 +1,29 @@
 package ihm;
 
 import java.util.logging.Logger;
-import java.util.logging.Level;
 import java.awt.Color;
 import java.awt.Dimension;
 import java.awt.event.ActionEvent;
 import java.awt.event.ActionListener;
 import java.awt.event.MouseEvent;
 import java.awt.event.MouseListener;
 import java.awt.event.MouseMotionListener;
-import java.io.BufferedWriter;
 import java.io.File;
-import java.io.FileWriter;
 import java.io.IOException;
+import java.nio.file.StandardOpenOption;
 
 import com.google.common.annotations.VisibleForTesting;
 import javax.swing.*;
 
 import logic.TileType;
 import logic.Controller;
 import logic.LevelFile;
+import one.util.streamex.IntStreamEx;
 
 public class Editor extends JFrame implements MouseListener, MouseMotionListener {
+
+    private final LevelFile levelFile;
+
     @VisibleForTesting
     enum Component {
         // Tool buttons
@@ -63,18 +65,13 @@ TileType getContent() {
 
 	public Editor (int rowCount, int columnCount, String name) throws IOException  {
 
-		FileWriter levelWriter = new FileWriter(new File(new File(".").getCanonicalPath() + "/levels/" + name + ".txt"));
-		BufferedWriter lowerWriter = new BufferedWriter(levelWriter);
-		for (int i = 0; i < rowCount; i++ ) {
-			lowerWriter.write("_".repeat(columnCount));
-			lowerWriter.newLine();
-		}
-		lowerWriter.close();
-		levelWriter.close();
+        levelFile = LevelFile.of(name);
+        initializeEmptyLevel(rowCount, columnCount);
 
-		controller = new Controller(new File(new File(".").getCanonicalPath() + "/levels/" + name + ".txt").getPath());
+        controller = new Controller(new File(new File(".").getCanonicalPath() + "/levels/" + name + ".txt").getPath());
 		windowWidth = controller.warehouse.getColumns() * TILE_SIZE;
         windowHeight = controller.warehouse.getLines() * TILE_SIZE;
+
         this.setDefaultCloseOperation( JFrame.EXIT_ON_CLOSE );
         this.setTitle("Sokoban v1.0 par Gabriel FARAGO");
         this.setPreferredSize(new Dimension(windowWidth + 150, Math.max(windowHeight + 150, 330)));
@@ -171,34 +168,16 @@ public void actionPerformed(ActionEvent e) {
 						}
 
 						if (line.length() == columnCount && i+1 == columnCount) {
-							try {
-								FileWriter levelWriter = new FileWriter(new File(new File(".").getCanonicalPath() + "/levels/" + name + ".txt"));
-								BufferedWriter lowerWriter = new BufferedWriter(levelWriter);
-								lowerWriter.write(line);
-								lowerWriter.close();
-								levelWriter.close();
-								line = "";
-							} catch (IOException e1) {
-								LOGGER.log(Level.SEVERE, "Error while writing level file: " + name, e1);
-							}
+							LevelFile.of(name).write(line);
+							line = "";
 						}
 						else if (line.length() == columnCount) {
-							try {
-								FileWriter levelWriter = new FileWriter(new File(new File(".").getCanonicalPath() + "/levels/" + name + ".txt"), true);
-								BufferedWriter lowerWriter = new BufferedWriter(levelWriter);
-								lowerWriter.newLine();
-								lowerWriter.write(line);
-								lowerWriter.close();
-								levelWriter.close();
-								line = "";
-							} catch (IOException e1) {
-								LOGGER.log(Level.SEVERE, "Error while writing level file: " + name, e1);
-							}
+							LevelFile.of(name).write(System.lineSeparator() + line, StandardOpenOption.APPEND);
+							line = "";
 						}
 					}
 					dispose();
                     new HomeWindow();
-
                 }
 				else {
 					invalid_level.setText("Invalid level!");
@@ -222,6 +201,14 @@ else if (line.length() == columnCount) {
         this.setVisible( true );
 	}
 
+    private void initializeEmptyLevel(int rowCount, int columnCount) {
+        String content = IntStreamEx
+                .range(rowCount)
+                .mapToObj(i -> "_".repeat(columnCount))
+                .joining(System.lineSeparator());
+        levelFile.write(content);
+    }
+
     private void createQuitButton(String name) {
         JButton quit = new JButton("Quit");
         quit.setName(Component.QUIT_BUTTON.name());

src/main/java/logic/LevelFile.java

@@ -1,27 +1,23 @@
 package logic;
 
+import org.jspecify.annotations.NullMarked;
+
 import java.io.File;
 import java.io.IOException;
 import java.nio.file.Files;
+import java.nio.file.OpenOption;
 import java.nio.file.Path;
 import java.util.logging.Level;
 import java.util.logging.Logger;
 
-/**
- * Thrown when an error occurs during level file operations.
- */
-class LevelFileException extends RuntimeException {
-    public LevelFileException(String message, Throwable cause) {
-        super(message, cause);
-    }
-}
-
 /**
  * Simple utility class for handling level file paths.
  */
+@NullMarked
 public class LevelFile {
     private static final Logger LOGGER = Logger.getLogger(LevelFile.class.getName());
     private static final String LEVELS_DIR = "levels";
+
     /**
      * Creates a new LevelFile instance for the given level name.
      * @param levelName the name of the level (without .txt extension)
@@ -45,6 +41,21 @@ public boolean delete() {
         }
     }
 
+    /**
+     * Writes content to the level file with the specified options.
+     * @param content The content to write to the file
+     * @param options Options specifying how the file is opened
+     */
+    public void write(String content, OpenOption... options) {
+        try {
+            Files.createDirectories(getLevelsDirectory());
+            Files.writeString(getFilePath(), content, options);
+        } catch (IOException e) {
+            String errorMsg = "Error while writing to level file: %s".formatted(levelName);
+            LOGGER.log(Level.SEVERE, errorMsg, e);
+        }
+    }
+
     private final String levelName;
 
     private LevelFile(String levelName) {