mam/mamweb
11
0
Fork 0
Code Issues Pull requests 8 Projects Releases Wiki Activity

soucet_bodu #70

Merged
ledoian merged 2 commits from soucet_bodu into master 2024-11-05 22:27:55 +01:00
Conversation 0 Commits 2 Files changed 2 +1 -2
1 changed files with 1 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files
Showing only changes of commit 0e83f96318 - Show all commits

3
odevzdavatko/views.py
View file

@ -354,10 +354,9 @@ class PrehledOdevzdanychReseni(ListView):
for rocnik, hodnoceni in groupby(ctx['object_list'], lambda ho: ho.deadline_body.cislo.rocnik if ho.deadline_body is not None else None):
suma_bodu = 0
hodnoceni = list(hodnoceni)
hodnoceni[0].body = None
for i in hodnoceni :
if i.body != None : suma_bodu += i.body
ledoian commented 2024-11-05 22:26:19 +01:00
Review
Copy link

Jen poznámka: None je lepší porovnávat pomocí is (resp. is not): if i.body is not None: ...

(Je to běžnější kód a možná lehce rychlejší)

Jen poznámka: `None` je lepší porovnávat pomocí `is` (resp. `is not`): `if i.body is not None: ...` (Je to běžnější kód a možná lehce rychlejší)
podle_rocniku.append((rocnik, list(hodnoceni),suma_bodu))
podle_rocniku.append((rocnik, hodnoceni, suma_bodu))
ctx['podle_rocniku'] = reversed(podle_rocniku) # Od nejnovějšího ročníku
# TODO: Umožnit stažení / zobrazení řešení