Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

로드맵 서비스 로직 개선 #1609

Open
nuyh99 opened this issue Oct 25, 2023 · 0 comments
Open

로드맵 서비스 로직 개선 #1609

nuyh99 opened this issue Oct 25, 2023 · 0 comments
Assignees
@nuyh99
Labels
BE 로드맵 로드맵 스쿼드 관련 이슈

Comments

@nuyh99
Copy link
Contributor

nuyh99 commented Oct 25, 2023

요구 사항

  • 커리큘럼에 해당하는 모든 키워드를 가져와야 합니다.
  • 각 키워드별 퀴즈의 개수가 필요합니다.
  • 각 퀴즈별 현재 유저가 몇 개를 완료했는 지 필요합니다.

문제 상황

지금은 5개의 Repository를 가지고 한 메서드에서 모두 사용하고 컬렉션 API로 조립하는 중입니다.
현재 로직의 흐름을 매우 읽기 힘듭니다.

개선 방향

SQL로 미리 가공해서 가져오는 방식으로 리팩토링할 예정입니다.
@nuyh99 nuyh99 added BE 로드맵 로드맵 스쿼드 관련 이슈 labels Oct 25, 2023
@nuyh99 nuyh99 self-assigned this Oct 25, 2023
nuyh99 added a commit that referenced this issue Nov 1, 2023
@nuyh99
feat: 유저 ID로 답변 개수 조회 기능 구현
dc4fae9 
#1609
nuyh99 added a commit that referenced this issue Nov 1, 2023
@nuyh99
test: 유저 ID로 답변 개수 조회 기능 테스트
11e87dc 
#1609
nuyh99 added a commit that referenced this issue Nov 1, 2023
@nuyh99
test: 커리큘럼의 모든 키워드 조회 테스트
bc41ccb 
#1609
nuyh99 added a commit that referenced this issue Nov 1, 2023
@nuyh99
refactor: 커리큘럼 ID의 모든 키워드 가져오는 로직 JPQL 사용해서 리팩토링
5faa0fb 
#1609
nuyh99 added a commit that referenced this issue Nov 1, 2023
@nuyh99
refactor: Dto -> Map 변환 로직 Stream API 사용
746ee8c 
#1609
nuyh99 added a commit that referenced this issue Nov 1, 2023
@nuyh99
refactor: 키워드 조회 시 추천 포스트 fetch join -> batch size 사용
4eac034 
#1609
@nuyh99 nuyh99 mentioned this issue Nov 1, 2023
Merged
nuyh99 added a commit that referenced this issue Nov 1, 2023
@nuyh99
style: doneQuizCount -> answeredQuizCount로 용어 변경
c730607 
#1609
nuyh99 added a commit that referenced this issue Nov 22, 2023
@nuyh99
style: 정적 팩토리 메서드 네이밍 변경
d94a493 
#1609
nuyh99 added a commit that referenced this issue Nov 22, 2023
@nuyh99
refactor: 진도율 setter 제거 후 dto 내부에서 매핑하도록 변경
e9cc339 
#1609
@nuyh99 nuyh99 mentioned this issue Nov 22, 2023
Closed
nuyh99 added a commit that referenced this issue Nov 23, 2023
@nuyh99
feat: 유저 ID로 답변 개수 조회 기능 구현
3c10bee 
#1609
nuyh99 added a commit that referenced this issue Nov 23, 2023
@nuyh99
test: 유저 ID로 답변 개수 조회 기능 테스트
9223670 
#1609
nuyh99 added a commit that referenced this issue Nov 23, 2023
@nuyh99
test: 커리큘럼의 모든 키워드 조회 테스트
d6cf99f 
#1609
nuyh99 added a commit that referenced this issue Nov 23, 2023
@nuyh99
refactor: 커리큘럼 ID의 모든 키워드 가져오는 로직 JPQL 사용해서 리팩토링
4e1ad8e 
#1609
nuyh99 added a commit that referenced this issue Nov 23, 2023
@nuyh99
refactor: Dto -> Map 변환 로직 Stream API 사용
0469d57 
#1609
nuyh99 added a commit that referenced this issue Nov 23, 2023
@nuyh99
refactor: 키워드 조회 시 추천 포스트 fetch join -> batch size 사용
5455a51 
#1609
nuyh99 added a commit that referenced this issue Nov 23, 2023
@nuyh99
style: doneQuizCount -> answeredQuizCount로 용어 변경
8a9faff 
#1609
nuyh99 added a commit that referenced this issue Nov 23, 2023
@nuyh99
style: 정적 팩토리 메서드 네이밍 변경
9a36df8 
#1609
nuyh99 added a commit that referenced this issue Nov 23, 2023
@nuyh99
refactor: 진도율 setter 제거 후 dto 내부에서 매핑하도록 변경
89ef1da 
#1609
@nuyh99 nuyh99 mentioned this issue Nov 23, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@nuyh99 nuyh99

Labels
BE 로드맵 로드맵 스쿼드 관련 이슈
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@nuyh99