PHP OO Duda sobre POO en PHP

[capixi]

Estoy creando una clase para listar los contactos de mi agenda, necesito listar los contactos automaticamente y no llamandolos uno por uno puesto que para listarlos tengo un botón de un formulario que lo hará.
Para listar los contactos automaticamente se me ha ocurrido hacer un bucle for y dentro de este generar la llamada al método listar_contactos();

Al ejecutar el código el navegador me muestra:
$mi_agenda1 -> listar_contactos();
$mi_agenda2 -> listar_contactos();
$mi_agenda3 -> listar_contactos();
$mi_agenda4 -> listar_contactos();

Esto es lo que contiene la variable $llamar, pero al invocar esa llamada no me muestra los contactos.
¿Alguien me podria explicar por que sucede esto y como podria solucionarlo?

¡Gracias por adelantado!

Código PHP:

  class agenda{
     function inicializar($nom, $apel, $tel)
     {
      $this->nombre=$nom;
      $this->apellidos=$apel;
      $this->telefono=$tel;
     }
     
     function listar_contactos(){
           echo $this->nombre;
           echo '<br>';
           echo $this->apellidos;
           echo '<br>';
           echo $this->telefono;
           echo '<br>';
     }
}//fin class
$mi_agenda1 = new agenda();
$mi_agenda1 -> inicializar('Ana','Maria Sellés','966854213');
$mi_agenda2 = new agenda();
$mi_agenda2 -> inicializar('Jose','Lorenzo','966141213');
$mi_agenda3 = new agenda();
$mi_agenda3 -> inicializar('Mariano','Moreno','961021213');
$mi_agenda4 = new agenda();
$mi_agenda4 -> inicializar('Beatriz','Salamanca','960001213');
 
for ($i=1; $i<=4; $i++)
{
  settype($llamar,"object");
  $llamar="\$mi_agenda".$i." -> "."listar_contactos();";
  echo "<br>";
  echo $llamar; //para comprobar el contenido
  $llamar;//deberia listar contactos pero no hace nada :(
} 
   

[ocp001a]

Evidentemente no te va a mostrar los contactos, pues nunca los mandas a llamar.
Lo que hacer es imprimir una cadena de texto de lo que podría ser una llamada.
Si bien no me parece en absoluto correcto, si cambias el
echo $llamar;
por
eval($llamar);
funcionaría.
No obstante lo adecuado es que tu clase guarde los contactos en un arreglo y los administre conforme agregues nuevos contactos.

Yo me he visto en la necesidad de hacer llamadas eval pero en clases con recursión y uso de librerías excel, pero lo recomendable es evitar en lo posible el uso de eval.

[_Ruben_]

Veo errores de diseño en la clase.

Tienes una clase agenda y para cada contacto crear una agenda y le agregas el contacto, entonces tienes 4 agendas con 1 contacto cada agenda. Debes tener 1 agenda con 4 contactos.

El método listar_contactos() debe listar los 4 contactos de 1 agenda (que es como debes tenerlo).

[capixi]

Gracias por contestarme tan rápido, lo que me aconsejais me ha resultado muy adecuado y efectivamente con eva()l funciona el código pero reconozco que no es el metodo adecuado para lo que quiero.
Estoy empezando con la POO y no tengo mucha idea aun, me podriais guiar sobre como declarar un array con los datos de contacto dentro de una clase?cómo quedaria el constructor?

[mogurbon]

asi se haria lo que querias en un inicio

Código PHP:

  for ($i=1; $i<=4; $i++)
{
 
  ${mi_agenda.$i}->listar_contactos();
  
} 
   

[_Ruben_]

La clase debería ser así (es recomendable tener una segunda clase):

Código PHP:

  class contacto {
     private $nombre;
     private $apellido;
     private $telefono;
 
    function __construct($nombre, $apellido, $telefono)
    {
      $this->nombre = $nombre;
      $this->apellido = $apellido;
      $this->telefono = $telefono;
    }
 
   function mostrar()
   {
     echo $this->nombre." - ".$this->apellido." - ".$this->telefono;
   }
}
 
class agenda{
     private $contactos;
 
     function __construct()
     {
       $this->contactos = array();
     }
 
     function add($nombre, $apellido, $telefono) 
     {
       $this->contactos[count($this->contactos)] = new contacto($nombre, $apellido, $telefono);
     } 
      
     function listar_contactos()
     { 
       foreach($contacto as $this->contactos)
        echo $contacto->mostrar()."<br>";
     } 
} 
   

Y para construirlo todo sería:

Código PHP:

  $mi_agenda = new agenda(); 
$mi_agenda->add('Ana','Maria Sellés','966854213');
$mi_agenda->add('Jose','Lorenzo','966141213');
$mi_agenda->add('Mariano','Moreno','961021213');
$mi_agenda->add('Beatriz','Salamanca','960001213');
$mi_agenda->listar_contactos(); 
   

Si no me he equivocado en algo.

[Vitesse92]

el codigo de _Ruben_ esta correcto :) y demuestra una buena clase pero en la programacion orientada a objetos, creo, un objeto no imprime los datos de manera directa, en todo caso se relaciona con otro objeto encargado de ello y o se devuelve la variable a imprimir :)
pero en fin, no soy un experto y no se si lo que digo este correcto para este caso.
tampoco hay que marear al chico :p

[_Ruben_]

Iniciado por Vitesse92

el codigo de _Ruben_ esta correcto :) y demuestra una buena clase pero en la programacion orientada a objetos, creo, un objeto no imprime los datos de manera directa, en todo caso se relaciona con otro objeto encargado de ello y o se devuelve la variable a imprimir :)
pero en fin, no soy un experto y no se si lo que digo este correcto para este caso.
tampoco hay que marear al chico :p

Efectivamente, en concreto la clase contacto debería tener 3 métodos para obtener el valor de cada uno de los atributos. Y la clase agenda debería devolver una lista de contactos.

También habría que comprobar que los valores de nombre, apellido y teléfono son lógicos, es decir, el nombre y apellido solo tienen letras, y el teléfono es un número de teléfono que pueda existir. En esas comprobaciones lanzar excepciones si no se cumplen, y también proporcionar métodos que puedan comprobar esos valores (para verificarlos antes de la llamada y que no salte la excepción).

Pero todo eso ya forma parte de una solución más larga que considero que corresponde a capixi implementar si realmente lo quiere hacer orientado a objetos.