search

ronituohino / ohte-sudoku

Ohjelmistotekniikka 2022 kevät harjoitustyö
MIT License
0 stars 0 forks source link

Koodikatselmointi #1

Open lauritus opened 2 years ago

lauritus commented 2 years ago

Viikon 5 release on ladattu 2.5. klo 13.00

Mielenkiintoinen ja hauska sovellus! (Vaikka en sudoja ulkomuisista edes itse osaa täyttää). Ohjelman suorittaminen onnistui kuten pitikin annetuilla ohjeilla.

Koodin rakenne on mukavan siisti ja erinäiset toiminnot on jaoteltu omiin luokkiinsa ja tiedostoihin kuten pitääkin. Docstringejä ei ainakaan vielä ole, mutta ne ovatkin vasta viikon 6 kohdalla materiaalissa. Erityismaininta muuten lisenssistä repossa.

Varmati hyvä valinta käyttää itse sudokuun pygamea verrattuna tkinteriin, jolla vastaavaa ei varmaan saisi kasaan edes kuudessa viikossa.

Pylintin arvosana todella hyvä (9.85) ja testien haaraumakattavuuskin lähes 60% Näiden suorittamiseen komennot toimivat myös kuten pitivät.

Luokat tekevät pääosin mukavasti vain niille tarkoitettuja asioita. Ehkä itse vielä siirtäisin esimerkiksi esimerkiksi .sudoku tiedostojen parsinnan omaan "utiliinsa" (nyt luokan Sudoku initissä).

Projektissa näyttää esiintyvän jokunen print() komento, nämä vaihtaisin myös itse pyhtonin loggingiin mikäli ne ovat vielä tarpeellisia. (Lukuisia etuja mm. ei yritetä bufferoida kuten tiedostoon kirjoitettaessa jos ajetaan ilman interaktiivista terminaalia, loglevelit)

Hyvää työtä ja kiva sovellus!

ronituohino commented 2 years ago

Kiitos palautteesta, otan sen huomioon jatkokehityksessä!