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

[fix] : 가장 많이 되는 시간이 이어지지 않는 문제를 해결한다 #99

Merged
merged 2 commits into from
Oct 25, 2024

Conversation

bbbang105
Copy link
Collaborator

✅ PR 유형

어떤 변경 사항이 있었나요?

  • 새로운 기능 추가
  • 버그 수정
  • 코드에 영향을 주지 않는 변경사항(오타 수정, 탭 사이즈 변경, 변수명 변경)
  • 코드 리팩토링
  • 주석 추가 및 수정
  • 문서 수정
  • 빌드 부분 혹은 패키지 매니저 수정
  • 파일 혹은 폴더명 수정
  • 파일 혹은 폴더 삭제

📝 작업 내용

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

  • 가장 많이 되는 시간이 이어지지 않는 문제를 해결한다
  • record는 불변 객체인데, 기존 DTO class 처럼 내부 필드를 변경하려 한 것이 문제
  • 새로운 record 객체를 반환하고 -> 리스트 마지막 인덱스에 set하는 것으로 문제를 해결함

✏️ 관련 이슈

본인이 작업한 내용이 어떤 Issue Number와 관련이 있는지만 작성해주세요


🎸 기타 사항 or 추가 코멘트

  • record는 불변 객체임을 명심하자
  • 되도록이면 DTO 객체를 바꾸는 로직은 지양해야할 것 같다
  • 배포 전 테스트를 충분히 진행하자

@bbbang105 bbbang105 added 🚨 fix 버그 수정 / 에러 해결 😵‍💫 sangho 상호 PR labels Oct 25, 2024
@bbbang105 bbbang105 self-assigned this Oct 25, 2024
@bbbang105 bbbang105 changed the title [fix] : 가장 많이 되는 시간이 이어지지 않는 문제를 해결한다 (record는 불변 객체) [fix] : 가장 많이 되는 시간이 이어지지 않는 문제를 해결한다 Oct 25, 2024
@bbbang105 bbbang105 merged commit 0ac7d6a into develop Oct 25, 2024
1 check passed
@bbbang105 bbbang105 deleted the fix/#98/most branch October 25, 2024 04:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🚨 fix 버그 수정 / 에러 해결 😵‍💫 sangho 상호 PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[fix] : 가장 많이 되는 시간이 이어지지 않는 문제를 해결한다
1 participant