katri96 / tsoha-bootstrap Goto Github PK

_{Ladattu 18:30 / 23.9.2017}

Koodista:

• Koodi on kaiken kaikkiaan hyvin sisennettyä, selkeää ja ymmärrettävää. 👍
• Niin model, view, kuin controller luokat ovat edistyneet MVC-mallin mukaisesti.
• Polut (routes.php) vaikuttavat loogisilta.
• create_tables.sql kertoo, että sovellukseen on alustettu viisi tietokantataulua. Dokumentoinnissa raaka-aineen ja reseptin välillä oli monesta moneen suhde, joten niiden välille kannattaa luoda vielä liitostaulu.

Sovelluksesta:

• Tässä vaiheessa sovelluksessa navigointi oli vielä hieman hankalaa. Tsoha-Bootstrap -pohjan mukana tulleen ylänavin tai muun vastaavan säilyttäminen sovelluksen kehitysvaiheessa olisi yksinkertaistanut sivuilla liikkumista.
• Katselmoinnin hetkellä vain pari sovelluksen näkymistä oli toiminnassa.
• /resepti sivu sisältää ilmeisesti vielä Kint tulostuksia, mitkä vähän rikkovat ulkoasua.

Muut:

• Dokumentointi oli hyvää ja selkeää.
• README filun ”Linkki suunniteltuihin sivuihin” osion linkit voisi nimetä niiden sovelluksen näkymien mukaan joihin linkit johtavat. Lähinnä selkeyden vuoksi. Kannattaa tähän myös lisätä muita valmiita näkymiä, jos on.

Yleisiä huomioita

• Sovellus vaikuttaa keskeneräiseltä. Ainakaan tominallisuudet eivät olleet vielä valmiita. Edellinen "commit" taisi olla 7pv. sitten, joten sovelluksen jatkokehityksen uskon siirtyneen tulevaisuuteen.
• Oli hyvä, että näkymien, kontrollerien, attribuuttien jne. nimet olivat pääosin kaikki suomeksi. Alussa meinaa helposti mennä sekaisin.
• Muuten voin yhtyä 1. koodikatselmoinnin kommentteihin.

Huomioita koodista

• Luokkien ja metodien nimet suht loogisia
• Koodi sisennetty pääosin hyvin
• Monta toiminnallista kokonaan toteuttamatta esim. kirjautuminen ja reseptin lisääminen. Sovellus siis keskeneräinen.
• HTML-tiedostoissa ei ilmeisesti ole otettu bootstrap-layouttia käyttöön? Näin olleen esim. navigaatio yläpalkista puuttuu. Bootstrap-layout hoitaa aika monta juttua automaattisesti.