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: Reservation hasConflictWith method 로직 수정 #747

Merged
merged 4 commits into from
Nov 25, 2021

Conversation

sakjung
Copy link
Collaborator

@sakjung sakjung commented Nov 18, 2021

구현 기능

  • 기존 hasConflictWith 로직: 실제로 기존 예약과 시간이 겹치는 모든 케이스를 다 찾아내서 검증
  • 변경된 (현재) hasConflictWith 로직: 시간이 겹치지 않는 케이스들을 찾아서 그것의 역을 구하는 식으로 검증

시간이 겹치지 않는 경우에 대한 시각화

hasconflictwith

공유하고 싶은 내용

  • 시간이 겹치지 않는 경우가 케이스가 훨씬 깔끔하고 찾기 쉽다는 것을 발견했습니다. 그래서 리팩터링 진행해봤습니다
  • 기존의 검증로직은 복잡하기도 하고 LocalDateTime의 쓰레기 API의 한계로 어쩔 수 없이 중복해서 검증이 들어가는 부분이 발생했습니다.
  • 현재의 로직은 훨씬 간단하고 중복 로직도 존재하지 않아서 더 좋은 방법이라고 생각합니다

Close #738

Copy link
Collaborator

@xrabcde xrabcde left a comment

Choose a reason for hiding this comment

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

index.html 빼주세요 ㅋㅋㅋㅋ
코드라인 2700줄 추가돼서 깜짝 놀랐네

Copy link
Collaborator

@xrabcde xrabcde left a comment

Choose a reason for hiding this comment

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

ㅇㅍㄼ~~~

@sakjung
Copy link
Collaborator Author

sakjung commented Nov 25, 2021

ZZIMKKONG SONARQUBE

All passes! - Read the report here!

Copy link
Collaborator

@dusdn1702 dusdn1702 left a comment

Choose a reason for hiding this comment

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

굿 좋습니다~!

@xrabcde xrabcde merged commit 5028b8a into dev Nov 25, 2021
@xrabcde xrabcde deleted the feat/738-existing-reservation-validation branch November 25, 2021 14:59
This was referenced Apr 27, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[BE] Reservation에서 hasConflictWith 메서드 내부 로직 개선
4 participants