Non greedy regexp eat too much

Question

i have a string

<div class="line"><input name="sid" value="" type="hidden"><img src="1.jpg" height="40" width="180"><input name="word" size="30" maxlength="50" value="" type="text"></div>

and i trying to find and replace whole input with name = word, with this regexp:

/<input .+?word.+?>/ui

preg_match returns this:

array (
  0 => '<input name="sid" value="" type="hidden"><img src="1.jpg" alt="CAPTCHA" height="40" width="180"><input name="word" size="30" maxlength="50" value="" type="text">',
)

please help me to write working regexp

Sergey Khalitov · Accepted Answer · 2017-05-29T19:14:49.180

0

Use this one: /<input[^>]+name="word".+?>/ui

In your pattern it finds first <input and then until word... If you use <input[^>]+ it will find <input and than group of simbols except closing > until name="word".

edited May 29 '17 at 19:14

answered May 29 '17 at 19:11

Sergey Khalitov

987
7
17

You still have a greedy match there. – AlVaz May 29 '17 at 19:14
Thank you very much! :-* – Aleksandr Bannov May 29 '17 at 19:14
@AleksandrBannov please check the best answer if you think it so – Sergey Khalitov May 29 '17 at 19:16

score 0 · Answer 2 · answered May 29 '17 at 19:14

0

Use this one: /<input ([^>]+)?name=['"]?word['"]?([^>]+)?>/ui

answered May 29 '17 at 19:14

AlVaz

768
1
6
21

Non greedy regexp eat too much

2 Answers2