Sovellus toimii tämänhetkisillä toiminnoilla moitteetta. Rakenne on vielä vähän sekava, mutta se tuskin on olennaista vielä tässä vaiheessa (esim. home-linkki home-sivulla, index-otsikon alla ei ole mitään). Pääasia on kuitenkin toimivuus. Käyttäjien luonti onnistuu ja myöskin ryhmät sekä invitet toimivat. Viesteissä lähettäjä sekä lähetysaika olisivat olennaisia vielä puuttuvia ominaisuuksia selkeyden kannalta, mutta tietokannan rakenteen perusteella vaikuttaa, että ne mahdollisesti lisätään myöhemmin.
Koodi vaikuttaa selkeältä ja tiiviiltä. Myöskin tietoturvaa on jo osin käsitelty, kuten salasanojen hashaus, sql parametrisointi jne. Sovelluksen eri kansioissa oli samalla nimellä eri tiedostoja kuten groups.py monessa kansiossa, tämä aiheutti lievää hämmennystä koodia tulkitessa. Mahdollisten virhetilanteiden käsittelyä voisi harkita Try-except-rakenteella, sillä jos oletettaisiin että sovellus laajenee ja skaalautuu suuremmaksi niin myöskin satunnaisten virheiden mahdollisuus kasvaa (esim. palvelin alhaalla, käyttäjä poistetaan ryhmästä samalla hetkellä, kun hän lähettää viestiä jne.) Tietorakenne vaikuttaa selkeältä ja riittää hyvin sovellukselle, mutta suuremmilla tietomäärillä sitä voisi ehkä optimoida lisäindekseillä. Kokonaisuutena vaikuttaa oikein hyvältä tässä vaiheessa.
DoebelnAtz
commented
1 week ago
Sovellus toimii tämänhetkisillä toiminnoilla moitteetta. Rakenne on vielä vähän sekava, mutta se tuskin on olennaista vielä tässä vaiheessa (esim. home-linkki home-sivulla, index-otsikon alla ei ole mitään). Pääasia on kuitenkin toimivuus. Käyttäjien luonti onnistuu ja myöskin ryhmät sekä invitet toimivat. Viesteissä lähettäjä sekä lähetysaika olisivat olennaisia vielä puuttuvia ominaisuuksia selkeyden kannalta, mutta tietokannan rakenteen perusteella vaikuttaa, että ne mahdollisesti lisätään myöhemmin.
Koodi vaikuttaa selkeältä ja tiiviiltä. Myöskin tietoturvaa on jo osin käsitelty, kuten salasanojen hashaus, sql parametrisointi jne. Sovelluksen eri kansioissa oli samalla nimellä eri tiedostoja kuten groups.py monessa kansiossa, tämä aiheutti lievää hämmennystä koodia tulkitessa. Mahdollisten virhetilanteiden käsittelyä voisi harkita Try-except-rakenteella, sillä jos oletettaisiin että sovellus laajenee ja skaalautuu suuremmaksi niin myöskin satunnaisten virheiden mahdollisuus kasvaa (esim. palvelin alhaalla, käyttäjä poistetaan ryhmästä samalla hetkellä, kun hän lähettää viestiä jne.) Tietorakenne vaikuttaa selkeältä ja riittää hyvin sovellukselle, mutta suuremmilla tietomäärillä sitä voisi ehkä optimoida lisäindekseillä. Kokonaisuutena vaikuttaa oikein hyvältä tässä vaiheessa.