Closed mimatik closed 1 month ago
@mimatik obecně fajn, ale mám pár připomínek:
Player.blit()
/ Monster.blit()
(společný název Entity
) byly doteď metody bez parametru. Tj. entita sama věděla, jak se má vykreslit (nebo třeba nevkreslit). Nově to předáváš jako parametr. Ono to funguje, možná je to i správně, ale každopádně je to poměrně výrazná změna toho, jak uvažujeme o kompetencích Game
vs Entity
.
(Trochu související otázka je, kdo má na sebe navázané aktuální souřadnice Entity - ta entita samotná, nebo Game, nebo by to měla na sebe navázaná Map?)
Při zachování stávající logiky (všechno o sobě ví Entity
) by Entity.blit()
byla bez parametru a vykreslila se na self.game.dynamic_surface
.
Layer
má metody draw
a blit
, ale nejsem si jistý, že správně rozumím tomu pojmenování.
Případá mi nekonzistentní s tím, jak metodu blit
používáme u Player
/Monster
/Map
. U nich to znamená vykresli sám sebe, tady to najednou znamená vykresli někoho jiného. Skoro mi připadá, že by se měly jmenovat obráceně.
Game
podobně - draw
by se imo měla jmenovat blit
?
@x0nix trochu jsem to poladil podle pripominek.. takhle ti to uz dava smysl?
@mimatik perfekt.
Vzhledem k tomu, ze uz jsem byl sam sobe otravny s tema hernima vrstvama, tak jsem se rozhodl jen nekibicovat a take pridat prispevek k vasi hre. Co si od teto upravy slibuji:
background_layer
), monstra a hrace (uprostreddynamic_layer
) a nad tim potrebuji interakcni UI (dialogy, ovladaci prvky, ohraniceni hry apod. mapr.ui_layer
). Pripadne muzu pridat dalsi vrsty, pokud to bude davat smysl.