Implement backend cheat validation#155
Conversation
|
the-only-queen-anna
left a comment
the-only-queen-anna
left a comment
There was a problem hiding this comment.
Grundsätzlich würde ich sagen, das passt so, ich hab dir ein paar Vorschläge noch kommentiert, eventuell findest du ja was davon sinnvoll. Besonders das Logik in den Service verschieben wäre glaub ich besser.
| result.message(), | ||
| ChatMessageType.SYSTEM); | ||
|
|
||
| messagingTemplate.convertAndSendToUser( |
There was a problem hiding this comment.
Hast du getestet, ob das so funktioniert? Ich hatte mit sendToUser schon öfters Probleme
| * @param message the cheat request containing lobby, username and selected cards | ||
| */ | ||
| @MessageMapping("/cheat") | ||
| public void useCheat(CheatCardMessage message) { |
There was a problem hiding this comment.
Für einen Controller ist hier ziemlich viel Logik drin. Am besten das meiste in den GameService verschieben, dann brauchst du auch nicht die zusätzliche Dependency zum LobbyService.
| * | ||
| * @param message private message for the player | ||
| */ | ||
| public record CheatResult(String message) {} No newline at end of file |
There was a problem hiding this comment.
Wenn du nur einen String drin hast, könntest du theoretisch auch die Wrapper-Klasse weglassen. Vor allem, weil das ja rein backend-intern ist
2 participants
Overview
Implemented the backend logic for the cheat functionality.
Features
Validation Rules
Closes #123