Чому RegExp з глобальним прапором дає неправильні результати?

Яка проблема з цим регулярним виразом, коли я використовую глобальний прапор та прапор, нечутливий до регістру? Запит - це введений користувачем вхід. Результат повинен бути [правдивим, правдивим].

var query = 'Foo B';
var re = new RegExp(query, 'gi');
var result = [];
result.push(re.test('Foo Bar'));
result.push(re.test('Foo Bar'));
// result will be [true, false]

var reg = /^a$/g;
for(i = 0; i++ < 10;)
   console.log(reg.test("a"));

RegExpОб'єкт відстежує , lastIndexде сталося збіг, так і на наступних матчах він буде стартувати з останнього використаного індексу, замість 0. Зверніть увагу:

var query = 'Foo B';
var re = new RegExp(query, 'gi');
var result = [];
result.push(re.test('Foo Bar'));

alert(re.lastIndex);

result.push(re.test('Foo Bar'));

Якщо ви не хочете вручну скидати lastIndexзначення 0 після кожного тесту, просто зніміть gпрапор.

Ось алгоритм, який диктують характеристики (розділ 15.10.6.2):

RegExp.prototype.exec (рядок)

Виконує збіг регулярного вираження рядка проти регулярного виразу і повертає об’єкт масиву, що містить результати відповідності, або null, якщо рядок не збігається. Шукає рядок ToString (рядок) для появи шаблону регулярного вираження таким чином:

1. Нехай S - значення ToString (рядок).
2. Нехай довжина буде довжиною S.
3. Нехай lastIndex є значенням властивості lastIndex.
4. Нехай я - значення ToInteger (lastIndex).
5. Якщо глобальна властивість хибна, нехай i = 0.
6. Якщо довжина I <0 або I>, то встановіть lastIndex на 0 і поверніть null.
7. Виклик [[Матч]], надавши йому аргументи S і i. Якщо [[Матч]] повернув помилку, перейдіть до кроку 8; інакше нехай r є його державним результатом і переходимо до кроку 10.
8. Нехай i = i + 1.
9. Перейдіть до кроку 6.
10. Нехай це значення r endIndex.
11. Якщо глобальна властивість вірна, встановіть lastIndex на e.
12. Нехай n - довжина масиву r захоплення. (Це те саме значення, що і NCapturingParens 15.10.2.1.)
13. Повернути новий масив із такими властивостями:
    • Властивість індексу встановлюється на позицію відповідного підрядка в повному рядку S.
    • Властивість вводу встановлено на S.
    • Властивість довжини встановлено на n + 1.
    • Властивість 0 встановлюється на підстрокову підстроку (тобто частину S між зміщенням i включно та зміщенням e виключно).
    • Для кожного цілого числа i такого, що I> 0 і I ≤ n, встановіть властивість під назвою ToString (i) на i-й елемент масиву r's captures.

Ви використовуєте один RegExpоб'єкт і виконуєте його кілька разів. Після кожного наступного виконання він продовжується з останнього індексу матчу.

Вам потрібно "скинути" регулярний вираз, щоб почати з початку перед кожним виконанням:

result.push(re.test('Foo Bar'));
re.lastIndex = 0;
result.push(re.test('Foo Bar'));
// result is now [true, true]

Сказавши, що може бути зручніше створювати новий об'єкт RegExp кожен раз (накладні витрати мінімальні, оскільки RegExp кешується в будь-якому випадку):

result.push((/Foo B/gi).test(stringA));
result.push((/Foo B/gi).test(stringB));

RegExp.prototype.testоновлює lastIndexвластивість регулярних виразів, щоб кожен тест починався там, де зупинився останній. Я б запропонував використовувати, String.prototype.matchоскільки він не оновлює lastIndexвластивість:

!!'Foo Bar'.match(re); // -> true
!!'Foo Bar'.match(re); // -> true

Примітка: !!перетворює його в булевий, а потім інвертує булевий, щоб він відображав результат.

Крім того, ви можете просто скинути lastIndexвластивість:

result.push(re.test('Foo Bar'));
re.lastIndex = 0;
result.push(re.test('Foo Bar'));

Видалення глобального gпрапора виправить вашу проблему.

var re = new RegExp(query, 'gi');

Має бути

var re = new RegExp(query, 'i');

Потрібно встановити re.lastIndex = 0, тому що, якщо g flag regex відстежує останній збіг, тому тест не буде переходити на тестування тієї ж строки, для цього вам потрібно зробити re.lastIndex = 0

var query = 'Foo B';
var re = new RegExp(query, 'gi');
var result = [];
result.push(re.test('Foo Bar'));
re.lastIndex=0;
result.push(re.test('Foo Bar'));

console.log(result)

Використання прапора / g повідомляє йому продовжувати пошук після попадання.

Якщо збіг вдається, метод exec () повертає масив і оновлює властивості об'єкта регулярного вираження.

Перед першим пошуком:

myRegex.lastIndex
//is 0

Після першого пошуку

myRegex.lastIndex
//is 8

Видаліть g, і він завершує пошук після кожного дзвінка до exec ().

У мене була функція:

function parseDevName(name) {
  var re = /^([^-]+)-([^-]+)-([^-]+)$/g;
  var match = re.exec(name);
  return match.slice(1,4);
}

var rv = parseDevName("BR-H-01");
rv = parseDevName("BR-H-01");

Перший дзвінок працює. Другий дзвінок не робить. sliceОперація скаржиться на нульове значення. Я припускаю, що це через re.lastIndex. Це дивно, тому що я б очікував, що RegExpкожен раз буде виділено нове, коли функція викликається, а не поділяється на кілька викликів моєї функції.

Коли я змінив його на:

var re = new RegExp('^([^-]+)-([^-]+)-([^-]+)$', 'g');

Тоді я не отримую lastIndexефекту утримування. Це працює так, як я б очікував.