Domaci ukol - vypocet dane

[Kubejda]

@cattitude

[cattitude]

Super, díky. Jakmile na to mrknu, napíšu ti sem postřehy. Pokud nějaké budou potřeba 😁

[cattitude]

Ahoj @Kubejda, mrknul jsem na tvůj úkol a vypadá fajn.:) Použití tříd, dědičnosti a objektů je správné, tady nemám výhrady.

Co se mi líbí už trochu méně, jsou metody calculate_tax(), konkrétně:

• Ve třídě Estate u ní dost opakuješ kód. Nešlo by si třeba přímo v definici třídy vytvořit slovník, do kterého by sis dala typy pozemků jako klíče a koeficienty jako hodnoty? Pak by ses mohla všech těch podmínek zbavit, a navíc přidání dalšího typu by bylo jednodušší.:)
• U Residence bych podmínku změnil na if ... else ..., přece jenom zde pracujeme s pravdivostní hodnotou, tedy buď je to True a nebo je to False. Když pak napíšeš if self.commercial: ..., bude to úplně profi. 😁

Jinak myslím úplně v pohodě, dobrá práce.:) Úkol beru jako hotový, nicméně kdybys na to ještě mrkla, zkusila trochu učesat a udělala nový commit, určitě by to nebylo na škodu.

Adam