fixed anagram puzzle not saving in the game

CSE-AashrithA · CSE-AashrithA · commit 1ae7137902ac · 2025-11-23T18:41:43.000-05:00
diff --git a/src/main/java/com/escapegame/AnagramPuzzleController.java b/src/main/java/com/escapegame/AnagramPuzzleController.java
@@ -6,9 +6,13 @@
 import java.io.IOException;
 import java.io.OutputStream;
 import java.net.URL;
+import java.text.Normalizer;
+import java.util.Locale;
 import java.util.Properties;
 import java.util.ResourceBundle;
 
+import com.model.User;
+
 import javafx.fxml.FXML;
 import javafx.fxml.Initializable;
 import javafx.scene.control.Alert;
@@ -22,9 +26,12 @@
 
 /**
  * Anagram puzzle controller — scrambled LPAEP -> APPLE (category Fruit)
- * Manages the anagram puzzle UI: background loading, attempts/hints state,
- * simple persistence to a properties file, and navigation on solve/quit.
- * UI references are checked for null to be defensive when used from FXML.
+ * Improvements:
+ *  - per-user save file (.escapegame_anagram_<userid>.properties)
+ *  - debug prints showing the save path and loaded state
+ *  - safe parsing of integer properties
+ *  - normalization of user input for robust matching
+ *  - clearSaveForCurrentUser() helper for testing
  */
 public class AnagramPuzzleController implements Initializable {
 
@@ -54,16 +61,9 @@ public class AnagramPuzzleController implements Initializable {
         "apple fruit"
     };
 
-    private final File SAVE_FILE = new File(System.getProperty("user.home"), ".escapegame_anagram.properties");
-
     private static final String RESOURCE_PATH = "/images/background.png";
     private static final String DEV_FALLBACK = "file:/mnt/data/Screenshot 2025-11-22 202825.png";
-     /**
-     * Initialize the puzzle screen: load background image (resource then fallback),
-     * set initial UI labels, draw hearts and restore saved progress.
-     * @param location  not used
-     * @param resources not used
-     */
+
     @Override
     public void initialize(URL location, ResourceBundle resources) {
         System.out.println("AnagramPuzzleController.initialize() start");
@@ -73,7 +73,7 @@ public void initialize(URL location, ResourceBundle resources) {
             URL res = getClass().getResource(RESOURCE_PATH);
             if (res != null) {
                 Image img = new Image(res.toExternalForm());
-                backgroundImage.setImage(img);
+                if (backgroundImage != null) backgroundImage.setImage(img);
                 loaded = true;
             }
         } catch (Exception ex) {
@@ -83,7 +83,7 @@ public void initialize(URL location, ResourceBundle resources) {
         if (!loaded) {
             try {
                 Image img = new Image(DEV_FALLBACK);
-                backgroundImage.setImage(img);
+                if (backgroundImage != null) backgroundImage.setImage(img);
                 loaded = true;
             } catch (Exception ex) {
                 System.err.println("Error loading dev fallback image: " + ex.getMessage());
@@ -104,19 +104,73 @@ public void initialize(URL location, ResourceBundle resources) {
         if (hintsLabel != null) hintsLabel.setText(hintsLeft + " hint(s) available");
         if (scrambledLabel != null) scrambledLabel.setText("Scrambled: LPAEP");
         if (categoryLabel != null) categoryLabel.setText("Category: Fruit");
-        if (promptLabel != null) promptLabel.setText("Prompt: Unscramble the letters to find the word");
+        if (promptLabel != null) promptLabel.setText("Prompt: Unscramble the letters to find the word: LPAEP");
         refreshHearts();
+
+        // debug: show exactly which save file will be used
+        System.out.println("DEBUG: Anagram save path -> " + getSaveFileForCurrentUser().getAbsolutePath());
+
         loadSave();
 
+        // If save marked solved=true, automatically clear it (one-time reset) and re-enable UI
+        if (solved) {
+            System.out.println("DEBUG: Saved state indicates solved=true — clearing save to reset puzzle.");
+            clearSaveForCurrentUser();
+            solved = false;
+            attemptsLeft = 3;
+            hintsLeft = 3;
+            nextHintIndex = 0;
+            if (hintsLabel != null) hintsLabel.setText(hintsLeft + " hint(s) available");
+            if (statusLabel != null) statusLabel.setText("");
+            if (btnSubmit != null) btnSubmit.setDisable(false);
+            if (btnHint != null) btnHint.setDisable(false);
+            if (answerField != null) answerField.setDisable(false);
+            refreshHearts();
+        }
+
         System.out.println("AnagramPuzzleController.initialize() done");
     }
-    /**
-     * Draw hearts representing remaining attempts and disable inputs when none remain.
-     */
+
+    private File getSaveFileForCurrentUser() {
+        String userId = "guest";
+        try {
+            User u = null;
+            try {
+                u = App.getCurrentUser();
+            } catch (Throwable t) {
+                // fallback to reflection if needed
+                try {
+                    java.lang.reflect.Method gu = App.class.getMethod("getCurrentUser");
+                    Object userObj = gu.invoke(null);
+                    if (userObj instanceof User) {
+                        u = (User) userObj;
+                    } else if (userObj != null) {
+                        try {
+                            java.lang.reflect.Method getId = userObj.getClass().getMethod("getUserId");
+                            Object idObj = getId.invoke(userObj);
+                            if (idObj != null) {
+                                final String idStr = idObj.toString();
+                                u = new User() { @Override public String getUserId() { return idStr; } };
+                            }
+                        } catch (Throwable ignored) { }
+                    }
+                } catch (Throwable ignored) { }
+            }
+            if (u != null && u.getUserId() != null && !u.getUserId().trim().isEmpty()) {
+                userId = u.getUserId().trim();
+            }
+        } catch (Throwable t) {
+            // ignore, use guest
+        }
+        String clean = userId.replaceAll("\\s+", "_").toLowerCase(Locale.ROOT);
+        String filename = ".escapegame_anagram_" + clean + ".properties";
+        return new File(System.getProperty("user.home"), filename);
+    }
+
     private void refreshHearts() {
         if (heartsBox == null) return;
         heartsBox.getChildren().clear();
-        for (int i = 0; i < attemptsLeft; i++) {
+        for (int i = 0; i < Math.max(0, attemptsLeft); i++) {
             Label heart = new Label("\u2764");
             heart.setStyle("-fx-text-fill: #ff4d6d; -fx-font-size: 20px;");
             heartsBox.getChildren().add(heart);
@@ -126,26 +180,24 @@ private void refreshHearts() {
             if (answerField != null) answerField.setDisable(true);
         }
     }
-    /**
-     * Called when the user submits an answer. Validates input, checks accepted
-     * answers, updates state, shows alerts, saves progress and navigates on success.
-     */
+
     @FXML
     private void onSubmit() {
         if (solved) {
             if (statusLabel != null) statusLabel.setText("You already solved the puzzle.");
             return;
         }
 
-        String answer = (answerField == null || answerField.getText() == null) ? "" : answerField.getText().trim().toLowerCase();
+        String raw = (answerField == null || answerField.getText() == null) ? "" : answerField.getText();
+        String answer = normalize(raw);
         if (answer.isEmpty()) {
             if (statusLabel != null) statusLabel.setText("Please enter an answer.");
             return;
         }
 
         boolean ok = false;
         for (String a : ACCEPTED_ANSWERS) {
-            if (a.equals(answer)) { ok = true; break; }
+            if (normalize(a).equals(answer)) { ok = true; break; }
         }
 
         if (ok) {
@@ -158,7 +210,7 @@ private void onSubmit() {
             saveProgress();
             try { App.setRoot("opened5"); } catch (IOException e) { e.printStackTrace(); }
         } else {
-            attemptsLeft--;
+            attemptsLeft = Math.max(0, attemptsLeft - 1);
             refreshHearts();
             if (attemptsLeft <= 0) {
                 if (statusLabel != null) statusLabel.setText("No attempts left. The correct answer was: \"apple\".");
@@ -171,9 +223,7 @@ private void onSubmit() {
             }
         }
     }
-    /**
-     * Shows the next hint (if available), updates hint counters and saves progress.
-     */
+
     @FXML
     private void onHint() {
         if (solved) {
@@ -186,62 +236,63 @@ private void onHint() {
         }
 
         String hint = HINTS[Math.min(nextHintIndex, HINTS.length - 1)];
-        nextHintIndex++;
-        hintsLeft--;
+        nextHintIndex = Math.min(nextHintIndex + 1, HINTS.length);
+        hintsLeft = Math.max(0, hintsLeft - 1);
         if (hintsLabel != null) hintsLabel.setText(hintsLeft + " hint(s) available");
         new Alert(Alert.AlertType.INFORMATION, hint).showAndWait();
         saveProgress();
     }
-    /** Saves progress to the user's properties file and updates the status label. */
+
     @FXML
     private void onSave() {
         if (statusLabel != null) statusLabel.setText(saveProgress() ? "Progress saved." : "Save failed.");
     }
-    /** Quit action: navigate back to the previous/opened4 screen. */
+
     @FXML
     private void onQuit() {
-        try { App.setRoot("opened4");
-        } catch (IOException e) { 
-            e.printStackTrace(); 
-        }
+        try { App.setRoot("opened4"); } catch (IOException e) { e.printStackTrace(); }
     }
-    /**
-     * Persist puzzle state to a properties file under the user's home directory.
-     * @return true if save succeeded, false otherwise
-     */
+
     private boolean saveProgress() {
         try {
             Properties p = new Properties();
             p.setProperty("attemptsLeft", String.valueOf(attemptsLeft));
             p.setProperty("hintsLeft", String.valueOf(hintsLeft));
             p.setProperty("solved", String.valueOf(solved));
             p.setProperty("nextHintIndex", String.valueOf(nextHintIndex));
-            try (OutputStream os = new FileOutputStream(SAVE_FILE)) {
+            File out = getSaveFileForCurrentUser();
+            try (OutputStream os = new FileOutputStream(out)) {
                 p.store(os, "Anagram puzzle save");
             }
+            System.out.println("DEBUG saved -> " + out.getAbsolutePath());
             return true;
         } catch (Exception e) {
             System.err.println("Save error: " + e.getMessage());
             return false;
         }
     }
-    /**
-     * Load saved puzzle state from properties file if present and update UI.
-     */
+
     private void loadSave() {
         try {
-            if (!SAVE_FILE.exists()) return;
+            File f = getSaveFileForCurrentUser();
+            if (!f.exists()) {
+                System.out.println("DEBUG: save file not found -> " + f.getAbsolutePath());
+                return;
+            }
             Properties p = new Properties();
-            try (FileInputStream fis = new FileInputStream(SAVE_FILE)) {
+            try (FileInputStream fis = new FileInputStream(f)) {
                 p.load(fis);
             }
-            attemptsLeft = Integer.parseInt(p.getProperty("attemptsLeft", "3"));
-            hintsLeft = Integer.parseInt(p.getProperty("hintsLeft", "3"));
-            solved = Boolean.parseBoolean(p.getProperty("solved", "false"));
-            nextHintIndex = Integer.parseInt(p.getProperty("nextHintIndex", "0"));
+            attemptsLeft = getSafeInt(p.getProperty("attemptsLeft"), attemptsLeft);
+            hintsLeft = getSafeInt(p.getProperty("hintsLeft"), hintsLeft);
+            solved = Boolean.parseBoolean(p.getProperty("solved", String.valueOf(solved)));
+            nextHintIndex = getSafeInt(p.getProperty("nextHintIndex"), nextHintIndex);
             if (hintsLabel != null) hintsLabel.setText(hintsLeft + " hint(s) available");
             refreshHearts();
 
+            System.out.println("DEBUG loaded - attemptsLeft=" + attemptsLeft
+                + " hintsLeft=" + hintsLeft + " nextHintIndex=" + nextHintIndex + " solved=" + solved
+                + " (from " + f.getAbsolutePath() + ")");
             if (solved) {
                 if (btnSubmit != null) btnSubmit.setDisable(true);
                 if (answerField != null) answerField.setDisable(true);
@@ -252,4 +303,39 @@ private void loadSave() {
             System.err.println("Load save failed: " + e.getMessage());
         }
     }
+
+    private int getSafeInt(String s, int fallback) {
+        if (s == null) return fallback;
+        try {
+            return Integer.parseInt(s.trim());
+        } catch (NumberFormatException ex) {
+            System.err.println("Invalid integer in save file: \"" + s + "\"; using fallback " + fallback);
+            return fallback;
+        }
+    }
+
+    private static String normalize(String raw) {
+        if (raw == null) return "";
+        String n = Normalizer.normalize(raw, Normalizer.Form.NFKC)
+                .toLowerCase(Locale.ROOT)
+                .replaceAll("[^\\p{L}\\p{N}\\s]", " ")
+                .trim()
+                .replaceAll("\\s+", " ");
+        if (n.startsWith("a ")) n = n.substring(2).trim();
+        else if (n.startsWith("an ")) n = n.substring(3).trim();
+        else if (n.startsWith("the ")) n = n.substring(4).trim();
+        return n;
+    }
+
+    private boolean clearSaveForCurrentUser() {
+        File f = getSaveFileForCurrentUser();
+        if (f.exists()) {
+            boolean deleted = f.delete();
+            System.out.println("DEBUG: clearSaveForCurrentUser() deleted=" + deleted + " -> " + f.getAbsolutePath());
+            return deleted;
+        } else {
+            System.out.println("DEBUG: clearSaveForCurrentUser() not found -> " + f.getAbsolutePath());
+            return true;
+        }
+    }
 }
