Witam. Mam pytanie dotyczące tego co umieścić w poszczególnych częściach kodu:
Przykładowo mam managed beana JSF a w nim metodę do nawigacji wywołującą metodę tworzącą użytkownika:
public String createNewAccount() {
FacesContext context = FacesContext.getCurrentInstance();
Map requestParameterMap = (Map) context.getExternalContext().getRequestParameterMap();
try {
String userRole = requestParameterMap.get("userRole").toString();
String active = requestParameterMap.get("active").toString();
Boolean act = Boolean.parseBoolean(active);
user.setRole(userRole);
user.setActive(act);
if ((user.getEmail() != null) && (userDao.findEmailExist(user.getEmail()))) {
sendErrorMessageToUser("Użytkownik z podanym adresem email istnieje w bazie");
return null;
} else {
userDao.create(user);
}
} catch (EJBException e) {
sendErrorMessageToUser("Błąd rejestracji");
return null;
}
return "newAccountCreated";
}
Wyczytałem, że managed beany JSF powinny być tylko pośrednikiem między kodem java a stroną i tutaj moje pytanie czy takie robienie weryfikacji poprawności w beanie JSF jest poprawne? Czy należałoby utworzyć np. warstwę serwisową, w które taka weryfikacja się dokona? Np. coś takiego:
@Stateless
public class UserDao implements UserDaoLocal {
@PersistenceContext
private EntityManager em;
private User user;
@Override
public void create(User user) {
em.persist(user);
}
i do tego warstwę serwisową np. coś takiego:
@Stateless
public class UserDaoService implements UserDaoServiceLocal {
@EJB
private UserDaoLocal userDao
private User user;
@Override
public String create(User user) {
if(findEmailExist(user.email) {
return "emailExist";
} else {
userDao.create(user);
return "create";
}
}
@Override
public boolean findEmailExist(String email) {
try {
Query q = em.createNamedQuery("User.findByEmail");
q.setParameter("email", email);
user = (User) q.getSingleResult();
return true;
} catch (Exception e) {
System.out.println("Błąd email: " + e.getMessage());
return false;
}
}
i wtedy w beanie jsf tylko coś takiego:
public String createNewAccount() {
FacesContext context = FacesContext.getCurrentInstance();
Map requestParameterMap = (Map) context.getExternalContext().getRequestParameterMap();
try {
String userRole = requestParameterMap.get("userRole").toString();
String active = requestParameterMap.get("active").toString();
Boolean act = Boolean.parseBoolean(active);
user.setRole(userRole);
user.setActive(act);
String result userDaoService.create(user);
if (result = "emailExist") {
sendErrorMessageToUser("Użytkownik z podanym adresem email istnieje w bazie");
return null;
} else
return result;
}
} catch (EJBException e) {
sendErrorMessageToUser("Błąd rejestracji");
return null;
}
}