Замена Regex не выполняется во всех совпадениях

Question

Может ли кто-нибудь понять, почему следующий код не выполняет операции регулярного выражения на всех {{...}} местах-заполнителях? Строка ввода ниже - это просто урезанная версия оригинала.

https://jsfiddle.net/2L12jr3u/2/

var input = "{{ %1$@ }}/{{ %1$@ }} ({{ %2$@ }}) {{ %1$@ }} {{ %1$@ }} {{ %1$d }} {{ %1$@ }} of {{ %2$d }} of {{ %3$d }}"; 

var regex = /(\{\{ \%(\d)\$(.) \}\})/g; 

var match; 
while (match = regex.exec(input)) { 
    console.log(match); 
    input = input.replace(match[0], '%@' + match[2]); 
} 

console.log(input); 

Answer 1

Поскольку вы используете exec, чтобы соответствовать несколько экземпляров, в каждой итерации он начинает поиск с last matched index:

Если ваш обычный экспресс-экспресс n использует флаг «g», вы можете использовать метод exec() несколько раз, чтобы найти последовательные совпадения в одной строке. Когда вы это сделаете, поиск начинается с подстроки str, заданной свойством lastIndex регулярного выражения (test() также увеличивает свойство lastIndex).

В вашем случае это намного проще и чище использовать String.prototype.replace метод:

var input = "{{ %1$@ }}/{{ %1$@ }} ({{ %2$@ }}) {{ %1$@ }} {{ %1$@ }} {{ %1$d }} {{ %1$@ }} of {{ %2$d }} of {{ %3$d }}"; 
 

 
var regex = /(\{\{ %(\d)\$(.) \}\})/g; 
 

 
input = input.replace(regex, '%@$2'); 
 

 
alert(input);

Answer 2

Это происходит потому, что вы измените переменную input в то время как exec не закончил. Индекс продвигается вперед, но строка становится короче.

Добавить еще одну переменную, или завернуть в отдельную функцию, или использовать replace как предложено @dfsq:

var input = "{{ %1$@ }}/{{ %1$@ }} ({{ %2$@ }}) {{ %1$@ }} {{ %1$@ }} {{ %1$d }} {{ %1$@ }} of {{ %2$d }} of {{ %3$d }}"; 
 

 
var regex = /(\{\{ \%(\d)\$(.) \}\})/g; 
 
var output = input; 
 
var match; 
 

 
while (match = regex.exec(input)) { 
 
    console.log(match); 
 
    output = output.replace(match[1], '%@' + match[2]); 
 
} 
 

 
alert(output);

Answer 3

Сначала я предлагаю, чтобы удалить ненужные группы в регулярном выражении,

/\{\{ %(\d)\$. \}\}/g вместо от /(\{\{ %(\d)\$(.) \}\})/g. Затем, чтобы заменить вы можете использовать более короткий подход (а также более ясно, на мой взгляд):

var input = "{{ %1$@ }}/{{ %1$@ }} ({{ %2$@ }}) {{ %1$@ }} {{ %1$@ }} {{ %1$d }} {{ %1$@ }} of {{ %2$d }} of {{ %3$d }}"; 
var output = input.replace(/\{\{ %(\d)\$. \}\}/g, "%@$1"); 

Окончательное значение выходе %@1/%@1 (%@2) %@1 %@1 %@1 %@1 of %@2 of %@3