Цикл в цикле. Квадратичная сложность. Ее можно было довольно легко избежать)

В этой фунцкии есть ошибки...

Давай по пунктам, что бы исправить эти моменты:

1. Изменены название функции и входящие параметры. Теперь функция может принимать бесконечное к-во параметров. Если бы на проекте применялось контрактное программирование - нужно было бы заключать новый контракт и импактить все области приложения в которых используется эта функция.
2. Был ли резон записывать функцию в переменную? Ничего против не имею, результат будет тем же, прост интересно)
3. indexesArr ничего не говорит о том что именно содержится в переменной. Вероятно как и letters
4. Была убрана проверка первого параметра, т.е. было изменено поведение функции. Опять же, импакт.

В целом решение неплохое. но оно на грани))
Отсутствие промежуточных результатов и сахар немного затрудняют чтение функции. Была бы в ней больше операций и уже нужно было бы рефакторить