{"action":"opened","issue":{"url":"https://api.github.com/repos/TaikaMurmeli/Tappajapupun-seikkailut-osa-1-Kaalimaan-kirous/issues/2","labels_url":"https://api.github.com/repos/TaikaMurmeli/Tappajapupun-seikkailut-osa-1-Kaalimaan-kirous/issues/2/labels{/name}","comments_url":"https://api.github.com/repos/TaikaMurmeli/Tappajapupun-seikkailut-osa-1-Kaalimaan-kirous/issues/2/comments","events_url":"https://api.github.com/repos/TaikaMurmeli/Tappajapupun-seikkailut-osa-1-Kaalimaan-kirous/issues/2/events","html_url":"https://github.com/TaikaMurmeli/Tappajapupun-seikkailut-osa-1-Kaalimaan-kirous/issues/2","id":72268497,"number":2,"title":"Koodikatselmointi II","user":{"login":"caniblossom","id":11410076,"avatar_url":"https://avatars.githubusercontent.com/u/11410076?v=3","gravatar_id":"","url":"https://api.github.com/users/caniblossom","html_url":"https://github.com/caniblossom","followers_url":"https://api.github.com/users/caniblossom/followers","following_url":"https://api.github.com/users/caniblossom/following{/other_user}","gists_url":"https://api.github.com/users/caniblossom/gists{/gist_id}","starred_url":"https://api.github.com/users/caniblossom/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/caniblossom/subscriptions","organizations_url":"https://api.github.com/users/caniblossom/orgs","repos_url":"https://api.github.com/users/caniblossom/repos","events_url":"https://api.github.com/users/caniblossom/events{/privacy}","received_events_url":"https://api.github.com/users/caniblossom/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":0,"created_at":"2015-04-30T20:09:18Z","updated_at":"2015-04-30T20:09:18Z","closed_at":null,"body":"Projekti ladattu: 30.4.2015, kello 21:26\r\n\r\n### Koodi\r\nPelin domainikoodia on kiva lukea - siinä on selkeä rakenne, ja uusien vihollisten ja esineiden lisääminen näytti helpolta.\r\n\r\nGameState ja GameStateManager näyttävät idealtaan kivoilta. Tarkemmin katsoessa jäi fiilis, että pelitilojen logiikan ja piirtämisen voisi ehkä erottaa toisistaan - mutta eipä se pahalta näyttänyt nykyisessäkään muodossa.\r\n\r\nSuurinta hämmennystä herättivät GameStateManagerissa olevat for -loopit run() -kutsujen ympärillä. Koitin poistaa ne ja ongelmia ilmeni :D Tästä jäi pieni purkkafiilis, mutta ei mitään vakavaa. Joku TODO tai muu kommentti voisi olla hyvä tässä kertomaan, että mitä pitäisi tehdä tai että miksi näin on.\r\n\r\nKoodi oli kuitenkin enimmäkseen siistiä ja sitä oli mukava lukea. Olin myös yllättynyt miten paljon toiminnallisuutta on saatu mahtumaan suhteellisen pieneen määrään koodia. Tästä jäi tunne, että jotain on tehty oikein.\r\n\r\n### Dokumentaatio\r\nJavadocia näytti olevan kaikissa oleellisissa paikoissa riittävästi. Tykkäsin taistelutilanteelle tehdystä sekvenssikaaviosta - en tiedä onko se ajan tasalla, mutta sen vilkuilu auttoi kuitenkin ymmärtämään koodia.\r\n\r\n### Havaittuja pikkubugeja (ehkä)\r\n- Kaikki mustat puput (Höpö?) katosivat kun olin voittanut yhden, sama olio kaikissa käytössä?\r\n- Alueenvaihtopupun kohdalla alue vaihtuu myös seinää päin kävellessä.\r\n\r\n### Muuta\r\nMahtava peli <3 Ainoa mikä harmitti oli se, että kun lopulta sain ydinlatauksen, niin ei ollut mitään kunnon vihuja joihin käyttää sitä :(\r\n"}}