Review

[andrivg]

Almennar athugasemdir

• Fæ villu þegar ég keyri mvn test

  [ERROR] Failed to execute goal org.apache.maven.plugins:maven-surefire-plugin:3.2.2:test (default-test) on project AudioPlayer3:
  [ERROR]
  [ERROR] Please refer to C:\Users\Lenovo\Documents\Kennsla\HÍ\HBV202G_2024V\students_final\sub17\AudioPlayer3\target\surefire-reports for the individual test results.
  [ERROR] Please refer to dump files (if any exist) [date].dump, [date]-jvmRun[N].dump and [date].dumpstream.
  [ERROR] There was an error in the forked process
  [ERROR] Unable to create test class 'askrifandiTest'

• Prófin (testin) tilheyra engum pökkum. Eðlilegt er að test og main möppustrúkturinn sé symmetrískur. Prófanirnar eru fyrir vinnslu pakkann, svo eðlilegt væri að það væri mappa v og undir hann mappa vinnsla sem innihéldi prófin.
• Eftirfarandi plugin eru ekki með nýjustu útgáfu

  [INFO] The following dependencies in Dependencies have newer versions:
  [INFO]   org.junit.jupiter:junit-jupiter-api .............. 5.10.0 -> 5.11.0-M1
  [INFO]   org.junit.jupiter:junit-jupiter-engine ........... 5.10.0 -> 5.11.0-M1
  [INFO]   org.openjfx:javafx-controls ............................ 21 -> 23-ea+3
  [INFO]   org.openjfx:javafx-fxml ................................ 21 -> 23-ea+3
  [INFO]   org.openjfx:javafx-media ............................... 21 -> 23-ea+3

• Fæ Maven can't find the file 'spdx.org\licenses\MIT.html' on the system. þegar ég opna AudioPlayer3/target/site/licenses.html eftir að keyra mvn site. Þið hefðuð einfaldlega uppfært pom.xml með eftirfarandi:

    <licenses>
        <license>
          <name>MIT</name>
          <url>LICENSE</url>
        </license>
    </licenses>

• Javadoc report ekki í mvn site.
• UML klasarit ekki í mvn site.
• Ég finn ekkert um Design Patterns í README.md.
  • Sýnist þið vera að reyna að nota Observer Patternið, en mér sýnist það ekki vera útfært alveg rétt. Klasinn ObservableAskrifandi er að implementa AskrifandiObserver og er líka með tilviksbreytu sem er listi og inniheldur AskrifandiObserver stök. Er klasinn þá bæði observer og observable?
• I README.md er vitnað í createjar.cmd og runjar.cmd, en skjölin í repo-inu eru packagejar.sh og runjar.sh.

  UML

• Ég sé að á mörgum stöðum eru þið að gefa tilviksbreytunum eitthvert gildi þegar þær eru fastar. Nafnið á breytunni ætti að vera nógu lýsandi, svo ekki á að setja inn gildi.

  Lagalistar

• Fastarnir eru með gildin líka skráð beint inní klasann í klasaritinu. Hefði verið betra að merkja þá með {readOnly}. Var þetta eitthvað sem Umlet tólið gerði sjálfkrafa?
• listar er búinn til með java syntax og 2 stök. Betra hefði verið að lýsa tveggja staka sambandinu á Association örinni með multiplicity.

  Örvar

• Ekki rétt ör frá PlayerControllerInterface til PlayerController. I raun og veru er ekkert samband á milli klasans og interface, en mig grunar PlayerController hafi átt að implementa PlayerControllerInterface.
  • Sama á við um ListiControllerInterface og ListiController.
• Vantar Implements ör frá ObservableAskrifandi til AskrifandiObserver.
• Örin frá PlayerController til ViewSwitcher stemmir ekki. PlayerController hefur enga tilviksbreytu sem geymir ViewSwitcher.

[Elma-KG]

Hérna varðandi tilviksbreyturnar með fasta, þá tókum við þetta verkefni frá Ebbu og hún hefur viljað fá þá svona þess vegna gáfum við þeim gildi.

Ég er ekki alveg að fatta með að ekki hægt sé að keyra mvn test, því ég hef náð því í gegnum README.md þegar ýtt er þar á mvn test, er ég eitthvað að misskilja þetta

[andrivg]

Hérna varðandi tilviksbreyturnar með fasta, þá tókum við þetta verkefni frá Ebbu og hún hefur viljað fá þá svona þess vegna gáfum við þeim gildi.

Þessi athugasemd hjá mér er undir kaflanum UML. Þið eigið ekki að setja inn Gildið á breytunni inní UML klasaritið. . Ég er ekki að gagnrýna kóðann sjálfann, bara svo það sé skýrt. Heldur að þið setið gildin inní klasaritið.

Ég er ekki alveg að fatta með að ekki hægt sé að keyra mvn test, því ég hef náð því í gegnum README.md þegar ýtt er þar á mvn test, er ég eitthvað að misskilja þetta

• Hvaða stýrikerfi eru þið að nota?
• Ef þið farið í rót verkefnisins í terminal/cmd og skrifið mvn test fáið þið enga villu?

Það var enginn frádráttur þó ég fékk þessa villu. Ég leyfði ykkur að njóta vafans, og hugsaði að þetta gæti verið út af stýrikerfinu mínu sem þetta virkaði ekki hjá mér. Hef verið að fá þessa villu hjá fleirum sem eru að nota module-info og javafx.

[Elma-KG]

ahhh okey skil, takk fyrir útskýringuna. En ég fæ allavega ekki villu þegar ég keyri það kannski gæti verið eitthvað hjá mér sem sýnir ekki villuna allavega núna en þá er það bara mitt vesen:)