perander/otm-project

otm-project's Issues

Koodikatselmointi

Projekti ladattu 30.4.2018 klo 9:27

Kommentteja:

• Luokat, metodit, muuttujat ja parametrit on nimetty kuvaavasti
• Ohjelma on hyvin jaettu selkeisiin pakkauksiin (domain, dao ja ui), joilla kaikilla on omat erilliset vastuualueet
• Yleisesti koodi on helppolukuista ja metodien tarkoitukset ymmärtää hyvin
• DiaryUi:n start-metodi on aika pitkä ja kaikki eri näkymät luodaan sen sisällä. Metodin toiminnallisuutta voisi ehkä jakaa pienempiin metodeihin, joissa eri näkymät tai asetelmat luodaan? Metodin koodia on kyllä kommentoitu niin, että sitä on helppo lukea myös nykyisessä muodossa.
• EntryDao-, FoodDao- ja UserDao- luokkien metodeissa mahdollisia poikkeuksia on käsitelty try ... catch - rakenteen avulla, mutta niin, että catch - osuus puuttuu ja pelkkä try on kirjoitettu. Esimerkkinä EntryDaon findOne-metodi, jossa lukee "try (Connection connection = database.getConnection())", mutta catch puuttuu. Kaikki nämä metodit myös heittävät jo valmiiksi SQLExceptionin, joten niiden pitäisi toimia myös kokonaan ilman try ... catch - rakennetta.