PHP preg_match from string

Question

I have this string :

$content = 'Hello <!--blapi[getinfoprix("prix_p1"=>"1-6048df6;image/purchase-small.png"]--> Hello<!--blapi[prix_p1->description]-->';

How can i get the two string  in an array[2]?

I've made this :

$pattern = '/<!--blapi\[(.*)\]-->/sU';
preg_match($pattern, $content, $matches);

But I have this result :

array(2) {
  [0]=>
  string(74) "<!--blapi[getinfoprix("prix_p1"=>"1-6048df6;image/purchase-small.png")]-->"
  [1]=>
  string(60) "getinfoprix("prix_p1"=>"1-6048df6;image/purchase-small.png")"
}

I don't understand why it's ignoring the second string ...

I've used the flag "U". Maybe there is a better pattern for what I want to do?

EDITION : I expect this result :

Array
(
    [0] => getinfoprix("prix_p1"=>"1-6048df6;image/purchase-small.png"]
    [1] => prix_p1->description
)

anubhava · Accepted Answer · 2014-02-22 13:26:09Z

2

This preg_match_all should work:

$content = 'Hello <!--blapi[getinfoprix("prix_p1"=>"1-6048df6;image/purchase-small.png"]--> Hello<!--blapi[prix_p1->description]-->';
if ( preg_match_all('/<!--.*?\[(.*?)\]-->/', $content, $matches) )
   print_r($matches[0]);

OUTPUT

Array
(
    [0] => getinfoprix("prix_p1"=>"1-6048df6;image/purchase-small.png"
    [1] => prix_p1->description
)

edited Feb 22, 2014 at 13:26

answered Feb 22, 2014 at 12:54

anubhava

790k67 gold badges603 silver badges671 bronze badges

Sign up to request clarification or add additional context in comments.

2 Comments

Portekoi Over a year ago

thank's but look my edit please. Only need this kind of string :

anubhava Over a year ago

Your edit is still unclear on what output are you expecting. Can you show that in your questio.

CodeAngry · Accepted Answer · 2014-02-22 13:54:02Z

1

$pattern = '~<!--(blapi\[(?:.*?)\])-->~si';

Does this pattern produce the expected results? I understand you want to capture the blapi part too. But not sure...

Changed .*U to .*? and added i for case-insensitive at the end. The inner blapi is a non-capture group and the blapi[...] is now the capture group.

Also avoid wrapping a regex in / as it conflicts with URLs and HTML. Use ~ as it's seldom used and much safer. It's not nice to escape http:// to http:\/\/ just because of the wrap character.

You also need preg_match_all as preg_match capture only one match. It's mostly used for match testing, single-match search but not multiple match search.

edited Feb 22, 2014 at 13:54

answered Feb 22, 2014 at 13:34

CodeAngry

13.1k3 gold badges54 silver badges65 bronze badges

Collectives™ on Stack Overflow

PHP preg_match from string

2 Answers 2

OUTPUT

2 Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

OUTPUT

2 Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related