Skip to content

Vertaisarviointi #2

@lporanta

Description

@lporanta

Määrittelydokumentti on ytimeikäs. Aiheesta olisi ollut mielenkiintoista lukea enemmänkin. Muutamia kirjoitusvirheitä löytyi. Kevyt oikoluku riittänee korjaamaan asian. Valmiin ohjelman toiminta on kuvattu mukavan selkeästi. Lopullisessa dokumentissa on varmaan sitten enemmän tekstiä?

BinomialTree:ssä ja FibonacciTree:ssä ei ole kommentteja ollenkaan. Koodi on kuitenkin sellaisenaan melko ymmärrettävää. Lopullisessa palautuksessa on varmaan hyvä kertoa edes jotain myös kommenttien muodossa. Heaps sen sijaan on hyvin perusteellisesti kommentoitu. Huomasin kuitenkin, että kekototeutuksissa käytetään vielä ArrayListiä. Sen korvaaminen omalla toteutuksella on silti ainoastaan pieni muotoseikka ja todennäköisesti jo työn alla. Ohjelman rakenne on myös yksinkertainen ja selkeä. Kaikki vaikuttaisi olevan käyttöliittymäpuolta lukuunottamatta hyvällä mallilla.

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions