{"action":"opened","issue":{"url":"https://api.github.com/repos/konstaoj/Sampleri/issues/1","labels_url":"https://api.github.com/repos/konstaoj/Sampleri/issues/1/labels{/name}","comments_url":"https://api.github.com/repos/konstaoj/Sampleri/issues/1/comments","events_url":"https://api.github.com/repos/konstaoj/Sampleri/issues/1/events","html_url":"https://github.com/konstaoj/Sampleri/issues/1","id":65915996,"number":1,"title":"Koodikatselmointi","user":{"login":"aqyakush","id":6067046,"avatar_url":"https://avatars.githubusercontent.com/u/6067046?v=3","gravatar_id":"","url":"https://api.github.com/users/aqyakush","html_url":"https://github.com/aqyakush","followers_url":"https://api.github.com/users/aqyakush/followers","following_url":"https://api.github.com/users/aqyakush/following{/other_user}","gists_url":"https://api.github.com/users/aqyakush/gists{/gist_id}","starred_url":"https://api.github.com/users/aqyakush/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/aqyakush/subscriptions","organizations_url":"https://api.github.com/users/aqyakush/orgs","repos_url":"https://api.github.com/users/aqyakush/repos","events_url":"https://api.github.com/users/aqyakush/events{/privacy}","received_events_url":"https://api.github.com/users/aqyakush/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":0,"created_at":"2015-04-02T12:31:17Z","updated_at":"2015-04-02T12:31:17Z","closed_at":null,"body":"download 2:00 PM 1.4.2015\r\n- aluksi oli vaikea lukea koodia, kun en tiennyt mitä tekee mitä. Olisi hyvää että olisi enemmän kommenteja. Main luokassa niitä on jonkun verran\r\n mutta muissa luokissa niitä on tosi vähän. Kommentit helpotaisi koodin lukemista\r\n\r\n- Luokan Sample testit isSamplePlayableTest ja SampleStopisWorking eivät menee läpi. Huomasin ainakin että testit menevät läpi,\r\njos isSamplePlayableTest:n viimeinen koodirivissa(alkuperäinen koodi AssertTrue(untitled.getClip().isActive())) muutetaan\r\nAssertTrue(!untitled.getClip().isActive()). Ja SampleStopisWorking:n viimeinen kommentti (alkuperäinen koodi AssertTrue(!untitled.getClip().isActive()))  muutettaan AssertTrue(untitled.getClip().isActive()). En osaa sanoa johtuuko se virheestä koodissa vai testissa, ainaki testit menevät läpi kun tehdään yllämainittut muutokset. Toivotovasti tästä on apua\r\n\r\n- Luokat, metodit, attribuutit, parametrit ja muuttujat ovat hyvät. Metodit ovat lyhyitä ja hoittavat yksi\r\ntehtävät per metodi. Ei ole toistuvaa koodia. Jokaisella luokkalla hoitaa vain yhtä asiaa. Koodissa oli huomautettu virhetilanteita.\r\nKoodi noutaa Clean coodin ehtoja.\r\n\r\n- Vaikuttaa siltä että ohjelman logiikka on halussa, niin lähiaikana voisi rupea tekemään ohjelman graafinen käyttöliittymä "}}