Bizon
Období: Týden 20/2026 (od 17.05.) Skupina: tria URL: https://github.com/stepanlenczowski-debug/Bizon.git
1. Splnění základních podmínek
| Podmínka | Status |
|---|---|
| Alespoň 3 commity v daném týdnu | ❌ NESPLNĚNO |
| Rozestup alespoň 12h mezi 3 commity | ❌ NESPLNĚNO |
| Projektová dokumentace (README, projekt.md) | ✅ SPLNĚNO |
2. Přehled commitů
- 2026-05-18 08:27: přidáni českých komentářu
- 2026-05-18 08:30: Dokončení merge s origin/main
3. Výsledky AI analýzy kódu
| Metrika | Hodnota |
|---|---|
| Celkové body (AI) | 14 / 20 b. (70%) |
| Náročnost projektu (Index) | 3 |
| Počet commitů | 2 |
| Změněné soubory | 3 |
| Počet přid./odeb. řádků kódu | 607 |
Komentář a zpětná vazba od AI
Zde je zhodnocení tvé práce za uplynulý týden na projektu Šachy a údržbě repozitáře:
1. Smysluplnost popisů u commitů [1 bod]
Záznamy o změnách jsou tento týden velmi stručné. Obsahují pouze informaci o dokončení merge a přidání komentářů. Postrádám commit, který by popisoval samotnou implementaci rozsáhlé logiky v souboru sachy.py. Příště se pokus commity více atomizovat (rozdělit na menší části) a popsat, co konkrétně jsi v kódu vytvořil (např. "Implementace pohybu věže a střelce" místo obecného merge).
2. Smysluplnost komentářů v kódu [4 body]
V souboru sachy.py jsi přidal základní české komentáře, které pomáhají v orientaci (označení bloků pro jednotlivé figurky). Oceňuji i snahu o překlad dokumentačních řetězců v adresáři chess. Komentáře jsou však spíše popisné (co kód dělá na povrchu) než vysvětlující (proč je použita daná logika, zejména u složitější rošády). V souboru chess/chess.py je navíc kód useknutý v polovině slova: ["R", "N", "B", "Q", "K", "B <span style='color: red;'>zde je chyba</span> (pravděpodobně chyba při kopírování nebo neúplný commit).
3. Kvalita a srozumitelnost dokumentace a struktura repozitáře [1 bod]
Dokumentace k projektům (Pexeso, Time Dodge, Generátor hesel) je zpracována velmi kvalitně a přehledně. Musím však strhnout body za strukturu repozitáře:
- Soubor v rootu: Soubor /podinky.md se nachází přímo v hlavní složce, což porušuje pravidlo, že každý projekt/soubor musí mít svou složku.
- Nepořádek v repozitáři: V adresáři /chess/ jsi ponechal složku __pycache__, která do repozitáře nepatří (měla by být v .gitignore).
- Duplicita: V repozitáři máš dvě složky pro šachy (/Šachy/ a /chess/), což působí chaoticky.
4. Logika řešení a přemýšlení studenta v Pythonu [8 bodů]
Logika v souboru sachy.py je na vysoké úrovni. Implementace pohybů všech figurek, detekce šachu (is_in_check) a správa stavu pro rošádu (pomocí příznaků _moved) svědčí o dobrém algoritmickém myšlení.
- Pochvala: Použití isupper() a islower() pro rozlišení stran a efektivní iterace směrů u věže a střelce.
- Nedostatek: Funkce get_valid_moves vrací všechny teoreticky možné pohyby, ale v šachu jsou tahy validní pouze tehdy, pokud po nich král nezůstane v šachu. Tvůj kód sice detekuje šach pro zvýraznění, ale nezamezuje hráči provést tah, který ho do šachu vystaví, nebo z něj neuteče. To je častá, ale podstatná logická chyba v implementaci šachových pravidel.
Celkem získáno bodů: [14 / 70%] ze 20 možných
DIFFICULTY_INDEX: [3]
Kontrolní otázka k obhajobě
V hlavní herní smyčce sice střídáš hodnotu v proměnné
turn, ale mohl bys mi ukázat, kde přesně v kódu kontroluješ, zda vybraná figura skutečně patří hráči, který je zrovna na řadě, a jak by se program zachoval, kdybych se teď pokusil pohnout soupeřovou figurou?