Koodikatselmointi

[ellimansikka]

Projekti ladattu: 3.5.2018 klo 13:23

Koodikatselmointi

Koodi on dao:n ja domain:in luokissa erittäin selkeää ja helposti ymmärrettävää. Käyttöliittymäkin on melko selkeä. Metodit on myös nimetty järkevästi. Luokat on sijoiteltu järkevästi pakkauksiin, mutta pakkaukset olisi voitu nimetä ohjelmannimi.dao ja vastaavasti .domain ja .ui .

Parannusehdotuksia

• Kun käyttäjä lisätään olisi hyvä saada jonkinlainen viesti tai tieto, että käyttäjä on lisätty. Ihan tekstikenttien tyhjentyminenkin riittäisi.
• Käyttäjän tarkastelusta ei pääse pois muuten kuin sulkemalla sovelluksen, mutta se taisi olla vielä kesken. Eli kannattaa laittaa paluu nappi siihen.
• Ostostilastot-näkymässä asiakkaiden nimi-kohdassa näkyy kaikki asiakkaan tiedot eli nimi, id, saldo ym. mutta taulukon ruutu jossa ne lukevat on liian pieni, jotta ne kaikki näkyisivät. Se kannattaisi varmaankin muuttaa suuremmaksi tai sitten näyttää siinä ruudussa vain asiakkaan nimi.
• Käyttäjien saldon ja tuotteiden määrän väheneminen voisi periaatteessa olla jossain muualla kuin käyttöliittymässä sillä se ehkä kuuluu enemmän logiikan puolelle. Voisi siis olla jotkin metodit jossakin luokissa jotka vähentäisivät määrää tai saldoa.

Tuli vain mieleen että koodihan oli tarkoitus kirjoittaa englanniksi ja sinulla suurimmalla osalla luokista ja metodeista ym. on suomenkieliset nimet. En tiedä kannattaako niitä alkaa enää muuttelemaan mutta ajattelin huomauttaa kuitenkin.

Hyvältä kuitenkin vaikuttaa ja koodi on tosi selkeää. Ohjelmakin toimii hyvin.

[Kallmark]

Kiitokset palautteesta, etenkin parannusehdotuksista. Kaikkia niistä en olisi itse keksinytkään, joten ovat varmasti hyödyksi.