{"action":"created","issue":{"url":"https://api.github.com/repos/VesaPuu/wordmaster/issues/1","labels_url":"https://api.github.com/repos/VesaPuu/wordmaster/issues/1/labels{/name}","comments_url":"https://api.github.com/repos/VesaPuu/wordmaster/issues/1/comments","events_url":"https://api.github.com/repos/VesaPuu/wordmaster/issues/1/events","html_url":"https://github.com/VesaPuu/wordmaster/issues/1","id":81373403,"number":1,"title":"Koodikatselmointi","user":{"login":"savuvaris","id":12268625,"avatar_url":"https://avatars.githubusercontent.com/u/12268625?v=3","gravatar_id":"","url":"https://api.github.com/users/savuvaris","html_url":"https://github.com/savuvaris","followers_url":"https://api.github.com/users/savuvaris/followers","following_url":"https://api.github.com/users/savuvaris/following{/other_user}","gists_url":"https://api.github.com/users/savuvaris/gists{/gist_id}","starred_url":"https://api.github.com/users/savuvaris/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/savuvaris/subscriptions","organizations_url":"https://api.github.com/users/savuvaris/orgs","repos_url":"https://api.github.com/users/savuvaris/repos","events_url":"https://api.github.com/users/savuvaris/events{/privacy}","received_events_url":"https://api.github.com/users/savuvaris/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":1,"created_at":"2015-05-27T09:46:59Z","updated_at":"2015-05-27T14:49:17Z","closed_at":null,"body":"- Katselmoitava koodi ladattu GITistä 27.5. kello 11.27.\r\n- Koodi kääntyi ja tulosti logiin yhden sanan diagonaalisesti.\r\n- Koodia ei ole vielä kauheasti, joten tässä yleisempi kysymys:\r\n\t- Onko tarkoituksena että taulukossa on vain yksi sanakirjasta poimittu sana, ja muut taulukon paikat täytetään satunnaisilla kirjaimilla?\r\n- Itse koodista keksin tälläiset kommentit:\r\n\t- Luokkajako tuntuu järkevältä tässä projektin vaiheessa. Lisää luokkia tarvittaneen vielä.\r\n\t- Metodit ovat myös rakennettu järkevästi ja noudattavat 'single responsibility' rakennetta.\r\n\t- Sana-luokan teeAnagrammi- ja permutoi-metodit ilmeisesti tällä hetkellä kääntää sanan nurinpäin, jos oikein ymmärsin(?). Onko tarkoitus laajentaa tätä siten, että tuloksena olisi oikea anagrammi (eli kirjaimet sekaisin)? Jos ei, ja tavoite onkin että kirjaimet ovat takaperoisessa järjestyksessä, niin sille löytyisi lyhyempikin tapa. \r\n\t- Sana-luokan r-parametrin voisi nimetä kuvaavammin. Tosin sitä ei taideta koodissa nyt käyttääkkään.\r\n\t- Peli-luokan sanastaMerkkitaulukko-metodi voisi ehkä olla Sana-luokassa, koska sitä käytetään myös siellä. (teeAnagrammi-metodissa)\r\n\t- Luokat ovat samassa paketissa. Niitä voisi jakaa useampiin pakkauksiin kun projekti etenee.\r\n\t- Käyttöliittymästä ei ole vielä koodia, joten kannattaa muistaa sitä rakentaessa, että erottaa käyttöliittymän ja muun pelilogiikan toisistaan."},"comment":{"url":"https://api.github.com/repos/VesaPuu/wordmaster/issues/comments/105941364","html_url":"https://github.com/VesaPuu/wordmaster/issues/1#issuecomment-105941364","issue_url":"https://api.github.com/repos/VesaPuu/wordmaster/issues/1","id":105941364,"user":{"login":"VesaPuu","id":12413517,"avatar_url":"https://avatars.githubusercontent.com/u/12413517?v=3","gravatar_id":"","url":"https://api.github.com/users/VesaPuu","html_url":"https://github.com/VesaPuu","followers_url":"https://api.github.com/users/VesaPuu/followers","following_url":"https://api.github.com/users/VesaPuu/following{/other_user}","gists_url":"https://api.github.com/users/VesaPuu/gists{/gist_id}","starred_url":"https://api.github.com/users/VesaPuu/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/VesaPuu/subscriptions","organizations_url":"https://api.github.com/users/VesaPuu/orgs","repos_url":"https://api.github.com/users/VesaPuu/repos","events_url":"https://api.github.com/users/VesaPuu/events{/privacy}","received_events_url":"https://api.github.com/users/VesaPuu/received_events","type":"User","site_admin":false},"created_at":"2015-05-27T14:49:17Z","updated_at":"2015-05-27T14:49:17Z","body":"Kiitos palautteesta! On tarkoitus, että taulukossa on useita sanoja. Toistaiseksi olen kokeillut vain yhden sanan sijoittamista. Sanat sijoittuvat taulukkoon, kuten Wordbase-pelissä niin, että seuraava kirjain voi olla missä tahansa suunnassa ympäröivissä kahdeksassa ruudussa (reunat tietenkin huomioiden). Se, miten sanojen sijoittelu käytännössä kannattaa tehdä, on vielä auki. Mielestäni teeAnagrammi ja permutoi-metodit kyllä tuottavat kaikki teeAnagrammi-metodille parametriksi annetun sanan kirjainten järjestykset, esim. AALLOKKO, AALLOKOK, AALLOKKO, AALLOKOK, AALLOOKK, AALLOOKK, AALLKOKO, AALLKOOK, AALLKKOO, AALLKKOO, AALLKOOK jne."}}