jonne-sotala / starsudoku Goto Github PK

Projekti (ei hotfix) ladattu 03.12.2020, kello 18.40.

Koodikatselmus:

• sudoku.py-moduuli tuntuu hieman irralliselta, varsinkin kun sudoku.py ei sisällä yhtäkään luokkametodia. Ehkä luokkaan voisi siirtää jotain metodeja sudoku_solver.py-moduulista, esim arvojen muuttamiseen tai palauttamiseen liittyviä? Muutenhan kaiken voisi säiliöä vain sudoku_solver.py-moduulissa, tehden sudoku.py hyödyttömän.

• solution.py-moduulin tuntuu myös hieman irralliselta. Käyttäjän pisteet ja ajan voisi sijoittaa suoraan user.py-moduliin pitkin peliä ja sitä kautta tilastoihin?

• Miksi main.py metodissa kutsutaan kahta eri näkymää silmukassa? Olisiko tarpeen luoda oma run()-metodi, joka alottaisi alkuvalikosta, ja valinnoilla siirtyisi toisiin näkymiin?

• Jokainen näkymä näyttää luovan oman silmukkansa. Harkitsisin sitä, että luodaan yksi pääsilmukka, missä vaihdetaan, että mitä näkymää päivitetään? Se vaikutti olevan suositus, kun itse tutustuin Pygameen, mutta toisaalta on mainittu, että silmukoiden määrä ei haittaa mitään.

• game.py sisältää paljon toisteista koodia, ainakin metodeissa check_events() ja insertion_mode(). Voisi olla aika refaktoroida, esimerkiksi luomalla sanakirja, missä on {K_n: n} tms. Näppäimen merkkijonoesityksen saa haettua esim: pygame.key.name(event.key).

• Pipfile ei sisällä pakkausta atomicwrites. Testit sitä kuitenkin vaatii, ja ajattelin mainita siitä. Jos on vahingossa asennettu lokaalina tms, ja sitä kautta unohtunut. :)

Traceback (most recent call last):
  File "C:\Users\xxx\AppData\Local\Programs\Python\Python39\lib\runpy.py", line 197, in _run_module_as_main
    return _run_code(code, main_globals, None,
  File "C:\Users\xxx\AppData\Local\Programs\Python\Python39\lib\runpy.py", line 87, in _run_code
    exec(code, run_globals)
  File "C:\Users\xxx\.virtualenvs\ot-harjoitustyo-viikko5-xqGucHQw\Scripts\pytest.exe\__main__.py", line 4, in <module>
  File "c:\users\xxx\.virtualenvs\ot-harjoitustyo-viikko5-xqguchqw\lib\site-packages\pytest\__init__.py", line 5, in <module>
    from _pytest.assertion import register_assert_rewrite
  File "c:\users\xxx\.virtualenvs\ot-harjoitustyo-viikko5-xqguchqw\lib\site-packages\_pytest\assertion\__init__.py", line 8, in <module>
    from _pytest.assertion import rewrite
  File "c:\users\xxx\.virtualenvs\ot-harjoitustyo-viikko5-xqguchqw\lib\site-packages\_pytest\assertion\rewrite.py", line 300, in <module>
    from atomicwrites import atomic_write
ModuleNotFoundError: No module named 'atomicwrites'

Mutta yhteenvetona koodi on tosi siistiä muuten, ja jokainen luokka ja metodi on nimetty järkevästi, niin on helppo seurata koodin kulkua. Hieno projekti ja pähee grafiikka! 👍

Ps. Solution-luokassa on yksi muuttuja asetettu näin: self.sudoko = sudoku.
Pss. .pylintrc on disabloitu aika paljon juttuja, kuten mm. docstringit.