itsmejr257 / pe

0 stars 0 forks source link

Missing alt frame for play() method in StockStorage #14

Open itsmejr257 opened 4 months ago

itsmejr257 commented 4 months ago

Within the play() method, there appears to be two different instances which can happen, when the stocks available is empty and when it is not.

In the case where the stocks available is empty, an extra method appears to be ran called setUp(). Would it be better to include a alt frame to handle the two difference instances

Here is the diagram i am referring to :

image.png

Here is the code snippet from StockStorage i am referring to :

image.png

nus-pe-script commented 4 months ago

Team's Response

Screenshot 2024-04-21 at 2.15.29 PM.png

The purpose of this sequence diagram is to show to the command is being invoked and what class is involved in this invocation. Adding extra information to the play method will not be completely necessary.

Items for the Tester to Verify

:question: Issue severity

Team chose [severity.VeryLow] Originally [severity.Medium]

Reason for disagreement: [replace this with your explanation]