Koodikatselmointi

[Eetusav]

Projekti ladattu 6.10. n. klo 12.00 Oletan, että jotkin työn osat ovat vielä vaiheessa ja siksi toiminnallisuus on osin outoa. Esimerkiksi kaikki yläreunan linkit ohjaavat samalle ./users osoitteeseen. Yritän keskittyä toiminnallisuuksiin, jotka ovat valmiita.

Sisään- ja uloskirjautuminen toimivat nätisti. Kun painaa sisäänkirjautuneena oikeassa yläkulmasta omaa profiilia, niin onko tarkoituksena, että aukeaa editointi ikkuna? Olisiko ehkä parempi, että olisi jokin view-tyyppinen sivu ja se sisältäisi redirectin oman profiilin editointiin.

Ei varmaan ole myöskään järkevää, että kaikki käyttäjät pystyvät editoimaan toisten profiileita.

Tietokantakaaviosta uupuvat vielä ainakin Viesti/Message ja Keskustelu/Conversation taulut. Itse sain kehotuksena, että olisi hyvä jos työ sisältäisi ainakin yhden monesta-moneen yhteyden taulujen välillä.

Html dokumenttien näyttää selkeältä ja nimeäminen on ymmärrettävää. Users_show ja user/show.html nimeämisestä en ole ihan varma, kun sinne kuitenkin tehdään myös post-tyyppisiä pyyntöjä.

Kivat nuo messaget, mitkä näytetään virheellisen rekisteröitymisen yhteydessä. (Esim. Passwords does not match!). Kannattaisiko myös lisätä, ettei ole mahdollista tehdä kahta profiilia samalla sähköpostilla?

Kannattaa varmaan lisätä abstrakti luokka Base, niinkuin viikon 4 materiaalin alussa. Ei tarvitse joka tauluun laittaa erikseen id, date-modified... jne.

Kaiken kaikkiaan työ näyttää ihan hyvältä. Tarvitsee vain vielä lisää työtä.

[joonissi]

Kiitos rakentavasta palautteesta!