This is cool, but it's not easy to read. Code should be as simple and easy for readability and maintainability.

For performance, you should use a StringBuilder to avoid creating new string objects, instead of using the + operator.

true this is a good case of bad pratices the code should be/try to be easy to read to any person new to the code

Difficult to read

Mine is similar :)

thanks.

Clever solution. We can use result.setCharAt(index, ')'); to make less discussions about StringBuilder::replace

4 cicles for counting 4 vars? :X

So many votes for this being best practice. This is not best practice. Readability > Complexity.

Do only I have to my_eyes.split() to read that code?

i tends to be for integers by convention. This exercise deals with lists, so a different letter like x would be more appropriate.

If you want ridiculous, check mine out, lol

And we all seek ridiculous one-liners in solutions section - that's a lot of fun.

In my opinon, his solution is easier to read thatn yours, and you have used too many unneccesary lambda functions further increasing the complexity. The other solution did the same thing without the use of multiple or no lambda functions :)

Oh my god