Koodikatselmointi

[wirefox]

Projekti ladattu koodikatselmointia varten su 29.11.2015 klo 23.00.

Yleistä projektista:

• Paljon olet ehtinyt tehdä, hyvää työtä! Sovelluksen käyttäminen on mielestäni loogista.

  Dokumentoinnista:

• Dokumentointi on selkeä ja antaa hyvän kuvan sovelluksesta, sen käyttäjistä sekä käyttötapauksista. Dokumentointi on hyvin pitkä ja perusteellinen. Käyttöoikeusroolit ovat selkeät.
• Järjestelmän komponentit -kuva on hieman hankalasti hahmotettavissa. Sitä voisi kunnolla yksinkertaistaa, jotta se aukeaisi katsojalle nopeammalla vilkaisulla.
• Jotkut dokumentoinnin kohdista ovat hieman vanhentuneita, kun sovellus on edennyt, mutta dokumentaatiota on varmastikin vielä tarkoitus päivittää.

  Clean codesta ym.:

• MVC-mallia on toteutettu tässä hyvin. Kontrollerit, mallit ja näkymät ovat omissa hakemistoissaan.
• koodissa on paljon kommentteja; ehkä ihan kaikkia kommentteja ei tarvitsisi.
• onko koodin sisennys joka puolella kunnossa? Tätä voisi tarkastella, jotta koodista tulisi vielä helpommin luettavaa.
• nimeämistä voisi paikoin vielä selventää, ja kirjoittaa esim. Metodien nimet joko suomeksi tai englanniksi. Nyt on paikoin molempia käytetty, esim. Public static function kayttaja_find (Kayttaja.php).

  Varastojen listaus:

• id-numero on nyt linkki, mikä ei johda mihinkään. Ehdotus/kysymys: tarvitseeko tässä olla linkkiä, koska rivillä on nappi, jolla pääsee varaston tietoihin(?)

  Tuotteiden listaus:

• id-numero on nyt linkki, mikä ei johda mihinkään. Ehdotus/kysymys: tarvitseeko tässä olla linkkiä, koska rivillä on nappi, jolla pääsee tuotteen tietoihin(?)

  Lisää uusi tuote:

• ehdotus: voisiko päivämäärän ja ajan lisäys tulla automaattisesti, eli käyttäjän ei tarvitsisi sitä itse antaa? Se lisäisi käytettävyyttä.
• uutta tuotetta syöttäessä, jos tapahtuu virhe, niin käyttäjän lomakkeelle jo syöttämät arvot eivät palaudu takaisin lomakkeelle, vaan kaikki pitää syöttää uudelleen. Jos ehdit, niin tätä voisi kehittää, eli kun sivu luodaan uudestaan, annetaan sille käyttäjän ensimmäisellä kerralla antamat syötteet.
  • jos näin on muissakin lisäyslomakkeissa, niin sama pätee niihinkin (en testannut näitä)

    Navigaatio:

• Navigaatio on nyt pääsivulle-napin kautta monessa paikassa. Se toimii näin. Halutessaan navigaatiota eri osioiden pääsivuille voisi harkita toteutettavaksi ylös navigaatiopalkkiinkin, jotta pääsisi suoraan yhdellä klikkauksella haluamansa osion pääsivulle.

  Toiminnot/napit:

• Kaikki napit, joita kokeilin, toimivat mielestäni niinkuin pitääkin. Järkeviä virheviestejäkin sain niistä kohdin, missä toimintoja testailin. Sisään- ja uloskirjautumiset toimivat mielestäni.

  Tsemppiä projektin loppurutistukseen!

[rotsten]

Kiitos kattavasta ja rakentavasta palautteesta!