Почему регулярное выражение в цикле не находит совпадения, а отдельно находит?

Question

[Михаил]

Здравствуйте!
Пишу регулярку для получения цен из текстов:

function test_price_min(){

    $post = '
Текстиль
32-37/8
975₽
';

    $result_match = preg_match('/Цена\d{1,6}[рр₽][^aак]|Цена\d{1,6}[^aак]|[^-р]\d{1,6}[рр₽][^aак]|ц\d{1,6}[^в]/ui', $post, $matches_gross);

    if ($result_match == true) {

    echo '<pre>';
    print_r($matches_gross);
    echo '</pre><br>';

    }

    }

      test_price_min();

Часть текста с различными условиями в цикле находит без проблем, а с некоторыми проблема.

Comments

[dodo512]

а с некоторыми проблема

А где пример текста с которым нет совпадений?

[maksam07]

Лучше напиши примеры текста и какой результат у каждого хочешь видеть, а то щас не очень понятно

[Михаил]

maksam07,

С 'этого текста цену получаю при помощи кода выше в виде 975₽

Текстиль
32-37/8
975₽

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

[Михаил]

Код в цикле:

foreach ($wp_query->posts as $post) {

    $post_content_filtered = json_decode($post->post_content_filtered,'true',512); // десериализовали общий массив

    // удаляем все точки (некоторые цены отделяют точками)
    $matches_gross = mb_ereg_replace('\.', ' ', $post_content_filtered['dop_text_post']['detail_post_text']);

    // удаляем все двоеточия (некоторые перед ценой ставят двоеточия)
    $matches_gross = mb_ereg_replace(':', ' ', $matches_gross);

    // заменяем все звездочки на бкву х
    $matches_gross = mb_ereg_replace('\*', 'х', $matches_gross);

    // удаляем все пробелы (включая табуляцию и концы строк)
    $matches_gross = preg_replace('/\s+/', '', $matches_gross);


    // смотрим очищенный текст
    echo '<pre>'; print_r($matches_gross); echo '</pre><br>';

    // пробую преобразовать в строку (не все данные находит в цикле)
    $matches_gross = (string)$matches_gross;

    // достаем сумму цены с текстом (брутто)
    $result_match = preg_match('/Цена\d{1,6}[рр₽][^aак]|Цена\d{1,6}[^aак]|[^-р]\d{1,6}[рр₽][^aак]|ц\d{1,6}[^в]/iu', $matches_gross, $matches_gross_result );

    echo 'брутто: <pre>'; print_r($matches_gross_result); echo '</pre><br>';

    if($result_match == true) {

    // echo 'Найдено!<br>';

    // достаем сумму цены с текстом (нетто)
    $result_clear_match = preg_match('/([^xх\/][1-9]\d{1,6}(?!х)(?!x)(?!ш))/ui', $matches_gross_result[0], $matches_net );

    // если цена с текстом получена
    if ($matches_net[0]) {

    echo '<pre>'; print_r($matches_net); echo '</pre>';

    // оставляем только цифры
    $result_clear_match = preg_match('/\d{2,6}/ui', $matches_net[0], $matches_net );

    echo ' Цена чистая: '.  $matches_net[0] .'<br>';
    echo '<pre>'; print_r($post->guid); echo '</pre>';
    echo '<pre>'; print_r($post->ID); echo '</pre>';
    echo '<pre>'; print_r($post->post_content); echo '</pre>';
    }

    // если цена не получена
    else { echo ' Пустой массив! ' .$post->ID . '<br>';
    echo '<pre>'; print_r($matches_gross); echo '| </pre><br>';
    echo '<pre>'; print_r($post->guid); echo '</pre>';

    }

    // вставляем цену в базу данных
    $insert_array = ['post_id' => $post->ID, 'post_price' => $matches_net[0],];
    $res_insert = $wpdb->insert( 'wp_posts_extend', $insert_array );

    // echo '<pre>'; var_dump($res_insert); echo  $post->ID . ' <hr></pre>';

    echo '<hr>';

    }

    // если цена с текстом не получена
    else { echo 'Цена не найдена
    <br>';

    echo '<pre>'; print_r($post->guid); echo '</pre>';
    echo '<pre>'; print_r($post->ID); echo '</pre>';
    echo '<pre>'; print_r($post->post_content); echo '</pre>';
    echo '<hr>';

    }

    }

    }

    test_price();

[Михаил]

dodo512, ответил ниже в комментариях

[dodo512]

// смотрим очищенный текст
    echo '<pre>'; print_r($matches_gross); echo '</pre><br>';

    // пробую преобразовать в строку (не все данные находит в цикле)
    $matches_gross = (string)$matches_gross;

А зачем попытка преобразовать в строку. Там что кроме обычных строк встречается ещё что-то?

if($result_match == true) {
 ...
// если цена с текстом не получена
else { echo 'Цена не найдена
<br>';

echo '<pre>'; print_r($post->guid); echo '</pre>';
echo '<pre>'; print_r($post->ID); echo '</pre>';
echo '<pre>'; print_r($post->post_content); echo '</pre>';
echo '<hr>';

}

Тут нет var_dump($matches_gross) для случая когда совпадение не найдено.
Нужно выяснить что именно передавалось в preg_match.

[ksnk]

Попробуй вот такую регулярку

$result_match = preg_match('/Цена(\d{1,6})[рр₽](?:[^aак]|$)|Цена(\d{1,6})(?:[^aак]|$)|[^-р](\d{1,6})[рр₽](?:[^aак]|$)|ц(\d{1,6})(?:[^в]|$)/iu', $post, $matches_gross_result );

    if ($result_match == true) {
        $pattern= array_shift($matches_gross_result); // $pattern - найденная подстрока, вдруг зачем то пригодится
        $matches=implode('',$matches_gross_result); // найденная цена - собственно без дополнительной магии.
...

Основной смысл - если строка поиска заканчивается сразу после указания цены - старая регулярка не сработает из за [^aаk] или [^в]. И зачем ковырять цену второй регуляркой, если она и так может быт выковыряна в первой.

Использование mb_ereg_replace сделано из каких высоких соображений ? Или так исторически сложилось?

[Александр Торопов]

У вас в $result_match будет массив, а вы его сравниваете с boolean

[dodo512]

в $result_match будет массив

Александр Торопов, с чего вдруг там появится массив???

[Михаил]

Александр Торопов,
preg_match() возвращает 1, если параметр pattern соответствует переданному параметру subject, 0 если нет, или FALSE в случае ошибки.

Массив возвращает $matches_gross.

[Михаил]

ksnk, Спасибо за внимание, ваш совет и вопрос полезны для размышления, увидел недочеты.

[Михаил]

dodo512, проверка есть, она выше в коде:

echo 'брутто: <pre>'; print_r($matches_gross_result); echo '</pre><br>';

она для обоих случаев

[Михаил]

Спасибо за внимание, вы обратили внимание на детали которые я не замечал, но суть вопроса в осталась незамеченной))

Дело в том, что регулярное выражение работает с данным текстом (матчит результат), но если этот текст проверять отдельно:

Текстиль
32-37/8
975₽

если этот же текст проверяется в массиве с другими текстами, то не матчится. Вот в чем проблема.

[dodo512]

Михаил, если результат разный, то напрашивается вывод что в preg_match передаёте разный текст.
'975₽'; и '975₽ '; это разные строки. Всего один лишний пробел.
https://onlinephp.io/c/c3195

проверка есть, она выше в коде:

echo 'брутто: '; print_r($matches_gross_result); echo '
';

она для обоих случаев

В $matches_gross_result результаты поиска.
Логировать нужно ещё и проверяемый текст, а он находится в $matches_gross.
Нужно использовать var_dump($matches_gross);
С print_r не особо заметно лишние символы после 975₽

[Михаил]

dodo512,

если результат разный, то напрашивается вывод что в preg_match передаёте разный текст.
'975₽'; и '975₽ '; это разные строки. Всего один лишний пробел.

Это позже проверю, очень похоже что так. Спасибо!
Надо мне быть внимательнее.

[dodo512]

Михаил, почему-то в вопросе

= '
Текстиль
32-37/8
975₽
';

Тут несколько переводов строк хотя в коде есть удаление всех пробельных символов.

// удаляем все пробелы (включая табуляцию и концы строк)
$matches_gross = preg_replace('/\s+/', '', $matches_gross);

После этой строки текст должен стать таким
='Текстиль32-37/8975₽';

[Михаил]

dodo512, Спасибо вам! Дело было в символе перевода строки. Регулярное выражение не учитывало конец строки и не находило цену.

Answer 1

[Indread]

Регулярное выражение может не находить совпадений в цикле, если используется флаг `g`, так как он изменяет позицию поиска между вызовами, что приводит к пропуску совпадений. В цикле важно либо использовать регулярное выражение без этого флага, либо сбрасывать позицию поиска вручную. Также проблемы могут возникать, если строка изменяется в процессе цикла или если неправильно обновляются индексы при выполнении поиска. Чтобы избежать таких ошибок, следует тщательно контролировать позицию поиска и логику цикла, а также убедиться, что регулярное выражение корректно настроено для многократных итераций.