nus-cs2103-AY2021S2 / forum

10 stars 0 forks source link

Clarification for Class diagram - Person, Inbox, Message #346

Closed tlylt closed 3 years ago

tlylt commented 3 years ago

For this exercise on class diagram, is it ok to

damithc commented 3 years ago

In general, it is safer not to put something into the diagram unless there is clear evidence of its existence.

  • state multiplicity of Message as * (from Inbox)

Should be OK. Although the description doesn't mention this specifically, we can't expect it to be anything else.

  • state multiplicity of Inbox as 1 (from Person)

I would be vary of using 1 unless there is a strong indication to support that it has to be exactly one.

  • make Message and its read method abstract?

There is no indication in the description that they are. Of course if there is code, we can check and choose accordingly.

In all three cases above, either way should not affect exam marks though.

tlylt commented 3 years ago

Roger, thanks prof.