Przegląd projektu

Rejestracja:prawie 8 lat
Ostatnio:około 7 lat
Postów:10

Witam, od jakiegoś czasu pisze swój projekt w cpp, tzn. gierke typu roguelike. Grafiki toto nie ma za bardzo, i zbyt grywalne nie jest, bo staram sie tworzyc jakby 'szkielet' aplikacji, na którym bedzie mozna zbudować pełnoprawna gre. Chciałbym żebyście mi podpowiedzieli, gdzie popełniam błędy, czy kod jest jakotako czytelny, co można by poprawić w ogólnej metodyce, konwencjach itd. itp. Mam nadzieje, że wybrałem dobry dział, jestem tu nowy ;)

repozytorium z projektem(VS 2015):
https://github.com/KarolDrach/RPG

LukeJL

2017-05-22 11:36

LukeJL2017-05-22 11:36

Rejestracja:około 11 lat
Ostatnio:minuta
Postów:8423

Tak na oko to zbyt wiele rzeczy robisz to na ifach/switch case, zamiast skorzystać z polimorfizmu, np. tutaj:
https://github.com/KarolDrach/RPG/blob/master/Equipment.cpp#L130

i ogólnie za bardzo wchodzisz w szczegóły, zamiast zrobić kolekcję* Itemów to tworzysz ileś zmiennych:

Kopiuj

Item* armor, *shoes, *lefthand, *righthand, *head, *ring;

a jeśli tworzysz ileś zmiennych to potem musisz switch/case robić...

** piszę "kolekcja", bo nie wchodzę już w szczegóły implementacyjne, czy to będzie zwykła tablica, wektor, mapa, zbiór czy jeszcze jakaś inny cudowny rodzaj kolekcji w C++.*

No i wszystko ci się miesza, np. wywołania ChangeColor walają się po całym projekcie. Wyświetlany kolor czegoś tam raczej jest szczegółem wizualnym, o którym obiekty takie jak Hero czy Quest w ogóle nie powinny wiedzieć.