Sure, my foul

This isn't possible in any current version of C#.

2^31 = 2147483648

(DateTime.MaxValue - DateTime.MinValue).TotalDays = 3652059 (less than 2^31)

cycleLength is int, which has a max value of 2147483647 (less than 2^31)

There is no possible way to pass this method values that will cause the error you describe.

Agreed, but sometimes practicing this type of coding style in easier task make it easy on harder tests with a max char length

It's excellent. Less code and clever solution

Minified code is not optimal for reading ^_^

.

Used same approach for names. Thanks for the help.

\o/

working.

But now that I see the tests: you have the same problem about names. The chances of collisions are really low, but your tests could generate 2 times the same name => use the same tactic (with the set)

Thanks, for feedback, I have now changed it to use a Set.

XD

worse than before...:

{"name":"dnpk8suxz1","party":"Republicans"}, => no vote at all

what are you doing to produce this? you should be using a set that you fill with random numbers as long as you don't have the neeeded number of data

Thanks, fixed, please review.

that part seems handled, but now you have again duplicated numebrs of votes:

{"name":"tfmmwodtk5","votes":752,"party":"Republicans"},
...,
{"name":"yyczu3rt4cg","votes":752,"party":"Democrats"},

Thanks, Blind4Basics, fixed the issue and also made my reference function different.