{"action":"opened","issue":{"url":"https://api.github.com/repos/Ooppa/jTODO/issues/1","labels_url":"https://api.github.com/repos/Ooppa/jTODO/issues/1/labels{/name}","comments_url":"https://api.github.com/repos/Ooppa/jTODO/issues/1/comments","events_url":"https://api.github.com/repos/Ooppa/jTODO/issues/1/events","html_url":"https://github.com/Ooppa/jTODO/issues/1","id":56580189,"number":1,"title":"Koodikatselmointi","user":{"login":"Rautiainen","id":10555275,"avatar_url":"https://avatars.githubusercontent.com/u/10555275?v=3","gravatar_id":"","url":"https://api.github.com/users/Rautiainen","html_url":"https://github.com/Rautiainen","followers_url":"https://api.github.com/users/Rautiainen/followers","following_url":"https://api.github.com/users/Rautiainen/following{/other_user}","gists_url":"https://api.github.com/users/Rautiainen/gists{/gist_id}","starred_url":"https://api.github.com/users/Rautiainen/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Rautiainen/subscriptions","organizations_url":"https://api.github.com/users/Rautiainen/orgs","repos_url":"https://api.github.com/users/Rautiainen/repos","events_url":"https://api.github.com/users/Rautiainen/events{/privacy}","received_events_url":"https://api.github.com/users/Rautiainen/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":0,"created_at":"2015-02-04T20:08:07Z","updated_at":"2015-02-04T20:08:07Z","closed_at":null,"body":"Koodikatselmointi 4.2. 22:05\r\n\r\n4.2.2015\r\n\r\nYleistä\r\n\r\n-Käyttöohjeet ja testausdokumentaatio ovat tyhjiä\r\n-Ohjelma kääntyy ja sovelluksen ikkuna aukeaa. File-menun toiminnot eivät kuitenkaan toimi, enkä löytänyt muitakaan tapoja luoda uusia todo-listoja. \r\n-Kaikki 43 yksikkötestiä menevät läpi. \r\n-Koodikatselmointia helpottaa jos dokumentoiti on yhtä laajaa tai laajempaa kuin olemassaoleva projekti. Ne funktionaalisuudet jotka eivät ole valmiina voi esimerkiksi merkata jotenkin. \r\n\r\nAihemäärittelystä\r\n\r\n-Epäselvää mihin käyttäjä tarvii tietokannan luomisvaihtoehtoa. Yleensä tällaisissa sovelluksissa ei ole mahdollisuutta muuttaa tietokantaa suoraan. \r\n\r\nLuokkakaaviosta\r\n\r\n-Sieltä täältä puuttuu sulkuja, englantia ja suomea sekaisin. Tämä ei tietenkään ole kovin merkittävää...\r\n-Epäilen voiko olla kompositioriippuvuus (salmiakki) itseensä, ts. refleksiivinen. Käsittääkseni kompositiossa oliot eivät voi olla olemassa ilman toisiaan, ja molemmat syntyvät ja kuolevat samanaikaisesti. Eli refleksiivisen kompositioriippuvuuden tapauksessa oliota luodessa sille syntyisi samalla refleksiivinen olio, jolle syntyisi samalla refleksiivinen olio jne, aina äärettömyyteen asti. \r\n\r\nKoodista\r\n\r\nKoodi on selkeää ja hyvin muotoiltua. Sovellus ja domain on eroteltu järkevästi. Luokat tuntuvat olevan aika lailla valmiita, jäljellä lähinnä vain käyttöliittymän rakentaminen. Javan GUI-luokat eivät ole minulle kovin tuttuja, joten en osaa kommentoida tarkemmin käyttöliittymän toteutus järkevä. Tässä on selvästi asiaa jota ei ole käsitelty ohjelmoinnin perus- ja jatkokurssilla ja jota en ole kompetentti arvioimaan. \r\n\r\nHyvää työtä!"}}