damykk
Období: Týden 22/2026 (od 31.05.) Skupina: tria URL: https://github.com/DamianDampha/damykk.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 | ❌ NESPLNĚNO |
| Projektová dokumentace (README, projekt.md) | ✅ SPLNĚNO |
2. Přehled commitů
- 2026-06-01 06:44: Začátek projektu J.A.R.V.I.S
- 2026-06-05 16:59: Udělal jsem si krypto penezenku
- 2026-06-05 17:00: Update README.md
3. Výsledky AI analýzy kódu
| Metrika | Hodnota |
|---|---|
| Celkové body (AI) | 15 / 20 b. (75%) |
| Náročnost projektu (Index) | 4 |
| Počet commitů | 3 |
| Změněné soubory | 5 |
| Počet přid./odeb. řádků kódu | 155 |
Komentář a zpětná vazba od AI
Ahoj! Jako tvůj asistent jsem prošel tvůj repozitář a kód, který jsi za poslední týden vytvořil. Oceňuji tvůj posun k pokročilejším tématům, jako je práce s API a ukládání dat do JSON souborů.
Zde je tvé hodnocení za uplynulý týden:
1. Smysluplnost popisů u commitů [2 body]
Tvé popisy commitů jsou srozumitelné a jasně říkají, co se v daném kroku stalo (např. „Udělal jsem si krypto penezenku“). Bod strhávám za příliš obecný commit „Update README.md“, který u tak velkého projektu, jako je dokumentace, neříká, co konkrétně bylo přidáno nebo opraveno.
2. Smysluplnost komentářů v kódu [5 bodů]
V souboru penezenka.py používáš komentáře vhodně k oddělení logických bloků a vysvětlení simulace trhu. V projektu jarvis.py oceňuji vysvětlivky u API parametrů. Bod strhávám za to, že v kódu penezenka.py chybí dokumentační řetězce (docstrings) u funkcí, které by vysvětlily parametry a návratové hodnoty, což by u takto komplexního kódu bylo profesionálnější.
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 repozitáře:
* Soubory v rootu: Soubory /generator_hesel.py a /Generator_hesel_projekt.md leží volně v hlavní složce, což je nepřípustné. Každý projekt musí mít vlastní složku.
* Chyby v názvech: Složka s kryptopeněženkou se jmenuje <span style='color: red;'>krypto_penezekna</span> (překlep v názvu).
* Nekonzistence v README.md: V tabulce projektů máš u čísla 20 uveden soubor Ai.py, ale ve složce TEST AI se soubor jmenuje jarvis.py. Také máš v tabulce dvakrát číslo 19.
* Chyba v README: Hned na prvním řádku máš překlep: <span style='color: red;'>f##</span> Projekty.
4. Logika řešení a přemýšlení studenta v Pythonu [8 bodů]
Tady jsi odvedl skvělou práci. Projekt kryptopeněženky ukazuje dobrou znalost:
* Práce s daty: Správné načítání a ukládání stavu peněženky do crypto_data.json.
* Logiky: Funkce pro nákup a prodej správně kontrolují zůstatek uživatele a validují vstupy.
* Integrace API: V projektu J.A.R.V.I.S. jsi úspěšně integroval knihovnu openai a ošetřil jsi chyby při komunikaci se serverem pomocí try-except.
* Drobné varování: Máš v kódu jarvis.py přímo vložený API klíč. To je bezpečnostní riziko (nikdy se nesmí nahrávat na GitHub), příště použij proměnné prostředí nebo externí konfigurační soubor.
Celkem získáno bodů: [15 / 75%] z 20 možných
Slovní zhodnocení: Tvůj technický pokrok v Pythonu je vynikající. Kryptopeněženka je logicky velmi dobře postavená a integrace AI je moderní přístup. Nicméně tvůj projekt začíná trpět velkým nepořádkem v souborech a dokumentaci. Prosím, příští týden se zaměř na "úklid": přesuň zapomenuté soubory do složek, oprav si překlepy v názvech adresářů a sjednoť názvy souborů s tím, co píšeš v README.
DIFFICULTY_INDEX: [4]
Kontrolní otázka k obhajobě
V souboru
penezenka.pyve funkciukaz_penezenkupoužíváš podmínkuif krypto != 'USD'– mohl bys mi vysvětlit, co přesně by se v programu stalo na následujícím řádkuhodnota = mnozstvi * TRH[krypto], kdyby tam tato podmínka nebyla?