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(#59) : 약 복용률 계산하는 메서드 추가, #61

Merged
merged 6 commits into from
May 20, 2024
Merged

Conversation

j2noo
Copy link
Contributor

@j2noo j2noo commented May 19, 2024

📝 작업 내용

  • patient 테이블에 아침,점심,저녁,토탈 약 복용율 컬럼을 넣엇습니다.
  • 약 복용 정보인 takingMedicine이 추가될 떄 마다 업데이트 됩니다

💬 ETC.

  • 기타 공유사항에 대해 작성해 주세요.

#️⃣ 연관된 이슈

resolves: #이슈번호, #이슈번호

Copy link
Contributor

@lsm-del lsm-del left a comment

Choose a reason for hiding this comment

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

수고했으~!

@@ -8,7 +8,7 @@ spring:

jpa:
hibernate:
ddl-auto: update
ddl-auto: create
Copy link
Contributor

Choose a reason for hiding this comment

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

이거 create말고 기존 update 계속 유지해야 할 것 같아

Copy link
Contributor

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);
Copy link
Contributor

Choose a reason for hiding this comment

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

비즈니스 로직이 만약 아침 약이 없으면 중요도 0으로 들어가고 아침 약이 존재하면 중요도가 무조건 >=1인거야?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

어어 맞아. 중요도를 입력하지 않으면 해당 약이 없는거!

Copy link
Contributor

@lsm-del lsm-del left a comment

Choose a reason for hiding this comment

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

수고했으~!

@lsm-del lsm-del merged commit fd51411 into master May 20, 2024
1 check passed
@j2noo j2noo self-assigned this May 20, 2024
@lsm-del lsm-del deleted the feat/59 branch May 27, 2024 01:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants