-
Notifications
You must be signed in to change notification settings - Fork 1
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
Deduplicate - factorial #190
Comments
I have no good idea how to pick a candidate to keep:
I think kata 5. and 6. can stay as sufficiently different to be kept, especially in languages where bigint types are not the default integer types. |
I vote to keep 3 and 6 only. 6 should be updates to handle large factorials. I personally don't thinks that large factorials (5) add anything new to this task: There are already many iterative factorial implementations in 3. 7 is a borderline kata. It may stay. But it seems to be overranked and its factorial definition for negative numbers is easy to implement. |
I like 7! Vote to keep 3, 6, and 7 and retire the others. |
My vote: |
Remove 1, 4, 5 as not having enough translations and 7 as being overranked |
No one voted to keep 1., 4., or 5., so I think these are good to go. I am yet not sure what to do with 2. and 7., let's see if there are any other votes for these two. |
I guess 2. may be applicable for those langauges where large numbers aren't supported (not those ones which may have reaaaally big numbers like JS or Python), for C/C++/Java without specialized imports, maybe? I guess it may stay here, as well as 6. – factorials for negatives are pretty interesting lol. (I wonder if anybody thought about factorials with float arguments – like, |
From wiki list
Recursion #1 - FactorialFactorial Factorynull
Big FactorialPartial conclusion:
I will try to get some more votes for kata 2. and 7. and see what users want to do with them.
The text was updated successfully, but these errors were encountered: