Skip to content

Commit

Permalink
CLAP-310 Refactor: response 매핑 application 레이어로 옮기기
Browse files Browse the repository at this point in the history
<footer>
- 관련: #380
  • Loading branch information
joowojr committed Feb 6, 2025
1 parent a86a574 commit 20abbdc
Show file tree
Hide file tree
Showing 3 changed files with 11 additions and 11 deletions.
Original file line number Diff line number Diff line change
@@ -1,12 +1,12 @@
package clap.server.adapter.inbound.web.admin;

import clap.server.adapter.inbound.web.dto.admin.response.FindAllDepartmentsResponse;
import clap.server.application.mapper.DepartmentResponseMapper;
import clap.server.application.port.inbound.admin.FindAllDepartmentsUsecase;
import clap.server.common.annotation.architecture.WebAdapter;
import io.swagger.v3.oas.annotations.Operation;
import io.swagger.v3.oas.annotations.tags.Tag;
import lombok.RequiredArgsConstructor;
import org.springframework.http.ResponseEntity;
import org.springframework.security.access.annotation.Secured;
import org.springframework.web.bind.annotation.GetMapping;
import org.springframework.web.bind.annotation.RequestMapping;
Expand All @@ -23,10 +23,7 @@ public class FindDepartmentController {
@Operation(summary = "부서 조회 API")
@Secured("ROLE_ADMIN")
@GetMapping("/departments")
public List<FindAllDepartmentsResponse> findAllDepartments() {
return findAllDepartmentsUsecase.findAllDepartments()
.stream()
.map(DepartmentResponseMapper::toFindAllDepartmentsResponse)
.toList();
public ResponseEntity<List<FindAllDepartmentsResponse>> findAllDepartments() {
return ResponseEntity.ok(findAllDepartmentsUsecase.findAllDepartments());
}
}
Original file line number Diff line number Diff line change
@@ -1,9 +1,9 @@
package clap.server.application.port.inbound.admin;

import clap.server.domain.model.member.Department;
import clap.server.adapter.inbound.web.dto.admin.response.FindAllDepartmentsResponse;

import java.util.List;

public interface FindAllDepartmentsUsecase {
List<Department> findAllDepartments();
List<FindAllDepartmentsResponse> findAllDepartments();
}
Original file line number Diff line number Diff line change
@@ -1,9 +1,10 @@
package clap.server.application.service.admin;

import clap.server.adapter.inbound.web.dto.admin.response.FindAllDepartmentsResponse;
import clap.server.application.mapper.DepartmentResponseMapper;
import clap.server.application.port.inbound.admin.FindAllDepartmentsUsecase;
import clap.server.application.port.outbound.member.LoadDepartmentPort;
import clap.server.common.annotation.architecture.ApplicationService;
import clap.server.domain.model.member.Department;
import lombok.RequiredArgsConstructor;

import java.util.List;
Expand All @@ -14,8 +15,10 @@ public class FindAllDepartmentsService implements FindAllDepartmentsUsecase {
private final LoadDepartmentPort loadDepartmentPort;

@Override
public List<Department> findAllDepartments() {
return loadDepartmentPort.findActiveDepartments();
public List<FindAllDepartmentsResponse> findAllDepartments() {
return loadDepartmentPort.findActiveDepartments().stream()
.map(DepartmentResponseMapper::toFindAllDepartmentsResponse)
.toList();
}

}

0 comments on commit 20abbdc

Please sign in to comment.