jekiiski
commented
10 years ago Kiitokset koodikatselmoinnista.
Näinhän siinä kävi, että kun suunnittelee projektia mielessään niin ei tule kirjattua itselleen selviä asioita koodiin.
Eli jos palkkaa ulkomaisen työntekijän on riski, että työnjälki ei kelpaa rakennustarkastajalle (jota ei vielä ole implementoitu) ja tällöin työn joutuu tekemään uudelleen. Em. syystä siis metodi lisaalaajuutta(). Samaten tuo pätevyyden "ostaminen" työntekijälle liittyy tilanteeseen, jossa työmaa vaatii erityisen pätevyyden, kuten esim. sähkötyöt tai putkityöt. Ilman tarvittavaa pätevyyttä voi rakennustarkastaja tulla jälleen kylään. Pätevyyden hankkiminen työntekijälle lisää myös työntekijän tehokkuutta hieman.
Ulkomaisten työntekijöiden käyttöön liittyy vielä veronkiertämisen kiinnijäämisen riski. Eli mikäli työmaalla on työskennellyt ulkomainen työntekijä, voi verottaja mätkäistä veronkierrosta sakot.
Peli siis on valintoja kotimaisen ja ulkomaisen työvoiman käytöstä siten, että pelaajalle tulisi mahdollisimman suuri voitto työmaasta. On myös mahdollista, että jollain työmaalla ei voittoa voi tehdä käyttämällä kotimaista työvoimaa.
Aloitin katselmoinnin 18:43 21.9.2014
Testit Raksamanagerin logiikka oli testattu selkeästi ja kattavasti.
CLI Käyttöjärjestelmän koodaus oli helposti ymmärrettävää ja johdonmukaista. Pelaaminen onnistui nopeasti syöttämällä valintaa vastaavan numeron. Eirtyiskiitos valikon valintojen tulostuksesta jokaisen valinnan jälkeen.
Pelaaja Metodit hyvin lyhyitä ja ytimekkäistä. Kokonaisuutena hyvin selkeä.
Peli Metodit lyhyitä ja nimet kuvaavia. Teki koodin ymmärtämisestä nopeaa. Ainoastaan Seuraavaksi()-metodin toiminta jäi mietityttämään ja olisin halunnut nähdä tämän toteutuksen. Arpoo ilmeisesti satunnaisgeneraattorilla uudet työmaat ja työntekijät valittavaski ja työmaat edistyvät.
Satunnaisgeneraattori Nimeäminen voisi olla luovempaa. Esimeriksi työmaiden nimet voisi arpoa jostain pitkästä yrityslistasta?
Työmaa Mitä lisaalaajuutta() tekee? En löytänyt vastausta koodista.
Työntekijä Selkeä ja pätevyysidea on hauska. Mietin että pitäisikö ulkomaisen työntekijän kouluttaminen olla hankalampaa? Jo kahdella pätevyydellä ulkomainen on yhtä tehokas kuin kotimainen, mutta 400 halvempi/vuoro?