Detail reportu: kumor

Týden 11/2026 (od 15.03.) – Výsledky AI analýzy a hodnocení odevzdaného kódu.

📁 Archiv-2026
Zpět

kumor

Období: Týden 11/2026 (od 15.03.) Skupina: tria URL: https://github.com/sebastianszkandera/kumor

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) ❌ NESPLNĚNO

2. Přehled commitů

3. Výsledky AI analýzy kódu

Metrika Hodnota
Celkové body (AI) 12 / 20 b. (60%)
Náročnost projektu (Index) 4
Počet commitů 7
Změněné soubory 18
Počet přid./odeb. řádků kódu 2663

Komentář a zpětná vazba od AI

Dobrý den, jako tvůj pedagogický asistent jsem prošel tvou práci za uplynulý týden. Vidím velký posun v náročnosti tvých projektů, zejména v aplikaci pro předpověď počasí, ale zároveň se stále dopouštíš hrubých chyb v organizaci celého repozitáře.

Zde je tvé hodnocení:

1. Smysluplnost popisů u commitů [2 body]:

Commity jsou věcné a popisují, co jsi v danou chvíli udělal (např. přesun souborů, vytvoření nové aplikace). Bod strhávám za překlep v klíčovém commitu („wheather app“) a za občasnou strohost. Snaž se také v commitech nepoužívat slangové výrazy pro názvy složek.

2. Smysluplnost komentářů v kódu [2 body]:

V kódu weather_app.py jsi sice použil logické rozdělení do sekcí (např. # Side bar, # Forecast section), ale zcela zde chybí dokumentační řetězce (docstrings) u tříd a metod. Pro takto rozsáhlý kód (přes 400 řádků) je nezbytné vysvětlit, co jednotlivé funkce dělají a jaké mají parametry. Kód je sice čistý, ale pro cizího čtenáře hůře pochopitelný bez jediného vysvětlujícího komentáře k logice API nebo vláken.

3. Kvalita a srozumitelnost dokumentace a struktura repozitáře [0 bodů]:

V této kategorii musím být přísný, protože jsi porušil základní pravidla organizace: - Soubory v rootu: Soubory /weather_app.py a /guitar_app.py leží volně v hlavní složce, což je nepřípustné. Každý projekt musí mít vlastní složku. - Nonsens názvy: Ve složce /bordel/ (mimochodem velmi neprofesionální název) a /kumor/ se nacházejí soubory jako pp.py, test.py a dokonce soubor s názvem H. To svědčí o nepořádku v projektu. - Dokumentace: README u projektu filament je naopak vynikající a velmi detailní. Bohužel celková struktura repozitáře toto pozitivum přebíjí.

4. Logika řešení a přemýšlení studenta v Pythonu [8 bodů]:

Zde tě musím pochválit. Aplikace weather_app.py vykazuje vysokou úroveň programátorského myšlení: - Použití knihovny requests pro komunikaci s API. - Implementace vícevláknového zpracování (threading), což brání zamrzání GUI při stahování dat. - Práce s JSON formátem pro ukládání historie a oblíbených položek. - Pokročilé GUI v tkinter využívající stylování ttk. - Logika přepínání uživatelů a dynamické generování prvků předpovědi je zpracována velmi dobře. Menší logickou duplicitu vidím v metodě _switch_user, kde znovu ověřuješ existenci uživatele v seznamu, i když to už pravděpodobně proběhlo v _new_user.

Celkem získáno bodů: [12 / 60%] ze 20 možných

Slovní shrnutí: Tvůj technický progres v Pythonu je obdivuhodný, pouštíš se do komplexních úloh (API, GUI, JSON databáze). Nicméně tvůj repozitář působí chaoticky. Do příště ukliď složku "bordel", přesuň hlavní aplikace do vlastních složek a odstraň nesmyslné soubory (pp.py, H). Pokud zapracuješ na dokumentaci kódu (komentářích), bude to výborné.

DIFFICULTY_INDEX: [4]

Kontrolní otázka k obhajobě

V metodě __init__ nastavuješ self.data_file napevno na 'weather_app_default.json', ale v metodách _new_user ani _switch_user tuto proměnnou nikde neaktualizuješ – jakým způsobem tedy tvoje aplikace zajistí, že se historie vyhledávání nového uživatele uloží do jeho vlastního JSON souboru a nepřepíše data toho výchozího?