Game/Assets/Scripts/BroomSweeping.cs

@@ -36,7 +36,7 @@ public class BroomSweeping : MonoBehaviour
         if (dirtSweepCount < dirtRequired || _taskCompleted) return;
 
         _taskCompleted = true;
-        GameManager.Instance.FloorSweptTaskComplete();
+        GameManager.Instance.FloorSweepedTaskComplete();
 
         storyPanelUI.SetActive(true);
         storyText.text = "I hope the house is clean enough now so I don't get scolded later...";

Game/Assets/Scripts/CanvasFade.cs

@@ -18,7 +18,7 @@ public class CanvasFade : MonoBehaviour
     {
         if (GameManager.Instance != null)
         {
-            var dayNumber = GameManager.Instance.CurrentDay;
+            var dayNumber = GameManager.Instance.currentDay;
             dayText.text = "Day " + dayNumber;
         }
 

Game/Assets/Scripts/DayIncrementPlaygroundScript.cs

@@ -32,11 +32,11 @@ public class DayIncrementPlaygroundScript : MonoBehaviour
             Debug.Log("initialisation wait...");
             yield return new WaitForSeconds(3);
 
-            Debug.Log($"hello vro, day is {GameManager.Instance.CurrentDay}");
+            Debug.Log($"hello vro, day is {GameManager.Instance.currentDay}");
 
             // get game manager and then increment the day
             GameManager.Instance.IncrementDay();
-            Debug.Log($"post-increment day is {GameManager.Instance.CurrentDay}");
+            Debug.Log($"post-increment day is {GameManager.Instance.currentDay}");
 
             // change to the next scene
             // ReSharper disable once Unity.LoadSceneDisabledSceneName
@@ -48,7 +48,7 @@ public class DayIncrementPlaygroundScript : MonoBehaviour
             Debug.Log("initialisation wait...");
             yield return new WaitForSeconds(3);
 
-            Debug.Log($"hello vro, day is {GameManager.Instance.CurrentDay}");
+            Debug.Log($"hello vro, day is {GameManager.Instance.currentDay}");
         }
 
         yield return 0;

Game/Assets/Scripts/GameManager.cs

@@ -1,13 +1,13 @@
 /*
- * Author: Reza, Wai Lam, Mark
+Author: Reza and Wai Lam
- * Date: 3/2/25
+Date: 3/2/25
- * Description: To keep track of tasks, which level the player is at, and game mechanics
+Description: To keep track of tasks, which level the player is at, and game mechanics
- */
+*/
 
 using System.Collections;
 using System.Collections.Generic;
-using TMPro;
 using UnityEngine;
+using TMPro;
 using UnityEngine.SceneManagement;
 
 public class GameManager : MonoBehaviour
@@ -17,28 +17,51 @@ public class GameManager : MonoBehaviour
     /// </summary>
     public static GameManager Instance;
 
+    // current day, publicly readable, privately settable
+
+    // for public access, setting is a no-op
+    public int currentDay { get; private set; } = 1;
+
+    // Tracks GoToSchool task status
+    private bool goToSchool = false;
+    private bool bedroomCleaned = false;
+    private bool teethBrushed = false;
+    private bool floorSweeped = false;
+
+    private string lastSceneName;
+
+    private bool hasIncrementedToday = false;
+
     // Defines UI references
     [Header("UI References")] public GameObject storyPanelUI;
     public TMP_Text storyText;
 
     // Queue for managing messages
-    private readonly Queue<string> _messageQueue = new();
+    private Queue<string> messageQueue = new Queue<string>();
-    private bool _bedroomCleaned;
+    private bool isMessageActive = false;
-    private bool _floorSwept;
 
-    // Tracks GoToSchool task status
+    /// <summary>
-    private bool _goToSchool;
+    ///     Checks if tasks are completed
+    /// </summary>
+    public bool IsBedroomCleaned()
+    {
+        return bedroomCleaned;
+    }
 
-    private bool _hasIncrementedToday;
+    public bool IsTeethBrushed()
-    private bool _isMessageActive;
+    {
+        return teethBrushed;
+    }
 
-    private string _lastSceneName;
+    public bool IsFloorSweeped()
-    private bool _teethBrushed;
+    {
+        return floorSweeped;
+    }
 
-    // current day, publicly readable, privately settable
+    public bool IsGoToSchool()
-
+    {
-    // for public access, setting is a no-op
+        return goToSchool;
-    public int CurrentDay { get; private set; } = 1;
+    }
 
     /// <summary>
     ///     Enforces singleton behavior; sets doesn't destroy on load and checks for multiple instances
@@ -62,57 +85,39 @@ public class GameManager : MonoBehaviour
         }
 
         Debug.Log($"game manager ({GetInstanceID()}) is initialising itself!");
-        CurrentDay = 1;
+        currentDay = 1;
 
         // Try to find UI elements if not set
-        if (storyPanelUI == null) storyPanelUI = GameObject.Find("StoryPanelUI");
+        if (storyPanelUI == null)
+        {
+            storyPanelUI = GameObject.Find("StoryPanelUI");
+        }
 
-        if (storyText == null) storyText = GameObject.Find("StoryText").GetComponent<TMP_Text>();
+        if (storyText == null)
+        {
+            storyText = GameObject.Find("StoryText").GetComponent<TMP_Text>();
+        }
 
         SceneManager.sceneLoaded += OnSceneLoaded;
     }
 
     // Update is called once per frame
-    private void Update()
+    void Update()
     {
         // Continuously check and display queued messages
-        if (!_isMessageActive && _messageQueue.Count > 0)
+        if (!isMessageActive && messageQueue.Count > 0)
         {
-            var nextMessage = _messageQueue.Dequeue();
+            string nextMessage = messageQueue.Dequeue();
             StartCoroutine(DisplayMessage(nextMessage));
         }
     }
 
-    /// <summary>
-    ///     Checks if tasks are completed
-    /// </summary>
-    public bool IsBedroomCleaned()
-    {
-        return _bedroomCleaned;
-    }
-
-    public bool IsTeethBrushed()
-    {
-        return _teethBrushed;
-    }
-
-    public bool IsFloorSwept()
-    {
-        return _floorSwept;
-    }
-
-    public bool IsGoToSchool()
-    {
-        return _goToSchool;
-    }
-
     /// <summary>
     ///     Queues a message to be displayed
     /// </summary>
-    // ReSharper disable once MemberCanBePrivate.Global
     public void QueueMessage(string message)
     {
-        _messageQueue.Enqueue(message);
+        messageQueue.Enqueue(message);
     }
 
     /// <summary>
@@ -120,82 +125,82 @@ public class GameManager : MonoBehaviour
     /// </summary>
     private IEnumerator DisplayMessage(string message)
     {
-        _isMessageActive = true;
+        isMessageActive = true;
         storyPanelUI.SetActive(true);
         storyText.text = message;
         yield return new WaitForSeconds(7f); // Wait for 7 seconds before hiding
         storyPanelUI.SetActive(false);
-        _isMessageActive = false;
+        isMessageActive = false;
     }
 
-    // log the players choices before leaving the house (for future Firebase tracking)
+    // Logs player's choices before leaving the house (for future Firebase tracking)
     public void LogPlayerChoices()
     {
         Debug.Log("Player is trying to leave the house. Task Completion Status:");
-        Debug.Log("Bedroom Cleaned: " + _bedroomCleaned);
+        Debug.Log("Bedroom Cleaned: " + bedroomCleaned);
-        Debug.Log("Teeth Brushed: " + _teethBrushed);
+        Debug.Log("Teeth Brushed: " + teethBrushed);
-        Debug.Log("Floor Swept: " + _floorSwept);
+        Debug.Log("Floor Sweeped: " + floorSweeped);
-        Debug.Log("Go To School: " + _goToSchool);
+        Debug.Log("Go To School: " + goToSchool);
     }
 
     // Checks if all tasks are done before player can go to school
-    // ReSharper disable once MemberCanBePrivate.Global
     public void AreTasksDone()
     {
-        if (_bedroomCleaned && _teethBrushed && _floorSwept)
+        if (bedroomCleaned && teethBrushed && floorSweeped)
+        {
             QueueMessage("I think I did everything... I think I can leave for school now");
+        }
     }
+    
+    // mark : u can track whether they want to do their tasks, some people may be unmoticvated to 
+    // do the tasks in game, then we can ask them irl why they didnt do the task
 
-    // mark: u can track whether they want to do their tasks, some people may be unmotivated to 
+    // Tracks if bedroom is cleaned or not
-    // do the tasks in game, then we can ask them irl why they didn't do the task
-
-    // Tracks if the bedroom is cleaned or not
     public void BedroomTaskComplete()
     {
-        _bedroomCleaned = true;
+        bedroomCleaned = true;
         AreTasksDone();
     }
 
-    // Tracks if teeth are brushed or not
+    // Tracks if teeth is brushed or not
     public void BrushTeethTaskComplete()
     {
-        _teethBrushed = true;
+        teethBrushed = true;
         AreTasksDone();
     }
 
-    // Tracks if the floor has been swept or not
+    // Tracks if floor is sweeped or not
-    public void FloorSweptTaskComplete()
+    public void FloorSweepedTaskComplete()
     {
-        _floorSwept = true;
+        floorSweeped = true;
         AreTasksDone();
     }
 
     public void GoToSchoolTaskComplete()
     {
-        _goToSchool = true;
+        goToSchool = true;
     }
 
     // Increments the current day by 1
     public void IncrementDay()
     {
-        if (_hasIncrementedToday) return; // Prevents multiple increments
+        if (hasIncrementedToday) return; // Prevents multiple increments
-        _hasIncrementedToday = true;
+        hasIncrementedToday = true;
-
+        
-        CurrentDay++;
+        currentDay++;
-        Debug.Log("Day incremented to: " + CurrentDay);
+        Debug.Log("Day incremented to: " + currentDay);
-        if (CurrentDay > 3) LoadCallingScene();
+        if (currentDay > 3) LoadCallingScene();
     }
 
-    private void OnSceneLoaded(Scene scene, LoadSceneMode mode)
+    void OnSceneLoaded(Scene scene, LoadSceneMode mode)
     {
-        _hasIncrementedToday = false; // Allows the day to be incremented again in the next transition
+        hasIncrementedToday = false; // Allows the day to be incremented again in the next transition
     }
 
     // Loads the callingChoice scene when Day 3 is completed
     private void LoadCallingScene()
     {
         Debug.Log("Loading Calling Scene: callingChoice");
-        Debug.LogError("not ready yet");
+        SceneManager.LoadScene("CallingChoice");
-        // FIXME: SceneManager.LoadScene("CallingChoice");
     }
 }