olegtervo / fastreserve Goto Github PK

Koodi näyttää ymmärrettävältä ja se on selkeästi järjestetty omiin kokonaisuuksiinsa. Tosin routet näyttävät olevan modelien sisällä, erottaisin routet omaksi kokonaisuudeksi pois modeleista, jotta niiden vastuulla on vain toimia controllerina ja modelit voivat toimia vain modeleina.

Tällä tavalla, jos routet kutsuvat modeleita vain rajapinnan kautta tyyliin saveItem, getItem, jne, ei niiden tarvitse tietää mitään modelien sisäisestä implementaatiosta ja koodi on modulaarisempaa ja eri kokonaisuuksien, kuten modelien tai controllereiden muuttaminen on joustavampaa.

Orders -modelissa funktio newOrderInGroup tekee useamman tietokantaoperaation, mikä on vähän vaivalloista. Tällaisista tilanteista pääsee kätevästi eroon käyttämällä jotakin ORM-kirjastoa, kuten SqlAlchemyä. Kirjastoilla saa helpommin ja nopeammin toteutettua monen suhde moneen / yhden suhde moneen / ym relaatiot ja voi välttyä isojen sql-kyselyiden kirjoittamiselta, kun kirjasto hoitaa mappaukset puolestasi aika vaivattomasti.

Kokeilin applikaatiota kirjautumalla sisään sekä admin-käyttäjänä, että luomalla oman käyttäjän. Kaikki operaatiot toimivat kuten pitää ja applikaation käyttö tuntui selkeältä. Hyvää työtä, jatka samaan malliin!

Hei,

Kävin kirjautumassa sovellukseesi. Kirjautuminen vaikuttaa toimivan. Ilmeisesti ei muuta toiminnallisuutta ole joten ei siitä enempää.
-koodi vaikuttaa siistiltä enkä ainakaan huomannut siinä virheitä.
-README.md ei ainakaan minulle avaa mitä toimintoja olet tarkkaan ottaen suunnitellut toteuttavasi. Siihen varmaan kannattaisi kirjata tarkka suunnitelma ehkä jopa siinä järjestyksessä miten asiat tuntuisi järkevältä toteuttaa.
-schema.sql sisältää vasta 2 taulua. Kannattaa aloittaa työ siitä että miettii miten asiat tietokantaan tallennetaan. Se selventää sovelluksen rakennetta kokonaisuutena.
Lähtisin itse liikkeelle pistämällä edellä mainitut filet vastaamaan suunnitelmaasi niin on paljon helpompi edetä määrätietoisesti kohti valmista sovellusta.

Työn iloa! :)

Terveisin,
Jorma