tl0 / tsoha Goto Github PK

Projekti ladattu 31.4 klo. 14:30

Työ näyttäisi edistyvän vakaasti ja MVC-malli alkaa kivasti näkyä rakenteessa. Varsinaista koodaustahan ei vielä tässä vaiheessa mielettömiä ole, mutta se mitä löytyy vaikuttaa ihan hyvin toteutetulta. Listaustesti ei tosin vaikuta listaavan mitään.

Sisään- ja uloskirjautuminen onnistui ja juurikaan muuta toiminnallisuutta ei varsinaisten sivujen kanssa olekaan vielä toteutettu.

Jostain jos pitää nipottaa niin dokumentaation tietosisältökartoituksessa on people-taulu kahdesti mainittuna.

Kaikkiaan näyttää kuitenkin projekti muotoutuvan ihan kivan näköisesti. Olemassaoleva koodi vaikuttaa ainakin siistimmältä kuin mitä oma refaktorointia kipeästi kaipaava räpellykseni.

Ihan jees!

Ladattu iltapäivällä 14.4

• Tietokantaa kutsutaan monessa paikkaa, esim
  $st = $db->prepare("INSERT INTO account (email, password) VALUES (?, ?)");
  $st->execute(array($email, $ps));
  Nämä samat kutsut voisi laittaa johonkin erilliseen metodiin, joka olisi sitten ainoa tietokantaa kutsuva metodi. Tällöin voisi myös varmistaa, että käytetään parametetrisoitua kyselyä, mikä toisi lisäsuojaa sql injektioita vastaan.
• Toisteisia ja välillä pitkiä if lauseita, esim user.php:ssa
  $allow_edit = (Account::getLoggedInAccount()->getAdmin() || Account::getLoggedInAccount()->getPerson()->getId() == $_GET['id']);

if(Account::getLoggedInAccount()->getAdmin() || Account::getLoggedInAccount()->getPerson()->getId() == $_GET['id'] || Person::getUserByID($_GET['id'])->getPublished())

$allow_edit:issa katotaan monia asioita jotka katotaan heti uudelleen if-lauseessa. Lauseen ehdot voisi ottaa pitkissä if-lauseissa ulkopuolelle ja nimetä ne, jolloin niistä tulisi selvempiä, tyyliin

if($ehto || $ehto2 || $ehto3)

voisi ehkä selvemmin olla

$kayttajaOnKirjautunut = $ehto || $ehto2 || $ehto3;
if($kayttajaOnKirjautunut)

• Tiedostoissa Login.php ja Register.php on kutsuja tietokantaan, jossa haetaan/luodaan accountit. Näiden luonti ja haku kuulunee Account luokkaan, MVC-mallin mukaisesti.
• Myös näkymässä userdetailbox.php on kutsu tietokantaan. Niiden tulisi olla vain mallissa.
• Koodi oli melko siistiä kuitenkin, sitä oli helppo lukea.
• Koodi oli jaetta hyvin siten, että tiedostot eivät olleet liian pitkiä.