[Fix] - equipmentUI 부분 오류 수정 - #62
Conversation
There was a problem hiding this comment.
Code Review
This pull request introduces several UI and manager updates, including the addition of the EquipManager prefab, adjustments to skill ranges in asset files, and robust binding of EquipmentManager to GameManager to handle player changes. It also adds defensive null checks and initialization logic in UI slot scripts (SelectSlot.cs, Slot.cs) to prevent runtime errors. The review feedback highlights a potential memory leak in EquipmentManager.cs due to Unity's overloaded == operator on destroyed objects, suggesting the use of ReferenceEquals. Additionally, it recommends moving icon assignment in Slot.cs to RefreshIcon() to avoid initialization order issues, and points out a redundant null check in EquipmentManager.cs.
Important
The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.
개요
장비 UI 부분에 있던 오류를 수정하였습니다
수정 사항