Koodikatselmointi
Opened this issue · 0 comments
Blodir commented
projekti ladattu 30.1.15 14:15
Moi,
muusikkona arvostan projektisi aihetta suuresti!
- Tämä ehkä on enemmänkin toiminnallinen kysymys, mutta näyttää siltä että Metronomisi tukee vain neljäsosa pohjaisia tahtilajeja ("kovakoodattu"), ehkä olisi järkevää toteuttaa ohjelma niin, että kahdeksasosa (esim 7/8) ja jopa kuudestoistaosa pohjaisten tahtilajien käyttö olisi mahdollista? Tietenkin tätä voi simuloida tuplaamalla tempon, mutta se ei ole kovin intuitiivista - ainakaan näillä muuttujanimillä ('quarters')
- 'Rod' ei ole kovin kuvaava nimi luokalle joka soittelee äänifiluja
- metodi 'setBpm' ei täytä Clean Code periaatteita (tekee kahta eri asiaa)
- Testejä ei ole kovin kattavasti
Kaikenkaikkiaan hyvin aluilla oleva projekti. Opin jotain uutta koodistasi :)