Почему не срабатывает регулярка?

Question

[maiskiykot]

Приветствую. Что-то сегодня день какой-то - понедельник, наверное. Пытаюсь разобрать строку, точнее набор строк типа:

<a href="/kontrolno-izmeritelnyie-priboryi" itemprop="item" title="Контрольно-измерительные приборы" class="breadcrumbs__item-link" data-v-71>

Создал для этого регулярку:

preg_match_all("!href=\"(.*)\".*title=\"(.*)\".*breadcrumbs__item-link!siU",$str,$matches))

Однако она собака не ловит ни Х. Если сокращаю левую часть до title, то title ловит отлично, а вот ссылку захватывать ни в какую не хочет. Что я тут делаю не так? Всем спасибо!

Answer 1

[Adamos]

Если ищешь что-то в кавычках, нужно использовать не \"(.*)\", а \"([^"]*)\"

Comments

[maiskiykot]

Ругается. На кавычку, наверное?

[Adamos]

maiskiykot, я на работе пью растворимый, гущи нет, гадать не на чем.

[AUser0]

maiskiykot, ну так новую двойную кавычку-то тоже нужно экранировать:
\"([^\"]+)\"

[Adamos]

AUser0, проще не использовать двойные кавычки для регулярки, обойтись одинарными. Сразу отпадет необходимость их экранировать.

[WbICHA]

Тогда уж \"(.*?)\"

[AUser0]

Adamos, а я привык для текста всегда использовать двойные кавычки, как раз из-за врождённого экранирования, и на всех языках.

Answer 2

[Дмитрий]

(href|title)="([^"]+)"
https://regex101.com/r/EpfgZV/1

Comments

[maiskiykot]

Мне надо и то и то захватить

[Дмитрий]

maiskiykot, дык он и title захватит и href - на ссылку то кликните

[WbICHA]

"([^"]+)"

Почему бы просто нежадный поиск не использовать?

[Дмитрий]

WbICHA, а зачем?

[WbICHA]

Дмитрий, красивее, короче, понятнее.

[Дмитрий]

WbICHA, ну честно говоря мне понятнее мой вариант - вырезаем все кроме. дело вкуса

Answer 3

[PavelMos]

.*?

href\=\"(.*?)\".*title=\"(.*)\".*breadcrumbs__item-link

https://regex101.com/r/wDkzxW/1

Answer 4

[dodo512]

Тут классика жанра "catastrophic backtracking" аж 4 раза используется .* и флаг /s.
Когда сокращаете левую часть до title убираете один .* и pcre.backtrack_limit вам хватает.
Нужно ограничить поиск с помощью [^"]* и [^>]* или убрать флаг /s.

preg_match_all('!href="([^"]*)"[^>]*title="([^"]*)"[^>]*breadcrumbs__item-link!siU',$str,$matches))

Answer 5

[maiskiykot]

На самом деле нужно было выбросить siU