Teda, mám otestováno jen odevzdávání správných a špatných odpovědí, zatím nevím, co všechno dalšího to má umět, byť nečekám, že to někde selže…
To je podle mě asi jedno, spíš mi chybí, exporty čeho to jsou, případně v jakém formátu…
Tyhle ify vypadají dost zběsile. Chápu správně, že se prakticky jen mění, jaký QuerySet předhodíme dataOsobCsvResponse a pak možná filename?
Nechceš použít fields.join(",")? Tohle mě neuráží, ale .join zní jakože to ušetří 5 řádků a dvě logické chyby…
Nevidím evidentní chyby, ale mám z toho kódu dost pocit, že sahat do toho bude strašně nepohodlné a náchylné na chyby, hlavně kvůli kopírovanému kódu, předávání magických číselných konstant a nezamýšlení se nad chybovými stavy.
Vlastně mi přijde, že je tam klidně NějakouDobu™ můžeme nechat – pokud někomu fungovaly / vyhovovaly staré exporty lépe (nevím přesně proč a jak), tak ať je klidně ještě…
Noooo, asi to lepší nebude… Ideálně někam poznamenat moje výtky k dokumentaci, ale jinak asi mergenout…
Přijde mi dobré mít v tomhle místě při čtení kódu mít kontext, že template tu Znalost rozepisuje nezávisle, a tím pádem ve formuláři se má vložit neviditelně.
Ale je to…
Uh, ale jména těch tříd dost držíme konzistentní a taky velmi jasná (i bez dekorátoru). Ale OK, aspoň to přidává odkaz ig
@zelvuska už nějak bastlil autogenerovanou dokumentaci, zkuste to asi nějak poladit, ať to dopadne nějak konzistentně (hlavně ať se případně změny stylu dají dělat na jednom místě nad (aspoň trochu) strukturovanými daty a ne všude v kódu…)
Tohle asi funguje (pokud to něco vrací…). Samozřejmě to naráží na to, že „kdo řešil co v nějakém čísle“ je strašně špatně definované… (Technicky je tohle dotaz na to, kdo má nějaké body zadané k nějakému deadlinu, může dávat lepší smysl se ptát na řešitele, kteří poslali řešení mezi vydáním nějakého čísla a jeho (nejzazším) deadlinem, což je o trochu přesnější ale naopak je potřeba řešit případný překryv deadlinů čísel a asi to za to nestojí…)
Ale jak koukám výš, tak tam je to taky takhle… 🤷
No, člověk odmaturuje v květnu/červnu… Takže pokud je jaro, tak pokud je rok_maturity > aktuální, pokud je podzim, tak ≥, a pokud je červen, tak 🤷 je jaro, protože pro velkou část věcí nás stejně asi zajímá předěl o letních prázdninách a ne samotný okamžik maturity…
Zatím nemám zkouknuté ty „typované“ views…