Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Cvičení Počasí – nejasné zadání #749

Open
FilipJirsak opened this issue Apr 13, 2024 · 1 comment
Open

Cvičení Počasí – nejasné zadání #749

FilipJirsak opened this issue Apr 13, 2024 · 1 comment

Comments

@FilipJirsak
Copy link
Contributor

FilipJirsak commented Apr 13, 2024

Není jasné, co se má procvičovat ve cvičení Počasí.

Zadání a aktuální zdrojový kód je popsaný tak, že mají objekt s daty – pak mohou zobrazit detail jednoho dne pomocí pouhého přístupu přes klíč pole (ale přes dynamický klíč, tedy notace s hranatými závorkami – což by neškodilo si zopakovat). Pak je ovšem nešikovný výpis všech sedmi dnů, které by museli vypsat ručně – klidně zavoláním funkce, ale ta funkce by se musela ručně zavolat sedmkrát a den v týdnu předávat jako parametr.

Lektor to na lekci (údajně) uvedl tak, že správně má být v zadání pole. Což by dávalo smysl jako procvičení polí, která se brala nedávno. Ale zase je pak problém se zobrazením detailu jednoho dne, protože to vyžaduje buď použití metody find, kterou neznají (ale asi by měli znát, viz #748), nebo cyklu for (který ale prakticky také neznají z důvodů popsaných též v #478). Tato varianta by zároveň byla hezkou přípravou na Filmbox.

Na druhou variantu je PR od @kareldohnal do materiálů ke cvičením: Czechitas-podklady-WEB/cviceni-pocasi#1. Myslím, že je potřeba se nejdřív dohodnout, co se má v tom cvičení vlastně procvičovat a podle toho pak upravit zadání i podklady.

@FilipJirsak
Copy link
Contributor Author

FilipJirsak commented Apr 13, 2024

Osobně jsem pro to učit předtím find a toto cvičení pak upravit na variantu s polem a find pro detail dne, jak je to v PR Czechitas-podklady-WEB/cviceni-pocasi#1.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant