-
Notifications
You must be signed in to change notification settings - Fork 2
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(#59) : 약 복용률 계산하는 메서드 추가, #61
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
수고했으~!
src/main/resources/application.yml
Outdated
@@ -8,7 +8,7 @@ spring: | |||
|
|||
jpa: | |||
hibernate: | |||
ddl-auto: update | |||
ddl-auto: create |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
이거 create말고 기존 update 계속 유지해야 할 것 같아
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
다른 commit에서 update로 다시 수정했구나 확인 완료!
int realCount = realBreakfastCount + realLunchCount + realDinnerCount; | ||
|
||
// 아침, 점심, 저녁에 먹어야하는 복용 횟수 | ||
int totalBreakfastCount = prescription.getBreakfastImportance() == 0 ? 0 : (prescription.getPeriod() + 1); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
비즈니스 로직이 만약 아침 약이 없으면 중요도 0으로 들어가고 아침 약이 존재하면 중요도가 무조건 >=1인거야?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
어어 맞아. 중요도를 입력하지 않으면 해당 약이 없는거!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
수고했으~!
📝 작업 내용
patient
테이블에 아침,점심,저녁,토탈 약 복용율 컬럼을 넣엇습니다.takingMedicine
이 추가될 떄 마다 업데이트 됩니다💬 ETC.
#️⃣ 연관된 이슈
resolves: #이슈번호, #이슈번호