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

nested objects -> array of objects #1

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

kareldohnal
Copy link

Oprava na pole objektů s id, aby odpovídalo znalostem z lekcí.

Oprava na pole objektů s id, aby odpovídalo znalostem z lekcí.
@FilipJirsak
Copy link

Chápal jsme původní zadání tak, že mají na stránce s přehledem počasí za týden těch sedm dní vyjmenovat. Teď to vyžaduje znalost metody find(), která se na lekci nikdy neprobírala. Je uvedená jen jako alternativa ve FilmBoxu, bez vysvětlení, jen s odkazem na dokumentaci.

@kareldohnal
Copy link
Author

Chápal jsme původní zadání tak, že mají na stránce s přehledem počasí za týden těch sedm dní vyjmenovat. Teď to vyžaduje znalost metody find(), která se na lekci nikdy neprobírala. Je uvedená jen jako alternativa ve FilmBoxu, bez vysvětlení, jen s odkazem na dokumentaci.

No, na lekci jim to bylo vysvětleno Kubou tak, že je chyba v zadání a má to být pole objektů. Zároveň, aby na úvodní stránce vypsaly karty se dny, tak bez pole neudělají forEach. O .find() bych se chtěl taky pobavit, aby se zařadilo do lekce, alespoň letmo, ale nutné k vyřešení není (stejně jako ve Filmboxu). Ale nevím, jak by to měli řešil jako nested objekt. Ne, že by to nešlo vyřešit, ale nedává mi to smysl.

Copy link

@FilipJirsak FilipJirsak left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ale nevím, jak by to měli řešil jako nested objekt.

Prostě by ten seznam pro celý týden vytvořily ručně:

vypsatDen('pondeli')
vypsatDen('utery')
vypsatDen('streda')

atd.

Ale myslím, že se musíme dohodnout, jak to má být, protože je to změna toho, na co je cvičení zaměřené. Já jsem také pro udělat z toho cvičení na pole a find, ale udělejme to jako vědomou změnu materiálů a ne jako opravu zdrojového repository. Protože při změně na pole je potřeba změnit i zadání a taky by bylo vhodné, kdyby účastnice znaly find.
Založil jsem na to issue Czechitas-podklady-WEB/daweb-vyuka#749 a souvisí s tím i Czechitas-podklady-WEB/daweb-vyuka#748 na find.

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

Successfully merging this pull request may close these issues.

2 participants