Можно ли улучшить это регулярное выражение замены подстроки?

Question

Мне нужно заменить подстроку из некоторой строки. Я уже создал исправленный код для этого. Но я не уверен, что это лучший способ. Пожалуйста, смотрите код ниже:

var str = 'test ruby,ruby on rails,ruby,'
var substr = 'ruby';
var reg = new RegExp(',' + substr + ',|^' + substr + ',', 'gi');
str.replace(reg, ','); //returns "test ruby,ruby on rails,"

Answer 1

Вы можете немного его укоротить:

var reg = new RegExp('(^|,)' + substr + ',', 'gi');

Answer 2

Если ваша подстрока не сгенерирована программно или не основана на вводе пользователем, на мой взгляд, ее легче читать, если вы определяете регулярное выражение в javascript с помощью операторов / /.

Чтобы вы могли переопределить reg как:

reg = /,ruby,|^ruby,/gi;

Answer 3

Попробуйте это:

var reg = new RegExp("(^|,)" + substr + "(,|$)", "gi");

Answer 4

Чтобы уточнить, о чем упоминает Шон, если вы на самом деле программно генерируете свои строки, вы можете проверить этот вопрос SO , который имеет функцию для экранирования строк регулярного выражения для javascript. Отличная вещь! Удачи:)