Skip to content

Koodikatselmointi #1

@ahjyrkia

Description

@ahjyrkia

Projekti ladattu: 3.2.2015 klo 21.55

Positiivista:

  • Ohjelman rakenne on selkeä: luokat jaettu järkevästi paketteihin ja luokilla selkeät tehtävät.
  • Metodit eivät ole liian isoja ja hyvin kuvaavasti nimettyjä.
  • Ohjelman koodi on laadultaan tehokasta ja selkää. Ei turhaa toistoa sekä noudattaa Clean Code-periaatteita.
  • Testejä löytyy kattavasti ja jokaiselle luokalle. Menevät myös heittämällä läpi.
  • Pelin aihe vaikuttaa kiinnostavalta ja varmasti hauskalta toteuttaa. Pelin luonteen takia lisäominaisuuksien keksimisellä on vain taivas rajana.

Parannusehdotuksia:

  • Hyppäämistoiminto ohjelmassasi toimii niin että hahmo liikkuu tasaisesti ylös ja sitten alas. Ehkä painovoiman luominen peliin toisi hyppäämiseen luonnollisemman liikkeen. Tällä myös ratkaistaisiin se että hahmo ei jäisi ikinä ilmaan leijumaan tyhjän päälle.
  • Esteisiin kannattaa lisätä x- ja y-koordinaattien lisäksi halkaisija. Se auttanee kun pitää selvittää osuuko hahmo esteeseen (jos siis esteistä on tulossa erimuotoisia tulevaisuudessa).
  • Painovoiman ja esteiden kanssa temppuiluun löytyy ehkäpä hyödyllisiä vinkkejä täältä: http://mooc.cs.helsinki.fi/peliohjelmointi/materiaali/roller-graafinen-tasohyppely
  • Lisäominaisuusidiksiä: ehkäpä pelin nopeus voisi kiihtyä sen edetessä pidemmälle tai ehkä mainitsemasi kolikot voisivat toimia myös lisäelämänä vähän niin kuin Sonic-pelisarjassa?

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