Controller에서 요청값이 같고 응답값이 조금 다른 요청을 처리하는 방법

Question

OptionDetail.java&#64;Entity &#64;Getter public class OptionDetail extends BaseTimeEntity { &#64;Id &#64;GeneratedValue(strategy &#61; GenerationType.IDENTITY) private Long optionDetailId; &#64;Column(nullable &#61; false) private String name; &#64;Column(nullable &#61; false) private Integer stockQuantity; &#64;Column(nullable &#61; false, precision &#61; 12, scale &#61; 2) private BigDecimal additionalPrice; &#64;Column private String photo; &#64;ManyToOne(fetch &#61; FetchType.LAZY) &#64;JoinColumn(name &#61; &#34;option_id&#34;, nullable &#61; false) private Option option; }Option.java&#64;Entity &#64;Getter &#64;Table(name &#61; &#34;options&#34;) public class Option extends BaseTimeEntity { &#64;Id &#64;GeneratedValue(strategy &#61; GenerationType.IDENTITY) private Long optionsId; &#64;Column(nullable &#61; false) private String name; &#64;OneToMany(mappedBy &#61; &#34;option&#34;) private List<OptionDetail> optionDetails; &#64;ManyToOne(fetch &#61; FetchType.LAZY) &#64;JoinColumn(name &#61; &#34;product_id&#34;, nullable &#61; false) private Product product; } 이와 같이 Option 엔티티와 OptionDetail 엔티티는 일대다 관계입니다.외부로부터 전달받은 optionId 를 가지고 있는 OptionDetail 의 특정 필드들을 반환하는데 상황은 2가지 입니다.name, photo name이러한 요청들을 어떻게 처리할지 아래 2가지 방법들을 두고 고민중입니다...Controller에 2개의 매핑 메서드를 둬서 별도로 처리Controller에 매핑 메서드 1개만 두고 요청값에 별도의 summarized 와 같은 boolean 필드를 두어 판단둘 중 어떤 방법이 좋을까요??

カワウソ · Accepted Answer

응답값 자체가 달라진다면 합치는 것보단 나누는게 나중에(?) 문서화 하실때 편하실 겁니다.스웨거 문서로 자동화할 가능성이 높은데 그렇게 유연하지 않습니다.

kenu,전독시 · Answer

둘 다 동작한다면 별 차이 없을 것 같습니다.

집킨 · Answer

entity를 컨트롤러에서 DTO 역할로 쓰는 것 자체가 일단 잘못된 설계입니다별도 DTO를 만들면 OptionBase 같은 클래스를 만들고 상속으로 처리하면 아주 간단해지는 문제죠

디엔디 · Answer

하나의 함수는 한가지 일만 한다는 원칙으로 봤을 때boolean 을 받고 if 문으로 구분하는 것 보다는 함수를 나누는게 좋다고 생각합니다.즉 매핑을 나누는게 좋은 방식으로 보입니다.

wharup · Answer

REST API는 대상 리소스가 무엇인지가 중요합니다.말씀하신 케이스는 조회하는 대상이 다르기 때문에 REST API를 분리하고각 클래스를 반환하는 Contoller 메소드를 만드시면 됩니다.[GET] /api/options -> OptionController.getOptions[GET] /api/option-details -> OptionController.getOptionDetailss

kmw1218 · Answer

다들 자세한 답변 너무 감사드립니다...! 문서화, SRP 등을 고려했을 때 함수를 분리하는게 맞는 것 같네요 ㅎㅎ

Controller에서 요청값이 같고 응답값이 조금 다른 요청을 처리하는 방법

6개의 답변

Q&A 목록