{"action":"opened","issue":{"url":"https://api.github.com/repos/NRoos/lazy-stronghold/issues/1","repository_url":"https://api.github.com/repos/NRoos/lazy-stronghold","labels_url":"https://api.github.com/repos/NRoos/lazy-stronghold/issues/1/labels{/name}","comments_url":"https://api.github.com/repos/NRoos/lazy-stronghold/issues/1/comments","events_url":"https://api.github.com/repos/NRoos/lazy-stronghold/issues/1/events","html_url":"https://github.com/NRoos/lazy-stronghold/issues/1","id":172323973,"number":1,"title":"Koodikatselmointi","user":{"login":"Meicceli","id":6899132,"avatar_url":"https://avatars.githubusercontent.com/u/6899132?v=3","gravatar_id":"","url":"https://api.github.com/users/Meicceli","html_url":"https://github.com/Meicceli","followers_url":"https://api.github.com/users/Meicceli/followers","following_url":"https://api.github.com/users/Meicceli/following{/other_user}","gists_url":"https://api.github.com/users/Meicceli/gists{/gist_id}","starred_url":"https://api.github.com/users/Meicceli/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Meicceli/subscriptions","organizations_url":"https://api.github.com/users/Meicceli/orgs","repos_url":"https://api.github.com/users/Meicceli/repos","events_url":"https://api.github.com/users/Meicceli/events{/privacy}","received_events_url":"https://api.github.com/users/Meicceli/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":0,"created_at":"2016-08-21T14:37:37Z","updated_at":"2016-08-21T14:37:37Z","closed_at":null,"body":"# Koodikatselmointi, Tiralabra - loppukesä 2016\r\nTyö ladattu 21.08. kello 15:40\r\n\r\n## Main.py\r\nMain.py:n rakenne on helposti ymmärretävissä, joskin koodin tyylityksessä on hieman hiottavaa. Voisit koota metodien ulkopuolella olevan koodin uusiin metodeihin. Suosittelen tämän lisäksi myös luomaan erillisen main metodin, joka hoitaa uusien metodien kutsumisen kunhan kutsut main metodia esimerkiksi tiedoston viimeisellä rivillä. Tällöin koodia ei ole irrallaan metodien ulkopuolella, ehkä globaaleja muuttujia lukuunottamatta.\r\n\r\nKannattaa myös perehtyä tarkemmin Python projektien luomiseen, ja ottaa erityisesti selvää siitä, mitä tiedostot `__init__.py` ja `__main__.py` ovat.\r\n\r\n## Player.py\r\nTämä on hyvin selkeä luokka. Luokan alustavassa metodissa on kuitenkin (ainakin toistaiseksi) turha muuttujan alustus `self.character`. Jos et tarvitse sitä jatkossa, niin sen voisi poistaa.\r\n\r\n## Room.py\r\nSelkeää koodia. Voisit vielä kommentoida valitsemiasi rajoja checkarea-metodissa (-3 ja +3) ja mihin alustusmetodissa kysyttyä colors parametria tullaan tarvimaan.\r\n\r\n## Worm.py\r\nfloodfill metodi kaipaa ehdottomasti lisää kommentointia. Muuten ei moitittavaa.\r\n\r\n## Level.py\r\nPääosin ymmärrettävää koodia. Tässä vielä muutama vinkki:\r\n- rangelle ei tarvitse antaa ala-rajaa, jos haluaa aloittaa iteroimisen nollasta (esim. rivi 20)\r\n- moniosaiset if ehdot kannattaa ehkä tyyli- ja ymmärrettävyyssyistä pilkkoa useammalle riville esim. jokainen `or <ehto>` omalle rivilleen rivillä 23.\r\n- metodien kuvaukset kannattaa asettaa metodin ensimmäiselle riville `\"\"\"` tagien sisään, ks. [Docstring Conventions](https://www.python.org/dev/peps/pep-0257/)\r\n\r\n## Projektin suorittamisesta\r\nPelin pelaaminen toimii koneellani odotetusti. Hahmoa pystyy liikuttamaan, ja kenttää paljastuu lisää hahmoa liikuttaessa. Hahmo on kuitenkin jumissa aloitushuoneessa, mutta tämä lienee kehityksen alla. Testitkin menevät kaikki läpi."}}
{"actor":{"display_login":"Meicceli"}}