viipu/linkkikirjasto-tsoha

Tietokantasovellus-harjoitustyö, joulu 2016

linkkikirjasto-tsoha's Issues

Koodikatselmointi

Projekti ladattu 3.1.2016 klo. 11.30
Sivujen painikkeisiin voisi lisätä linkit toimiviin sivuihin, kuten lainojen ja aineiston listaukseen sekä kurssin esittelysivulle (myöskään tiedoston README.md linkki aineiston sivulle ei toiminut). Koodi näyttää siistiltä ja MVC-mallin mukaiselta, tosin joissakin kommenteissa puhutaan peleistä. Kontrollerin items_controller viimeiset metodit (metodista items_list lähtien) ovat ilmeisesti jääneet suunnitteluvaiheesta mutta joka tapauksessa ne vaikuttavat ylimääräisiltä tai olevan väärässä kontrollerissa. Kaikki on johdonmukaisesti englanniksi ja asiat on muutenkin nimetty selkeästi ja kuvaavasti. Lisäksi itse katselmointi oli helppo tehdä kun näkymissä muun muassa ilmoitettiin, minkä asioiden ei ole vielä tarkoitus toimia.

Koodikatselmointi

Koodi ladattu keskivikona 11.1.2017 kello 9:45.

Dokumentaatio

Dokumentaatio oli todella siisti ja hyvin tehty, täytynee itse ottaa tästä mallia omani kehittämiseen :D. Aihe tosin vaikutti aika vaativalta, kannattaa vielä harkita, jos haluat jättää siitä osan pois. Työssä tuntui olevan vielä paljon hommia, joten kannattaa priorisoida lisättäviä asioita ja pudottaa pois vähemmän tärkeitä. Esim. tällä hetkellä et käytä Loan-taulua ollenkaan, joten voisit esimerkiksi jättää sen pois ja keskittyä vain näyttämään, lisäämään ja muokkaamaan aineistoa saadaksesi valmiimman kokonaisuuden.

Käyttöoikeudet

Käyttöoikeudet oli varmistettu hyvin, en onnistunut pääsemään minnekään minne ei pitäisi päästä.

Aineiston lisääminen ja muokkaaminen

Järjestelmän validointi on ilmeisesti vielä kesken, onnistuin lisäämään aineistoa, jonka nimi on suuri määrä välilyöntejä ja samoin tyyppi ja muut tiedot. Näin aineisto tuli näkyviin, muttei sitä pysty klikkamaan näkymässä. Tyypissä olisi hyvä tsekata, että tämä on jotakin sallituista tyypeistä, jolloin aineistoa ei vahingossa (tai tahallaan) voi lisätä väärään tyyppiin. Voit myös lisätä added-päivämäärän käyttäen PHP:n metodia DATE(format) tai harkita rivin poistamista jos et tarvitse sitä.

Käyttäjän kannalta olisi myös hyvä käsitellä vääristä syötteistä nousevat PDOExceptionit ohjaamalla käyttäjä takaisin aineistonlisäys-näkymään ja muistamalla vanhat valinnat (tai luomalla vastaavat validaattorit). Esim. liian pitkä string syöte tai kirjailijaa ei valittu. Onnistuin myös jotenkin vääriä syötteitä antamalla päätymään takaisin aineiston lisäysnäkymään niin, että kirjailija oli kadonnut.

Aineiston poistaminen

Lisätyn aineiston poistaminen ei tällä hetkellä onnistu, sillä siihen on vielä viittaus itemauthor-taulusta. Tämä kannattaa korjata poistamalla ensin vastaavat itemauthor-taulun rivit ennen aineiston poistoa destroy-metodissa. Poisto kuitenkin toimi malleilla, joilla ei ollut viittauksia itemauthor-taulusta.

Lisäysehdotuksia

Kannattanee ensin tehdä jo aloitetut työt loppuun, mutta jos aikaa riittää voisit lisätä työhön kirjailijan/käyttäjän lisäyssivut.

Tsemppiä viimeiseen viikkoon!
Sami Salonen