0

I'm trying to run a function whenever there's a [%xxx%] (acting as a placeholder, if you will), e.g.:

Bla bla bla blabla. Blablabla bla bla.
[%hooray%]
Blabla hey bla bla. [%yay%] blabla bla.

I'm pretty much a PHP beginner, but I managed to crack my head through and come up with the following (yay to me - I somehow managed to understand the basics of regular expressions!):

$maintext = preg_replace("#\[%(.{1,20})%\]#", "display_products('$1')"), $maintext);
echo $maintext; 

I tried working with the e modifier, I tried using preg_replace_callback, and it all does somewhat work - but I don't need the function to run before I echo the $maintext variable.

Any help on this guys?

edit The regex isn't the issue - that's working fine. I just need to figure out how to run the function only when I echo $maintext... preg_replace_callback runs the function immediately...

4
  • Try the %s and %d/%f modifier Commented Jul 18, 2012 at 8:29
  • Ugh, you should really stay away from /e but use preg_replace_callback instead. Commented Jul 18, 2012 at 8:29
  • @qaisjp how do I do that? I've got no issues with the regex - that's working fine. I just need to figure out how to run the function only when I echo $maintext... Commented Jul 18, 2012 at 8:35
  • yep. definitely don't use the /e modifier. It's considered very bad practice, and will be formally deprecated in the next version of PHP. Commented Jul 18, 2012 at 11:41

3 Answers 3

0

Because PHP is a single threaded procedural scripting language1, it doesn't work in the way you want it to.

It is not possible to trigger the function to run only when you call echo2, what you need instead is an extra variable.

$newVariable = preg_replace_callback("#\[%(.{1,20})%\]#", function($matches) {
  // Do your thang here
}, $maintext);

// $maintext remains the same as it was when you started, do stuff with it here

// When you come to output the data, do...
echo $newVariable;

There are other approaches to this problem, like wrapping the above code in a function that can be called on demand and using the output buffering callback (see footnote #2), but reading between the lines I think all of this would be overkill for what is a relatively simple problem.

For the record, I think your regex would be better if you narrow the allowed content down a it, it will potentially match characters you don't want it to. I suspect that this would be better:

#\[%(\w{1,20})%\]#

This will allow only the characters a-zA-Z0-9_ in the placeholder.


1 Others may tell you PHP is now an object oriented language, but they are wrong. It still fundamentally works the same underneath and its still a scripting language, but now provides many OO-features.
2 It is possible to do something very close to this, but it is very rarely ever a good idea and is far too advanced for dealing with this problem.


EDIT

It is important to note that when using either method to pass replacements through a callback (e modifier or preg_replace_callback()) the callback function should return the new string, rather than outputting it directly.

This is because the code in any statement is executed from the inside out, and the "inner" echo statement (in the callback function) will be executed before the "outer" echo statement (on the line where preg_replace() is called, so the order in which things are output will be incorrect.

For example:

$str = "This is my string which contains a %placeholder%";

echo preg_replace_callback('/%(\w+)%/', function($matches) {
  echo "replacement string";
}, $str);
// Wrong - outputs "replacement stringThis is my string which contains a "

echo preg_replace_callback('/%(\w+)%/', function($matches) {
  return "replacement string";
}, $str);
// Right - outputs "This is my string which contains a replacement string"
Sign up to request clarification or add additional context in comments.

12 Comments

thanks @DaveRandom - I gave your code a try but I'm getting this error: Catchable fatal error: Object of class Closure could not be converted to string in XXX (on the same line as the preg_replace function)...
@AKG My bad, I left it as preg_replace instead of preg_replace_callback. Code above updated, try it again...
@DaveRandom I#d like to know about this "possible to do something very close". Is there any way to get in contact? Or can you provide a link to that? It sounds very interesting and I've never heard of that before!
@Mohammer If you pass a callback to the first argument of ob_start() and a very low integer (minimum sensible value is 2) to the second argument, the data will be passed through the callback function every time the length buffer exceeds the second argument (in bytes). Since you will rarely if ever output less than 2 bytes at a time, you are effectively calling the callback every time you output anything.
@DaveRandom - thanks for your help, but I can't seem to wrap my head around this. Whenever I use preg_replace_callback or preg_replace with e modifier, it will run immediately - meaning that I don't have the function run at the place I want it. This is the pattern essentially: Text | Function | More text | Function
|
0

If you have a script that uses echo many times and you want the printed text modified, you can use output buffering:

ob_start();
/*
 * Lots of php code
*/

$maintext = ob_get_clean(); // now you capture the content in $maintext 

// than you modify the output
$maintext = preg_replace("#\[%(.{1,20})%\]#", "display_products($1)", $maintext);
echo $maintext;

Comments

0

ob_start accepts a callback function, you can read more on this topic : Making all PHP file output pass through a "filter file" before being displayed

Comments

Your Answer

By clicking “Post Your Answer”, you agree to our terms of service and acknowledge you have read our privacy policy.

Start asking to get answers

Find the answer to your question by asking.

Ask question

Explore related questions

See similar questions with these tags.