Remove low level details in activity diagram

AY2122S1-CS2103-W14-1 · Nov 7, 2021 · 9f738c3 · 9f738c3
1 parent 36bfcf5
commit 9f738c3
Show file tree

Hide file tree

Showing 5 changed files with 12 additions and 25 deletions.
diff --git a/docs/diagrams/AddMedicalEntryFeatureActivityDiagram.png b/docs/diagrams/AddMedicalEntryFeatureActivityDiagram.png
diff --git a/docs/diagrams/AddMedicalEntryFeatureActivityDiagram.puml b/docs/diagrams/AddMedicalEntryFeatureActivityDiagram.puml
@@ -2,18 +2,14 @@
 
 start
 :User inputs Add Medical Feature command;
-if (patientIndex && medicalPrefix is present?) then (no)
-: throw ParseException;
-else (yes)
-
-if (patientIndex is within Patient Book) then (yes)
-: Add medical entry into specified patient's medical history;
-: Produce patient copy with modified medical history;
+if () then ([user input is valid])
+: Find specified target patient from Patient Book;
+: Produce patient copy of target patient;
+: Add medical entry into patient copy's medical history;
 : Replace target patient in Patient Book with the patient copy;
 : Display updated Patient Book;
-else (no)
-: throw CommandException;
-endif
+else ([else])
+: Display Error Message;
 endif
 stop
 

diff --git a/docs/diagrams/AddMedicalEntryFeatureSequenceDiagram.puml b/docs/diagrams/AddMedicalEntryFeatureSequenceDiagram.puml
@@ -42,7 +42,6 @@ PatientBookParser --> AddressBookParser : cmd
 deactivate PatientBookParser
 AddressBookParser --> LogicManager : cmd
 deactivate AddressBookParser
-
 LogicManager --> AddMedicalEntryCommand : execute(model)
 activate AddMedicalEntryCommand
 

diff --git a/docs/diagrams/DeleteMedicalEntryFeatureActivityDiagram.png b/docs/diagrams/DeleteMedicalEntryFeatureActivityDiagram.png
diff --git a/docs/diagrams/DeleteMedicalEntryFeatureActivityDiagram.puml b/docs/diagrams/DeleteMedicalEntryFeatureActivityDiagram.puml
@@ -2,22 +2,14 @@
 
 start
 :User inputs Delete Medical Feature command;
-if (patientIndex && medicalIndex is present?) then (no)
-: throw ParseException;
-else (yes)
-if (patientIndex is within Patient Book) then (no)
-: throw CommandException;
-else (yes)
-: Find patient from Patient Book via the patientIndex;
-: Produce patient copy;
-if (medicalIndex is within the length of patient's medical history?) then (yes)
-: Modify patient copy medical history to delete the medicalIndex;
+if () then ([user input is valid])
+: Find specified target patient from Patient Book;
+: Produce patient copy of target patient;
+: Delete the specified medical entry from the patient copy's medical history;
 : Replace target patient in Patient Book with the patient copy;
 : Display updated Patient Book;
-else (no)
-: throw CommandException;
-endif
-endif
+else ([else])
+: Display Error Message;
 endif
 stop