Nataliia71 / Ukol1

0 stars 0 forks source link

review k ukolu 1 #1

Open KristynaSodomkova opened 1 week ago

KristynaSodomkova commented 1 week ago

Review

KristynaSodomkova commented 1 week ago

Na řádce 6 by neměla třída Property dědit od Locality.

Na řádku 7: nepotřebuješ name a locality_coefficient, ponech pouze locality.

Řádek 8 můžeš vymazat, protože třída Property nebude dědit z Locality, a proto není potřeba použít metodu super.

Přidej třídu Estate, která bude také dědit od Property a bude pracovat s estate_type_coefficients. Tato třída bude mít parametry locality, estate_type a area, dále metodu calculate_tax a str.

Třída Residence bude dědit od Property a v init bude řešit locality, area, commercial. Dále bude obsahovat metodu calculate_tax a str.

Je vidět, že jsi si úkol vypracovala sama a moc si toho cením. U mě máš splněno. Vypracuj ale prosím tyto připomínky.

Nataliia71 commented 1 week ago

Děkuji. Bude opraveno! ☺️Dobrou noc.

Вт, 12 нояб. 2024 г. в 23:01, KristynaSodomkova @.***>:

Na řádce 6 by neměla třída Property dědit od Locality.

Na řádku 7 https://www.google.com/maps/search/Na+%C5%99%C3%A1dku+7?entry=gmail&source=g: nepotřebuješ name a locality_coefficient, ponech pouze locality.

Řádek 8 můžeš vymazat, protože třída Property nebude dědit z Locality, a proto není potřeba použít metodu super.

Přidej třídu Estate, která bude také dědit od Property a bude pracovat s estate_type_coefficients. Tato třída bude mít parametry locality, estate_type a area, dále metodu calculate_tax a str.

Třída Residence bude dědit od Property a v init bude řešit locality, area, commercial. Dále bude obsahovat metodu calculate_tax a str.

Je vidět, že jsi si úkol vypracovala sama a moc si toho cením. U mě máš splněno. Vypracuj ale prosím tyto připomínky.

— Reply to this email directly, view it on GitHub https://github.com/Nataliia71/Ukol1/issues/1#issuecomment-2471672596, or unsubscribe https://github.com/notifications/unsubscribe-auth/BMAA4ILFTQQ3CWBV5WHQVLL2AJ3EZAVCNFSM6AAAAABRVBTPACVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDINZRGY3TENJZGY . You are receiving this because you are subscribed to this thread.Message ID: @.***>