Refactoring

src/main/java/com/lions/dev/service/UsersService.java

@@ -29,26 +29,29 @@ public class UsersService {
    * @return L'utilisateur créé.
    */
   public Users createUser(UserCreateRequestDTO userCreateRequestDTO) {
+    // Vérification si l'email existe déjà
+    Optional<Users> existingUser = usersRepository.findByEmail(userCreateRequestDTO.getEmail());
+    if (existingUser.isPresent()) {
+      throw new IllegalArgumentException("Un utilisateur avec cet email existe déjà.");
+    }
+
     Users user = new Users();
     user.setNom(userCreateRequestDTO.getNom());
     user.setPrenoms(userCreateRequestDTO.getPrenoms());
     user.setEmail(userCreateRequestDTO.getEmail());
     user.setMotDePasse(userCreateRequestDTO.getMotDePasse()); // Hachage automatique
 
-    // Vérifier si le profile image est défini, sinon attribuer une image par défaut
-    if (userCreateRequestDTO.getProfileImageUrl() == null
-        || userCreateRequestDTO.getProfileImageUrl().isEmpty()) {
-      user.setProfileImageUrl("https://via.placeholder.com/150"); // Assigner une image par défaut
-    } else {
-      user.setProfileImageUrl(userCreateRequestDTO.getProfileImageUrl());
-    }
-
-    // Vérifier si le rôle est défini, sinon attribuer un rôle par défaut
-    if (userCreateRequestDTO.getRole() == null || userCreateRequestDTO.getRole().isEmpty()) {
-      user.setRole("USER"); // Assigner un rôle par défaut
-    } else {
-      user.setRole(userCreateRequestDTO.getRole());
-    }
+    // Logique pour l'image et le rôle par défaut.
+    user.setProfileImageUrl(
+        userCreateRequestDTO.getProfileImageUrl() != null
+        ? userCreateRequestDTO.getProfileImageUrl()
+        : "https://via.placeholder.com/150"
+    );
+    user.setRole(
+        userCreateRequestDTO.getRole() != null
+        ? userCreateRequestDTO.getRole()
+        : "USER"
+    );
 
     usersRepository.persist(user);
     System.out.println("[LOG] Utilisateur créé : " + user.getEmail());