Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Koodikatselmointi #1

Closed
ainokuos opened this issue May 3, 2021 · 1 comment
Closed

Koodikatselmointi #1

ainokuos opened this issue May 3, 2021 · 1 comment

Comments

@ainokuos
Copy link

ainokuos commented May 3, 2021

Koodikatselmointi

Projekti ladattu 3.5.2021 klo 20:00

Yleistä

  • Koodi on jaettu selkeästi hakemistoihin eri toiminnallisuuksien perusteella
    - Luokat toteuttavat vain yhtä toimintoa
    - Metodit ja funktiot pysyvät yksinkertaisina
  • Sovelluslogiikka on erillään käyttöliittymästä
  • Pylintin antaman arvosanan mukaan koodin laatu on erittäin hyvää lukuunottamatta muutamaa muuttujaa

Huomioita

  • src hakemistossa on vielä tiedostoja joita voisi jakaa erillisiin hakemistoihin, kuten olitkin jo maininnut arkkitehtuurikuvauksessa
  • Hakemisto dialogs toistuu kahdesti ensin gui-hakemistossa ja myöhemmin myös gui/ui-hakemistossa, joka on jokseenkin hämäävää
    - Hakemistot sisältävät myös lähes samannimiset tiedostot

Toiminnasta

En saanut ohjelmaa toimimaan fuksiläppärilläni. Yritin myös etätyöpöydällä ajaa viimeisimmän releasen koodia, mutta virheilmoituksena sain:
ModuleNotFoundError: No module named 'gui.models.saved_address_model'
Models hakemistosta on siis jäänyt ilmeisesti pois tuo moduuli.

Fuksiläppärillä ohjelma kyllä aukesi, mutta kaatui tiedoston valintaan, mikä taitaa johtua fuksiläppärin käyttöoikeuksien puutteesta. Tässä olisi voinut olla virheilmoitus ohjelman kaatumisen sijaan.

@vikke1234
Copy link
Owner

Joo siis olin unohtanu committaa ton filen ni sen takii ei toiminu. Ja syy miks siin on aika samoi tiedostoi siin ui folderis on kosk ne on vaa autogeneroitui ja ei mikään muu paitsi ne mitkä on siin gui kansios pitäs käyttää niit.

@ainokuos ainokuos closed this as completed May 4, 2021
@ainokuos ainokuos reopened this May 4, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants