[ARCH] Replace SendMessage anti-pattern with interfaces or C# events #15

New issue

Open

opened 2026-03-12 00:03:05 +00:00 by lavarius · 0 comments

lavarius commented

2026-03-12 00:03:05 +00:00

Owner

🟠 High — Architecture

Files: Interactable.cs, InteractionManager.cs, CharacterZoneChanger.cs

SendMessage("MethodName") is used for communication between components. This has no compile-time safety — typos in the string silently break things, and it's impossible to find callers via IDE.

Fix: Replace with C# interfaces or events:

public interface IInteractable { void OnInteract(); }
public interface IZoneAware { void OnZoneChanged(ZoneType zone); }

Or use UnityEvent fields for designer-friendly wiring.

## 🟠 High — Architecture **Files:** `Interactable.cs`, `InteractionManager.cs`, `CharacterZoneChanger.cs` `SendMessage("MethodName")` is used for communication between components. This has no compile-time safety — typos in the string silently break things, and it's impossible to find callers via IDE. **Fix:** Replace with C# interfaces or events: ```csharp public interface IInteractable { void OnInteract(); } public interface IZoneAware { void OnZoneChanged(ZoneType zone); } ``` Or use `UnityEvent` fields for designer-friendly wiring.