Diskuze – Lekce 8 - Aréna s mágem (dědičnost a polymorfismus)

Ahoj Rostislave, děkujeme za podnět, který jsem předal k řešení.

Ahoj Jaroslave, děkujeme za podnět, který jsem předal k řešení.

Davide,

Console.WriteLine(((Mag)b).GrafickaMana());

je správně, protože prvně proběhne přetypování ((Mag)b) a teprve posléze můžeš použít metodu b.GrafickaMana(); + to celé obaleno do výpisu Console.Write­Line();

Díky moc za odpověď.

Jedna drobnost: Grafická mana se nezobrazuje jako prázdná, i když prázdná je. Je to kvůli podmínce v metodě GrafickyUkazatel:

if (pocet == 0 && Nazivu())

Jestliže jsme tuto metodu abstrahovali od života bojovníka a udělali ji univerzální, aby mohla zobrazovat i cokoli jiného - v našem případě manu - neměla by dle mého názoru už mít nic společného s testováním života. Proto jsem podmínku změnil takto:

if (pocet == 0 && aktualni > 0)

Teď mi vše funguje, jak má - u života se zobrazí vždy aspoň jeden dílek (pokud bojovnik není mrtvý) a u many žádné dílky, pokud je prázdná.

Přemýšlím, jestli toto je reálné navržení hry nebo je tento guláš jen kvůli příkladům dědičnosti a polymorfismusu. Ještě před pár lekcemi se čtenář dozví, jak je OOP přehledné a ideální, protože neexistují komplikovaně větvené struktury a pak najednou: Kousek třídy si duplikujeme, kousek si přetvoříme, nějakou metodu si vycucneme odjinud, jinou si raději založíme jinde... tohle bych chtěl jednou po někom opravovat v rozsáhlejším projektu. Není prostě lepší napsat trochu objemnější kód, vše si v klidu definovat jinak, třeba zdrojáky nějakých objektů (herních tříd mág, bojovník), zdrojáky procesů a stavů (boj, obrana, vliv prostředí atd.) a volat si do nějakých smysluplných tříd?