You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Ohjelman eri osat oli jaettu loogisesti omiksi kokonaisuuksikseen ja eri tietokantatauluihin liittyvät toiminnallisuudet oli helppo löytää. Mietitytti tosin, tarvitseeko liitostauluille välttämättä luoda omia kansioitaan ja models-tiedostojaan.
Ohjelman koodi oli helppolukuista. Muuttujat, funktiot ja metodit oli loogisesti nimetty, eikä niiden toimintaa tarvinnut arvuutella. Luokat kuvasivat tarkasti tietokantakaaviossa määriteltyjä tauluja ja SQL-kyselyt olivat järkeviä. Lisäksi koodi oli hyvin sisennetty.
Tiedostossa application/pizzas/views.py sekä kansion application/templates/pizzas html-tiedostoissa oli jonkun verran toisteista koodia liittyen pizzan neljään täytteeseen. Näissä voisi yrittää vielä refaktoroida. Jinjan makroihin löytyy ohje täältä.
Kokeilin ohjelmaa omalla koneellani sekä herokussa.
Huomioita ohjelman käytöstä:
Ohjelman ulkoasu oli hyvin siisti ja bootstrapin komponentteja oli hyödynnetty erinomaisesti. Erityisesti pidin siitä, miten ”jumbotronit”, laatikkoelementit ja symbolien käyttö napeissa eläivöittivät sivuja.
Käyttäjäroolit ja niiden oikeudet toimivat ongelmitta. Ensimmäisen admin-oikeuden asettamiseen liittyen oli asennusohjeeseen tosin lipsahtanut pieni virhe. SQLitessa boolean-sarakkeeseen asetettava arvo on 1 tai 0.
Adminin toiminnot löytyivät helposti, ja täytteiden ja pizzojen luominen, muokkaaminen ja poistaminen oli yksinkertaista. Käyttäjäpuolella pizzojen tilaaminen onnistui hyvin, mutta suuremmalle pizzavalikoimalle toivoisi näkymäksi perinteisempää listaa, josta koko tarjonta olisi helposti nähtävissä yhdellä silmäyksellä.
Ihmettelin alkuun, kun olin lisännyt ananaksen ainekseksi ja pizzaan, mutta aineksen nimeen sidottu jumbotron ei näkynyt. Tajusin tämän johtuvan siitä, että olin kirjoittanut ananaksen isolla alkukirjaimella. Voisikin olla hyvä, jos ainesten nimet muunnettaisiin tietokantaan siirrettäessä aina kokonaan pienistä kirjaimista koostuviksi, etenkin kun tiettyihin merkkijonoihin on liitetty erikoistoiminnallisuutta. Lisäksi voisi olla hyvä estää samojen ainesten lisääminen tietokantaan useampaan kertaan, mikä on tällä hetkellä mahdollista.
Kokonaisuudessaan projektisi näyttää todella hyvältä. Tsemppiä viimeiseen rutistukseen!
The text was updated successfully, but these errors were encountered:
Koodikatselmointi
Projekti ladattu 25.4.2020 klo 16:27
Huomioita koodista:
Ohjelman eri osat oli jaettu loogisesti omiksi kokonaisuuksikseen ja eri tietokantatauluihin liittyvät toiminnallisuudet oli helppo löytää. Mietitytti tosin, tarvitseeko liitostauluille välttämättä luoda omia kansioitaan ja models-tiedostojaan.
Ohjelman koodi oli helppolukuista. Muuttujat, funktiot ja metodit oli loogisesti nimetty, eikä niiden toimintaa tarvinnut arvuutella. Luokat kuvasivat tarkasti tietokantakaaviossa määriteltyjä tauluja ja SQL-kyselyt olivat järkeviä. Lisäksi koodi oli hyvin sisennetty.
Tiedostossa application/pizzas/views.py sekä kansion application/templates/pizzas html-tiedostoissa oli jonkun verran toisteista koodia liittyen pizzan neljään täytteeseen. Näissä voisi yrittää vielä refaktoroida. Jinjan makroihin löytyy ohje täältä.
Kokeilin ohjelmaa omalla koneellani sekä herokussa.
Huomioita ohjelman käytöstä:
Ohjelman ulkoasu oli hyvin siisti ja bootstrapin komponentteja oli hyödynnetty erinomaisesti. Erityisesti pidin siitä, miten ”jumbotronit”, laatikkoelementit ja symbolien käyttö napeissa eläivöittivät sivuja.
Käyttäjäroolit ja niiden oikeudet toimivat ongelmitta. Ensimmäisen admin-oikeuden asettamiseen liittyen oli asennusohjeeseen tosin lipsahtanut pieni virhe. SQLitessa boolean-sarakkeeseen asetettava arvo on 1 tai 0.
Adminin toiminnot löytyivät helposti, ja täytteiden ja pizzojen luominen, muokkaaminen ja poistaminen oli yksinkertaista. Käyttäjäpuolella pizzojen tilaaminen onnistui hyvin, mutta suuremmalle pizzavalikoimalle toivoisi näkymäksi perinteisempää listaa, josta koko tarjonta olisi helposti nähtävissä yhdellä silmäyksellä.
Ihmettelin alkuun, kun olin lisännyt ananaksen ainekseksi ja pizzaan, mutta aineksen nimeen sidottu jumbotron ei näkynyt. Tajusin tämän johtuvan siitä, että olin kirjoittanut ananaksen isolla alkukirjaimella. Voisikin olla hyvä, jos ainesten nimet muunnettaisiin tietokantaan siirrettäessä aina kokonaan pienistä kirjaimista koostuviksi, etenkin kun tiettyihin merkkijonoihin on liitetty erikoistoiminnallisuutta. Lisäksi voisi olla hyvä estää samojen ainesten lisääminen tietokantaan useampaan kertaan, mikä on tällä hetkellä mahdollista.
Kokonaisuudessaan projektisi näyttää todella hyvältä. Tsemppiä viimeiseen rutistukseen!
The text was updated successfully, but these errors were encountered: