Tänään on ollut vähän kaikenlaista tässä, enkä muutenkaan ole varsinainen PHP-asiantuntija, joten tyydyn antamaan vähän pintapuolisemman katselmoinnin listamuodossa:
En päässyt kokeilemaan sisäänkirjautumista kun en tiennyt käyttäjätunnuksia ja salasanoja. (Ehkä ne olivat jossain mainittu, mutten niitä löytänyt.) Kuitenkin näin sivumennen mainiten huomasin, että kirjautumislomake näyttää virheilmoituksen puuttuvasta käyttäjätunnuksesta/salasanasta jo sivun ensimmäisellä latauskerralla, siis ennen kuin käyttäjä on mitään ehtinyt syöttää.
Koodin yleisilme oli hyvä, mutta olisin toivonut enemmän dokumentointia ja selkeästi nimettyjä aliohjelmia.
Esimerkiksi kontrollerit olivat minulle vähän vaikealukuisia (ehkä koska en PHP:ta kovin paljoa tunne). Nähdäkseni esimerkiksi tiedoston "Signin.php" voisi jakaa esimerkiksi kolmeen aliohjelmaan, _tarkastasyote, rekisteroi, ja kirjaudu, jotka tekisivät nimensä mukaisia asioita. Myös esimerkiksi tiedoston "Mypage.php" rungossa oleva koodi vaikuttaisi tekevän vähän päällekkäistä työtä funktion "deleteFromUser" kanssa. Voisikohan 'POST'-parametrin käsittelyn hoitaa pelkästään toisessa paikassa?
Tietokannan osalta en lähde toistamaan edellisessä katselmoinnissa esiin otettua kysymystä taulujen "Pokemon" ja "trainer_pokemon" osalta. Sen sijaan haluaisin tuoda esille, että ainakin SQL-lauseiden merkityksen hahmottamisen kannalta olisi hyvä jos yhdistelmätauluissa olisi kirjoitettu avainrajoitteet näkyviin. Esimerkiksi taulun "poke_moves" CREATE TABLE-lauseen loppuun voisi nähdäkseni lisätä rajoitteen PRIMARY KEY (poke_id, move_id) (ainakin jos käytössä on PostgreSQL).
Lopuksi vielä mainitsen että minua hieman jäi mietityttämään taulun "db_user" salasanakenttä. Miksi se on tasan 64 merkin merkkijono? Vai tallennetaanko siihen kenties vakiomittainen salattu salasana/salasanatiiviste? Tämä tietysti olisi ainakin tosielämässä varsin suotavaa.
Eipä tässä nyt enempää noussut asioita mieleen. Ihan toimivan näköistä settiä oli ja selvästi ohjeita oli noudatettu (joka todallakin on viisas veto :D). P.S. Latasin koodin siinä kahdentoista aikoihin päivällä.
P.P.S Unohdin mainita että olisi ehkä ajatuksellisesti selkeämpää jos tiedostossa "libs/common.php" funktio "isUserLogged" palauttaisi enumin kun vaihtoehtoja on kolme.*
Tänään on ollut vähän kaikenlaista tässä, enkä muutenkaan ole varsinainen PHP-asiantuntija, joten tyydyn antamaan vähän pintapuolisemman katselmoinnin listamuodossa:
CREATE TABLE
-lauseen loppuun voisi nähdäkseni lisätä rajoitteenPRIMARY KEY (poke_id, move_id)
(ainakin jos käytössä on PostgreSQL).Eipä tässä nyt enempää noussut asioita mieleen. Ihan toimivan näköistä settiä oli ja selvästi ohjeita oli noudatettu (joka todallakin on viisas veto :D). P.S. Latasin koodin siinä kahdentoista aikoihin päivällä.
P.P.S Unohdin mainita että olisi ehkä ajatuksellisesti selkeämpää jos tiedostossa "libs/common.php" funktio "isUserLogged" palauttaisi enumin kun vaihtoehtoja on kolme.*