preg_replace regex doesn't replace what it should

Asked May 25 '16 at 13:34

Active May 25 '16 at 13:34

Viewed 30 times

since hours I am on this problem:

// b) replace xls operation with fitting vars: E12 > var2233
preg_match_all('/[A-Z]{1,2}\d+/', $rowoperation["formulaxls"], $matches);
  $output = $rowoperation["formulaxls"];
  foreach ($matches[0] as $match) {
    $result2 = $db->query("SELECT varid FROM variablen WHERE varimportedindex = '".$match."' AND projectid = $pid");
    $rowoperation = $result2->fetch_assoc();
    $output = preg_replace("#".$match."#", "var".$rowoperation['varid']."", $output);
    $output = preg_replace('#\,#', ";", $output);
    #$output = str_replace($match, "var".$rowoperation['varid']."", $output);
    #$output = str_replace(',', ";", $output);
  }

Output of matches seem to be ok. But the replacement is not. I always got strange replacements like "Avar30031=var3424 var3612=var3424 var3613=var3424 Avar30051=var3424". So i dont't understand, why it replaces "Avar" in between. The Numbers are then also not right.

Is it possible, that it makes problem with a Excel cellindex A10 because of the "0"?

Thank you all experts...

Regards Olaf

asked May 25 '16 at 13:34

Olaf K.

Looks like you could leverage `preg_replace_callback`. – Wiktor Stribiżew May 25 '16 at 13:36
Yes i know, but this is not the problem at the moment i think. – Olaf K. May 25 '16 at 13:41
1

Try to replace `"#".$match."#"` with `"#\b".$match."\b#"`. – Casimir et Hippolyte May 25 '16 at 13:52
1

Can you provide some samples of `$output` in the first place? – Jan May 25 '16 at 13:58
WOW!!! It works with /b, but why. Thank you! – Olaf K. May 25 '16 at 14:56
1

@OlafK.: read about word-boundaries. Note that now you can use `strtr` or `str_replace` for the comma. – Casimir et Hippolyte May 25 '16 at 15:11
I have tried to repro, and it is not that easy without input text. If you think the word boundaries is all you missed, I suggest closing the question as a duplicate of [*Regex match entire words only*](http://stackoverflow.com/questions/1751301/regex-match-entire-words-only). – Wiktor Stribiżew May 27 '16 at 07:38

preg_replace regex doesn't replace what it should

0 Answers0