Skip to content

Latest commit

 

History

History
38 lines (25 loc) · 1.37 KB

MOTTATT-TILBAKEMELDING.md

File metadata and controls

38 lines (25 loc) · 1.37 KB
Raw

#Gitt tilbakemelding

For gruppe:

  • Alexander Bredesen
  • Håvard Nordlie Mathisen
  • Jason Nilsen

Generell tilbakemelding

  • Mappestrukturen på toppnivå er helt super og veldig oversiktlig, men det kunne vært delt inn i flere undermapper på de lavere nivåene (spesielt i cli-mappen). Dette hadde hjulpet på forståelsen av hvordan programmet henger sammen.

  • Kunne vært skrevet korte kommentarer for å beskrive hva de ulike klassene gjør. Det er noen ganger litt vanskelig for våre utrente kode-øyne å forstå hva klassene gjør.

  • Generelt er det veldig ryddig, fin og lettleselig kode, spesielt i cli-klassene.

  • Noen steder eksisterer det metoder som ikke blir brukt, og som dermed er unødvendige å ha med, jo mindre de planlegges å brukes ved en utvidelse.

  • Når vi prøver å kjøre mvn test, feiler det i "common". Vi får derfor ikke kjørt programmet eller testet det.

Hva lærte vi av koden?

  • Veldig flinke til å dele opp koden i naturlige klasser, slik at hver klasse gjør nøyaktig det den skal og ingenting mer. Dette er noe vi selv kunne vært bedre på.

Hva forstod vi ikke av koden?

  • Vi forstår ikke hvorfor dere har valgt å la Controlleren mellom databasen og serveren sette statuskoder i serveren. Vi hadde tenkt at dette er noe som burde foregå i serveren?

Skrevet av:

  • Stine Sandberg
  • Safet Zahirovic
  • Richard W. Zacho