Review 1: Первичное ревью

Assets/Scripts/DoorPassage.cs

@@ -1,5 +1,6 @@
 using UnityEngine;
 
+// review(24.05.2024): Стоит разместить в отдельном файле
 public enum Door
 {
     RoomDoor,

Assets/Scripts/ElectricalPanel.cs

@@ -20,6 +20,8 @@ private void Update()
 
     private void OnTriggerEnter2D(Collider2D other)
     {
+        // review(24.05.2024): if (!other.CompareTag("Player")) return;
+        // review(24.05.2024): А еще можно просто отнаследовать PlayerTrigger : BoxCollider и в нем реализовать одну и ту же проверку на игрока 
         if (other.CompareTag("Player") && other.gameObject.GetComponent<Player>().penguinName == PenguinNames.Kawazaki)
             isTriggered = true;
         else if (isStupid && other.CompareTag("Player"))
@@ -39,6 +41,7 @@ private void OnTriggerExit2D(Collider2D other)
         }
     }
 
+    // review(24.05.2024): Код дублируется. Стоит выделить класс, который занимается печатью текста
     private IEnumerator TypeLine()
     {
         panel.SetActive(true);

Assets/Scripts/Inventory.cs

@@ -24,7 +24,7 @@ public static void Add(Item item)
         if (item.itemName == ItemName.Crowbar)
             GameState.HaveCrowbar = true;
         if (OnItemChangedCallback != null)
-            OnItemChangedCallback.Invoke ();
+            OnItemChangedCallback.Invoke (); // review(24.05.2024): null-propagation
     }
     public static void Remove (ItemName itemName)
     {

Assets/Scripts/Item.cs

@@ -2,6 +2,7 @@
 using System.Collections.Generic;
 using UnityEngine;
 
+// review(24.05.2024): Стоит разместить в отдельном файле
 public enum ItemName {Nail, Brick, Screwdriver, Crowbar}
 
 public class Item : MonoBehaviour

Assets/Scripts/KeyboardGame/Key_1.cs

@@ -2,6 +2,7 @@
 using UnityEngine;
 using UnityEngine.UI;
 
+// review(24.05.2024): Название класса не соответствует названию файла
 public class ChangeImage : MonoBehaviour
 {
     [Header("Image")] [SerializeField] private Image image;

Assets/Scripts/KeyboardGame/TextPuzzle.cs

@@ -22,12 +22,14 @@ public class TextPuzzle : MonoBehaviour
 
     void Start()
     {
+        // review(24.05.2024): text ??= GetComponent<TMP_Text>();
         if (text == null)
             text = GetComponent<TMP_Text>();
     }
 
     void Update()
     {
+        // review(24.05.2024): Правильно ли я понял, что даже если TextPuzzle не активен, он все равно будет выполнять метод Update?
         if (text.text == answer)
             GameState.IsOverGameKeyboard = true;
         if (Input.GetKeyDown(KeyCode.Backspace) && text.text.Length > 0)
@@ -38,9 +40,12 @@ void Update()
 
         if (text.text.Length == answer.Length)
             return;
+        // review(24.05.2024): Если предполагается, что числа вводятся по одному, тогда зачем тут foreach?
+        // Можно же через FirstOrDefault() найти нужную кнопку и с ней работать
         foreach (var el in Alphabet)
         {
-            if (!Input.GetKeyDown(el)) continue;
+            if (!Input.GetKeyDown(el)) continue; // review(24.05.2024): На новую строку
+            // review(24.05.2024): Я бы выделил класс-хелпер, который бы умел KeyCodes.GetDigit(KeyCode key): KeyCode -> int и скрыл бы там всю эту сложность
             var nexIndex = (Int32.Parse((el.ToString()[^1]).ToString()) + 2) % Alphabet.Count;
             var newDigit = Alphabet[nexIndex].ToString()[^1];
             if (text.text.Length != 0)

Assets/Scripts/Player.cs

@@ -7,7 +7,7 @@ public class Player : MonoBehaviour
     public PlayerController controller;
 
     public bool isActive;
-    private Inventory inventory;
+    private Inventory inventory; // review(24.05.2024): А зачем это поле, если класс статический?
 
     private PlayerTrigger playerTrigger;