Koodikatselmointi: TelegramEventBot - projekti ladattu 22.4.2019 kello 7:20
Koodi oli selkeää ja metodit pilkottu järkeviin kokonaisuuksiin. Metodien ja attribuuttien nimeäminen oli järkevää ja niistä ymmärsi hyvin, mitä kyseinen metodi tekee, vaikka en perehtynyt TelegramLongPollingBot -luokan toimintaan, jonka tämän projektin luokka TelegramEventBot perii.
Nyt Dao -luokassa on aika paljon sovelluslogiikkaa, esimerkiksi metodeissa eventIsInDb ja isAttendingEvent. Ohjelman vastuiden ja rakenteen selkeyden kannalta voisi olla hyvä, että ohjelmalogiikka olisi ”service” luokassa ja Dao-luokassa tehtäisiin vain tietokantaoperaatioita. Lisäksi jäin miettimään isAttendingEvent metodia, että nyt Registration -taulun name attribuutti ei ole ainakaan tietokantatasolla merkitty uniikiksi: jos useammalla henkilöllä on sama nimi, toimiiko isAttenfingEvent -metodi oikein. En tosin ole varma hoidetaanko tämä jotenkin ohjelmalogiikassa.
Erittäin hyvää koodia kaiken kaikkiaan!
Ps. Repositoriosta puuttuu config.properties tiedosto, minkä vuoksi ohjelman suoritus ei onnistunut.