Koodikatselmointi

[koivkai]

Projekti ladattu 2.2. klo 22.18

-Seuraat clean code –periaatteita hyvin.
-Kommentit ovat hyviä
-Testit vaikuttivat toimivilta ja menivät kaikki läpi. Muista testata myös koko ohjelman toimintaa yksittäisten luokkien lisäksi.
-Kokeilin tekstikäyttöliittymää. Se ei toimi vielä kunnolla, mutta tällä tuskin on väliä jos teet kuitenkin graafisen käyttöliittymän.
-Sekoittat ikävästi suomea ja englantia koodissa, esim luokat ovat englanniksi mutta muuttujat suomeksi. Tämä vaikeuttaa koodin lukemista. Suossittelisin käännätämään kaiken yhdelle kielelle.
-En tiedä oikein mitään musiikista, mutta vaikkuttaa erikoiselta että song, drumPhrase ja hit liittyvät toisiinsa vain Machinen kautta, kun eivätkö biisit koostu fraasesta ja fraasit hiteistä? Ohjelman rakenteen ei tietenkään ole kuitenkaan pakko seurata todellisuutta joten tee mitä tahdot. Tai jätä kommitti huomiotta jos en ymmärtänyt musiikkitermejä oikein.

• Kaiken kaikkkiaan projektisi näyttää etenevän hyvin.

Ystävällisin terveisin,
Kaius