Skip to content

[7주차] 워크북 미션 -hoddak#18

Open
ghdudx2 wants to merge 2 commits into
mainfrom
feat/week7-hoddak2
Open

[7주차] 워크북 미션 -hoddak#18
ghdudx2 wants to merge 2 commits into
mainfrom
feat/week7-hoddak2

Conversation

@ghdudx2

@ghdudx2 ghdudx2 commented May 14, 2026

Copy link
Copy Markdown
Collaborator

No description provided.

@ghdudx2 ghdudx2 changed the title [7주차 [7주차] 워크북 미션 -hoddak May 14, 2026
@ghdudx2

ghdudx2 commented May 15, 2026

Copy link
Copy Markdown
Collaborator Author

뭔가 깃허브 푸쉬 과정에서 꼬인 것 같은데, 어디서부터 꼬였는지 모르겠습니다..

@drghdtjr

Copy link
Copy Markdown
Contributor

뭔가 깃허브 푸쉬 과정에서 꼬인 것 같은데, 어디서부터 꼬였는지 모르겠습니다..

아마 루트 디렉토리에서 출발해서 그런것 같습니다. 본인 하위 디렉토리에서 해보시면 될 수도있습니다.

@drghdtjr

Copy link
Copy Markdown
Contributor

네, 전체적으로 코드 확인했습니다.

Cursor Paging 구현 시 countQuery 분리 및 별점 정렬 상황에서 (score = cursorScore AND id < cursorId) 형태의 tie-break 조건을 사용하신 부분은 성능과 정렬 안정성을 함께 고려한 좋은 구현이라고 생각합니다.

GET 요청에 RequestBody 를 사용하는 부분은 HTTP Method 특성상 한 번 더 검토해보시면 좋을 것 같습니다. 현재 단계에서는 RequestParam 또는 PathVariable 기반으로 사용자 식별값을 전달하는 방향이 더 안전해 보입니다.
또한 Service 계층의 트랜잭션 정책(@transactional(readOnly = true) 활용), 사용되지 않는 DTO 정리 등의 부분도 함께 다듬으면 코드 완성도가 더 높아질 것 같습니다.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants