From d703b1f255d35cc8e36bd4781d67eeb9740c454b Mon Sep 17 00:00:00 2001 From: tomvahl Date: Wed, 25 Mar 2020 14:47:46 +0100 Subject: [PATCH] refactor Exceptions Co-Authored-By: andibuls Co-Authored-By: Lukas Ettel --- .../gruppen2/service/ControllerService.java | 43 ++++++++++--------- 1 file changed, 22 insertions(+), 21 deletions(-) diff --git a/src/main/java/mops/gruppen2/service/ControllerService.java b/src/main/java/mops/gruppen2/service/ControllerService.java index bd38ce7..1e57d6d 100644 --- a/src/main/java/mops/gruppen2/service/ControllerService.java +++ b/src/main/java/mops/gruppen2/service/ControllerService.java @@ -45,6 +45,26 @@ public class ControllerService { this.logger = Logger.getLogger("controllerServiceLogger"); } + /** + * Überprüft ob alle Felder richtig gesetzt sind. + * @param description + * @param title + * @param userMaximum + */ + private void checkFields(String description, String title, Long userMaximum ) { + if(description == null) { + throw new BadParameterException("Die Beschreibung wurde nicht korrekt angegeben"); + } + + if(title == null) { + throw new BadParameterException("Der Titel wurde nicht korrekt angegeben"); + } + + if (userMaximum == null) { + throw new BadParameterException("Teilnehmeranzahl wurde nicht korrekt angegeben"); + } + } + /** * Erzeugt eine neue Gruppe, fügt den User, der die Gruppe erstellt hat, hinzu und setzt seine Rolle als Admin fest. * Zudem wird der Gruppentitel und die Gruppenbeschreibung erzeugt, welche vorher der Methode übergeben wurden. @@ -65,17 +85,7 @@ public class ControllerService { userMaximum = 100000L; } - if(description == null) { - throw new BadParameterException("Die Beschreibung wurde nicht korrekt angegeben"); - } - - if(title == null) { - throw new BadParameterException("Der Titel wurde nicht korrekt angegeben"); - } - - if (userMaximum == null) { - throw new BadParameterException("Teilnehmeranzahl wurde nicht korrekt angegeben"); - } + checkFields(description, title, userMaximum); visibility = visibility == null; @@ -101,17 +111,8 @@ public class ControllerService { userMaximum = 100000L; } - if(description == null) { - throw new BadParameterException("Die Beschreibung wurde nicht korrekt angegeben"); - } + checkFields(description, title, userMaximum); - if(title == null) { - throw new BadParameterException("Der Titel wurde nicht korrekt angegeben"); - } - - if (userMaximum == null) { - throw new BadParameterException("Teilnehmeranzahl wurde nicht korrekt angegeben"); - } if (!file.isEmpty()) { try { userList = CsvService.read(file.getInputStream());