Koodikatselmointi #1
Description
Koodi ladattu pe 29.4. klo09:23.
Heippa!
Sovelluksesi aihe on todella piristävä! Kiitos selkeistä ohjeista heti sovelluksen käynnistämisestä alkaen.
Koodisi oli minun mielestäni hyvin siistiä ja kurssilla noudatettavien laatuvaatimusten mukaista. Nimeäminen oli johdonmukaista ja toteutettu PascalCase- ja snake_case- formaateissa.
Käyttöliittymä ja sovelluslogiikka oli eritytetty esimerkillisesti ja luokat olivat siististi omissa moduuleissaan. Tiedon haku ja tallentaminen oli toteutettu repositorioissa.
Kurssin tarjoamassa koodin laatuvaatimukset -dokumentissa (https://ohjelmistotekniikka-hy.github.io/python/koodin-laatuvaatimukset) mainitaan:
"Jos luokan metodin tai attribuutin on tarkoitus olla vain luokan sisäisessä käytössä, on tapana käyttää nimessä _- tai __-etuliitettä".
Jos olen ymmärtänyt oikein, niin kaksoisalaviiva piilottaa attribuutin asiakkaalta ja yksi alaviiva tekee siitä hieman yksityisemmän.
Jos koet, että tämä olisi relevanttia koodissassi, niin referenssisovelluksen luokasta LoginView näkee hyvin, miten luokan attribuutit on merkitty yhdellä alaviivalla: https://github.com/ohjelmistotekniikka-hy/python-todo-app/blob/master/src/ui/login_view.py
Mukavaa loppukurssia ja aurinkoista kevättä!
Terveisin,
Susanna Kinnunen