chore: register view to model #16
Conversation
mainStage is coming as a constructor parameter and it is entry stage provided by JavaFX. We do not change, and we do not want to change this reference anywhere.
We do not intent to instantiate abstract InteractiveGameObject type. Only more specific types should be instantiated.
This aims to extract the object creation logic from LocationView class
kkafar
force-pushed
the
@kkafar/register-view-to-model
branch
from
03ce151 to
6614408
Compare
This aims to extract the creation logic to separate class to make object
injection possible. (Object injection is needed as we need register view
to model)
| */ | ||
| public static GameObject fromConfig(GameObjectConfig config) { | ||
| // TODO: add case for Player type here! | ||
| switch (GameObject.Type.valueOf(config.getTypeString())) { |
There was a problem hiding this comment.
program się tu mi wywala, ale można szybko poprawić. wartości enumów są pisane wielkimi literami, a w plikach konfiguracyjnych jest małymi. valueOf zwraca wartości enumów na podstawie ich nazwy, musi się zgadzać dokładnie. jeśli w zamyśle było zwracanie na podstawie atrybutu asString, to trzeba napisać metodę w GameObject.Type do tego. oprócz tego w configu location-1 pojawia się typ 'interactive', trzeba zamienić raczej na 'dialog' i dodać wartości assetPath do wszystkich obiektów (bo zakładam, że obiekty są specyfikowane w plikach location, a katalogi objects są do wyrzucenia?)
| switch (GameObject.Type.valueOf(config.getTypeString())) { | |
| switch (GameObject.Type.valueOf(config.getTypeString().toUpperCase())) { |
There was a problem hiding this comment.
Nie, katalogi objects maja zostać: będzie możliwość specyfikowania obiektów zarówno w konfiguracji lokacji jak i w katalogu obejcts -- zacząłem to robić ale wypadło. Dokończę to możliwie szybko.
Description
Generalnie dodałem rejestrowanie obiektów
GameObjectViewdoGameObject. Teraz gdy kontroler zmodyfikuje obiekt ten może powiadomić swoją reprezentację o zaistniałych zmianach.Można to ewentualnie uprościć i w
GameObjectdodać pole z referencją do swojej reprezentacji, ale wydaje mi się, że obecne rozwiązanie jest bardziej elastyczne.How
Wyciągnąłem logikę tworzenia
GameObjectiGameObjectViewdo osobnych klas (factory methods), a dlaLocationModelzrobiłem buildera (jak teraz na niego patrzę to jest całkowicie zbędny, ale jak już go napisałem to nie kasowałem... :D), to pozwoliło na dodanie metody doInitializer, która rejestruje widoki do modelu.Nie testowałem tego jeszcze.
Checklist