isEven -- Modulo issue

[erictessin]

Test will fail for "should not use modulo" when the word "modulo" is included in comments.

[mybrainishuge]

Thanks for your feedback, @erictessin. Do you have a suggested solution for improving this test?

[erictessin]

Hey @mybrainishuge!

Maybe delete this line in the test?

expect(stringified).to.not.contain('modulo');

Unless there is a way to calculate the remainder using the word "modulo" that i don't know about, I don't see why including the word should make the test fail. But maybe I'm missing something?

[mybrainishuge]

One of the challenges is to write a function called modulo. The test you referenced is attempting to prevent you from using that function.

var isEven = function(n) {
    return modulo(n, 2) === 0
};

^^^ This misses the point of the exercise since isEven doesn't call itself. Keep in mind that these challenges aren't expected to be performant, but rather force you to utilize recursion. More specifically, each function must call itself.

What's a base case for even or odd and how do you reach that base case? Think about factorial if you've already implemented that one. Given n, what can you do to n repeatedly until the base case is reached to determine the answer?

Does that help?

[mybrainishuge]

I made a change to the check for modulo, but using it by name or operator is still forbidden when implementing isEven. I'm going to close this issue, but feel free to reach out if you have more questions.

[erictessin]

That does clear it up! Thank you Richard. I've been enjoying the exercises, they're super helpful. Best regards, Eric T

…

On Thu, Feb 11, 2021 at 5:57 PM Richard May ***@***.***> wrote: Closed #83 <#83>. — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#83 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/APZUN6JVST63OTIOGDM4J2DS6SDJLANCNFSM4XPXA6AA> .