Den maxSafeIndex könnte man als zusätzlichen member von State machen und im Konstruktor mit initialisieren, denn der Wert ändert sich während des gesamten Spiels nicht (zumindest für mein Verständnis). Um die Komplexität rauszunehmen, könntest du die bereits existierende GameLogicUtils Methode getAllFieldsWith(...) nutzen und dann die Länge des return Vektors als maxSafeIndex setzen.

Genauso könnte collarOnMap ein member von State sein, der initial false ist und in der execute im case // first character gets diamond collar auf true gesetzt wird.