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

[CS2113-W13-1] DoctorDuke #54

Open
wants to merge 390 commits into
base: master
Choose a base branch
from

Conversation

Jeraldchen
Copy link

Doctor Duke is a desktop app for diagnosing patients who have mild illnesses, optimized for use via a Command Line Interface (CLI). If you can type fast, Doctor Duke can diagnose illnesses and dispense medicine accordingly faster than visiting a General Practitioner.

Jeraldchen and others added 30 commits March 15, 2023 19:57
Added documentation to important methods and classes involving diagnosis.
Changed author comments to Javadoc @author tags.
Remove commented test code for diagnosis again.
Removed some redundancies.
Added an assertion in Diagnosis.java to prevent a divide by zero in the event the programmer created an Illness child with no symptoms.
Added logging to signal when diagnosis was ran, and what is the comparison value to each of the possible illnesses.
Fixed a typo on authorship @author tag.
Enable assertion in build.gradle
Prevent duplicate passwords and hash passwords
Added more medications for illnesses.
Allow user to see medicine prescription history.
Added 13 more options for the user to input when entering their symptoms.
Added a new function addSymptom to add users' symptoms to the symptoms ArrayList.
Added error handling in Patient.java in the event no medication is available for the illness.
Removed boilerplate lines in case statement when user inputs their symptoms and moved them to addSymptom
Fixed Insomnia.java to display the correct symptom.
Fixed order of @author tags in Javadoc to pass Gradlew checks.
Jeraldchen and others added 30 commits April 4, 2023 17:33
Changed text displaying symptoms in the UserGuide.md after it has been changed to lowercase.

Added note under login in UserGuide.md to explain why having multiple passwords for the same username to access different accounts is intentional.

Added note under "reporting symptoms" in UserGuide.md in the case that the patient's symptoms does not yield any suitable diagnosis.

Added separating line in Information.java when viewing symptom history in viewSymptomHistory().

Tried to fix image not showing in brennanzuz.md.
Fixed outdated output when displaying symptoms in UserGuide.md
Tried to fix image not showing in brennanzuz.md.
Tried to fix image not showing in brennanzuz.md.
Update project status on PPP
Formatted PPP
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.