Witam,
Bardzo proszę o ocenę 2 diagramów UML
0
2
Wszytkie te include do kosza. Include oznacza ze wykonanie jednego przypadku wymaga wykonania innego. U ciebie na przykład każdorazowe logowanie wymaga zakładania konta.
Umówienie nowej wizyty wymaga chyba też Pacjenta ;)
Z opisu wynika że pacjent ma dostęp do historii leczenia jeśli ma konto, a u ciebie nie ma on dostępu do takiego przypadku użycia.
Mylisz diagram klas z diagramem ERD. Co to za "id" w tych klasach? o_O
Pacjent rozszerza Ekonto? A jak pacjent nie ma Ekonta to co? Jeśli juz to byłaby tam jakaś agregacja czy kompozycja.
0
Bardzo dziękuję za odpowiedź i wskazówki. Spróbowałem to poprawić i dodałem również 2 inne diagramy - sekwencji i stanu.
Nie jestem przekonany co do nowych wersji, a najbardziej do diagramu sekwencji.
Czy mógłbym prosić ponownie o ocenę?
Z góry dziękuję.
1
Lepiej, ale nie rozumiem po co administrator ma mieć jakieś relacje generalizacji. Admin nikogo nie będzie leczył i dostępu do historii medycznej mieć nie powinien.
W diagramach klas nie podobają mi sie "logowania" w klasach domenowych. No bo jak ty sobie to wyobrażasz potem w kodzie?
Lekarz lek = new Lekarz();
lek.logowanie();
?
To byłby lekko dziwny kod ;)
W diagramie stanów nie bardzo rozumiem czemu TRZEBA się umówić po założeniu konta i TRZEBA iść na wizytę. Nie da się założyć konta tak po prostu, żeby mieć na później? ;)
Diagram sekwencji jest zupełnie bez sensu. Nie ma co opisywać nawet bo nic tam nie jest poprawnie.
0
Bardzo dziękuję za kolejne wskazówki. Bardzo mi to pomogło.
Zastanawiam się tylko czy w diagramie klas to Pacjent przegląda terminarz/historię czy klasa Ekonto skoro dostęp do tego mają tylko pacjenci z kontem?
Jeśli chodzi o diagram sekwencji to zrobiłem coś takiego, ma to trochę więcej sensu?
