Discussion of Past Year Paper

[ZhangAnli]

Hello! Here are my answers for this PYP. Please feel free to let me know what my mistakes are!

[tlylt]

Thanks for sharing!

• Inventory
  • should include a dependency arrow to Billable
  • I would use an arrow to indicate navigability from Inventory to Item
• StockItem
  • I would include the constructor
  • should include a dependency arrow to Rating
• Review
  • I would include the constructor
  • StockItem to Review, I used 0..1 instead as I feel that it might be possible if null is passed in, might need prof to confirm this
  • same as above for association to rating, I used 0..1
• Billable
  • If I'm not wrong, all methods are public within interface

PS: Just my 2 cents :)

[ong6]

To add to what @tlylt has mentioned,

I think it's possible for StockItem to have a composition to Review (since it seems unlikely that a review would exist without an item)

My attempt:

[tlylt]

To add to what @tlylt has mentioned,

I think it's possible for StockItem to have a composition to Review (since it seems unlikely that a review would exist without an item)

My attempt:

Btw your double arrow from Inventory to Item is wrong. If there is a stronger association indicated by the solid line, there is no need to include a dependency line anymore

[kouyk]

StockItem to Review, I used 0..1 instead as I feel that it might be possible if null is passed in

Even if null is passed in, a Review object will still be created, so it should be just 1.

[ong6]

Btw your double arrow from Inventory to Item is wrong. If there is a stronger association indicated by the solid line, there is no need to include a dependency line anymore

Oops forgot to erase it

[tlylt]

StockItem to Review, I used 0..1 instead as I feel that it might be possible if null is passed in

Even if null is passed in, a Review object will still be created, so it should be just 1.

@kouyk True, I think in this case StockItem to Review has to be 1. I think for Review to Rating, it could still be 0..1?Unless it does not matter if it is null or not

[ong6]

StockItem to Review, I used 0..1 instead as I feel that it might be possible if null is passed in

Even if null is passed in, a Review object will still be created, so it should be just 1.

@kouyk True, I think in this case StockItem to Review has to be 1. I think for Review to Rating, it could still be 0..1

Review to rating should also be 1, since the code indicates that there is a Rating inside review. I don't think null matters in this case. Maybe @damithc can clarify more?

[tlylt]

If Review class is simply like above, we all agree that it could be 0..1,
So either it means that if a variable is never explicitly initialized we can treat it as 0..1
or it means that if a variable could be assigned null, then we treat it as 0..1
That's my interpretation, will be good if prof clarifies

[damithc]

Review to rating should also be 1, since the code indicates that there is a Rating inside review. I don't think null matters in this case. Maybe @damithc can clarify more?

Wasn't this entire question discussed during tutorial 8? You may want to watch the week 8 tutorial recording. :-p

[tlylt]

Review to rating should also be 1, since the code indicates that there is a Rating inside review. I don't think null matters in this case. Maybe @damithc can clarify more?

Wasn't this entire question discussed during tutorial 8? You may want to watch the week 8 tutorial recording. :-p

Are tutorial recordings available for students to view? Doesn't seem to be able to find them.

[damithc]

Are tutorial recordings available for students to view? Doesn't seem to be able to find them.

In the MS Team for your tutorial group, there is a file called Recordings.docx that should contain all the tutorial recordings. If not found, ask the tutor to update it.

As the exam is tomorrow, here's the recording from my T8.

https://nus-sg.zoom.us/rec/share/Nk2-lh73_3zm5ij8kf5cipZwc1zpdWwrIVNz8L5Wki_I-Fq7MpV5P1KIjH4oqZ_H.h-TLRgoVjgwFsOSV

Access Passcode: cs2103-tutorial

[ong6]

According to the tut, 0..1 is fine for variables (40:00 mins) prof mentions this.

[prerthan99]

here's my attempt. ive fixed some of the 'bugs' but i think its pretty similar