Witam. Mam takiego JSFowego beana, w którym mam metodę tworzącą użytkownika:
public String createNewAccount() {
//pobranie parametrów sesji
String userRole = requestParameterMap.get("userRole").toString();
String active = requestParameterMap.get("active").toString();
Boolean act = Boolean.parseBoolean(active);
//ustawienie pobranych parametrów użytkownikowi
user.setRole(userRole);
user.setActive(act);
String result = userService.create(user); //wywołanie metody tworzącej nowe konto użytkownika
//sprawdzenie czy podane dane nie istnieją już w bazie
if (result.equals("emailExist")) {
sendErrorMessageToUser("Użytkownik z podanym adresem email istnieje w bazie");
return null;
} else if (result.equals("personalIdExist")) {
sendErrorMessageToUser("Użytkownik z podanym numerem pesel istnieje w bazie");
return null;
} else if (result.equals("phoneExist")) {
sendErrorMessageToUser("Użytkownik z podanym numerem telefonu istnieje w bazie");
return null;
} else if (result.equals("ok")) {
return "newAccountCreated";
} else {
sendErrorMessageToUser("Błąd dodawania użytkownika do bazy");
return null;
}
}
Wywoływana jest w niej metoda
userService.create(user);
która zwraca różne stringi, jeśli np. ktoś z podanym emailem, telefonem lub peselem istnieje w bazie, jeśli nie ma to zwraca "ok".
I tutaj moje pytanie bo ktoś mi kiedyś powiedział, że takie coś jest nieestetyczne, chodziło mianowicie o te Stringi oraz ify, że taki styl programowania jest na poziomie początkującego. Niestety nie miałem jak już dopytać tej osoby w jaki inny sposób to zrealizować. I tutaj moje pytanie do kogoś kto w tym siedzi. W jaki inny sposób można to zrealizować? Czym zastąpić te zwracane Stringi oraz te kilka ifów? Mi osobiście nic do głowy nie przychodzi.