Feature/1474 키워드의 퀴즈 목록을 조회할 때 각 퀴즈 별로 학습 여부를 알 수 있다 #1509
Conversation
| this.question = question; | ||
| } | ||
|
|
||
| public static EssayAnswerQuizResponse of(Quiz quiz) { |
There was a problem hiding this comment.
파라미터 한개일 경우에 정팩매에서 from 으로 사용하는 것으로 알고 있습니다!
무조건은 아니지만 이렇게 많이 쓰더라구요
| import org.springframework.data.jpa.repository.JpaSpecificationExecutor; | ||
| import wooteco.prolog.roadmap.domain.EssayAnswer; | ||
|
|
||
| public interface EssayAnswerRepository extends JpaRepository<EssayAnswer, Long>, | ||
| JpaSpecificationExecutor<EssayAnswer> { | ||
|
|
||
| @Query("select ea from EssayAnswer ea where ea.quiz.id = :quizId and ea.member.id = :memberId ") |
There was a problem hiding this comment.
끝 부분이라 상관없을 수 있는데 SQL 문에서 띄어쓰기도 중요한 것 같아서 마지막 띄어쓰기는 삭제해주시는게 어떨까요?
- 하나의 인자를 가지고 있는 정적 팩토리 메서드의 이름을 from으로 변경 #1474
| } | ||
|
|
||
| private boolean isLearning(Long memberId, Long quizId) { | ||
| if (memberId == null) { |
There was a problem hiding this comment.
| if (memberId == null) { | |
| if (isNull(memberId)) { |
개인 취향이지만 Objects의 정적 메서드를 쓰면 예쁠 것 같아요!
| return essayAnswerRepository.existsByQuizIdAndMemberId(quizId, | ||
| memberId); |
| //given | ||
| final long findQuizId = 1L; | ||
| final String findQuizQuestion = "question"; | ||
| when(essayAnswerRepository.existsByQuizIdAndMemberId(anyLong(), anyLong())) |
There was a problem hiding this comment.
BDDMockito.given() 을 사용하는 것은 어떨까요??
| final List<QuizResponse> responses = quizzes.stream().map(QuizResponse::of) | ||
| .collect(Collectors.toList()); | ||
| return new QuizzesResponse(keywordId, responses); | ||
| public static QuizzesResponse of(Long keywordId, List<QuizResponse> quizzes) { |
There was a problem hiding this comment.
정적 팩터리 메서드에서 추가적인 작업이 없으면 new QuizzesResponse() 로 가도 좋지 않을까요??
|
Kudos, SonarCloud Quality Gate passed!
|
#️⃣연관된 이슈
#1474
📝작업 내용
퀴즈 응답 DTO에 학습 여부 정보를 Boolean 값으로 추가했습니다.
학습 여부는 퀴즈에 대해 답변 게시글을 작성한 경우 true가 됩니다.