improvement: Context und Mapper eingebunden (WIP)

Thomas Ziemer
1 parent 33e38bd1
Showing 3 changed files with 34 additions and 18 deletions
src/main/java/net/ziemers/swxercise/lg/user/service/UserDtoToEntityContextService.java
src/main/java/net/ziemers/swxercise/lg/user/service/UserDtoToEntityMapper.java
src/main/java/net/ziemers/swxercise/lg/user/service/UserService.java
@@ -3,7 +3,8 @@ package net.ziemers.swxercise.lg.user.service;
 import javax.ejb.Stateless;
 import javax.inject.Inject;
-import net.ziemers.swxercise.db.dao.GenericDao;
+import net.ziemers.swxercise.db.dao.user.UserDao;
+import net.ziemers.swxercise.lg.model.user.Profile;
 import net.ziemers.swxercise.lg.model.user.User;
 import net.ziemers.swxercise.lg.user.dto.UserDto;
@@ -11,7 +12,7 @@ import net.ziemers.swxercise.lg.user.dto.UserDto;
 public class UserDtoToEntityContextService {
     @Inject
-    private GenericDao dao;
+    private UserDao dao;
     /**
      * Erzeugt den Aktualisierungskontext zur übergebenen Zielentität.
@@ -20,12 +21,22 @@ public class UserDtoToEntityContextService {
      * @return den erzeugten Kontext.
      */
     public UserDtoToEntityContext createContext(final UserDto dto) {
-      final UserDtoToEntityContext ctx = new UserDtoToEntityContext();
-
-      ctx.dto = dto;
-      ctx.user = dto.getEntityId() == null ? new User() : dao.findById(User.class, dto.getEntityId());
-
-      return ctx;
+        final UserDtoToEntityContext ctx = new UserDtoToEntityContext();
+
+        // das UserDTO in den Kontext füllen
+        ctx.dto = dto;
+
+        // einen neuen oder einen bereits existierenden Benutzer in den Kontext füllen
+        ctx.user = dao.findByUsername(dto.getUsername());
+        if (ctx.user == null) {
+            // es soll niemals ein Profil ohne Benutzername und Kennwort geben
+            final Profile profile = new Profile(dto.getUsername(), dto.getPassword());
+
+            // wir füllen das User-Objekt mit Method Chaining
+            ctx.user = new User(dto.getFirstname(), dto.getLastname())
+                    .withProfile(profile);
+        }
+        return ctx;
     }
 }
@@ -15,7 +15,7 @@ public class UserDtoToEntityMapper {
       ctx.user.setFirstname(ctx.dto.getFirstname());
       ctx.user.setLastname(ctx.dto.getLastname());
       //ctx.user.setProfile(ctx.dto.getProfile());
-      //ctx.user.setAddress(ctx.fto.getAddress());
+      //ctx.user.setAddress(ctx.dto.getAddress());
       return ctx;
     }
@@ -25,6 +25,12 @@ public class UserService {
     @Inject
     private SessionContext sessionContext;
+    @Inject
+    private UserDtoToEntityContextService ctxService;
+
+    @Inject
+    private UserDtoToEntityMapper mapper;
+
     /**
      * Meldet den Benutzer im SessionContext an.
      *
@@ -79,16 +85,15 @@ public class UserService {
      * @return die Id des neuen Benutzers, wenn die Erstellung erfolgreich war.
      */
     public Long createUser(final UserDto dto) {
-        // der Benutzer darf natürlich noch nicht existieren :)
-        User user = dao.findByUsername(dto.getUsername());
-        if (user == null) {
-            final Profile profile = new Profile(dto.getUsername(), dto.getPassword());
-
-            // wir füllen das User-Objekt mit Method Chaining
-            user = new User(dto.getFirstname(), dto.getLastname())
-                    .withProfile(profile);
+        final UserDtoToEntityContext ctx = ctxService.createContext(dto);
+        mapper.map(ctx);
+        return persistUserIfNew(ctx);
+    }
-            return dao.save(user);
+    private Long persistUserIfNew(final UserDtoToEntityContext ctx) {
+        // nur ein neuer Benutzer hat noch keine oid
+        if (ctx.user.getId() == null) {
+            return dao.save(ctx.user);
         }
         return null;
     }