ilou
Období: Týden 15/2026 (od 12.04.) Skupina: tria URL: https://github.com/cieslarova/ilou.git
1. Splnění základních podmínek
| Podmínka | Status |
|---|---|
| Alespoň 3 commity v daném týdnu | ✅ SPLNĚNO |
| Rozestup alespoň 12h mezi 3 commity | ✅ SPLNĚNO |
| Projektová dokumentace (README, projekt.md) | ✅ SPLNĚNO |
2. Přehled commitů
- 2026-04-14 16:14: Pridani faze 4 pro Coin Collector vcetne hodnoceni v realnem case
- 2026-04-15 17:32: Založení fáze 5: přídání základu pro implementaci rotace kamery
- 2026-04-17 21:12: přídání komentářů
3. Výsledky AI analýzy kódu
| Metrika | Hodnota |
|---|---|
| Celkové body (AI) | 18 / 20 b. (90%) |
| Náročnost projektu (Index) | 4 |
| Počet commitů | 3 |
| Změněné soubory | 5 |
| Počet přid./odeb. řádků kódu | 750 |
Komentář a zpětná vazba od AI
Ahoj, jsem tvůj pedagogický asistent a prostudoval jsem tvou práci na projektu Coin Collector a dalších souborech v repozitáři. Zde je tvé hodnocení za uplynulý týden:
1. Smysluplnost popisů u commitů [3 body]
Zprávy u commitů jsou jasné, stručné a popisují konkrétní změny (např. přidání komentářů, založení fáze 5). Oceňuji, že uvádíš i kontext (např. "včetně hodnocení v reálném čase"). To velmi usnadňuje orientaci v historii vývoje.
2. Smysluplnost komentářů v kódu [6 bodů]
Komentáře jsou na vynikající úrovni. Používáš docstringy pro třídy i metody, což je v Pythonu nejlepší praxe. V kódu vysvětluješ i méně zřejmé věci, jako je význam bitových masek pro kolize nebo matematický faktor pro simulaci kutálení kuličky. Pomáhá to pochopit záměr kódu.
3. Kvalita a srozumitelnost dokumentace a struktura repozitáře [2 body]
Dodržuješ stanovenou strukturu – každý projekt má svou vlastní složku a v rootu jsou pouze konfigurační či obecné dokumentační soubory. Dokumentace v souborech _projekt.md je velmi podrobná, obsahuje technické detaily i popisy funkcionalit, což je u středoškolského projektu nadstandardní.
4. Logika řešení a přemýšlení studenta v Pythonu [7 bodů]
Práce s frameworkem Panda3D je náročná a ty ji zvládáš velmi dobře. Oceňuji implementaci fyziky (kutálení kuličky pomocí rotace setHpr) a plynulé animace mincí pomocí funkce math.sin. Použití bitových masek pro kolize svědčí o hlubším porozumění tomu, jak herní enginy fungují.
Mám však několik technických výtek:
* Absolutní cesty: V souborech copy_tex.py a coin_collector_faze5.py používáš absolutní cesty k souborům (např. C:\Users\elen\...). zde je chyba – pokud si projekt stáhne někdo jiný, program mu okamžitě spadne, protože takovou složku v počítači nemá. Vždy používej relativní cesty vzhledem k umístění skriptu.
* Nekonzistence verzí: Soubor se jmenuje coin_collector_faze5.py, ale hned v prvním komentáři máš napsáno # coin_collector_faze4.py a v titulku okna také. Působí to zmatečně.
* Neúplný kód: Na úplném konci souboru coin_collector_faze5.py je kód useknutý: p_pos = self.player. zde je chyba. Pravděpodobně jsi zapomněl kód před commitem uložit nebo zkopírovat celý.
Celkem získáno bodů: [18 / 90%] ze 20 možných
STUPNICE NÁROČNOSTI (Index 1-5): DIFFICULTY_INDEX: [4]
Kontrolní otázka k obhajobě
V metodě
update_movementu třídyPlayerpoužíváš v rámcisetHprjako první parametrself.model– k čemu přesně tento zápis v Panda3D slouží a jak by se změnil vizuální efekt kutálení kuličky, kdybys tento první parametr vynechal?