Done. New fork for Scala translation
I'm getting "Description cannot be approved, recent changes from related record must be merged first."
I think you need to create a new fork.
Please review and approve Scala translation (https://www.codewars.com/kumite/5a715163fd8c06065d00006e?sel=5a715163fd8c06065d00006e)
I just saw this one too. I approved it to replace "Magnets". Next time ask before translating. Thanks for your time.
I have just discovered your translation when, having written mine, I wanted to publish it. I am very sorry but usually I translate myself my own katas when I know the language I am translating into and from my Java translation to Scala it is not very difficult... Maybe it would be better to ask before translating? Do you have any other translation of one of my katas that it is not approved yet? If so tell me and I will approve it instead of this one. Cheers!
Also I would like to mention the code's weak readibility.
Right. I know that but I mistakenly let it from another program. Unable to re-publish: CW Scala is so slow that it always times out.
Thanks for your post!
Not about the kata or solution but ...
"lazy" don't give you any advantages here. Because in the next line the whole expression will be executed completely.
If you want to filter your zipped lists step by step just to the first "right" item you should use "streams" in Scala.
What parameters should I pass to non-default constructors? Default? Null? Any values?
This algorithm has O(n^2) complexity due to IndexOf inside Where. Avoid this in production.