Pierdo variables usando una clase dentro de otra

[calamarjoe]

Buenos días,

Estoy empollándome esto de las clases en PHP y por lo visto se me ha escapado algo. Tengo dos clases, una con una función que me genera una cadena de texto y otra clase con una función que busca una cadena de texto en un texto más grande y la sustituye por la cadena generada por la primera clase.

El problema está en que todo funciona correctamente salvo porque las variables por las que está formada la primera cadena no aparecen. Se quedan en blanco.

El archivo es el siguiente:

Código PHP:

  <?php
 
class Flickr {
 
    private $apiKey = 'c933e0d66a6695d005d660392bc496f2';
 
    public function img($id)
    {
        $url = 'http://www.flickr.com/services/rest/?api_key=' . $this->apiKey . '&format=php_serial&method=flickr.photos.getInfo&photo_id=' . $id;
 
        $resultado = file_get_contents($url);
        $resultado = unserialize($resultado);
        $foto = $resultado[photo];
 
        $foto_url = 'http://farm' . $foto[farm] . '.static.flickr.com/' . $foto[server] . '/' . $foto[id] . '_' . $foto[secret] . $tam . '.jpg';
 
        $foto = '<a href="' . $foto[urls][url][0][_content] . '"><img src="' . $foto_url . '" alt="' . $foto[title][_content] . '" /></a>';
 
        return $foto;
    }
 
}
 
$flickr = new Flickr();
 
 
 
class BBcode {
 
    function __construct(&$flickr)
    {
        $this->flickr =& $flickr; 
    }
 
    function procesar($texto){
 
        $a = "/\[img\](.*?)\[\/img\]/is";
 
        $b = "$1";
        $b = $this->flickr->img($b);
 
        $texto = preg_replace($a, $b, $texto);
 
        return $texto;
    }
}
 
$texto  = 'Imagen 1:<br />';
$texto .= '[img]4398563145[/img]<br />';
$texto .= 'Imagen 2:<br />';
$texto .= '[img]4334520973[/img]';
 
echo '<br /><br />';
 
$bbcode = new BBcode($flickr);
 
echo $bbcode->procesar($texto);
 
?>   

Y esto me devuelve lo siguiente:

Código HTML:

Imagen 1:<br /><a href=""><img src="http://farm.static.flickr.com//_.jpg" alt="" /></a><br />Imagen 2:<br /><a href=""><img src="http://farm.static.flickr.com//_.jpg" alt="" /></a> 

A ver si alguien sabe qué es lo que he hecho mal y puede echarme un cable.

Un saludo.

[GatorV]

Lo que pasa es que en lugar de usar preg_replace, debes de usar preg_replace_callback para que se evalue el texto y extraer las ids y procesarlas con tu primera clase.

Saludos.

[calamarjoe]

Gracias por la respuesta tan rápida.

Mmmm... estoy mirando y no consigo acertar a usar preg_replace_callback(). No entiendo exactamente como debería de usarla aquí.

[GatorV]

Tienes que usarlo con tu segundo parámetro para pasarle un objeto valido del tipo callback donde llames a la función img de tu clase Flickr y con eso se pasa el id que hace match en tu patrón.

Saludos.

[calamarjoe]

Buf, no me sale. ¿Podrías poner un ejemplo o algo?

Creo que me iré a dormir un rato a ver si mañana me inspiro.

[GatorV]

En este post hay un ejemplo: http://www.forosdelweb.com/f18/preg_...ulares-797729/

[calamarjoe]

No se si he acabado de entenderlo. La teoría la entiendo, pero a la hora de implementarlo no sé cómo debería ser exactamente.

Lo he dejado de la siguiente manera:

Código PHP:

  class BBcode {

    function __construct(&$flickr)
    {
        $this->flickr =& $flickr;
    }

    function procesar($texto)
    {
        global $flickr;

        $a = "/\[img\](.*?)\[\/img\]/is";

        //$b = $this->flickr->img("$b");

        $texto = preg_replace_callback($a, array($this->flickr, 'img("$b")'), $texto);

        return $texto;
    }
} 
   

Me devuelve el mismo texto que tengo, sin sustituír las cadenas de texto :(

A parte del link que me dejaste miré [URL="http://www.forosdelweb.com/f68/preg_replace_callback-clase-718725/"]este post[/URL] y revisé el manual de PHP pero no consigo aclararme.

[GatorV]

El segundo parámetro es solamente la función que quieres llamar de tu clase, sin parámetros ni nada, revisa en el manual como es el tipo callback.

En el enlace que te deje viene muy bien puesto como es que tienes que llamar a la función si lo quieres hacer con una instancia de la clase.

Saludos.

[calamarjoe]

Claro, pero tengo que pasar parámetros, sino no tiene sentido llamar a la función.
Para que me genere la imagen tengo que pasarle el identificador. El problema es el cómo paso el identificador.

[pateketrueke]

Iniciado por calamarjoe

Claro, pero tengo que pasar parámetros, sino no tiene sentido llamar a la función.
Para que me genere la imagen tengo que pasarle el identificador. El problema es el cómo paso el identificador.

cual identificador??

recuerda que la función calllback se ejecuta en determinado contexto, si usas como función una llamada a algún método de tu mismo objeto.... pues.... es el mismo objeto!!

no necesitas pasar nada... lo único que se pasa a dicha función es un arreglo con los matches de la expresión... y solo eso...

por favor, consulta el manual
http://php.net/preg_replace_callback

[GatorV]

Exacto si lees el manual puedes ver que preg_replace_callback automaticamente envia como primer parámetro a tu función callback el array con los matches.

Pero NO es necesario definirlo como parte del callback, ya que el tipo Callback es un tipo especial de datos de PHP y solo es necesario definir la función.

Es decir, tu pones que llame a array($foo, 'bar') PHP va a evaluar y va a hacer esto $foo->bar($matches);

Saludos.

[calamarjoe]

Claro, pero en la primera clase (Flickr) cuando llamas a la función img necesitas pasarle un $id para que te devuelva el resultado.

En la segunda clase (BBcode), en la función procesar, primero busca el patrón (que es lo que me estais diciendo y eso ya funciona) y luego llama al primer objeto para hacer uso de su función img, la cual necesita un $id, que es la cadena que obtiene al pasar el patrón. Por tanto, si no paso ese $id la función img no puede funcionar.

Estamos hablando de cosas diferentes, creo.

[GatorV]

No, lo que pasa es que tu tienes que pasarle el $id que haces del match del patrón, es decir tu patron hace matches de [img][/img] extrae el id y quieres que lo pase a tu clase Flickr.

Para que eso funcione tienes es FORZOSO que uses preg_replace_callback(), para que se llame por cada reemplazo a tu funcion y regrese el resultado.

Saludos.

[calamarjoe]

Finalmente me ayudó un colega a solucionarlo. En ningún momento usó la función preg_replace_callback() y funciona a la perfección.

Así ha quedado:

Código PHP:

  <?php 
 
class Flickr { 
 
    private $apiKey = 'c933e0d66a6695d005d660392bc496f2'; 
 
    public function img($id) 
    { 
        $url = 'http://www.flickr.com/services/rest/?api_key=' . $this->apiKey . '&format=php_serial&method=flickr.photos.getInfo&photo_id=' . $id; 
 
        $resultado = file_get_contents($url); 
        $resultado = unserialize($resultado); 
        $foto = $resultado[photo]; 
 
        $foto_url = 'http://farm' . $foto[farm] . '.static.flickr.com/' . $foto[server] . '/' . $foto[id] . '_' . $foto[secret] . $tam . '.jpg'; 
 
        $foto = '<a href="' . $foto[urls][url][0][_content] . '"><img src="' . $foto_url . '" alt="' . $foto[title][_content] . '" /></a>'; 
 
        return $foto; 
    } 
 
} 
 
$flickr = new Flickr(); 
 
 
 
class BBcode { 
 
    function __construct(&$flickr) 
    { 
        $this->flickr =& $flickr;  
    } 
 
    function procesar($texto){ 
 
        $a = "/\[img\](.*?)\[\/img\]/is";  
 
        preg_match_all($a, $texto, $matches);
        
        for($i = 0; $i < count($matches[0]); $i++) { // matches[0] -> la línea completa, matches[1] -> el grupo capturado
            $real_deal = $this->flickr->img($matches[1][$i]);
            $texto = str_replace($matches[0][$i], $real_deal, $texto); // replaceamos y listo
        }
        
        return $texto; 
    } 
} 
 
$texto  = 'Imagen 1:<br />'; 
$texto .= '[img]4398563145[/img]<br />'; 
$texto .= 'Imagen 2:<br />'; 
$texto .= '[img]4334520973[/img]'; 
 
$bbcode = new BBcode($flickr); 
 
echo $bbcode->procesar($texto); 
 
?>   

[GatorV]

Te hubiera quedado más limpio usando preg_replace_callback pero bueno...

[calamarjoe]

Iniciado por GatorV

Te hubiera quedado más limpio usando preg_replace_callback pero bueno...

Es posible, pero aún no se cómo hacerlo. Por mucho que me digais que lea el manual de PHP (cosa que he hecho 20 veces) no consigo entenderlo.

De verdad aprecio vuestra ayuda, pero a falta de que alguien me ayude con preg_replace_callback() con lo explicado arriba lo dejaré así, que al menos funciona.

[GatorV]

Código PHP:

<?php  
 
class Flickr {  
 
    private $apiKey = 'c933e0d66a6695d005d660392bc496f2';  
 
    public function img($id)  
    {  
        $url = 'http://www.flickr.com/services/rest/?api_key=' . $this->apiKey . '&format=php_serial&method=flickr.photos.getInfo&photo_id=' . $id;  
 
        $resultado = file_get_contents($url);  
        $resultado = unserialize($resultado);  
        $foto = $resultado[photo];  
 
        $foto_url = 'http://farm' . $foto[farm] . '.static.flickr.com/' . $foto[server] . '/' . $foto[id] . '_' . $foto[secret] . $tam . '.jpg';  
 
        $foto = '<a href="' . $foto[urls][url][0][_content] . '"><img src="' . $foto_url . '" alt="' . $foto[title][_content] . '" /></a>';  
 
        return $foto;  
    }  
 
}  
 
$flickr = new Flickr();  
 
 
 
class BBcode {  
 
    function __construct(&$flickr)  
    {  
        $this->flickr =& $flickr;   
    }  
 
    function procesar($texto){  
 
        $a = "/\[img\](.*?)\[\/img\]/is";   
 
        $texto = preg_replace_callback($a, array($this, 'process_img'), $matches); 
         
        return $texto;  
    }
    
    public function process_img($matches)
    {
        $img_id = $matches[1];
        $this->flickr->img($img_id);
    }
}  
 
$texto  = 'Imagen 1:<br />';  
$texto .= '[img]4398563145[/img]<br />';  
$texto .= 'Imagen 2:<br />';  
$texto .= '[img]4334520973[/img]';  
 
$bbcode = new BBcode($flickr);  
 
echo $bbcode->procesar($texto);
 

[calamarjoe]

No me funciona :S

[GatorV]

Creo el indice es $matches[0] prueba cambiarlo

[calamarjoe]

No es eso, ya lo había probado. Al igual que cambiar el $matches de

Código PHP:

  $texto = preg_replace_callback($a, array($this, 'process_img'), $matches); 
   

por $texto.

[GatorV]

Perdon si es $matches[0] en el callback y en la linea es: $texto = preg_replace_callback($a, array($this, 'process_img'), $texto);

En la firma de preg_replace_callback es:

Código:

preg_replace_callback($pattern, $callback, $subject)