[SOLUCIONADO] preg_replace_callback bug?

[Fernand0]

Buenas, tengo un problema medio raro con el preg_replace_callback.. pego el codigo y el output para que lo vean...

(viendo el index.php y el output se ve que quiero lograr)

index.php

Código PHP:

<?php
    
    include 'replace.php';
    include 'replacer.php';
    
    $rplc = new Replace(
        '@\.([a-z_]+)@i',
        '["$1"]',
        '$color.name.lala'
    );
    
    echo "<br>replaced: " . $rplc->getReplaced();
    
    $rplcr = new Replacer(
        '@\.([a-z_]+)@i',
        '["$1"]',
        '$color.name.lala'
    );
    
    echo "<br>replacedR: " . $rplcr->getReplaced();
    
?>
 

replace.php

Código PHP:

<?php
        
        class Replace
        {
            
            private $pattern = null;
            
            private $replace = null;
            
            private $subject = null;
            
            private $capture = '@\$([0-9]{1,2})@i';
            
            private $perform = 'self::callback';
            
            
            public function __construct($pattern, $replace, $subject)
            {
                
                $this->pattern = $pattern;
                $this->replace = $replace;
                $this->subject = $subject;
                
                if(!$this->is_match())
                    return false;
                
                preg_replace_callback(
                    $this->pattern,
                    $this->perform,
                    $this->subject
                );
                
            }
            
            private function callback($matches)
            {
                
                $replaced = null;
                
                $callback = function($groups) use ($matches, &$replaced) {
                    
                    $match = $matches[$groups[1]];
                    
                    $replaced = str_replace(
                        $groups[0],
                        $match,
                        $this->replace
                    );
                    
                };
                
                preg_replace_callback(
                    $this->capture,
                    $callback,
                    $this->replace
                );
                
                $this->subject = str_replace(
                    $matches[0],
                    $replaced,
                    $this->subject
                );
                
            }
            
            private function is_match()
            {
                
                $count = preg_match(
                    $this->pattern,
                    $this->subject
                );
                
                return ($count !== 0)? true : false;
                
            }
            
            public function getReplaced()
            {
                
                return $this->subject;
                
            }
            
        }
    
?>
 

replacer.php

Código PHP:

<?php
    
        class Replacer
        {
            
            private $pattern = null;
            
            private $replace = null;
            
            private $subject = null;
            
            private $capture = '@\$([0-9]{1,2})@i';
            
            private $perform = 'self::callback';
            
            
            public function __construct($pattern, $replace, $subject)
            {
                
                $this->pattern = $pattern;
                $this->replace = $replace;
                $this->subject = $subject;
                
                if(!$this->is_match())
                    return false;
                
                $this->subject = preg_replace_callback(
                    $this->pattern,
                    $this->perform,
                    $this->subject
                );
                
            }
            
            private function callback($matches)
            {
                
                $callback = function($groups) use ($matches) {
                    
                    $match = $matches[$groups[1]];
                    
                    return str_replace(
                        $groups[0],
                        $match,
                        $this->replace
                    );
                    
                };
                
                $replaced = preg_replace_callback(
                    $this->capture,
                    $callback,
                    $this->replace
                );
                
                return str_replace(
                    $matches[0],
                    $replaced,
                    $this->subject
                );
                
            }
            
            private function is_match()
            {
                
                $count = preg_match(
                    $this->pattern,
                    $this->subject
                );
                
                return ($count !== 0)? true : false;
                
            }
            
            public function getReplaced()
            {
                
                return $this->subject;
                
            }
            
        }
    
?>
 

output:

Código HTML:

replaced: $color["name"]["lala"]
replacedr: $color$color["["name"]"].lala$color.name["["lala"]"]

Por alguna razon, el codigo correcto deberia ser el de "replacer.php", pero como ven en el output, devuelve cualquier cosa..

Tuve que dejar de usar el return en el preg_replace_callback.. y directamente en el callback asignarle el valor a parseado a una variable.

Si me dan una ayuda se los agradeceria

[pateketrueke]

¿Podrías simplificar todo tu código y sólo usar preg_replace_callback()?

Porque uno se pierde tratando de entender si el problema es tu diseño de OOP o cualquier otra cosa ahí.

[Fernand0]

A ver si esto ayuda...

Código PHP:

$subject = '$color.name';
    
    $callback = function($matches) use ($subject) {
    
        $callback2 = function($groups) use ($matches) {
            
            $match = $matches[$groups[1]];  // $groups[1] devuelve el '1' de '$1'
                                            // por tanto seria $matches[1]
                                            // equivalente a 'name' (sin el punto)
            
            return str_replace( // duevuelvo '["name"]' para $replaced
                $groups[0], // esto tomaria cada uno de los $1 $2 encontrados..
                $match, // seria 'name'
                '["$1"]' // se reemplaza '$1' por 'name'
            );
            
        };
        
        $replaced = preg_replace_callback( // consigo '["name"]'
            '@\$([0-9]{1,2})@i', // busca los $1 $2 $3 etc.. que uno ponga
            $callback2, // llama a $callback2
            '["$1"]' // $1 deberia ser remplazado por 'name'
        );
        
        return str_replace( // finalmente se reemplaza todo
            $matches[0],  // esto es ".name"
            $replaced, // esto es '["name"]'
            $subject // esto es '$color.name'
        );
        
    };
    
    $subject = preg_replace_callback(
        '@\.([a-z_]+)@i', // busca los ".algo" en este caso ".name"
        $callback, // llama a $callback
        $subject // string = '$color.name'
    );
    
    echo "<br>handed replaced: " . $subject;
 

output

Código HTML:

handed replaced: $color$color["["name"]"]
 

[pateketrueke]

¿Podrías explicar exactamente que buscas obtener?

[Fernand0]

Pense que se entendia.

La idea es encontrar un patron ('@\.([a-z_]+)@i') en una cadena ($color.name) y reemplazarlo por otra cadena (["..grupo 1 del patron encontrado.."] que se leeria como ["$1"]).

Entonces encuentro ".name" en "$color.name"... y reemplazo ".name" por ["name"]... el resultado esperado seria: $color["name"]... sin embargo me devuelve: $color$color["["name"]"]

No se si se entiende ahora, su funcionamiento seria parecido al de un RewriteRule de .htaccess.. que se puede utilizar el $1.. $2.. $3... para referirse a los grupos del regex

[pateketrueke]

Las capturas $1 $2 y así ya las soporta PCRE por defecto (\1 \2 etc.):

Código PHP:

$regex = '@\.([a-z_]+)@i';
$texto = '$color.name';
 
echo preg_replace($regex, '["\1"]', $texto); // $color["name"]
 

¿Necesitas algo más complejo?

Digo, sigo sin entender para que tanto código.

[Fernand0]

what a fail..

gracias, no sabia que soportaba capturing en el replacement con el preg_replace

Igual.. no se porque el codigo cuando por ejemplo, ya consigo ["name"].. hago un echo, y el echo devuelve ["name"]... ahora cuando uso un return desde el callback, despues echo preg_replace_callback... me devuelve ["["name"]"]...
No se cual es el error del codigo la verdad.. pero bueno, me facilitaste la vida, gracias :P

Saludos

[pateketrueke]

no se porque el codigo cuando por ejemplo, ya consigo ["name"].. hago un echo, y el echo devuelve ["name"]... ahora cuando uso un return desde el callback, despues echo preg_replace_callback... me devuelve ["["name"]"]

Eso se debe a lo siguiente:

preg_replace_callback() usa una función callback para el reemplazo, lo que sea que devuelvas se usará como reemplazo.

Tu error era que dentro del callback hacías tus propios reemplazos, y desde luego eso provocaba los reemplazos duplicados o mal formados, etc.

[Fernand0]

Si, pero no, algo raro pasa, por eso puse el codigo..

Si pongo al final

Código PHP:

preg_replace_callback(
        '@\.([a-z_]+)@i',
        $callback,
        $subject
    );
 

en vez de esto que venia poniendo

Código PHP:

$subject = preg_replace_callback(
        '@\.([a-z_]+)@i',
        $callback,
        $subject
    );
 

$subject no cambia.. sigue siendo

Código HTML:

$color.name
 

Por alguna razon no me hace el reemplazo.

[pateketrueke]

Pues no, nada raro, sencillamente hacías cosas de más, trucos chinos y no se que tantos hacks para intentar darle la vuelta a algo muy simple.

Básicamente estabas usando código sin razón ni beneficio alguno:

Código PHP:

<?php
 
$subject = '$color.name';
 
// use($subject) es innecesario
 
$callback = function($matches) {
 
  /* todo esto es innecesario...
 
    $callback2 = function($groups) use ($matches) {
 
        $match = $matches[$groups[1]];  // $groups[1] devuelve el '1' de '$1'
                                        // por tanto seria $matches[1]
                                        // equivalente a 'name' (sin el punto)
 
        return str_replace( // duevuelvo '["name"]' para $replaced
            $groups[0], // esto tomaria cada uno de los $1 $2 encontrados..
            $match, // seria 'name'
            '["$1"]' // se reemplaza '$1' por 'name'
        );
 
    };
 
    $replaced = preg_replace_callback( // consigo '["name"]'
        '@\$([0-9]{1,2})@i', // busca los $1 $2 $3 etc.. que uno ponga
        $callback2, // llama a $callback2
        '["$1"]' // $1 deberia ser remplazado por 'name'
    );*/
 
    // esto esta mal, tu no debes hacer ningun reemplazo,
    // debes dejar que preg_replace_callback haga su trabajo...
 
  /*
    return str_replace( // finalmente se reemplaza todo
        $matches[0],  // esto es ".name"
        $replaced, // esto es '["name"]'
        $subject // esto es '$color.name'
    );
    */
 
  // $matches[1] es "name"
  return '["' . $matches[1] . '"]';
};
 
$subject = preg_replace_callback(
    '@\.([a-z_]+)@i', // busca los ".algo" en este caso ".name"
    $callback, // llama a $callback
    $subject // string = '$color.name'
);
 
echo "<br>handed replaced: " . $subject;
 

Lee bien tu código y verás que siempre hiciste mal todo.

Y todo eso ocurre por no leer el manual.

[Fernand0]

Ahorrate las opiniones y suposiciones para quien te las pida...