search

wirefox / visailu

Eka kokonainen ohjelmointiprojektini, jossa syvennän ohpe-, ohja-, ja otm-kursseilla opittuja ohjelmointi- ja mallintamistaitoja ja opettelen ohjelmointiprojektin hyviä käytänteitä sekä versionhallintaa. Tammi-helmikuu 2015.
0 stars 0 forks source link

Koodikatselmointi #1

Open pparm opened 9 years ago

pparm commented 9 years ago

Projekti ladattu maanantaina 2.2.2015 klo 19.40.

Heti allkuun onnittelut sitkeydestä, jos gitHubin kanssa oli haasteita.

Sain avattua ohjelman ja ajettua sen NetBeansissä. Graafinen käyttöliittymää oltiin aloitettu suunnitella, mutta tällä hetkellä ohjelma toimi komentorivin kautta.

Huomioita koodin laatuvaatimuksista: Luokat. metodit. attribuutit ja muuttuja ovat nimetty johdonmukaisesti. Metodit ovat riittävän lyhyitä ja toteuttavat yhden tehtävän kerrallaan. Luokkat tekevät käytännössä yhtä asiaa. Eli asiat ovat mielestäni suhteellisen hyvässä kunnossa noiden suhteen. Heti kättelyssä luokan nimi vastausArpoja ihmetytti, mutta sekin on hyvä nimi, koska luokan avulla arvotaan vastauksia:)

Vaikka luokat ja muuttujat ovat nimetty mielestäni hyvin, ei koodia ole kommentoitu käytännössä lainkaan. Suosittelen kommentoinnin tarkentamista.

Luokat on jaettu sovelluslogiikka, sovellus.domain ja käyttöliittymä -paketteihin. Graafinen käyttöliittymää ei ole vielä tehty, joten on vaikea arvioida kuinka käyttöliittymän ja sovelluslogiikan rajapinta toteutetaan. Tämä lienee yksi kurssin haastavimpia osa-alueita.

Sovellus.domain paketti sisältää Kysymys ja Kysymyssarja luokat. En hahmota miksi ne ovat kyseisessä paketissa, eivätkä sovellus.logiikka paketissa. Siirtäisin ne logiikkapakettiin.

Ohjelmassa tallennetaan tiedostoja jonkin verran. Juuri tiedostonhallinta saattaa käytännössä aiheuttaa ongelmia. Varsinki jos projektin tekee omalla Windows koneella, suosittelen ohjelman käytännön testausta laitoksen koneella. Loppukäyttäjät majailevat laitoksella(tarkastajat).

Yksikkötestit ovat vielä vähän kesken ja rakentaisin ne valmiiksi mahdollisimman pian, jopa ennen käyttöliittymän rakennuksen aloittamista.

Kaiken kaikkiiaan näyttää siltä, että projekti on edennyt vaikka vaikka ilmeisesti teknisiä käynnistys vaikeuksia on ollut jonkin verran. Sain myös muutaman idean omaan projektiini katselmuksen aikana. Mielestäni projekti on riittävän laaja tämän kurssin kannalta. Luokkarakenne mahdollistaa myös pelin laajentamisen kohtuullisen helposti.

Tsemppiä projektin finalisointiin ja GUI:n rakentamiseen.

P

wirefox commented 9 years ago

Kiitos kommenteistasi ja tsempistä! Tsemppiä myös sinulle omaan projektiisi!