Adapting to DDD

Author: peterm85

src/main/java/com/example/code/apirest/controller/UserController.java

@@ -6,9 +6,9 @@
 
 public interface UserController {
 
-  public ResponseEntity<UserResponse> createUser(final UserRequest user);
+  ResponseEntity<UserResponse> createUser(final UserRequest user);
 
-  public ResponseEntity<UserResponse> getUserById(final Integer id);
+  ResponseEntity<UserResponse> getUserById(final Integer id);
 
-  public ResponseEntity<Void> modifyUser(final Integer id, final UserRequest user);
+  ResponseEntity<Void> modifyUser(final Integer id, final UserRequest user);
 }

src/main/java/com/example/code/apirest/controller/impl/UserControllerImpl.java

@@ -4,11 +4,12 @@
 import com.example.code.apirest.dto.UserRequest;
 import com.example.code.apirest.dto.UserResponse;
 import com.example.code.apirest.mapper.UserMapper;
+import com.example.code.application.dto.UserDto;
 import com.example.code.application.usecase.UserCreationUseCase;
 import com.example.code.application.usecase.UserModificationUseCase;
 import com.example.code.application.usecase.UserRetrieveUseCase;
-import com.example.code.domain.model.User;
 import jakarta.validation.Valid;
+import lombok.AllArgsConstructor;
 import lombok.extern.slf4j.Slf4j;
 import org.springframework.http.HttpStatus;
 import org.springframework.http.ResponseEntity;
@@ -20,33 +21,23 @@
 import org.springframework.web.bind.annotation.RequestMapping;
 import org.springframework.web.bind.annotation.RestController;
 
+@Slf4j
 @RestController
 @RequestMapping("/user")
-@Slf4j
+@AllArgsConstructor
 public class UserControllerImpl implements UserController {
 
-  private UserCreationUseCase userCreationUseCase;
-  private UserModificationUseCase userModificationUseCase;
-  private UserRetrieveUseCase userRetrieveUseCase;
-  private UserMapper userMapper;
-
-  public UserControllerImpl(
-      final UserCreationUseCase userCreationUseCase,
-      final UserModificationUseCase userModificationUseCase,
-      final UserRetrieveUseCase userRetrieveUseCase,
-      final UserMapper userMapper) {
-    this.userCreationUseCase = userCreationUseCase;
-    this.userModificationUseCase = userModificationUseCase;
-    this.userRetrieveUseCase = userRetrieveUseCase;
-    this.userMapper = userMapper;
-  }
+  private final UserCreationUseCase userCreationUseCase;
+  private final UserModificationUseCase userModificationUseCase;
+  private final UserRetrieveUseCase userRetrieveUseCase;
+  private final UserMapper userMapper;
 
   @Override
   @PostMapping
   public ResponseEntity<UserResponse> createUser(@Valid @RequestBody final UserRequest user) {
     log.info("Processing POST request with body {} ", user);
 
-    final User newUser = userCreationUseCase.createUser(userMapper.toUser(user));
+    final UserDto newUser = userCreationUseCase.createUser(userMapper.toUserCreationDto(user));
 
     return ResponseEntity.status(HttpStatus.CREATED).body(userMapper.toResponse(newUser));
   }
@@ -56,9 +47,9 @@ public ResponseEntity<UserResponse> createUser(@Valid @RequestBody final UserReq
   public ResponseEntity<UserResponse> getUserById(@PathVariable final Integer id) {
     log.info("Processing GET request by id {} ", id);
 
-    final User user = userRetrieveUseCase.getUserById(id);
+    final UserDto userDto = userRetrieveUseCase.getUserById(id);
 
-    return ResponseEntity.status(HttpStatus.OK).body(userMapper.toResponse(user));
+    return ResponseEntity.status(HttpStatus.OK).body(userMapper.toResponse(userDto));
   }
 
   @Override
@@ -67,7 +58,7 @@ public ResponseEntity<Void> modifyUser(
       @PathVariable final Integer id, @Valid @RequestBody final UserRequest user) {
     log.info("Processing PUT request with body {} ", user);
 
-    userModificationUseCase.modifyUser(id, userMapper.toUser(user));
+    userModificationUseCase.modifyUser(id, userMapper.toUserModificationDto(user));
 
     return ResponseEntity.status(HttpStatus.NO_CONTENT).build();
   }

src/main/java/com/example/code/apirest/mapper/UserMapper.java

@@ -2,13 +2,17 @@
 
 import com.example.code.apirest.dto.UserRequest;
 import com.example.code.apirest.dto.UserResponse;
-import com.example.code.domain.model.User;
+import com.example.code.application.dto.UserCreationDto;
+import com.example.code.application.dto.UserDto;
+import com.example.code.application.dto.UserModificationDto;
 import org.mapstruct.Mapper;
 
 @Mapper
 public interface UserMapper {
 
-  User toUser(UserRequest request);
+  UserCreationDto toUserCreationDto(UserRequest request);
 
-  UserResponse toResponse(User user);
+  UserModificationDto toUserModificationDto(UserRequest request);
+
+  UserResponse toResponse(UserDto userDto);
 }

src/main/java/com/example/code/application/dto/UserCreationDto.java

@@ -0,0 +1,17 @@
+package com.example.code.application.dto;
+
+import lombok.Data;
+
+@Data
+public class UserCreationDto {
+
+  private String name;
+
+  private String surname;
+
+  private String address;
+
+  private String city;
+
+  private String email;
+}

src/main/java/com/example/code/application/dto/UserDto.java

@@ -0,0 +1,19 @@
+package com.example.code.application.dto;
+
+import lombok.Data;
+
+@Data
+public class UserDto {
+
+  private Integer id;
+
+  private String name;
+
+  private String surname;
+
+  private String address;
+
+  private String city;
+
+  private String email;
+}

src/main/java/com/example/code/application/dto/UserModificationDto.java

@@ -0,0 +1,17 @@
+package com.example.code.application.dto;
+
+import lombok.Data;
+
+@Data
+public class UserModificationDto {
+
+  private String name;
+
+  private String surname;
+
+  private String address;
+
+  private String city;
+
+  private String email;
+}

src/main/java/com/example/code/application/mapper/UserDtoMapper.java

@@ -0,0 +1,10 @@
+package com.example.code.application.mapper;
+
+import com.example.code.application.dto.UserDto;
+import com.example.code.domain.model.User;
+import org.mapstruct.Mapper;
+
+@Mapper
+public interface UserDtoMapper {
+  UserDto toUserDto(User user);
+}

src/main/java/com/example/code/application/usecase/UserCreationUseCase.java

@@ -1,8 +1,9 @@
 package com.example.code.application.usecase;
 
-import com.example.code.domain.model.User;
+import com.example.code.application.dto.UserCreationDto;
+import com.example.code.application.dto.UserDto;
 
 public interface UserCreationUseCase {
 
-  User createUser(final User newUser);
+  UserDto createUser(final UserCreationDto userCreationDto);
 }

src/main/java/com/example/code/application/usecase/UserModificationUseCase.java

@@ -1,8 +1,8 @@
 package com.example.code.application.usecase;
 
-import com.example.code.domain.model.User;
+import com.example.code.application.dto.UserModificationDto;
 
 public interface UserModificationUseCase {
 
-  void modifyUser(final int userId, final User user);
+  void modifyUser(final int userId, final UserModificationDto userModificationDto);
 }

src/main/java/com/example/code/application/usecase/UserRetrieveUseCase.java

@@ -1,8 +1,8 @@
 package com.example.code.application.usecase;
 
-import com.example.code.domain.model.User;
+import com.example.code.application.dto.UserDto;
 
 public interface UserRetrieveUseCase {
 
-  User getUserById(final Integer id);
+  UserDto getUserById(final Integer id);
 }