Was macht getStatisticsByExerciseId in StatisticsResource?

Die Methode

    @GetMapping("/statistics/exercise/{id}")
    public ResponseEntity<StatisticsDTO> getStatisticsByExerciseId(@PathVariable Long id) {

scheint zuemlich kompliziert und hat keinen Kommentar. Sie scheint auch nie aufgerufen zu werden. Es gibt auch keine Tests dazu.

Was ist der Unterschied zu

    @GetMapping("/statistics/{id}")
    public ResponseEntity<StatisticsDTO> getStatistics(@PathVariable Long id) {

?

Bitte kommentieren, nutzen und testen, oder einfach löschen :-)

changed milestone to %2021 April Sprint Sharing

added low Priority label

 @GetMapping("/statistics/exercise/{id}")
    public ResponseEntity<StatisticsDTO> getStatisticsByExerciseId(@PathVariable Long id) {

ist der Rest Enpoint der aufgerufen wird, um die Statistiken für eine Aufgabe zu laden.

 @GetMapping("/statistics/{id}")
    public ResponseEntity<StatisticsDTO> getStatistics(@PathVariable Long id) {

Ist der Rest endpoint, wenn man aus der DB ein Statistikobjekt mit Hilfe der automatisch erzeugten id abrufen will.

Die Datenbank speichert eine Aufgabe mit der automatisch erzeugten ID, der Anzahl der Downloads, Anzahl der Views und der ExerciseId ab. Wenn die Details von einer Aufgabe abgerufen werden, dann wird die im Client vorhandene ExerciseId verwendet, um eine Restanfrage an den Server zu schicken der dann diese ExerciseId verwendet, um die Daten aus der DB abzufragen.

OK, alles klar.

Allerdings sollte man vielleicht die Parameter besser benennen:

`public ResponseEntity<StatisticsDTO> getStatisticsByExerciseId(@PathVariable Long id) {`
  -> `public ResponseEntity<StatisticsDTO> getStatisticsByExerciseId(@PathVariable Long exerciseId) {

`public ResponseEntity<StatisticsDTO> getStatistics(@PathVariable Long id) {`
  -> `public ResponseEntity<StatisticsDTO> getStatistics(@PathVariable Long dbId) {

Außerdem scheint es wohl keine Tests für getStatisticsByExerciseId zu geben?

Ja stimmt, hab mit commit tests hinzugefügt: https://sharing-codeability.uibk.ac.at/sharing/codeability-sharing-platform/-/commit/90839eff1bc4953583be431a645b896f75c387c5

closed