That's... possible. It was the case in the past, yes. I don't know if it got fixed or not. I guess not...

Note for future: create a new translation.

Thanks!

(Although I didn't seem to get any points for it. Do they go to the original translator?)

approved

the expectedSolution method should be private

Done.

the building of the random list of coins is a bit weird (why not directly an array?)

Done. Much simpler.

Not sure exactly what you mean here.

I guess it's just the diff that is not showing the next line (worst case, I can fix it after approval)

• the expectedSolution method should be private
• the building of the random list of coins is a bit weird (why not directly an array?) but at this point, "whatever", I guess.

Poke at me when the method is private.

I think the java code block is missing the closing backticks? (maybe not, it's not clear by looking at the diffs)

Not sure exactly what you mean here.

Why the instane if the method is static...? => remove the instance completely?

That was in the original translation that I forked. I have removed it.

the solution setup is not aligned with the current solution (no static method).

Fixed. Also added final to both parameters.

could we use a loop to write only once the code for assertions?

Done

• I think the java code block is missing the closing backticks? (maybe not, it's not clear by looking at the diffs)
• Why the instane if the method is static...? => remove the instance completely?
• the solution setup is not aligned with the current solution (no static method)
• could we use a loop to write only once the code for assertions?

same here

Random tests should be added.

See description. "No whitespaces/underscore"

Hi,

I don't know if you'll see this message, bensechrist, but your translation isn't approvable because of description conflicts and, moreover, you should have chosen to return a Map instead of this object/getter design, to be more consistent with other languages (that would ease the work for moderators, maintenance, ...). So, if I do not have news in the next few days, I'll reject your translation and make another one.

cheers

B4B

haha so much for "eval" not being included with JavaScript. When i first read the kata I thought it would be super easy because this solution was an option, but then i read that eval was disabled so I did a much longer one. Wish I would have just done this since obviously it's not disabled.