위키 및 답변 조회 기능을 구현한다. #186
위키 및 답변 조회 기능을 구현한다. #186
Conversation
Test Results100 tests 99 ✅ 6s ⏱️ Results for commit d3c0821. ♻️ This comment has been updated with latest results. |
| @Column(nullable = false, length = MAX_NAME_LENGTH) | ||
| private String name; |
| @JsonInclude(Include.NON_NULL) List<CommentResponse> comments, | ||
| @JsonInclude(Include.NON_NULL) Long commentCount |
There was a problem hiding this comment.
@JsonInclude(Include.NON_NULL) 구문은 해당 값이 null인 경우, 필드 노출을 하지 않는다는 의미일까요?
그렇다면, 위키 단건 조회와 위키 목록 조회에서 사용되는 응답을 한개의 응답 DTO로 통일한 것으로 보여요. 작성된 API 문서에는 위키 목록 조회에서 question detail이 존재하지 않는데요. 이 부분도 노출하지 않고, null 처리하는 것은 어떨까요?
There was a problem hiding this comment.
@JsonInclude(Include.NON_NULL)구문은 해당 값이 null인 경우, 필드 노출을 하지 않는다는 의미일까요?
네 맞습니다
question detail에도 반영했습니다
| Page<WikiSummaryWithCommentCount> pageResults = wikiRepository.pageByCategory(category, pageable); | ||
| List<WikiResponse> wikiResponses = pageResults.getContent() | ||
| .stream() | ||
| .map(w -> WikiResponse.withCommentCount(resolveAnonymousWiki(w.wikiSummary()), w.commentCount())) |
There was a problem hiding this comment.
저는 표현식 인자는 한개인데 길면 it을 사용하는게 더욱 잘 읽히는 것 같아요. 이 부분은 사소해서 편하게 반영 여부 결정해주셔도 됩니다.
|
@Transactional로 격리하던 것을 extension으로 변경