Правильно ли перевел Python-функцию в JS?

Question

[Сергей Ильин]

есть функция:

def text_cleaner_fio(text):
    text = re.sub( r'[\W]+', ' ', text ) # удаление лишних символов
    text = re.sub( r'([А-Я]{1}[а-яё]{0,23} [А-Я]{1}[а-яё]{0,23} [А-Я]{1}[а-яё]{0,23})|( [A-Z]{1}[a-z]{1,23} [A-Z]{1}[a-z]{1,23})$', ' FIO ', text ) # замена учеток
    return text

нужен JS.
правильно ли:

function text_cleaner_fio(text){
    text.replace(/[\W]+/, ' '); # удаление лишних символов
    text.replace(/([А-Я]{1}[а-яё]{0,23} [А-Я]{1}[а-яё]{0,23} [А-Я]{1}[а-яё]{0,23})|( [A-Z]{1}[a-z]{1,23} [A-Z]{1}[a-z]{1,23})$/, 'FIO'); # замена учеток
    return text;
}

Answer 1

[Дмитрий Дерепко]

Особенности регулярных выражений в том, что они представляют отдельный язык, на котором они написаны. То есть регулярное выражение в питоне = регулярному выражению на любом другом языке.

Ответ на ваш вопрос - да, правильно.

Еще можно объединить вызовы методов:

function text_cleaner_fio(text){
    return text
        .replace(/[\W]+/, ' ') # удаление лишних символов
        .replace(/([А-Я]{1}[а-яё]{0,23} [А-Я]{1}[а-яё]{0,23} [А-Я]{1}[а-яё]{0,23})|( [A-Z]{1}[a-z]{1,23} [A-Z]{1}[a-z]{1,23})$/, 'FIO'); # замена учеток
}

Comments

[Сергей Ильин]

по поводу того, что регулярки представляют отдельный язык - это понятно.
не ясен был синтаксис JS, ибо не писал на нем ничего никогда.

Спасибо за коррекцию кода - оч прикольно

[Сергей Ильин]

Алексей Аминодов-Борисов, да, спасибо ) понял, когда запустил код )