SeededRandomizer interface #166

Closed
DanielBruegmann opened this Issue Mar 6, 2014 · 2 comments

Comments

Projects
None yet
2 participants
Contributor

DanielBruegmann commented Mar 6, 2014

There is some commented out code for a method named nextInt in SeededRandomizer. Can it go? (I am guessing that the intention was to not have this method in order to force you to think about the upper bound, which I agree with). I am also wondering about why every method is called get... except the one for long?

@DanielBruegmann DanielBruegmann added this to the 0.4 milestone Mar 6, 2014

Contributor

DanielBruegmann commented Mar 6, 2014

I have some unpushed changes w.r.t. #30 that would be bad to push by themselves that change SeededRandomizer, so this should probably wait until I push them (I don't think it is worth it to create a branch).

Contributor

PascalCremer commented Mar 6, 2014

Yes, I think it can go. We should probably stick to the "next" notation since it is standard and more intuitive. I'm to blame for this inconsistency.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment