Показать сообщение отдельно
  #2 (permalink)  
Старый 08.02.2020, 18:02
Аватар для Aetae
Тлен
Отправить личное сообщение для Aetae Посмотреть профиль Найти все сообщения от Aetae
 
Регистрация: 02.01.2010
Сообщений: 6,590

Вон вверху есть "Аналоги функций PHP", чтоб тебе не писать своих костылей, и в частности str_replace.

А конкретно у тебя не так - всё. Это даже не javascript, на неряшливый набор букв.

Синтаксис:
1.String.prototype.str_replace_all () { ... } - такой конструкции в javascript нет.
2.this.replace( ... )); - в конце лишняя скобка.

Логика:
1. Чтобы получить результат - функция должна возвращать результат. Отсутствует return.
2. Метод replace не меняет исходную строку. Нет сохранения результата.
3. Строки попадающие в new RegExp должны быть экранированы. Если там попадутся спецсимволы всё упадёт.
4. Нет проверки переданных аргуметов.

Стиль:
1. Использование arguments нежелательно, если нужно получить массив входных аргументов используйте ...args.
2. Для именования в javascript принят camelCase, snake_case не рекомендован.
__________________
29375, 35

Последний раз редактировалось Aetae, 08.02.2020 в 18:20.
Ответить с цитированием