improvement: Kontext und Mapper für create- und updateUser() optimiert

Thomas Ziemer
1 parent 7a765da2
Showing 7 changed files with 53 additions and 48 deletions
src/main/java/net/ziemers/swxercise/lg/user/dto/UserDto.java
src/main/java/net/ziemers/swxercise/lg/user/service/EntityToUserDtoMapper.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
src/main/java/net/ziemers/swxercise/ui/UserViewController.java
 package net.ziemers.swxercise.lg.user.dto;
+import net.ziemers.swxercise.lg.model.user.User;
+
 import javax.validation.constraints.NotNull;
 /**
@@ -8,7 +10,7 @@ import javax.validation.constraints.NotNull;
  */
 public class UserDto {
-    private Long entityId;
+    private User user = null;
     @NotNull
     private String username;    // aus dem Profile
@@ -22,12 +24,12 @@ public class UserDto {
     private String mailaddress; // aus dem Profile
-    public Long getEntityId() {
-        return entityId;
+    public User getUser() {
+        return user;
     }
-    public void setEntityId(Long entityId) {
-        this.entityId = entityId;
+    public void setUser(User user) {
+        this.user = user;
     }
     public String getUsername() {
@@ -9,7 +9,7 @@ import net.ziemers.swxercise.lg.user.dto.UserDto;
 public class EntityToUserDtoMapper {
     /**
-     * Erzeugt ein DTO aus dem übergebenem Kontext.
+     * Erzeugt ein Data Transfer Object aus dem übergebenem Kontext.
      *
      * @param ctx der Kontext
      * @return das erzeugte DTO.
@@ -18,15 +18,15 @@ public class EntityToUserDtoMapper {
         final UserDto dto = new UserDto();
         if (ctx.user != null) {
-          mapEntityId(ctx, dto);
+          mapUser(ctx, dto);
           mapFirstname(ctx, dto);
           mapLastname(ctx, dto);
         }
         return dto;
     }
-    private void mapEntityId(EntityToUserDtoContext ctx, UserDto dto) {
-        dto.setEntityId(ctx.user.getId());
+    private void mapUser(EntityToUserDtoContext ctx, UserDto dto) {
+        dto.setUser(ctx.user);
     }
     private void mapFirstname(EntityToUserDtoContext ctx, UserDto dto) {
@@ -13,6 +13,6 @@ public class UserDtoToEntityContext {
     public Profile profile;
-    public Address address;
+    public Address address = null;
 }
@@ -22,20 +22,27 @@ public class UserDtoToEntityContextService {
      */
     public UserDtoToEntityContext createContext(final UserDto dto) {
         final UserDtoToEntityContext ctx = new UserDtoToEntityContext();
+        final User existingUser = dto.getUser();
         // das übergebene 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
-            ctx.profile = new Profile(dto.getUsername(), dto.getPassword());
+        // einen neuen Benutzer in den Kontext füllen
+        if (existingUser == null) {
+            // es darf nicht mehrere Benutzer mit dem selben Benutzernamen geben!
+            if (dao.findByUsername(dto.getUsername()) == null) {
+                // es soll niemals ein Profil ohne Benutzername und Kennwort geben
+                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(ctx.profile);
+            }
+        }
-            // wir füllen das User-Objekt mit Method Chaining
-            ctx.user = new User(dto.getFirstname(), dto.getLastname())
-                    .withProfile(ctx.profile);
-        } else {
+        // einen bereits existierenden Benutzer in den Kontext füllen
+        else {
+            ctx.user = existingUser;
             ctx.profile = ctx.user.getProfile();
             ctx.address = ctx.user.getAddress();
         }
@@ -11,26 +11,30 @@ import javax.ejb.Stateless;
 public class UserDtoToEntityMapper {
     /**
-     * Überträgt die Eigenschaften aus dem UserDto sowie dem zusätzlichen Kontext in die Zielentität.
+     * Überträgt die Eigenschaften aus dem UserDto sowie dem zusätzlichen Kontext in die Zielentitäten.
+     * Anzumerken ist, dass der Benutzername einer existierenden Profile-Entität niemals aktualisiert
+     * wird.
      *
      * @param ctx der Kontext mit den Eigenschaften und der Zielentität
      * @return den Kontext.
      */
     public UserDtoToEntityContext map(UserDtoToEntityContext ctx) {
-        // User-Objekt mappen
+        // ins User-Objekt mappen
         ctx.user.setFirstname(ctx.dto.getFirstname());
         ctx.user.setLastname(ctx.dto.getLastname());
-        // Profile-Objekt mappen, falls gegeben
+        // ins Profile-Objekt mappen, falls gegeben
         if (ctx.profile != null) {
             ctx.user.setProfile(ctx.profile);
+
+            // eventuell ein neues Kenntwort ins Profile-Objekt mappen
             if(ctx.dto.getPassword().length() > 0) {
                 ctx.profile.setPassword(ctx.dto.getPassword());
             }
             ctx.profile.setMailaddress(ctx.dto.getMailaddress());
         }
-        // Address-Objekt mappen, falls gegeben
+        // ins Address-Objekt mappen, falls gegeben
         if (ctx.address != null) {
             ctx.user.setAddress(ctx.address);
         }
@@ -81,20 +81,17 @@ public class UserService {
      * Zwischen der Groß- und Kleinschreibung wird nicht unterschieden.
      *
      * @param dto das {@link UserDto} enthält die Eigenschaften des zu erstellenden Benutzers
-     * @return die Id des neuen Benutzers, wenn die Erstellung erfolgreich war.
+     * @return <code>true</code>, wenn das Aktualisieren des Benutzers erfolgreich war.
      */
-    public Long createUser(final UserDto dto) {
+    public boolean createUser(final UserDto dto) {
+        // Kontext für den zu erstellenden Benutzer erzeugen; falls bereits ein Benutzer
+        // mit dem selben Benutzernamen existiert, wird kein Benutzerobjekt zurückgeliefert
         final UserDtoToEntityContext ctx = ctxService.createContext(dto);
-        mapper.map(ctx);
-        return persistUserIfNew(ctx);
-    }
-
-    private Long persistUserIfNew(final UserDtoToEntityContext ctx) {
-        // nur ein neuer Benutzer hat noch keine Id
-        if (ctx.user.getId() == null) {
-            return dao.save(ctx.user);
+        if (ctx.user != null) {
+            mapper.map(ctx);
+            return dao.save(ctx.user) != null;
         }
-        return null;
+        return false;
     }
     /**
@@ -107,8 +104,12 @@ public class UserService {
     public boolean updateUser(final Long id, final UserDto dto) {
         final User user = dao.findById(id);
         if (user != null) {
-            // TODO noch zu implementieren
-            return false;
+            // auf dieses Benutzerobjekt soll sich die Aktualisierung beziehen
+            dto.setUser(user);
+
+            final UserDtoToEntityContext ctx = ctxService.createContext(dto);
+            mapper.map(ctx);
+            return dao.saveOrUpdate(ctx.user) != null;
         }
         return false;
     }
@@ -120,16 +121,8 @@ public class UserService {
      * @return <code>true</code>, wenn das Aktualisieren des Benutzers erfolgreich war.
      */
     public boolean updateUser(final UserDto dto) {
-        // ist zurzeit ein Benutzer angemeldet, können wir ihn aktualisieren
         final User user = sessionContext.getUser();
-        if (user != null) {
-            // der Benutzername darf sich beim Aktualisieren nicht mehr ändern!
-            dto.withUsername(user.getProfile().getUsername());
-            final UserDtoToEntityContext ctx = ctxService.createContext(dto);
-            mapper.map(ctx);
-            return dao.saveOrUpdate(ctx.user) != null;
-        }
-        return false;
+        return updateUser(user.getId(), dto);
     }
     /**
@@ -89,9 +89,8 @@ public class UserViewController {
     @Produces(MediaType.APPLICATION_JSON)
     @RolesAllowed(RightState.Constants.ADMIN)
     public RestResponse createUser(UserDto dto) {
-        final Long id = userService.createUser(dto);
-        if (id != null) {
-            return new RestResponse(ResponseState.SUCCESS, String.valueOf(id));
+        if (userService.createUser(dto)) {
+            return new RestResponse();
         }
         return new RestResponse(ResponseState.ALREADY_EXISTING);
     }