0

Добрый день!

Необходимо строку перевести в нижний регистр и удалить все кроме цифр и символов, вот рабочий код:

 var str = "Some_string-9"
 str=str.toLocaleLowerCase();
 str=str.replace(/\W/g, "");
 str=str.replace(/_/g, ""); 
 //result: somestring9

Грамотно ли оформлять его подобным образом? В смысле в три строки такую простую операцию, спасибо!

5
  • одно слово: вкусовщина 29 окт 2017 в 21:16
  • Ясно=) Я начинающий программист, интересно нет ли в этом коде грубой стилистической ошибки. 29 окт 2017 в 21:19
  • Как минимум, обе регулярки можно объединить str.replace(/[\W_]/g, "").toLowerCase() 29 окт 2017 в 21:20
  • @ЕвгенийМошкин всё, что не трудно читать и приятно глазу - не является ошибкой 29 окт 2017 в 21:22
  • У вас ошибка в постановке условия. Если убрать "все кроме цифр и символов", то из Some_string-9 получится _-9.
    – Nick Volynkin
    30 окт 2017 в 2:45

1 ответ 1

1

Два ваших регулярных выражения можно запросто объединить в одно. В остальном - это все дело вкуса. Лично я написал бы вот так:

var str = 'Some_string-9';
var result = str.toLocaleLowerCase().replace(/[\W_]+/g, '');
1
  • 1
    toLocaleLowerCase, а потом удалить все нелатинские буквы.. ну а что? ))
    – Qwertiy
    30 окт 2017 в 0:07

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge you have read our privacy policy.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.