Koodikatselmointi
Opened this issue · 1 comments
sokkanen commented
.zip ladattu sunnuntaina 10.2.2019 klo 10:45
Lähdekoodi
- Ohjelman kansiorakenne on selkeä. Tiedostot on nimetty oikeaoppisesti.
- Koodisi on erittäin selkeää. Koodia tarkkailemalla selviää sen käyttötarkoitus. Olet monessa kohdassa jakanut koodin toiminnallisiin lohkoihin, mikä on erittäin hyvä ja selkeä tapa.
- Olet käyttänyt loginform.html:ssä vanhentuneeksi merkittyä "center" -elementtiä. Tyylit voi hoitaa css:llä tai bootstrapilla Codeply:n demo bootstrapin keskitystoiminnoista.
- Kokonaisuudessaan lähdekoodi on oikein luettavaa ja hyvää.
Toimivuus ja käytettävyys
- Ohjelman rakenne on selkeä, jolloin käyttäjä osaa tehdä perustemput ilman käyttöohjettakin.
- Projekti lähti toimimaan paikallisesti ilman ongelmia.
- Käyttäjähallinta (esim. sähköpostin ja salasanan päivítys) olisi hyvä. Tämä lienee tulossa seuraaviin versioihin.
- Rekisteröinti ja reseptin lisääminen validoivat syötteet, ja käyttäjälle annettava palaute on selkeä!
- Reseptin lisääminen onnistuu hienosti.
- ”All recipes” -sivun muotoilua voisi hieman viilata – ja varmasti viilaatkin. Nykyisellään yksinkertainen kiljureseptikin menee melkoiseksi tekstiseinäksi.
- Olisiko mahdollista erotella otsikko, kuvaus, ainesosat, ohje ja ”extra tips”? Tällöin listaukseen voisi lisätä vain otsikon ja mahdollisen kuvauksen, jolloin siitä tulisi selkeämpi. Esim:
| Pulla |
|---|
| Helppo pullaresepti opiskelijalle. |
Kokonaisuudessaan ohjelmasi näyttää ja tuntuu jo hyvältä. Tsemppiä!
jonitaajamo commented
Tarkka katselmointi, kiitos tästä!