Aozi
commented
10 years ago trainer_pokemon voisi muuten toimia välitauluna, mutta ideana oli siis että käyttäjä voi itse muuttaa omien pokemoniensa tietoja. Minkä takia se linkkitaulutus ei oikein toimi, tosin voi olal että sieltä voi karsia jotain tietoja pois ja hakea ne pokemon taulusta sen sijaan.
Tuo session start höslä oli enemmän siitä että itse väänsin nuo loppuun joskus perjantaina viideltä aamulla enkä siinä väsyneenä saanut sitä toimimaan niin tungin noita vähän joka paikaan ja unohdin selkeesti poistaa.
Kirjautuminen on joo vähän vaiheessa, tosin sen pitäisitoimia niin kauan kun tietokantaan syötetään käyttäjä, en kyllä kovin sitä kokeillu.
Projekti ladattu 3.6.2014 klo 22:02.
Koodi näyttää oikein hyvältä ja kurssin ohjeita on selkeästi noudatettu. Näkymät, kontrollerit ja mallit on jaettu kaikki selkeästi omiin kansioihinsa ja rakenteesta sai muutenkin hyvin selvää. Tietokantayhteyden avaaminen on omassa luokassaan eikä mikään muutenkaan leviä ympäriinsä.
Tietokannan rakenne aiheutti itsessäni hieman hämmennystä: miksi trainer_pokemon ja pokemon ovat omat taulunsa? Eikö trainer_pokemon voisi toimia selkeästi välitauluna niin että statsit haettaisiin pokemon-taulusta eikä trainer_pokemoniin tarvitsisi tallentaa duplikaattinä pokemonien tietoja? Toki saattaa olla hakujen kannalta optimaalisempaa. Muuten tietokantakin näytti oikein hyvältä, vaikka kaikkea ei vielä olekaan toteutettu.
Toinen juttu, jota kummastelin, oli pohja.php:sta löytyvä session_start -kutsu. Tämän ei kuuluisi täällä olla kyllä lainkaan? Aiheuttanee jotain bugejakin html-tagin jälkeen sijoitettuna.
Tietokannan sisällön parsiminen Pokéapin kautta näytti hauskalta. Kirjautumista en pahemmin päässyt testailemaan, näytti olevan vielä vähän kesken. Varmaan tästä johtuen SignIn-kontrolleri näytti jotenkin sotkuiselta, sitä voisi siistiä. Muuten muuttujien nimet ym. olivat ihan ok.
Kaiken kaikkiaan oikein kiva (aihekin hyödyllinen ja tärkeä!). Tekemistä toki riittää, mutta vielä ehtii. :) Good job!