Rating about conference-system

[autarchprinceps]

Der User kann seine Bewertung nicht abgeben, weil wenn er Sie noch nicht abgegeben hat, dann existiert anscheinend noch keine Bewertung in der currentConference.getConferenceRatingCollection()
Wenn Sie dort aber nicht existiert, dann schlägt ein .setRating mit NullPointerException fehl.
Wie erstellt man eine gänzlich neue Bewertung wenn noch keine existiert?

[mbasti]

Du hast die Anzahl von Sternen auf 6 reduziert. Kannst du mir erklären, wie das neue Rating funktioniert?

Bei doRate substrahierst du 6 von den momentanen Wert, also sind die möglichen Int-Werte -5 bis 0?
Das scheint mir etwas merkwürdig, was hast du dir dabei gedacht?

Ich weiß nicht mehr warum und macht auch keinen Sinn für mich ^^

Der User kann seine Bewertung nicht abgeben, weil wenn er Sie noch nicht abgegeben hat, dann existiert anscheinend noch keine Bewertung in der currentConference.getConferenceRatingCollection()
Wenn Sie dort aber nicht existiert, dann schlägt ein .setRating mit NullPointerException fehl.
Wie erstellt man eine gänzlich neue Bewertung wenn noch keine existiert?

Ja auf Null prüfen und ein Rating anlegen, oder wenn nicht Null das alte Rating ändern, würde ich sagen...

[autarchprinceps]

Ok, aber wie genau lege ich ein neues Rating an?

[mbasti]

Eigentlich funktioniert das Bewerten jetzt. Ich verstehe nur nicht was du hiermit machst:

Math.round(confRatings.stream().map((conferenceRating) -> conferenceRating.getRating()).reduce((a, b) -> a + b).orElse(ratingCount * 3) / ratingCount);

[autarchprinceps]

Ich summiere alle Bewertungen auf und teile durch die Anzahl der Bewertungen. Das ist das ganz normale arithmetische Mittel oder wolltest du einen Median benutzen?

[mbasti]

Ok, aber warum dieses ratingCount *3 ?

[autarchprinceps]

Das ist nur der default Wert. Der sollte nie in den Fall reingehen, da wir ja ihn vorher überprüfen.
Ich gucke nachher mal, ob wir das noch brauchen. Es kann sein, dass, wenn ich das wegmache, ein try catch nötig wäre.