в чём-то соглашусь с тобой, в чём то нет)
Логической ошибки там нет, т.к. функция режет строку на равные части по 3 символа) Для этого она и существует)) И если уж говорить о ресурсоёмкости (что в данном примере в принципе смешно), то мой код менее извращённый и в цикле не содержится лишних переменных (которые ты объявил я сам не знаю зачем). С твоей доработкой кода согласен, если автору нужно чтобы только после каждого 3-го символа ставилось тире, то ты да.
Насчёт двойных кавычек ... хехе, выразился конечно так....в таких ничтожных скриптах это не суть важно....а в серьёзных проектах...согласен одинарную кавычку использовать рациональней