-
Notifications
You must be signed in to change notification settings - Fork 0
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
caffeineBuzz won't ever print "Coffee" #3
Comments
it's codewars...kata I've done that has passed. The katas aren't perfect...quite a few are buggy, but that makes them fun (well, some of them...some of them are unable to be completed because of test errors). These are best done if you code them yourself :) Generally, the ones I have uploaded have passed all test cases, but this is a personal repo of my solutions. You're welcome to try your alternative on the website codewars.com. You can also try some of the more difficult ones, or different languages, etc (I've trained in 8 thus far). |
It's interesting that all the tests passed, do you mind posting the codewars link to this problem? I'm not too interested in doing coding exercises, I just looked at some of your work, noticed some issues, and couldn't help posting my feedback. |
I found the codewars link, and I can see why you'd code it the way you did. Note: I don't mean to be a dick, although I am jealous of the amount of work you've done here. |
Nah..you're not a dick Haha. I'd actually love to meet and follow your work, too. We're all in this together. I'm jealous of some of those people who are 1 dan, and their code is SO CLEAN!!! Aaaah..it's insane. I want to get that good..I have a looong way to go. It's nice to meet you..I hope we can be friends :) |
Looking at the CaffeineScript.py file, I don't think that the
return "Coffee"
case will ever be reached.This is due to the fact that there will never be a case where n % 4 == 0 is true, n % 3 == 0 is true, and n % 2 == 0 is false, because if a number is cleanly divisible by 4, it's always going to be cleanly divisible by 2.
I don't know the context of the exercise, so I'm not really sure what the expected behavior is. I would go exploring, but im too tired atm lol.
The text was updated successfully, but these errors were encountered: