gunnarbjo / finalass Goto Github PK

Almennar athugasemdir

• Það virðist vera svolítið mikill munur á framlagi þegar ég skoða commit history. Vonandi unnuð þið þetta í sameiningu.
• UML ekki í site.
• Javadoc report ekki í site.
• Það er beðið um tvær scriptur í verkefnalýsingunni. Önnur sem pakkar verkefninu í jar skrá, og önnur scripta semkeyrir verkefnið án þess að nota maven eða IDE, þ.e. keyrir jar skránna. Þetta hefur verið sameinað í eina scriptu.
• UML ekki í README.md
• LICENSE ekki í README.md.
• Allir klasar ættu að byrja með hástaf. bæði textUI og printText byrja með lágstaf.

UML

• Það vantar printText klasann í UML klasaritið.
• Það vantar E í mptyAutherListException. Kannski vandamál með exportið

LibrarySystem

• passa þarf að vera vandvirkur með hástafi og lágstafi. Það er munur á Boolean og boolean, það fyrra er hlutur og það seinna er frumtag. Kannski smáatriði, en í tilfellum þar sem Klasaritið er gert á undan forritun, þá gæti það valdið miskilningi.
• findStudentByName skilar Student en ekki User.
• findFacultyMemberByName skilar FacultyMember en ekki User.

Örvar

• Ég hefði sjálfur notast við association ör á milli testUI og LibrarySystem, en það er hægt að færa rök fyrir því að aggregation ör virki. Aftur á móti snýr örin vitlaust, tígullinn á ekki að vera á hausnum, heldur á skottinu. Tígullinn á semsagt að vera uppvið testUI klasann.
  • Það stendur líka text hjá örinni sem mætti fjarlægja.
• Tígullinn á ekki að vera fylltur í aggregation örvunum, ekki nema þetta eigi að vera composition ör allstaðar, sem ég myndi telja vitlaust.
• Það er enginn greinarmunur á Hausnum á Association örvunum og Inheritance örvunum. Hausinn er vitlaus í báðum þeim tilfellum.