Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Implicit requirement on class and variable names in lab04 #33

Open
tygq13 opened this issue Feb 18, 2022 · 12 comments
Open

Implicit requirement on class and variable names in lab04 #33

tygq13 opened this issue Feb 18, 2022 · 12 comments
Labels
💻 Lab 04 Queries with regards to Lab 04

Comments

@tygq13
Copy link

tygq13 commented Feb 18, 2022

Summary

Lab04 require "BookingStub" class but this is not mentioned in task.txt.

Description:

I fail at this question where it has implicit requirement on this class "BookingStub" and variable "list", but this requirement is not mentioned in the task.txt. I have my own implementation that can pass the sample test cases but failed at codecrunch. My worries is that in the PE where we only have one time to submit the code, how can we know the class names and variable names required by the grader which is not explicitly mentioned in task.txt?

image

Code Snippets

Attach code snippets (if any)

Screenshots

Screenshots (if any)

@tygq13 tygq13 added the 💻 Lab 04 Queries with regards to Lab 04 label Feb 18, 2022
@tygq13 tygq13 changed the title Implicit requirement on class and variable by grader in lab04 Implicit requirement on class and variable names in lab04 Feb 18, 2022
@jingyulim1
Copy link

Isn't this due to runtime error? I am facing the same issue and it shows runtime error at the top.

image

@alantay11
Copy link

I'm not sure why it's happening, but I think it's trying to go through a bunch of test cases in a list and for some reason the list can't be loaded. I didn't have to implement class BookingStub so I don't think it's required.

@szeying02
Copy link

I didn't implement the class BookingStub and variable list. I think those are part of the private sample cases within code crunch that tests for edge cases. It's possible that they failed to run as a result of runtime error.

@feederalert
Copy link

image

I'm facing a similar issue for test case 3. Not sure why a sort method for class BookingStub has to be implemented?

@kayyenl
Copy link

kayyenl commented Feb 18, 2022

Gave my TA a ping to notify on this issue. Was also affected by these testcases after 2/5. Likely that the past iterations of this Lab was done using class BookingStub instead of Booking?

@wayneonn
Copy link

image

I'm facing a similar issue for test case 3. Not sure why a sort method for class BookingStub has to be implemented?

Currently facing the same issue too, wondering why there has to be a sort method.

@inas0ng
Copy link

inas0ng commented Feb 19, 2022

hi! I might be entirely wrong, but I think that BookingStub may be extending on your Booking.java class and is attempting to sort certain bookings to check your code. Not sure if this is why the error occurs, but maybe you can check on whether you have implemented Comparable and coded the compareTo method correctly?

@alantay11
Copy link

Is the class/interface that NormalCab is a child of named Driver? If not you can try renaming it.

@jingyulim1
Copy link

Is the class/interface that NormalCab is a child of named Driver? If not you can try renaming it.

This worked for me, seems like we have to use specific Class names in the PE exam as well (I named it CabDriver instead of Driver originally)

@weihan10
Copy link

Is the class/interface that NormalCab is a child of named Driver? If not you can try renaming it.

This works for me as well. Probably from past iterations, the cabs must inherit/implement the Driver class/interface and then there is probably some modification from the past that makes us don't know that we have to use that specific naming.

Also another one is to rename your jsh from level 5 to level5.jsh.

This worked for me, seems like we have to use specific Class names in the PE exam as well (I named it CabDriver instead of Driver originally)

As for this, I think it's not always the case, feels like a clarity issue to me 😕

@wayneonn
Copy link

image

I'm facing a similar issue for test case 3. Not sure why a sort method for class BookingStub has to be implemented?

The test case worked without submitting the Comparator interface. It is conflicting with Codecrunch's own. Hope it works for you

@feederalert
Copy link

feederalert commented Feb 19, 2022

image
I'm facing a similar issue for test case 3. Not sure why a sort method for class BookingStub has to be implemented?

The test case worked without submitting the Comparator interface. It is conflicting with Codecrunch's own. Hope it works for you

Thank you so much! It worked after I dropped my Comparable interface from the submission files. I misunderstood the text to mean that I had to create my own Comparable interface.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
💻 Lab 04 Queries with regards to Lab 04
Projects
None yet
Development

No branches or pull requests

9 participants