Tere!

Yleiskatsaus

MVC-mallin toteutus näytti ihan hyvältä, tosin voisi hakemistoa selkeyttää jos controlleri tiedostotkin olisivat omassa kansiossaan. Tai vaihtoehtoisesti jos rootissa ei olisi muita kuin controlleri tiedostoja, nythän siellä on seassa yhteystestejä yms.

Käyttöystävällisyys ja ulkoasu

Huomasin että sinulla tällä hetkellä haku nimellä on case-sensitive ja toimii vain kokonaisilla sanoilla. Case-sensitivity on helppo korjata käyttämällä postgresql:n ilike komentoa = vertailun sijaan, kts. http://www.postgresql.org/docs/8.3/static/functions-matching.html

The key word ILIKE can be used instead of LIKE to make the match case-insensitive according to the active locale. This is not in the SQL standard but is a PostgreSQL extension.

Halutessaan voi hakusanaan lisätä eteen ja taakse %, jolloin haku etsii myös kohteita jotka sisältävät haetun merkkijonon. Esimerkki hausta

if ($nimi !== "") {
    $sql .= "SELECT * FROM pokemonlaji WHERE nimi ILIKE ? INTERSECT ";
    $parametrit[] = '%' . $nimi . '%';
}

Mahdollisesti kannattaisi vilkaista autoloaderin tekemistä, niin pääsisi eroon suuresta osasta require komennoista. http://www.php.net/manual/en/language.oop5.autoload.php

Sinulla myös versionhallinnassa näkyvissä tietokantayhteys tiedostoissa on esillä omat tietokanta käyttäjätunnuks ja salasana, nämä kannattaa poistaa vaikket niitä varmaan muualla käytä kun ne luotiin automaattisesti. Postgresql käyttäessä usersilla voit käyttää molemmissa vaikka null ja tietokanta silti toimii.

Erroreista voisi tehdä siistimmät laittamalle ne yhden alert boxin sisään, tällä hetkellä esim. pokémonia lisätessä voi pahimmillaan tulla 8 erillistä alert boxia jotka vievät yksinään jo lähes puolet sivusta. Esimerkki miten voisit kyseisen virhe ilmoituksen tehdä, (esimerkissä vaihdoin myös niin että <div class="alert"> ... </alert> on <div class="container"> ... </div> sisällä, jotta se myös on sisennetty niin kuin muu sivusto eikä ole laidasta laitaan.

// Näin saisit viestit samaan alert boxiin, sitten vain listausta tai rivivaihtoa yms. oman mielen mukaan :)
<div class="alert alert-danger">
<?php foreach($data->virheet as $key => $viesti): ?>
    <?php echo $viesti; ?>
<?php endforeach; ?>
</div>

Muilta osin lisäys, poisto ja muokkaus onnistui mallikkaasti.

Koodi

Hyvä että olet osan yleisimmistä koodin pätkistä koonnut functions tiedostoon! Modeleissa voisit kokeilla

foreach ($kysely->fetchAll(PDO::FETCH_OBJ) as $tulos) {
    $kayttaja = new Kayttaja();
    $kayttaja->setId($tulos->id);
    $kayttaja->setTunnus($tulos->tunnus);
    $kayttaja->setSalasana($tulos->salasana);

    $tulokset[] = $kayttaja;
}

sijasta

return $kysely->fetchAll(PDO::FETCH_CLASS, __CLASS__);

Joka palauttaa kaikki löydetyt oliot ja asettaa id, tunnus ja salasana automaattisesti kunhan luokan attribuutit on samannimisiä kuin taulukossa olevat kentät.

Hyvin näyttää työ edenneen, kohta se onkin jo valmis! 👍