Koodikatselmointi 1

[RadicalOyster]

Repositorio ladattu Githubista 23.04.2021 kello 12:18

Yleiskatsaus

Ohjelman toimivuus näyttää olevan kohdallaan. Käyttöliittymä on selkeä ja
helppokäyttöinen. Testejä on hyvä määrä ja kattavuus näyttää todella hyvältä.
Koodin rakenne on selkeää, koodia on helppoa seurata ja koodi on hyvin
kommentoitua. Projektin rakenne on ainakin omaan aika keskeneräiseen
rakenteeseen verrattuna myös hyvin selkeä. Luokat on järkevällä tavalla
jaettu paketteihin. Omat tietorakenteet vaikuttavat hyvin toteutetuilta.

Parannusehdotuksia

• Aloitus- ja loppupisteiden tarkka määrittely on hiirellä vaikeaa. Käyttäjänä
  haluaisin voida määritellä pisteet tekstisyötteenä. Käyttöliittymä voisi ehkä
  myös näyttää kursorin nykyisen sijainnin kun käyttäjä siirtä kursoria kartalla.

• Kun pisteet on määritelty, voisi ohjelma myös merkitä pisteet ennen kuin käyttäjä
  laskee polun. Tämä auttaisi hahmottamaan pisteiden sijainnin suhteessa toisiinsa.

• Käyttäjän kannalta "run tests"-toiminto ei ole järin intuitiivinen. Lyhyellä,
  suoralla ja esteettömällä reitilläkin kone hyrrää suht kauan eikä käyttäjälle
  välttämättä ole selvää mitä taustalla tapahtuu. Testien suorittamisen päätyttyä
  ei myöskään ole selvää suoritettiinko testit onnistuneesti eikä minne testien
  tulokset päätyvät.

Loppukommentit

Projekti näyttää jo tässä vaiheessa todella hyvältä. Kaikki perustoiminnallisuudet
on toteutettu ja toimivat eikä käyttöliittymäkään vaadi kuin pientä viilausta.
Mukava nähdä muiden toteutuksia samalta aihealueelta ja tämän tasoinen projekti
antaa itsellekin motivaatiota suorittaa oma projektini loppuun. Onnea vaan
loppuviikoille!

[k0psutin]

Kiitos katselmoinnista @RadicalOyster!

UI onkin jäänyt pienemmälle huomiolle projektissa ja sen parantelu olikin seuraavana vuorossa. Mm. alku- ja loppupisteet piirtyvät kartalle ja ''run tests'' on nimetty uusiksi ''run performance tests''. Se ajaa siis ennaltamäärätyt testit (resources\data.json), eikä siihen voi vaikuttaa UI:ssa.