Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Koodikatselmointi #2

Open
kafenoir opened this issue Apr 25, 2020 · 0 comments
Open

Koodikatselmointi #2

kafenoir opened this issue Apr 25, 2020 · 0 comments

Comments

@kafenoir
Copy link

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!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant