Kuten readmessa on kerrottu näyttää sovellus vielä olevan hyvin alkuvaiheessa. Tapahtuman lisäys ei toimi vielä. Omistajan, tilin tai osakkeen lisäys ei vielä näytä tekevän mitään. Sivut näyttävät kuitenkin hyviltä ja linkit avautuvat sujuvasti.
Yksi parannusehdotus voisi olla, että etusivulla voisi olla lyhyt kuvaus, miten sovellusta tulee käyttää ja sivuja voisi myös siistiä näyttämään paremmalta. Esimerkiksi linkkien sijaan voitaisiin tehdä niistä painikkeita. Kun omistajan, tilin tai osakkeen lisää voisi tulostua ilmoitus onnistuneesta lisäyksestä. Jos omistajaa, tiliä tai osaketta ja näiden nimeä ei ole annettu voisi tulostua virhe viesti, joka sanoo tapahtuman epäonnistuneen sekä syyn. Saman voisi tehdä tapahtuman lisäykselle, kun tämä toiminnallisuus on saatu valmiiksi. Rekisteröinnin ja kirjautumisen voisi myös toteuttaa.
Sovelluksen koodi
Route.py ja event.py näyttävät hyvältä. Kommentoidun koodin voisi toki jättää pois, jotta koodi olisi siistimpää, tai selittää kommentoidun koodin toiminnallisuus paremmin. Monet html tiedostot näyttävät olevan tyhjiä. Näiden lisäämistä GitHubiin voisi välttää, jotta koodista tulisi selkeämpää.
Parannusehdotukset:
Yksityiskohtaisempia kommentteja tai docstringejä kuvaamaan funktioiden tarkoitusta ja toiminnallisuutta.
Tulostuslausekkeiden (print) korvaaminen logging lausekkeilla, jotta voit hallita tulostusta paremmin.
Virheenkäsittelyn parantaminen.
Käyttäjien syötteiden asianmukainen validointi. Varmista esimerkiksi, että syötetty päivämäärä on oikeassa muodossa ennen sen käsittelyä.
CSRF-suojauksen käyttöönotto lomakkeissa, erityisesti POST-pyyntöjen osalta.
Tietokannan rakenne
Virhe "relation 'events' does not exist", esiintyy kun tietokannan alustaa. Tämä osoittaa, että "pairing"-tauluun on yritetty luoda "table reference", joka viittaa "events"-tauluun, mutta "events"-taulua ei ole määritelty annetussa skeemassa. Tämän ratkaisemiseksi on varmistettava, että viitattu taulukko on olemassa. Tässä tapauksessa on ehkä luotava "events"-taulu, jossa yhdistyvät sekä "buy_events"- että "sell_events"-taulujen sarakkeet ja rakenne, ennen kuin viittaa siihen "pairing"-taulussa.
Tietokannasta on vaikea antaa muita kommentteja sillä, niissä ei vielä ole paljoa tietoa.
Muuten koodi näyttää hyvältä ja se on jaettu hyvin eri hakemistoihin.
Sovellus tuotannossa
Kuten readmessa on kerrottu näyttää sovellus vielä olevan hyvin alkuvaiheessa. Tapahtuman lisäys ei toimi vielä. Omistajan, tilin tai osakkeen lisäys ei vielä näytä tekevän mitään. Sivut näyttävät kuitenkin hyviltä ja linkit avautuvat sujuvasti.
Yksi parannusehdotus voisi olla, että etusivulla voisi olla lyhyt kuvaus, miten sovellusta tulee käyttää ja sivuja voisi myös siistiä näyttämään paremmalta. Esimerkiksi linkkien sijaan voitaisiin tehdä niistä painikkeita. Kun omistajan, tilin tai osakkeen lisää voisi tulostua ilmoitus onnistuneesta lisäyksestä. Jos omistajaa, tiliä tai osaketta ja näiden nimeä ei ole annettu voisi tulostua virhe viesti, joka sanoo tapahtuman epäonnistuneen sekä syyn. Saman voisi tehdä tapahtuman lisäykselle, kun tämä toiminnallisuus on saatu valmiiksi. Rekisteröinnin ja kirjautumisen voisi myös toteuttaa.
Sovelluksen koodi
Route.py ja event.py näyttävät hyvältä. Kommentoidun koodin voisi toki jättää pois, jotta koodi olisi siistimpää, tai selittää kommentoidun koodin toiminnallisuus paremmin. Monet html tiedostot näyttävät olevan tyhjiä. Näiden lisäämistä GitHubiin voisi välttää, jotta koodista tulisi selkeämpää.
Parannusehdotukset:
Yksityiskohtaisempia kommentteja tai docstringejä kuvaamaan funktioiden tarkoitusta ja toiminnallisuutta.
Tulostuslausekkeiden (print) korvaaminen logging lausekkeilla, jotta voit hallita tulostusta paremmin.
Virheenkäsittelyn parantaminen.
Käyttäjien syötteiden asianmukainen validointi. Varmista esimerkiksi, että syötetty päivämäärä on oikeassa muodossa ennen sen käsittelyä.
CSRF-suojauksen käyttöönotto lomakkeissa, erityisesti POST-pyyntöjen osalta.
Tietokannan rakenne
Virhe "relation 'events' does not exist", esiintyy kun tietokannan alustaa. Tämä osoittaa, että "pairing"-tauluun on yritetty luoda "table reference", joka viittaa "events"-tauluun, mutta "events"-taulua ei ole määritelty annetussa skeemassa. Tämän ratkaisemiseksi on varmistettava, että viitattu taulukko on olemassa. Tässä tapauksessa on ehkä luotava "events"-taulu, jossa yhdistyvät sekä "buy_events"- että "sell_events"-taulujen sarakkeet ja rakenne, ennen kuin viittaa siihen "pairing"-taulussa.
Tietokannasta on vaikea antaa muita kommentteja sillä, niissä ei vielä ole paljoa tietoa.
Muuten koodi näyttää hyvältä ja se on jaettu hyvin eri hakemistoihin.