soucet_bodu #70

Merged
ledoian merged 2 commits from soucet_bodu into master 2024-11-05 22:27:55 +01:00
2 changed files with 9 additions and 3 deletions
Showing only changes of commit e660a96df2 - Show all commits

View file

@ -12,7 +12,7 @@
<br>
{% for rocnik, hodnoceni in podle_rocniku %}
{% for rocnik, hodnoceni, suma_bodu in podle_rocniku %}
<h1>Ročník {{ rocnik }}</h1>
<table class="moje_reseni plne_ohranicena_tabulka">
<tr>
@ -33,7 +33,7 @@
</tr>
{% endfor %}
</table>
<p>Celkový počet bodů {{suma_bodu}}</p>
<br>
{% endfor %}

View file

@ -352,7 +352,13 @@ class PrehledOdevzdanychReseni(ListView):
# Chceme to mít seřazené, takže místo comphrerehsion ručně postavíme pole polí. Django templates neumí použít OrderedDict :-/
podle_rocniku = []
for rocnik, hodnoceni in groupby(ctx['object_list'], lambda ho: ho.deadline_body.cislo.rocnik if ho.deadline_body is not None else None):
podle_rocniku.append((rocnik, list(hodnoceni)))
suma_bodu = 0
hodnoceni = list(hodnoceni)
hodnoceni[0].body = None
for i in hodnoceni :
Review

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ší)
if i.body != None : suma_bodu += i.body
podle_rocniku.append((rocnik, list(hodnoceni),suma_bodu))
ctx['podle_rocniku'] = reversed(podle_rocniku) # Od nejnovějšího ročníku
# TODO: Umožnit stažení / zobrazení řešení
return ctx