Koodikatselmointi

Ladattu 2.9.2019 klo 20.53

Yleistä

Projektisi on todella laaja, joten sitä on hieman vaikea sisäistää täysin. Koodi on selkeää ja helposti ymmärrettävää, mutta johtuen sovelluksen laajuudesta, helpottaisi jos sitä olisi kommentoitu hieman enemmän. Esimerkiksi match.views -tiedoston kommentoidut osuudet auttoivat koodin ymmärtämistä huomattavasti. Niiltä osin, jotka olet ehtinyt saada valmiiksi työsi vaikuttaa laadukkaalta ja toimivalta käyttötarkoitukseensa. Erityisesti ottelujen luonti ja niiden ajankohdan lisääminen on hyödyllinen ominaisuus erilaisten urheilusarjojen pyörittämiseen.

Tietokantarakenne

Tietokantarakenne vaikuttaa turhan monimutkaiselta taulujen roles ja account osalta. Miksi käyttäjän roolia ei ole ainoastaan tehty osaksi account-taulua tai vaihtoehtoisesti tehty roles-taulusta ainoastaan taulua, jossa eri roolit ovat ja account-taulusta olisi aina viite kyseisen käyttäjän rooliin. Olitkin kommentoinut tuota readmessa, joten asia on sinulla tiedossa.

Kauden lisäyksessä väärä url-osoite

Lokaalisti asiasta tulee seuraava virhe: werkzeug.routing.BuildError: "Could not build url for endpoint 'match_list'. Did you mean 'match_start' instead?"
En tiedä olitko tietoinen tuosta ongelmasta, mutta jos tarkoituksena ei ole myöhemmin ohjata käyttäjää kyseiseen osoitteeseen, kannattaa korjata osoite.

Joukkueen lisääminen

Joukkueen voi käydä lisäämässä, vaikka ei olisikaan kirjautunut. Johtuuko siitä, että kapteenin roolia ei ole vielä luotu?

Käyttötapaukset

Kävin kaikki käyttötapaukset ja niiden koodin lävitse ja en löytänyt koodista virheitä. Validaatiot ovat kunnossa, login required -tagit on lisätty käyttötapausten ja sovelluksen kuvauksen mukaisesti ja haut toimivat. Lisäksi käyttötapaukset toimivat sekä Herokussa että lokaalisti, kuten pitääkin.

Käyttötapaukset-dokumentti

Tähän kannattaisi lisätä SQL-koodin eteen SQL-tagit, jotta koodi näkyisi oikeassa muodossa. Nyt varsinkin pidempien kyselyiden lukeminen on melko työlästä, kun kaikki rivinvaihdot ja korostukset puuttuvat.

Lopuksi

Todella hienosti toteutettu sovellus ja siinä mielessä mukava katselmoida, että se ilmeisesti otetaan joskus myös käyttöön, eikä sitä ole tehty ainoastaan tätä kurssia varten.