fix

blueberryPeak/Assets/Scripts/Player/DialogManager.cs

@@ -16,19 +16,6 @@ public class DialogManager : MonoBehaviour
     private Animator player;
     private Animator quester;
 
-    private void Awake()
-    {
-        if (Instance == null)
-        {
-            Instance = this;
-            DontDestroyOnLoad(gameObject);
-        }
-        else
-        {
-            Destroy(gameObject); // Ensure only one instance exists
-        }
-    }
-
     private void Start()
     {
         dialogPanel.SetActive(false); // Hide on start

blueberryPeak/Assets/Scripts/Player/PlayerInteraction.cs

@@ -10,12 +10,12 @@ public class PlayerInteraction : MonoBehaviour
     [SerializeField] private InputActionReference InteractAction;
     [SerializeField] private InputActionReference DialogAction;
     [SerializeField] private GameObject dialogManagerObject;
-    private List<GameObject> interactableObjects = new();
+    [SerializeField] private List<InventoryItem> inventory = new();
+    private readonly List<GameObject> interactableObjects = new();
     private int blueberryCount;
     private int dialogIndex;
 
     private DialogManager dialogManager;
-    [SerializeField] private List<InventoryItem> inventory = new();
 
     private GameObject speaker;
 
@@ -72,6 +72,7 @@ public class PlayerInteraction : MonoBehaviour
                 speaker = null;
                 dialogManager.HideDialog();
             }
+
             dialogIndex++;
         }
     }
@@ -133,6 +134,7 @@ public class PlayerInteraction : MonoBehaviour
             }
         }
     }
+
     public Dialog[] GetDialog()
     {
         return dialog;
@@ -142,6 +144,7 @@ public class PlayerInteraction : MonoBehaviour
     {
         dialog = savedDialog;
     }
+
     public int GetBlueberryCount()
     {
         return blueberryCount;
@@ -160,7 +163,7 @@ public class PlayerInteraction : MonoBehaviour
 
     public void AddToInventory(InventoryItem reward)
     {
-        InventoryItem item = inventory.Find(e => e.itemName == reward.itemName);
+        var item = inventory.Find(e => e.itemName == reward.itemName);
         if (item == null)
             inventory.Add(Instantiate(reward));
         else
@@ -171,7 +174,7 @@ public class PlayerInteraction : MonoBehaviour
     public void RemoveFromInventory(InventoryItem reward)
     {
         Debug.Log("Attempting to remove " + reward.name + " from inventory.");
-        InventoryItem item = inventory.Find(e => e.itemName == reward.itemName);
+        var item = inventory.Find(e => e.itemName == reward.itemName);
         if (item != null)
         {
             inventory.Remove(item);

blueberryPeak/Assets/Scripts/Player/PlayerMovement.cs

@@ -1,28 +1,31 @@
 using UnityEngine;
 using UnityEngine.InputSystem;
+using Event = AK.Wwise.Event;
 
 [RequireComponent(typeof(CharacterController))]
 public class PlayerMovement : MonoBehaviour
 {
-    [Header("Movement Settings")]
+    [Header("Movement Settings")] [SerializeField]
-    [SerializeField] private float movementSpeed = 5f;
+    private float movementSpeed = 5f;
+
     [SerializeField] private float rotationSpeed = 200f;
     [SerializeField] private InputActionReference movement;
     [SerializeField] private Transform cameraTransform;
-    [SerializeField] private Vector3 cameraOffset = new Vector3(0, 10, -10);
+    [SerializeField] private Vector3 cameraOffset = new(0, 10, -10);
     [SerializeField] private Animator _animator;
 
-    [Header("Footstep Sound Settings")]
+    [Header("Footstep Sound Settings")] [SerializeField]
-    [SerializeField] private AK.Wwise.Event footstepEvent;
+    private Event footstepEvent;
-    [SerializeField] private float footstepInterval = 0.5f; // seconds between footsteps
-    private float footstepTimer = 0f;
 
-    private Vector3 inputDirection = Vector3.zero;
+    [SerializeField] private float footstepInterval = 0.5f; // seconds between footsteps
-    private CharacterController controller;
 
     public bool moveAllowed = true;
+    private CharacterController controller;
+    private float footstepTimer;
 
-    void Start()
+    private Vector3 inputDirection = Vector3.zero;
+
+    private void Start()
     {
         controller = GetComponent<CharacterController>();
 
@@ -31,41 +34,28 @@ public class PlayerMovement : MonoBehaviour
         movement.action.canceled += OnMovementStop;
     }
 
-    void OnMovement(InputAction.CallbackContext context)
+    private void Update()
-    {
-        if (moveAllowed)
-        {
-            Vector2 inputVector = context.ReadValue<Vector2>();
-            inputDirection = new Vector3(inputVector.x, 0, inputVector.y);
-        }
-    }
-
-    void OnMovementStop(InputAction.CallbackContext context)
-    {
-        inputDirection = Vector3.zero;
-    }
-
-    void Update()
     {
         Cursor.lockState = CursorLockMode.Locked;
 
-        Vector3 cameraForward = cameraTransform.forward;
+        var cameraForward = cameraTransform.forward;
-        Vector3 cameraRight = cameraTransform.right;
+        var cameraRight = cameraTransform.right;
 
         cameraForward.y = 0;
         cameraRight.y = 0;
         cameraForward.Normalize();
         cameraRight.Normalize();
 
-        Vector3 move = (cameraForward * inputDirection.z + cameraRight * inputDirection.x).normalized * movementSpeed;
+        var move = (cameraForward * inputDirection.z + cameraRight * inputDirection.x).normalized * movementSpeed;
 
         controller.SimpleMove(move);
         _animator.SetFloat("Speed", move.magnitude);
 
         if (move.magnitude > 0.01f)
         {
-            Quaternion toRotation = Quaternion.LookRotation(move, Vector3.up);
+            var toRotation = Quaternion.LookRotation(move, Vector3.up);
-            transform.rotation = Quaternion.RotateTowards(transform.rotation, toRotation, rotationSpeed * Time.deltaTime);
+            transform.rotation =
+                Quaternion.RotateTowards(transform.rotation, toRotation, rotationSpeed * Time.deltaTime);
 
             // Footstep logic
             footstepTimer -= Time.deltaTime;
@@ -81,7 +71,7 @@ public class PlayerMovement : MonoBehaviour
         }
     }
 
-    void LateUpdate()
+    private void LateUpdate()
     {
         if (cameraTransform != null)
         {
@@ -89,4 +79,18 @@ public class PlayerMovement : MonoBehaviour
             cameraTransform.LookAt(transform.position);
         }
     }
+
+    private void OnMovement(InputAction.CallbackContext context)
+    {
+        if (moveAllowed)
+        {
+            var inputVector = context.ReadValue<Vector2>();
+            inputDirection = new Vector3(inputVector.x, 0, inputVector.y);
+        }
+    }
+
+    private void OnMovementStop(InputAction.CallbackContext context)
+    {
+        inputDirection = Vector3.zero;
+    }
 }