What's with the sample tests testing the function output with itself?
The only explanation I can think of is the author knows that there are only a few possible outputs and it's too easy to get the answer from the test results. If that's the case why are you publishing such an un-interesting kata in the first place?
you are correct. that part is redundant. also, one can terminate the program after the first text case and not even test for the second
Something like
if (word==true) { ... }when the variablewordis itself a boolean is redundant - you can just writeif (word) { ... }Edit: Ignore me if you were questioning the statement "don't use...", didn't take your meaning first at first read
I do agree there is alot wrong with this code. But what do you mean with "don't use the boolean property of the parameter"?
VERY slow? How is this very slow?
This is VERY slow. You do 2 checks, have too many curly braces, don't use the boolean property of the parameter etc.
Nice!
0 is divisible by 5 and not excluded.
What's with the sample tests testing the function output with itself?
The only explanation I can think of is the author knows that there are only a few possible outputs and it's too easy to get the answer from the test results. If that's the case why are you publishing such an un-interesting kata in the first place?
apostrophe?
Needs Random Tests.