Verfeinerungen

Thomas Ziemer
1 parent fce4ab74
Showing 9 changed files with 32 additions and 18 deletions
src/main/java/net/ziemers/swxercise/lg/user/dto/UserDto.java
src/main/java/net/ziemers/swxercise/lg/user/service/EntityToUserDtoContext.java
src/main/java/net/ziemers/swxercise/lg/user/service/EntityToUserDtoContextService.java
src/main/java/net/ziemers/swxercise/lg/user/service/EntityToUserDtoMapper.java
src/main/java/net/ziemers/swxercise/lg/user/service/RoleService.java
src/main/java/net/ziemers/swxercise/lg/user/service/UserDtoToEntityContext.java
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
@@ -11,16 +11,16 @@ public class UserDto {
     private Long entityId;
  
     @NotNull
-    private String username;
+    private String username;    // aus dem Profile
  
     @NotNull
-    private String password;
+    private String password;    // aus dem Profile
  
     private String firstname;
  
     private String lastname;
  
-    private String mailaddress;
+    private String mailaddress; // aus dem Profile
  
     public Long getEntityId() {
         return entityId;
 package net.ziemers.swxercise.lg.user.service;
  
 import net.ziemers.swxercise.lg.model.user.User;
+import net.ziemers.swxercise.lg.user.dto.UserDto;
  
 /**
  * Mapping-Kontext von {@link User} nach {@link UserDto}. Diese Klasse dient als Hilfe bei der
 package net.ziemers.swxercise.lg.user.service;
  
+import net.ziemers.swxercise.lg.model.user.User;
+import net.ziemers.swxercise.lg.user.dto.UserDto;
+
 import javax.ejb.Stateless;
  
 /**
 package net.ziemers.swxercise.lg.user.service;
  
+import net.ziemers.swxercise.lg.model.user.User;
 import net.ziemers.swxercise.lg.user.dto.UserDto;
  
 /**
- * Mapper zur Konvertierung von User-Entities in {@link UserDto}s.
+ * Mapper zur Konvertierung des Kontexts einer {@link User}-Entity in ein {@link UserDto}.
  */
 public class EntityToUserDtoMapper {
  
 package net.ziemers.swxercise.lg.user.service;
  
-import com.sun.mail.imap.Rights;
 import net.ziemers.swxercise.db.dao.user.RoleDao;
 import net.ziemers.swxercise.lg.model.user.Role;
 import net.ziemers.swxercise.lg.model.user.User;
 package net.ziemers.swxercise.lg.user.service;
  
+import net.ziemers.swxercise.lg.model.user.Address;
+import net.ziemers.swxercise.lg.model.user.Profile;
 import net.ziemers.swxercise.lg.model.user.User;
 import net.ziemers.swxercise.lg.user.dto.UserDto;
  
@@ -9,4 +11,8 @@ public class UserDtoToEntityContext {
  
     public User user;
  
+    public Profile profile;
+
+    public Address address;
+
 }
@@ -15,7 +15,7 @@ public class UserDtoToEntityContextService {
     private UserDao dao;
  
     /**
-     * Erzeugt den Aktualisierungskontext zur übergebenen Zielentität.
+     * Erzeugt den Erstellungs-/Aktualisierungskontext zum übergebenen Data Transfer Objekt der Zielentität.
      *
      * @param dto das Benutzer-DTO
      * @return den erzeugten Kontext.
@@ -30,11 +30,11 @@ public class UserDtoToEntityContextService {
         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());
+            ctx.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);
+                    .withProfile(ctx.profile);
         }
         return ctx;
     }
 package net.ziemers.swxercise.lg.user.service;
  
+import net.ziemers.swxercise.lg.user.dto.UserDto;
+
 import javax.ejb.Stateless;
  
 /**
- * Mapper zur Konvertierung von {@link UserDto}s in User-Entities.
+ * Mapper zur Konvertierung des Kontexts eines {@link UserDto}s in eine User-Entity.
  */
 @Stateless
 public class UserDtoToEntityMapper {
  
     /**
-     * Überträgt die Eigenschaften aus dem UserDto in die Zielentität.
+     * Überträgt die Eigenschaften aus dem UserDto sowie dem zusätzlichen Kontext in die Zielentität.
      *
      * @param ctx der Kontext mit den Eigenschaften und der Zielentität
      * @return den Kontext.
      */
     public UserDtoToEntityContext map(UserDtoToEntityContext ctx) {
-      ctx.user.setFirstname(ctx.dto.getFirstname());
-      ctx.user.setLastname(ctx.dto.getLastname());
-      //ctx.user.setProfile(ctx.dto.getProfile());
-      //ctx.user.setAddress(ctx.dto.getAddress());
-
-      return ctx;
+        ctx.user.setFirstname(ctx.dto.getFirstname());
+        ctx.user.setLastname(ctx.dto.getLastname());
+        if (ctx.profile != null) {
+            ctx.user.setProfile(ctx.profile);
+        }
+        if (ctx.address != null) {
+            ctx.user.setAddress(ctx.address);
+        }
+        return ctx;
     }
  
 }
@@ -7,7 +7,6 @@ import javax.ejb.Stateless;
 import javax.inject.Inject;
  
 import net.ziemers.swxercise.db.dao.user.UserDao;
-import net.ziemers.swxercise.lg.model.user.Profile;
 import net.ziemers.swxercise.lg.model.user.Role;
 import net.ziemers.swxercise.lg.model.user.User;
 import net.ziemers.swxercise.lg.user.dto.UserDto;
@@ -91,7 +90,7 @@ public class UserService {
     }
  
     private Long persistUserIfNew(final UserDtoToEntityContext ctx) {
-        // nur ein neuer Benutzer hat noch keine oid
+        // nur ein neuer Benutzer hat noch keine Id
         if (ctx.user.getId() == null) {
             return dao.save(ctx.user);
         }