jonitaajamo / ot-harjoitustyo

Telegram bot that keeps track of events in Telegram groups. Made for University of Helsinki course "Ohjelmistotekniikka".

Ohjelmistotekniikka

Telegram EventBot

Telegram EventBotilla voidaan ylläpitää tapahtumia ja niiden osallistujia Telegram keskusteluissa. Sovellus on toteutettu Helsingin Yliopiston tietojenkäsittelytieteen kandiohjelman kurssille "Ohjelmistotekniikka".

Dokumentaatio

• vaatimusmäärittely
• arkkitehtuuri
• työaikakirjanpito
• komentorivitoiminnot
• käyttöohjeet
• testausdokumentti

Releaset

• release viikko 5
• release viikko 6
• final release

ot-harjoitustyo's Issues

Koodikatselmointi

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.