northernlion / phppelikirjasto2017 Goto Github PK

Latasin tiedostot ja aloitin katselmoinnin torstaina 30.3 klo 14.

Ohjelman koodi oli selkeästi kirjoitettua ja helposti ymmärrettävissä. Ohjelman palaset oli fiksusti jaettu omiin kokonaisuuksiinsa esim. html-tiedostot oli jaettu kansioihin niiden roolin mukaan. Luokka routes.php, joka on väistämättä melko pitkä luokka, oli kommenttien avulla jaettu osiin, joka teki sen lukemisesta helpompaa. Ehkä model-luokat kuten game.php voisi kommenttien avulla jakaa osiin, jos siitä tulee tulevaisuudessa pidempi?

Kaikki luokat, aliohjelmat ja muuttujien nimet oli kirjoitettu englanniksi, joka on hyvä koodin selkeyden kannalta.

Jäljellä oli vielä luokkia kuten hello_world.php, jotka eivät tee mitään. Nämä tullaan varmaan poistamaan ennen projektin valmistumista.

Sivulla pelit/strategy oli linkki strategian julkaisseen pelaajan sivulle. Linkki oli rikki sillä sen pitäisi ilmeisesti osoittaa sivulle /pelit/user/1 eikä sivulle /pelit/player/1 ? Linkissä voisi lukea sen pelaajan nimi, jonka sivulle linkki johtaa. Tämän tosin vaatii ylimääräisen tietokantapyynnön pelaajan nimen saamiseen id:n perusteella.

Sivulla /pelit/club oleva linkki klubin sivulle ei jostain syystä toimi.

Pelin esittelysivulla ei näy peliin liittyviä strategioita, vaikka niiden pitäisi koodin perusteella siinä näkyä?

Kaiken kaikkiaan ohjelma näytti olevan jo hyvässä vaiheessa.