search

nus-cs2030 / 2021-s1

27 stars 48 forks source link

Project level 5 #519

Open shawnkai opened 4 years ago

shawnkai commented 4 years ago

Description

hi, i'm slightly clueless about why the customer number is mixed up like that for this test case, if anyone has any insights please do let me know! thanks in advance!

Topic:

Specific Topics this question is related to (If Any)

Screenshots (if any):

image

zechajw commented 4 years ago

There seems to be 2 possible issues.

1) The instantiation of your customer variable might have an issue, resulting in 25 being output as 3 during test cases. For this you need to check your Customer.java to see how you are storing the integer for customeridentifier

2) It might not be an issue of the customer number being mixed up, but has to do directly with the outputting of events, which means that your events are not being polled in the correct order in the priority queue.

SkyBlaise99 commented 4 years ago

You can try by adding if (customer.id == 3) system.out.println(...); and see if anything went wrong.

shawnkai commented 4 years ago

Thanks for the help guys! Turns out its because I did not handle the queue properly when server comes back from rest XD