Rewizja 1

[jchudzinski]

Proszę o dokonanie następujących poprawek:

• oczyszczenie maina (main powinien zawierać tylko inicjalizacje gry) > proszę w tym celu utworzyć klasę Scena do rysowania HUD'a, tworzenia okna, zawierającej silnik gry i obsługę klawiszy oraz klasy Gra do rysowania obiektów, animowania i przyjmowania komend
• pozbycie się wszelkich funckji, które są poza klasami (hermetyzacja)
• zastosowanie list inicjalizacyjnych w konstruktorach Na ten moment to tyle, ma Pan rozbudowany projekt i myślę, że taka refaktoryzacja kodu byłaby przydatna.

[JerZaw]

tylko doprecyzuję: mówiąc main ma Pan na myśli główną funkcję gry w game_jumping.cpp, a nie właściwą funkcję main (w main.cpp)?

Nie do końca rozumiem właśnie punkt pierwszy. Może Pan zajrzeć, oczyściłem main.cpp i wczytanie danych, oraz pozbyłem się pozaklasowych funkcji.

Teraz skoro mam tak naprawdę 3 gry w jednej, powinienem dla każdej zrobić klasę Scena i Gra? Nie do końca też rozumiem jakie funkcjonalności mają zawierać te klasy, np co ma pan na myśli pisząc, że klasa Gra powinna przyjmować komendy? Albo to znaczy, że w jednej klasie mam mieć obiekt i obliczać jak się porusza, a w drugiej zmieniać klatki jego animacji?

czy chodzi mniej więcej o coś takiego? https://github.com/pkaczmarek-put/PSIO_Z2020/blob/main/L4/sfml_project_init/main.cpp?fbclid=IwAR0yINJs3DDZapY0NToDwzfbSIvuqn1Gf06IgkqlCbgjn5cvQBfNArZ4FNk

[JerZaw]

Na podstawie wcześniej wysłanego linku oczyściłem maina game_jumping.cpp tworząc dwie klasy, wydaje mi się że to miałem zrobić, ale proszę o odzew czy o to chodziło i czy jeszcze coś do poprawy :)

[jchudzinski]

Miałem na cały plik main. Nadal znajdują się tam zmienne globalne, ale poza tym jest OK! Proszę jeszcze tylko ich się pozbyć i zwrócić uwagę na wycieki pamięci (miejsca, gdzie przydziela Pan pamięć i jej nie oczyszcza np. rand_plat - może Pan przejsć na uniqueptr lub ręcznie przeszukać kod i usunąć pamięć)

[JerZaw]

Dobrze, wydaje mi się ze usunąłem problem (przerobiłem główny wektor platformpointers na unique_ptr) i inne rzeczy doczyściłem, czekam na informację zwrotną