Game/Assets/Scripts/BrushTeeth.cs

@@ -5,39 +5,42 @@ Description: Bathroom interaction
 */
 
 using System.Collections;
-using TMPro;
+using System.Collections.Generic;
 using UnityEngine;
 using UnityEngine.UI;
 using UnityEngine.XR.Interaction.Toolkit;
 using UnityEngine.XR.Interaction.Toolkit.Interactables;
 using UnityEngine.XR.Interaction.Toolkit.Interactors;
+using TMPro;
 
 public class BrushTeeth : MonoBehaviour
 {
+    private GameManager gameManager;
+    
     public Slider progressBar; // Reference to the Slider (progress bar)
     public float progressTime = 5f; // Time for the progress bar to complete
+    private XRGrabInteractable grabInteractable;
+    private float timer = 0f;
+    private bool isGrabbing = false;
+    
+    private bool taskCompleted = false;
     
     // Defines UI references
-    [Header("UI References")] public GameObject storyPanelUI;
+    [Header("UI References")]
 
+    public GameObject storyPanelUI;
     public TMP_Text storyText;
     
     // Defines Audio References
-    [Header("Audio References")] public AudioSource audioSource;
+    [Header("Audio References")]
-
+    public AudioSource audioSource;
     public AudioClip brushingSound;
-    private GameManager _gameManager;
-    private XRGrabInteractable _grabInteractable;
-    private bool _isGrabbing;
 
-    private bool _taskCompleted;
+    void Start()
-    private float _timer;
-
-    private void Start()
     {
-        _grabInteractable = GetComponent<XRGrabInteractable>();
+        grabInteractable = GetComponent<XRGrabInteractable>();
 
-        if (_grabInteractable == null)
+        if (grabInteractable == null)
         {
             Debug.LogError("XRGrabInteractable component not found on the object!");
             return;
@@ -47,72 +50,77 @@ public class BrushTeeth : MonoBehaviour
         progressBar.gameObject.SetActive(false);
 
         // Subscribe to grab and release events
-        _grabInteractable.selectEntered.AddListener(OnGrab);
+        grabInteractable.selectEntered.AddListener(OnGrab);
-        _grabInteractable.selectExited.AddListener(OnRelease);
+        grabInteractable.selectExited.AddListener(OnRelease);
     }
 
-    private void Update()
+    void Update()
     {
-        if (!_isGrabbing || _taskCompleted) return;
+        if (isGrabbing && !taskCompleted)
+        {
+            timer += Time.deltaTime;
+            progressBar.value = timer / progressTime;
 
-        _timer += Time.deltaTime;
+            if (timer >= progressTime)
-        progressBar.value = _timer / progressTime;
+            {
-
+                CompleteProgress();
-        if (_timer >= progressTime) CompleteProgress();
+            }
-    }
+        }
-
-    private void OnDestroy()
-    {
-        if (_grabInteractable == null) return;
-        
-        _grabInteractable.selectEntered.RemoveListener(OnGrab);
-        _grabInteractable.selectExited.RemoveListener(OnRelease);
     }
 
     private void OnGrab(SelectEnterEventArgs args)
     {
         // Ignore if grabbed by a socket interactor
-        if (args.interactorObject.transform.GetComponent<XRSocketInteractor>() !=
+        if (args.interactorObject.transform.GetComponent<XRSocketInteractor>() != null)
-            null) return; // Do nothing if grabbed by a socket
+        {
+            return; // Do nothing if grabbed by a socket
+        }
 
         // Only show progress bar if NOT grabbed by a socket
         progressBar.gameObject.SetActive(true);
         progressBar.value = 0f;
-        _timer = 0f;
+        timer = 0f;
-        _isGrabbing = true;
+        isGrabbing = true;
         
         // Play brushing sound while toothbrush is grabbed, plays only if it isn't playing already
-        if (audioSource.isPlaying) return;
+        if (!audioSource.isPlaying)
+        {
+            audioSource.clip = brushingSound;
             
-        audioSource.clip = brushingSound;
+            // Loops the sound for as long as the toothbrush is held; allows editable progress time
+            audioSource.loop = true;  
             
-        // Loops the sound for as long as the toothbrush is held; allows editable progress time
+            audioSource.Play();
-        audioSource.loop = true;
+        }
-
-        audioSource.Play();
     }
 
     private void OnRelease(SelectExitEventArgs args)
     {
         // Stop progress when released, regardless of interactor type
         progressBar.gameObject.SetActive(false);
-        _isGrabbing = false;
+        isGrabbing = false;
-        _timer = 0f;
+        timer = 0f;
         
         // Stop the brushing sound when the toothbrush is released
-        if (audioSource.isPlaying) audioSource.Stop();
+        if (audioSource.isPlaying)
+        {
+            audioSource.Stop();
+        }
     }
 
     private void CompleteProgress()
     {
-        if (_taskCompleted) return;
+        if (taskCompleted) return;
         
-        _taskCompleted = true;
+        taskCompleted = true;
         progressBar.gameObject.SetActive(false);
-        _isGrabbing = false;
+        isGrabbing = false;
         
         // Stop the brushing sound when the task is completed
-        if (audioSource.isPlaying) audioSource.Stop();
+        if (audioSource.isPlaying)
+        {
+            audioSource.Stop();
+        }
         
         GameManager.Instance.BrushTeethTaskComplete();
         
@@ -125,6 +133,15 @@ public class BrushTeeth : MonoBehaviour
         Debug.Log("Progress completed!");
     }
 
+    private void OnDestroy()
+    {
+        if (grabInteractable != null)
+        {
+            grabInteractable.selectEntered.RemoveListener(OnGrab);
+            grabInteractable.selectExited.RemoveListener(OnRelease);
+        }
+    }
+    
     private IEnumerator ClearMessageAfterSeconds(float delay)
     {
         // Waits for delay to end and hides the UI

Game/Assets/Scripts/CanvasFade.cs

@@ -1,32 +1,32 @@
 /*
- * Author: Wai Lam
+ Author : Wai Lam
- * Date: 13/2/2025
+ Date : 13/2/2025
- * Description: Counting days
+ Description : Counting days
  */
-
-using System;
 using System.Collections;
-using TMPro;
+using System.Collections.Generic;
 using UnityEngine;
+using UnityEngine.SceneManagement;
+using TMPro;
+using System;
 
 public class CanvasFade : MonoBehaviour
 {
     public CanvasGroup dayPanel; // Assign the UI Panel
     public TextMeshProUGUI dayText; // Assign the TextMeshPro UI
     
+    public static event Action OnDayPanelHidden;
     private void Start()
     {
         if (GameManager.Instance != null)
         {
-            var dayNumber = GameManager.Instance.currentDay;
+            int dayNumber = GameManager.Instance.currentDay;
             dayText.text = "Day " + dayNumber;
         }
 
         StartCoroutine(FadeOutPanel());
     }
 
-    public static event Action OnDayPanelHidden;
-
     private IEnumerator FadeOutPanel()
     {
         dayPanel.alpha = 1f;
@@ -34,8 +34,8 @@ public class CanvasFade : MonoBehaviour
 
         yield return new WaitForSeconds(3f); // Display for 3 seconds
 
-        const float fadeDuration = 1.5f;
+        float fadeDuration = 1.5f;
-        var elapsedTime = 0f;
+        float elapsedTime = 0f;
 
         while (elapsedTime < fadeDuration)
         {
@@ -47,5 +47,6 @@ public class CanvasFade : MonoBehaviour
         dayPanel.gameObject.SetActive(false);
         
         OnDayPanelHidden?.Invoke();
+
     }
 }