Koodikatselmointi

Koodikatselmointi repolle https://github.com/KristianLauttamus/HospitalSchedule

Projekti ladattu 8.2.2016 klo. 14.29

Ilmeisesti kolmannen viikon PHP toiminnallisuutta ei ole vielä toteutettu ja sivut ovat staattisia. Esimerkkisivut ovat hyvän näköiset ja miellyttävät. Ilmeisesti monta sivua pitää vielä toteuttaa toiminnallisuuksineen.

Koodit on sisennetty ja aika lailla ohjeiden mukaan tehty. Itseäni tosin hieman kalvaa tab sisenykset välilyöntien sijaan. Omasta koodistanikin niitä kyllä varmasti löytyy kun editorit ja valmiit koodipätkät niitä aiheuttavat. - Ei siis juuri mitään ehdotuksia parantamiseksi.

Tietokanta vaikuttaa ihan hyvältä - vaikkei selvästikään lopullinen, ellei monia aihemäärittelyn ominaisuuksia karsita. Testidataa ei ollut kuin yksi rivi, vaikka tietokannassa on neljä taulua - lisää dataa!

Kokeilin mennä osoitteeseen https://hospital-schedule.herokuapp.com/tietokantayhteys
Mutta kyseinen osoite antaa virheilmoituksen. Tietokannassa herokussa on siis jotakin ongelmaa. - Kannattaa korjata tämä.