Koodikatselmointi II

[emmakamarainen]

Latasin projektin 12.10. klo 14.28.

Hauska idea! Sitsien järjestäjänä niin käytän tätä seuraavalla kerralla kun pitää plassata :D

• Pitkistä metodeista kun puhutaan, niin ainakin Plassaaja- ja SitsaajienRyhmittaja- luokissa on muutama huiman pitkä metodi, joidenka tajuamiseen menee aikamoinen tovi minulta. Kyseisissä metodeissa näyttää olevan huima määrä parametreja joita metodille annetaan, ja pohdin voisikohan niitä jakaa useammiksi metodeiksi? Tai jotenkin selkeyttää. Tosin toki tässä vaihtoehtoja on aikamoinen määrä kaikkine kaveri- ja avec-toiveineen...
• Kokeilin ohjelmaa, juutuin lisäilemään sitsaajia sillä niiden lukumäärä ei ollut neljällä jaollinen. Helpottaisi jos tulostuksessa näkyisi montako sitsaajaa on sillä hetkellä lisätty. Olisin tahtonut kokeilla, miltä tulostus näyttää mutta sain ArrayListin nullPointerexceptionin. Tämä lienee kuitenkin työn alla oletan :)
• Sain myös errorin kun clean and buildasin, Plassaajan testi OnkoJokaToinenSitsaajaMies oli kyseessä. Muuten testaus näytti oikein hyvältä ja kattavalta.
• Olen vähän sekaisin siitä mitä missäkin järjestyksessä tapahtuu, joten olisin kaivannut sekvenssikaavioita.
• Tuntuu myös, että ohjelmassa on eri luokissa suuri määrä listoja joita käydään läpi (esim. asetaKaveriListastaJosSitsaajallaJaAvecillaEiToiveita), ja näissä metodeissa on hieman copy-pastea (voisi olla oma metodi esim. sille joka testaa onko sitsaaja jo laitettu paikalleen eli sitsaaja on jo ryhmitetyllä listalla). Metodi lyhenisi jos saisi muutamat if:it pois sieltä.
• Sitsi-ilmot ovat saatavissa excel-tiedostona (ainakin jos e-lomaketta on käytetty), olisi mahtavaa jos voisi vain syöttää tiedoston ja ohjelma lukisi sen ja sen perusteella loisi istumajärjestyksen. Ja sitten olisi vielä käsin mahdollista muuttaa järjestystä. Tämä nyt tuli vain mieleeni, siksi kerroin :D

Toivottavasti saat valmiiksi!

[horttanainen]

Kiitos koodikatselmoinnista!

Sain kuin sainkin plassaajan toimimaan ja parannusehdotuksesi toteutettu. Plassaaja plassaa nyt toimivasti jos sitä haluat kokeilla.

Lopullinen version olen kaavaillut tulostavan excel-taulukkoon ja omaavan graafisen käyttöliittymän. Aion myös tutkia tuon tiedostonsyöttö mahdollisuuden.