I guess you assume that “SimpleRandomString” is supposed to be wrapped up in another function that validates “size” argument first, and handles the following special cases:

size = (2147483647 + 1)

…or would you increase complexity of “SimpleRandomString” function?
——————————–
I’m sorry for a few tests down here…

I’ve just submitted my comment and noticed that entire lines containing “prohibited” characters were wiped out.

So I’m not sure which way it will work and try a few.

size = = 0
“size equals zero”

size <= -1
“size is equal to or less than”

size >= (2147483647 + 1)
“size is equal to or greater than”

Thanks,
Albert

[Bj's Reply] Hi Albert, I combined both your comments to avoid redundancy.

In answer to your question, the answer is no. The SimpleRandomString method does not need increased complexity to handle your ‘special case tests.’

If you call the SimpleRandomString method and pass a 0 or any value less than 0 then what do you get in return?

You get an empty string! Basically, you are getting exactly what you ask for.

Now, if you try to pass it a value greater than 2147483647 then your code will likly not compile. And even if it did (say you weren’t checked mode), you can’t pass an argument greater than 2147483647 to a type int parameter in strongly typed languages such as C#. So, if you try to pass an argument greater than 2147483647 it will actually be your code that throws an exception rather than the SimpleRandomString method.

These ‘special tests’ are in fact all false positives where you think you are getting an error from my code, but in fact the test inputs are based on false assumptions.