ES6 klasy - pomoc oraz ukierunkowanie

Question

Witam,

jestem w trakckie pisania prostej gierki w JS opartej na klasach.Chciałbym, aby ktoś mnie wskazał, jakie błędy popełniam w OOP i dobrych praktykach dotyczących klas w JS.Głównie nie jestem do końca pewny, czy wszystkie funkcji i zmienne znajdują się w odpowiednich klasach, oraz jak zmniejszyć powtarzalność kodu.

https://github.com/aqfgit/action-game - klasy znajdują się w /src

Answer 1

Klasy zdecydowanie za dużo robią, np. klasa Game nie powinna się zajmować generowaniem planszy. Od tego stworzyłbym osobną klasę, GameBoard. Tak samo w Player nie powinno być obsługi inputu ze strony użytkownika – to też przerzuciłbym do osobnej klasy.

Zależności są sztywno ustalone, np. klasa Player ma de facto zaszytą integrację z klasą Weapon i dodatkowo na sztywno określa, jaką broń ma użytkownik. Wszystkie zależności bym wstrzykiwał do konstruktora przy pomocy mechanizmu DI. Tak samo informacje o ekwipunku – osobna klasa wstrzykiwana do konstruktora.

Comments

Dzięki za odpowiedź.

1.Załóżmy, że mam klasę GameBoard, która reneruje obiekty.Jak miałby te obiekty do niej dodać?Zrobić tak jak teraz, że zamiast gameObjects mieć w Game to  mam je w GameBoard i tam dodać do tablicy na sztywno wszystkie obiekty?Czy da to się zrobić automatycznie, w bardziej elegancki sposób?
2.Mógłbyś zobrazować jak takie wstrzykiwanie zależonści miałoby wyglądać?

---

ad. 1) Miej metodę addObject, która będzie dodawać te obiekty.

ad. 2) 

const inventory = new Inventory();
const player = new Player( inventory );
const game = new Game( player );

 

---

Ok, zaktualizowałem kod na Githubie, posługując się Twoimi sugestiami.Mógłbyś zerknąć i zobaczyć, czy o coś takiego chodziło?Masz jeszcze jakieś uwagi/wskazówki?

---

No wygląda to lepiej, ale np. czemu Inventory może posiadać tylko jedną broń?

---

Więcej nie uniesie xd

Answer 2

Bez różnicy czy to JS, PHP czy C# zasada jest ta sama. Jedna klasa, jedno zadanie. Wiec rozbijaj klasy że tak sparafrazuje. "Mała klasa mały problem, duża klasa duży problem" ;)