{"action":"created","issue":{"url":"https://api.github.com/repos/Samppaa/Matkansuunnittelija/issues/1","repository_url":"https://api.github.com/repos/Samppaa/Matkansuunnittelija","labels_url":"https://api.github.com/repos/Samppaa/Matkansuunnittelija/issues/1/labels{/name}","comments_url":"https://api.github.com/repos/Samppaa/Matkansuunnittelija/issues/1/comments","events_url":"https://api.github.com/repos/Samppaa/Matkansuunnittelija/issues/1/events","html_url":"https://github.com/Samppaa/Matkansuunnittelija/issues/1","id":161023524,"number":1,"title":"Koodikatselmointi","user":{"login":"salsam","id":14109970,"avatar_url":"https://avatars.githubusercontent.com/u/14109970?v=3","gravatar_id":"","url":"https://api.github.com/users/salsam","html_url":"https://github.com/salsam","followers_url":"https://api.github.com/users/salsam/followers","following_url":"https://api.github.com/users/salsam/following{/other_user}","gists_url":"https://api.github.com/users/salsam/gists{/gist_id}","starred_url":"https://api.github.com/users/salsam/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/salsam/subscriptions","organizations_url":"https://api.github.com/users/salsam/orgs","repos_url":"https://api.github.com/users/salsam/repos","events_url":"https://api.github.com/users/salsam/events{/privacy}","received_events_url":"https://api.github.com/users/salsam/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":1,"created_at":"2016-06-18T12:59:22Z","updated_at":"2016-06-21T18:43:52Z","closed_at":null,"body":"Projekti ladattu 18.06.2016 kello 14:20.\r\n\r\nKoodi oli kauttaaltaan clean code-vaatimusten mukainen, enkä huomannut rakenteessa mitään parannettavaa. Testit olivat kattavat ja checkstyle-reporttikin puhdas. Ainoa huomautettava asia clean codesta on TravelPlancontrolerin pituus, joka menee 20 riviä yli vaatimusten, mutta luokan koodista näyttäisi puolet olevan javadoccia ja single responsibility näyttää myös toimivan.\r\n\r\nRakenne tuntui todella selkeältä ja tuskailtuani itse sen kanssa oli kiva nähdä malliesimerkki selkeästä rakenteesta. HTML/CSS-osuuden luulen ymmärtäneeni, vaikken ole ikinä ohjelmoinut CSS/HTML-koodia, joten eiköhän sekin ollut hyvin tehty. Tuo statuskoodi-idea näytti itseässä todella hyvältä ja aion itse panna sen muistiin, sillä se teki koodista paljon selkeämmän.\r\n\r\nGUI oli selkeä ja helpot virhetapaukset, kuten väärät syötteet oli käsitelty. GUIn luomassa HTML-tiedostossa ääkköset eivät tosin näkyneet oikein, joten sen voisi ehkä korjata määräämällä eri kielipaketin käyttöön.\r\n\r\nAinoa muu \"virhe\", jonka huomasin oli, ettei javadocissa ollut pilkkuja ennen esim. sanaa joka. Tämä tosin on jo aika pientä, joten ei muuta kuin peukkuja minulta :)."},"comment":{"url":"https://api.github.com/repos/Samppaa/Matkansuunnittelija/issues/comments/227533723","html_url":"https://github.com/Samppaa/Matkansuunnittelija/issues/1#issuecomment-227533723","issue_url":"https://api.github.com/repos/Samppaa/Matkansuunnittelija/issues/1","id":227533723,"user":{"login":"Samppaa","id":10501962,"avatar_url":"https://avatars.githubusercontent.com/u/10501962?v=3","gravatar_id":"","url":"https://api.github.com/users/Samppaa","html_url":"https://github.com/Samppaa","followers_url":"https://api.github.com/users/Samppaa/followers","following_url":"https://api.github.com/users/Samppaa/following{/other_user}","gists_url":"https://api.github.com/users/Samppaa/gists{/gist_id}","starred_url":"https://api.github.com/users/Samppaa/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Samppaa/subscriptions","organizations_url":"https://api.github.com/users/Samppaa/orgs","repos_url":"https://api.github.com/users/Samppaa/repos","events_url":"https://api.github.com/users/Samppaa/events{/privacy}","received_events_url":"https://api.github.com/users/Samppaa/received_events","type":"User","site_admin":false},"created_at":"2016-06-21T18:43:52Z","updated_at":"2016-06-21T18:43:52Z","body":"Kiitos kommenteista."}}
{"actor":{"display_login":"Samppaa"}}