1.2.1 #301

KWY0218 · 2024-08-05T13:08:36Z

변경된 회의 시간 추천 로직 반영

#289 [feat] 최적의 회의 시간 도출 클래스 틀 구현

given에서 List<TimeBlockDto>를 표현하기 위해서 수십줄이 필요했습니다. 이를 간소화하기 위해 테스트를 위한 코드인 List<TimeBlockDto>를 구현했습니다. 요일, 시작시간, 종료시간, 가중치, 유저 수를 넣으면 해당 값대로 List<TimeBlockDto>를 반환합니다.

각 요일들을 구분할 수 있는지 확인하는 테스트 코드

투 포인터 알고리즘을 활용했습니다. startIdx와 endIdx를 생성하여 연속된 값이면 continue를 했습니다. date가 다르거나 연속된 값이 아니라면 startIdx부터 endIdx까지 차를 구하고, 회의 진행 시간 보다 큰지 비교한다. 크다면 해당 값을 BestMeetingVo로 만들어 추가한다. 반복문이 끝난 후, 마지막 연속된 값이 회의 진행 시간보다 큰지 구분하여 추가한다

총 회의 시간의 가중치의 합을 구하는 로직, 회의 진행 시간을 만족하는지 구분하는 로직 추출

time block이 길수록 가중치가 더 커지기 때문에 평균값으로 수정

코드 중복 제거를 위해 ParameterizedTest 도입 displayname 명 수정

사용자가 16:00 ~ 16:30분을 체크한 경우 TIME_SLOT_16_00만 들어오는 것을 고려하지 못한 것에 대한 수정입니다.

#291 [feat] ContinuousMeetingTimeStrategy 구현체 코드 구현

회의 진행 시간 별 긴 블록을 구분하는 로직을 계산하기 위해 매개변수에 duration을 추가했습니다

각 회의 진행 시간마다 최적의 회의 시간을 1개 반환하는 테스트 케이스 최적의 회의 시간을 2개 반환하는 테스트 케이스

7명이 진행할 경우 테스트 케이스 추가

#293 [feat] BestMeetingTimeStrategy 구현체 로직 개발

#297 [feat] 신규 추천 로직 적용

#299 [feat] 이미 추천한 회의 시간대를 이후 조합에서 추천하지 않는 로직 추가

sohyundoh

굿. 고생하셨습니다!

KWY0218 and others added 30 commits July 31, 2024 14:57

#289 [feat] 최적의 회의 시간 도출에 필요한 인터페이스 및 구현체 생성

3aa67a5

#289 [feat] 최적의 회의 시간 도출 알고리즘 틀 구현

4186d92

#289 [feat] possible time case vo record로 수정

8e00b5e

#289 [chore] 클래스 명 수정

fb00bce

Merge pull request #290 from ASAP-as-soon-as-possible/feat/#289

afc533c

#289 [feat] 최적의 회의 시간 도출 클래스 틀 구현

#291 [test] BestMeetingTimeVo 하나를 반환하는 테스트 코드 작성

93735fc

#291 [test] 회의 진행 시간 별 결과를 반환하는 테스트 코드 작성

fbef13b

#291 [chore] 매개변수 명 수정

0b03045

#291 [test] 30분 밖에 없는 상황 테스트 코드 작성

47e5e47

#291 [test] 테스트 코드 추가

0714d27

각 요일들을 구분할 수 있는지 확인하는 테스트 코드

#291 [refactor] 일부 로직 함수로 추출

4a3a6f1

총 회의 시간의 가중치의 합을 구하는 로직, 회의 진행 시간을 만족하는지 구분하는 로직 추출

#291 [refactor] 가독성을 위해 if 문 내부 로직 함수로 추출

82593fe

#291 [chore] 함수 간 공백 추가

ffbdc9f

#291 [test] 가중치 테스트 추가

4cb9a78

#291 [feat] 모든 time block을 더해서 평균 값을 반환하도록 수정

7b9df2e

time block이 길수록 가중치가 더 커지기 때문에 평균값으로 수정

#291 [test] 가중치 기준으로 정렬 여부 확인 테스트 추가

fc4272d

#291 [feat] 반환하기 전 가중치 기준 내림차순으로 정렬

b52e236

#291 [refactor] 공통된 로직 함수로 추출

95232ed

#291 [refactor] 테스트 코드 리팩토링

ad2cfd8

코드 중복 제거를 위해 ParameterizedTest 도입 displayname 명 수정

#291 [feat] given 수정

e420021

사용자가 16:00 ~ 16:30분을 체크한 경우 TIME_SLOT_16_00만 들어오는 것을 고려하지 못한 것에 대한 수정입니다.

#291 [fix] 요구사항 반영에 대한 수정

991f441

#291 [test] 양 끝단의 time slot이 있는 경우 테스트 코드 작성

e964737

#291 [chore] time slot enum class import 수정

afc66f3

#291 [feat] time slot enum 내 index 속성 추가

c5a3893

#291 [fix] ordinal에서 index로 수정

e0bffd9

Merge pull request #292 from ASAP-as-soon-as-possible/feat/#291

87b804d

#291 [feat] ContinuousMeetingTimeStrategy 구현체 코드 구현

#293 [feat] duration 매개변수 추가

a2f24cd

회의 진행 시간 별 긴 블록을 구분하는 로직을 계산하기 위해 매개변수에 duration을 추가했습니다

#293 [test] 회의 진행 시간마다 테스트 코드 추가

204c3eb

각 회의 진행 시간마다 최적의 회의 시간을 1개 반환하는 테스트 케이스 최적의 회의 시간을 2개 반환하는 테스트 케이스

sohyundoh and others added 25 commits August 3, 2024 20:22

#295 [feat] MeetingTimeCasesStratagy 구현

338f730

#295 [test] 10명, 2명, 1명의 경우 테스트 코드 작성

7170f8e

#295 [test] 테스트 케이스 추가

1309dd7

7명이 진행할 경우 테스트 케이스 추가

#295 [feat] 함수 분리

0c5806a

#295 [test] 테스트 재작성

e1004aa

#295 [chore] 함수명 변경

b683ef8

#295 [test] 테스트 코드 클래스화 변경

24eda6c

#295 [feat] 코드리뷰 반영

db53098

#295 [test] 테스트 코드 수정

2857b70

#295 [test] 테스트 코드 의존성 수정

98906db

#295 [refactor] 미사용 import 문 제거

84dddd2

Merge pull request #294 from ASAP-as-soon-as-possible/feat/#293

eecb6af

#293 [feat] BestMeetingTimeStrategy 구현체 로직 개발

#295 [feat] while문 조건문 분리

03e980d

Merge pull request #296 from ASAP-as-soon-as-possible/feat/#295

554c59d

#297 [chore] 불필요한 파일 제거

48b7cde

#297 [feat] 새로운 추천 로직 빈 등록

3bb43ae

#297 [test] 신규 추천 로직에 기존 테스트 코드 적용

ca83b13

Merge pull request #298 from ASAP-as-soon-as-possible/feat/#297

46be6b5

#297 [feat] 신규 추천 로직 적용

#299 [test] 추가된 기능 요구사항 테스트 코드 작성

ef1bea6

#299 [test] 변경된 요구사항에 따른 조건 수정

849d35a

#299 [feat] 추천한 회의 로직 timeblocks 내에 제거하는 로직 추가

2bbe356

#299 [docs] read me 수정

de8ad7c

#299 [fix] 불변성을 보장하기 위해 toList()로 수정

ae086b0

Merge pull request #300 from ASAP-as-soon-as-possible/feat/#299

870ad87

#299 [feat] 이미 추천한 회의 시간대를 이후 조합에서 추천하지 않는 로직 추가

[chore] 배포 전 버전 수정

fe6ff0f

KWY0218 requested a review from sohyundoh August 5, 2024 13:08

KWY0218 assigned sohyundoh and KWY0218 Aug 5, 2024

sohyundoh approved these changes Aug 5, 2024

View reviewed changes

KWY0218 merged commit 623725e into main Aug 5, 2024
2 checks passed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

1.2.1 #301

1.2.1 #301

KWY0218 commented Aug 5, 2024

sohyundoh left a comment

1.2.1 #301

1.2.1 #301

Conversation

KWY0218 commented Aug 5, 2024

sohyundoh left a comment

Choose a reason for hiding this comment