Koodikatselmointi

[RIR]

Koodikatselmointi KalastusApu, projekti ladattu pe 23.9.2016 klo 18.50

Aiheen kuvaus ja luokkakaavio ovat mielestäni erittäin selkeitä ja suunnistankin ympäri ohjelmaa niiden perusteella.

Netbeansilla ajettaessa ohjelma ei käänny johtuen ongelmista testeissä. Heitti seuraavanlaista virheviestiä :

Failed__ to execute goal org.apache.maven.plugins:maven-compiler-plugin:3.3:testCompile (default-testCompile) on project KalastusApu: Compilation failure: Compilation failure:
aikapvm/KuukausiTest.java:[19,5] cannot find symbol
symbol:   class Kuukausi
location: class aikapvm.KuukausiTest
alue/AlueValintaTest.java:[18,5] cannot find symbol
symbol:   class AlueValinta
location: class alue.AlueValintaTest
aikapvm/KuukausiTest.java:[24,17] cannot find symbol
symbol:   class Kuukausi
location: class aikapvm.KuukausiTest
alue/AlueValintaTest.java:[22,22] cannot find symbol
symbol:   class AlueValinta
location: class alue.AlueValintaTest
alue/AlueValintaTest.java:[28,22] cannot find symbol
symbol:   class AlueValinta
location: class alue.AlueValintaTest
-> [Help 1]

To see the full stack trace of the errors, re-run Maven with the -e switch.
Re-run Maven using the -X switch to enable full debug logging.

For more information about the errors and possible solutions, please read the following articles:
[Help 1] _http://cwiki.apache.org/confluence/display/MAVEN/MojoFailureException

Mainissa näytti olevan jo kokeilua, mutta palautti kanssa virhettä :

Failed to execute goal org.codehaus.mojo:exec-maven-plugin:1.2.1:exec (default-cli) on project KalastusApu: Command execution failed. Process exited with an error: 1 (Exit value: 1) -> [Help 1]

To see the full stack trace of the errors, re-run Maven with the -e switch.
Re-run Maven using the -X switch to enable full debug logging.

For more information about the errors and possible solutions, please read the following articles:
[Help 1] http://cwiki.apache.org/confluence/display/MAVEN/MojoExecutionException

Mavenista johtuvaa?

Yleistä:

• Luokat ja pakkaukset ovat mielestäni erittäin selkeästi ja johdonmukaisesti nimettyjä ja ohjelman toimintaidea selviää niistä helposti. Samoin kaikki muuttujien ja metodien nimet ovat selkeitä ja kuvaavia. -Luokka- ja pakettijaot ovat johdonmukaiset ja loogiset.
• Rajapintojen käyttö kaikissa paketeissa on järkevän tuntuista ja toteuttaa Clean coden periaatetta hienosti. -Varsinkin kalalaji-paketin luokkien toteutus on erittäin optimaalisen tuntuinen. Aliluokkien käyttö eri lajeille ja rajapinta niiden käyttäytymiselle on helposti ymmärrettävää ja pitää huolen siitä, että luokkien vastuujako on järkevää.
• SesonkiNyt-luokan toteutus on myös selkeä ja helppo ymmärtää. Vaihtoehtoisesti public String getSesonki () metodin olisi ehkä voinut toteuttaa switch-lauseella. Tämä tosin lienee enemmänkin mielipidekysymys.

Ohjelman toteutus tähän mennessä kaikinpuolin sen verran selkeää, etten ainakaan itse osaa antaa sen kummemmin parannusehdotuksia. Luokkien ja pakettien vastuujako ja nimeämiskäytännöt kautta linjan noudattivat mielestäni hyvin clean code-periaatteita.

[mikkovalla]

Jes, kiitos palautteesta.

Toi Mavenin virhe on omituinen, antaa itsellä sen kanssa windows koneella muttei maceillä. Kokeilin vielä eri netbeans versioitakin.

getSesonki metodi voisi tosiaan olla parempi switch lauseella toteutettuna. Samoin testailen ENUM vakio arvojen käyttöä jotka voisivat olla loogisempia ja helpommin hallittavissa kuin tämän hetkiset String muuttujat.