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.

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

feat : 나의 풀이 목록 조회 API 추가 #156

Merged
merged 3 commits into from
Nov 10, 2024
Merged

feat : 나의 풀이 목록 조회 API 추가 #156

merged 3 commits into from
Nov 10, 2024

Conversation

rladmstn
Copy link
Contributor

@rladmstn rladmstn commented Nov 9, 2024

📌 Related Issue

close #155

🚀 Description

  • 나의 풀이 목록을 조회하는 API를 추가했습니다.
  • service 계층은 추가된 사항 없습니다. 기존에 풀이 목록 조회하던 service 메소드에 request로 닉네임만 현재 유저로 박아넣으면 재활용이 가능합니다.
  • controller단에서 내 풀이 목록 조회 API 추가와 함께 다른 API들 엔드포인트들을 좀 수정했습니다. (/api/solution/my-solutions은 좀 에바같아서)
  • 곧 API들 엔드포인트 대거 수정이 발생할 수 있을 것 같아서 지금은 임의로 이렇게 만들었고, 추후에 한 번에 제대로 수정하지 않을까 싶습니다.

📢 Review Point

📚Etc (선택)

  • 좋은 API 엔드포인트 설계 예시 중 하나가 이런게 있더라구요.
DB 상에서 book이라는 테이블을 사용한다고 가정해보자.
- 특정 책 하나를 가져오는 요청은 여러 책 중에서 하나를 가져오는 것 이므로, GET /books:id
- 책 전체를 가져오는 요청은 여러 책들의 리스트를 가져오는 것 이므로, GET /books
- 책 한권을 등록하는 요청은 여러 책 안에 한 권의 책을 임의의 ID를 부여하면서 추가하는 것 이므로, POST /books
- 특정 책의 내용을 변경하는 요청은 여러 책 안에 있는 한 권의 책을 변경하는 것 이므로, PUT /books:id
  • 좋은 예시같긴 한데 stackoverflow 보니 모두가 동의하는 사항은 아닌가봅니다. 하지만 단수형이든 복수형이든 하나로 통일해서 사용 해야 한다는게 공통 의견인 것 같아요.
  • 추후에 엔드포인트 설계를 하게 된다면 같이 고민해 볼 사항이 될 것 같습니다. 엔드포인트가 제일 어렵네요 :(

@rladmstn rladmstn added the new-feature 기능 추가 label Nov 9, 2024
@rladmstn rladmstn self-assigned this Nov 9, 2024
Copy link
Contributor

@s-hwan s-hwan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

API 설계 예시 좋은것 같아요 !
GET/solutions랑 Post/solutions 는 같이 사용할 수 있는거군요
요청이 달라서 가능한건가요??

Copy link
Contributor

@sh0723 sh0723 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

고생하셨습니다~!!

@sh0723 sh0723 merged commit f9ce138 into develop Nov 10, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
new-feature 기능 추가
Projects
None yet
Development

Successfully merging this pull request may close these issues.

feat : 나의 풀이 목록 조회 API
3 participants