views:

154

answers:

1

mmmh guys, i really hope my english is good enaught to explain what i need.

Lets take this example (that is just an example!) of code:

class Something(){
    public function Lower($string){
        return strtolower($string);
    }
}
class Foo{
    public $something;
    public $reg;
    public $string;
    public function __construct($reg, $string, $something){
        $this->something = $something;
        $this->reg = $reg;
        $this->string = $string;
    }
    public function Replace(){
        return preg_replace_callback($this->reg, 'Foo::Bar', $this->string);
    }
    public static function Bar($matches){
        /*
        * [...]
        * do something with $matches and create the $output variable
        * [...]
        */

        /*
        * I know is really useless in this example, but i need to have an istance to an object here
        * (in this example, the Something object, but can be something else!)
        */
        return $this->something->Lower($output);
    }
}
$s = new Something();
$foo = new Foo($myregexp, $mystring, $s);
$content = $foo->Replace();

So, the php manual say that to use a class method as callback in preg_replace_callback(), the method must be abstract.

I need to pass an instance of a previuosly initialized object (in the example, an instance of the Something class) at the callback function.

I tryed to use call_user_func(), but doesnt work (becose in this way i miss the matches parameter).

Is there a way to do that, or have i to separate the process (doing before preg_match_all, for each match retrieve the replace value, and then a simple preg_replace)?

edit: as a side-note, before the tom haigh answer, i used this work-around (in the example, this is the Replace method):

$has_dynamic = preg_match_all($this->reg, $this->string, $dynamic);
if($has_dynamic){
    /*
    * The 'usefull' subset of my regexp is the third, so $dynamic[2]
    */
    foreach($dynamic[2] AS $key => $value){
        $dynamic['replaces'][$key] = $this->Bar($value);
    }
    /*
    * ..but i need to replace the complete subset, so $dynamic[0]
    */
    return str_replace($dynamic[0], $dynamic['replaces'], $this->string);
}else{
    return $this->string;
}

Hope can help someone.

+1  A: 

It is hard to pass arguments to callbacks, but instead of this:

return preg_replace_callback($this->reg, 'Foo::Bar', $this->string);

You could make Bar() not static, and use this:

return preg_replace_callback($this->reg, array($this, 'Bar'), $this->string);

Then the callback function will be able to see $this

See 'callback' in Pseudo-types and variables

Also in PHP >=5.3 you could use anonymous functions/closures to pass other data to callback functions.

Tom Haigh
This works! I thought that the class method have to be static, dont remember where i did read it.. maybe i didnt at all and I messy with some sentences.Thanks
DaNieL