Koodikatselmointi

[gitblast]

Latasin projektin 25.4. klo 17.35

Kommentteja

Muuttujat, funktiot ym. on merkitty järkevillä ja kuvaavilla nimillä.

Dokumentaatio on yleisesti ottaen melko selkeä, mutta voisi ehkä olla hieman kattavampi. Esimerkiksi tietokannan rakennetta voisi olla järkevää selventää dokumentaatiossa.

Ylempään kommenttiin liittyen: Voi olla, että missaan jotain, mutta ymmärtääkseni ResultConditioning ja ResultStrength -taulujen sisällön voisi tallentaa suoraan Result -tauluun.

Uusia tuloksia lisätessä käyttäjän tulee antaa tulokset yhteen tekstikenttään tietyssä muodossa ja enterillä eroteltuina; käyttäjän kannalta olisi miellyttävämpää, että jokaiselle tulokselle olisi oma kenttänsä. Tämän voisi saada aikaan esimerkiksi näin:

new_results.html

{% for n in range(rounds) %}
    <input name="tulos{{n}}" type="text">
{% endfor %}

Nyt backendissä tuloksiin pääsisi käsiksi näin:

application/result/log/views.py

form = ResultForm(request.form)

for n in range(rounds):
    resultField = getattr(form, 'tulos' + str(n))
    result = resultField.data
    // tee jotain kyseisellä tuloksella

Herokussa sekä "Personal records" että "Activity stats" -linkkien klikkaaminen tuottaa Internal Server Errorin. Myöskään superuser -tunnukset eivät toimineet Herokussa.

Yleisesti ottaen projekti oli kurssimateriaalin käytäntöjen mukaan rakennettu ja kehitys hyvässä vaiheessa. Projektista löytyi myös monimutkaisempaa toiminnallisuutta kuten käyttäjien roolitus usergrouppien avulla ja salasanojen hashays ennen tietokantaan tallennusta, näistä iso plussa!

Kaiken kaikkiaan hyvää työtä.

[korolainenriikka]

Kiitos tuosta koodinpätkästä!

Pohdin muuttuvien tulosmäärien formia monta päivää, nyt toimii hieman tuosta muunnetulla systeemillä!