Koodi ladattu 2018-05-03 14:55

• CurrencyConverterService luokan konstruktorille injektoidaan DAO-oliot, mikä on merkki luokan yksilöllisestä vastuusta. Hyvää työtä!
• Myös JavaDocit on kirjoitettu hyvin. Arvostan selkeitä kuvauksia erityisesti CurrencyConverter-luokan osalta. Metodien lisäksi voisit kuvata myös itse luokan toiminnan kuvauksen: mistä kyseinen luokka on vastuussa?
• Checkstyle ilmoitti ExchangeRate-luokan hieman ylipitkästä (22 riviä) public boolean equals(Object obj) -metodista. Nuo pari riviä voi säästää esimerkiksi käyttämällä 'else if-lauseita tai muuntamalla if-lauseet switch-lauseeksi.
• Luokat ja metodit on nimetty selkeästi hyödyntäen camelCase-formaattia.
• Monissa luokissa, esimerkiksi SQLExchangeRateDao, on käyttämättömiä importteja, jotka voisi siistiä pois.
• Testit on toteutettu järkevästi ja ilman päällekkäisyyksiä.
  • Havaitsin yhden päällekkäisyyden: CurrencyConverterServiceTest, rivi 45: assertFalse(service.isReady()); sama asia testataan jo aiemmassa testissä.
• Pakkaukset on nimetty selkeästi ja eri asiaa hoitavat luokat on eritelty omiin pakkauksiinsa.

Kaikki parannusehdotukseni ovat suhteellisen pieniä, sillä koodin laatu on kauttaaltaan hyvä. Aina on silti jotain parannettavaa. Hyvää työtä ja tsemppiä viimeisen viikon loppurutistukseen!