Because:

1. there are specific methods that should be used for better code readability and maintenance (e.g. "find")
2. "let" is used instead of "const"
3. there are multiple exit points ("return")
4. just because it is performant does not mean it is best practice :)

Hey I am quite uneducated when it comes to best practice in conding. Could you explain to me why it is not best pracite even though it is the most performant solution?

Very nice kata! The most intuitive solution involving nested loops may not be the fastest.
One should try to stick to simple things and "memoize" them 😇

Quite slow solution unfortunately, but amongst the cleanest anyway :)

Not so "best practices" but it is the fastest solution! (by far...) Congrats!! :) :)