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

[#38] 활동 추가 API 구현 #54

Merged
merged 1 commit into from
May 5, 2024
Merged

[#38] 활동 추가 API 구현 #54

merged 1 commit into from
May 5, 2024

Conversation

KkomSang
Copy link
Collaborator

@KkomSang KkomSang commented May 5, 2024

#️⃣ 연관된 이슈

Resolves #38

📝 작업 내용

이번 PR에서 작업한 내용을 간략히 설명해주세요 (이미지 첨부 가능)

활동 추가 API를 구현했습니다.

  • /api/activities?userId=45 로 요청하면 다음과 같은 응답을 받을 수 있습니다.
image
{
  "success": true,
  "code": 0,
  "message": "활동을 성공적으로 추가했습니다.",
  "data": {
    "activityId": 98
  }
}

스크린샷 (선택)

💬 리뷰 요구사항(선택)

리뷰어가 특별히 봐주었으면 하는 부분이 있다면 작성해주세요

ex) 메서드 XXX의 이름을 더 잘 짓고 싶은데 혹시 좋은 명칭이 있을까요?

image
@Id
    @GeneratedValue(strategy = GenerationType.IDENTITY)
    @Column(name = "activity_id")
    private Long activityId;

위 코드는 Activity 도메인의 일부입니다. @GeneratedValue으로 인해 추가되는 활동의 activity_id가 현재 테이블에서 가장 큰 값 + 1이 아닌, 데이터베이스가 자동으로 생성한 id 값 중에서 최대값을 찾아서 +1을 한 값으로 저장되고 있습니다. 이 부분을 수정해야 하는지 고민이 됩니다...... @GeneratedValue가 없도록 수정한다면 위의 예시가 activity_id 98번이 아닌 8번으로 저장되었을 겁니다!

✅ Check List

  • PR 제목을 커밋 규칙에 맞게 작성했는가?
  • PR에 해당되는 Issue를 연결했는가?
  • 적절한 라벨을 설정했는가?
  • 작업한 사람을 모두 Assign했는가?

@KkomSang KkomSang added the feat 새로운 기능을 추가합니다 label May 5, 2024
@KkomSang KkomSang self-assigned this May 5, 2024
@KkomSang KkomSang requested a review from yel-m May 5, 2024 12:42
@yel-m
Copy link
Member

yel-m commented May 5, 2024

@KkomSang
넘 수고하셨습니다!! 👏🎃💛

PK 설정 시 특별한 경우가 아니라면 @id@GeneratedValue는 필수로 알고 있습니다! activityId에 자동으로 ID를 생성해주는 @GeneratedValue가 없다면 수동으로 ID를 할당해야 하는 문제가 생깁니당 저도 여러 데이터 생성 후 삭제하면 그 다음 생성되는 데이터의 id가 갑자기 너무 커져서 당황했었던 적이 있는데 결국 id는 객체를 서로 구분하기 위한 용도라고 생각하면 될 것 같아요!
어차피 activity를 보내줄 때에는 for문으로 id를 1부터 특정 숫자까지 돌리는 게 아니라 JPARepository를 사용해 필요한 객체를 가져오는 형식이므로, 실제로 ID 값이 얼마나 커지는지에 대해 신경 쓸 필요가 없을 것 같습니다!

저도 이해하지 못한 부분이 있을 수 있어서 혹시 제가 틀린 부분이 있거나 다른 의견이 있다면 코멘트 달아주세욥!

@yel-m yel-m merged commit ff2fc10 into develop May 5, 2024
1 check passed
@yel-m yel-m deleted the feat/#38 branch May 5, 2024 15:29
@KkomSang
Copy link
Collaborator Author

KkomSang commented May 5, 2024

자세한 설명 너무 감사합니다....🥲😍 해피해피호박입니다...🎃💕🤓

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feat 새로운 기능을 추가합니다
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants