You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
JsonAdaptedPerson class is currently bloated with attributes from all types of persons. As a result, it represents, for example, a doctor by setting all its non-doctor related attributes such as age, bloodType, etc to empty strings.
Proposed refactoring is to have JsonAdaptedPerson be an abstract class with JsonAdaptedDoctor/Patient/etc extending it. Naturally, this issue should be fixed after #102 .
Issue up for discussion.
The text was updated successfully, but these errors were encountered:
JsonAdaptedPerson class is currently bloated with attributes from all types of persons. As a result, it represents, for example, a
doctor
by setting all its non-doctor related attributes such asage
,bloodType
, etc to empty strings.Proposed refactoring is to have JsonAdaptedPerson be an abstract class with JsonAdaptedDoctor/Patient/etc extending it. Naturally, this issue should be fixed after #102 .
Issue up for discussion.
The text was updated successfully, but these errors were encountered: