Clase no muestra datos

[sergabrod]

Hola Claudio, si bien me parece que lo que posteó Enrique es mucho mas limpio y te sugiero que lo tomes en cuenta, es interesante detectar los errores que hacen que tu código (no muy limpio) no funcione, te marco algunas de las cosas que deberías chequear:

Código PHP:

<?php
class Indice {
        private $lista=array();
        private $titulo // te falta el punto y coma
 
 public function __construct($titulo,$lista)
  {
        $this->titulo=$titulo;
        $this->lista=$lista;
  }
 
  public function mostrar()
  {
    for($f=0;$f<count($this->lista);$f++)
    {
        echo '<h1>'$this->titulo'</h1>'; // te faltan los puntos de concatenación
        echo '<ul>';
        echo '<li>'$this->lista[$f]'</a></li>'; // te faltan los puntos de concatenación
      echo "</ul>";
    }
  }
} 
}// esta llave está demás
 
// y como ya todos te dijeron te falta el titulo como primer parámetro
 
$texto1=new Indice('Oportunidad Trabajo Independiente Herbalife');
$lista1=new Indice('!Inscríbete AHORA! para ser distribuidor de productos Herbalife.');
$lista1=new Indice('Puedes comenzar a trabajar desde que te inscribes.');
$lista1=new Indice('Tendrás flexibilidad para elegir el lugar y tiempo de trabajo');
$lista1=new Indice('Serás tu jefe y dueño de tu propio negocio.');
?>
 

Creo que no te muestra estos errores porque no tenes activado en tu php.ini que te los muestre por defecto, podrias agregarle al inicio de tu scritp un error_reporting(E_ALL | E_STRICT) para que te los muestre.

Pero supongamos que corregís todos los errores y le pasas el título a la creación del objeto:

Código PHP:

$texto1=new Indice('Mi titulo', 'Oportunidad Trabajo Independiente Herbalife');
 

Cuando haces esto le estas pasando como parámetro $lista el string 'Oportunidad Trabajo bla bla", pero si te fijas en el constructor tenes lo sgte:

Código PHP:

public function __construct($titulo,$lista)
  {
        $this->titulo=$titulo;
        $this->lista=$lista;
  }
 

Es decir le estas pasando al atributo "lista" de tu clase Indice, un string, que en php no es ni mas ni menos que un array de caracteres, por lo que cuando llames al metodo mostrar y se ejecute la línea

Código PHP:

echo '<li>' . $this->lista[$f] . '</a></li>';
 

lo que hará es imprimirte la primer letra del string o sea la 'O' de 'Oportunidad', ya que estas accediendo a un string con un subindice.
Podrías hacer dos cosas, dejarlo así y recibir un array completo en tu constructor como te lo indicó Enrique o bien cambiar el constructor para que reciba un elemento del array:

Código PHP:

public function __construct($titulo,$lista)
  {
        $this->titulo=$titulo;
        $this->lista[]=$lista; //aca todo el string pasa a ser un elemento
  }
 

y podés crear otro método :

Código PHP:

public function addItem($item)
  {
        $this->lista[]=$item; 
  }
 

para ir agregando items individuales una vez creado el objeto.

Otro punto es que por cada new que haces, creas un objeto diferente al otro:

Código PHP:

$texto1=new Indice('Oportunidad Trabajo Independiente Herbalife');
$lista1=new Indice('!Inscríbete AHORA! para ser distribuidor de productos Herbalife.');
$lista1=new Indice('Puedes comenzar a trabajar desde que te inscribes.');
$lista1=new Indice('Tendrás flexibilidad para elegir el lugar y tiempo de trabajo');
$lista1=new Indice('Serás tu jefe y dueño de tu propio negocio.');
 

Agregando el método en cuestión podrías tener:

Código PHP:

$texto = new Indice('Mi titulo', 'Oportunidad de trabajo');
$texto->addItem('!Inscríbete AHORA! para ser distribuidor de productos Herbalife.');
$texto->addItem('Puedes comenzar a trabajar desde que te inscribes.');
$texto->addItem('Tendrás flexibilidad para elegir el lugar y tiempo de trabajo');
echo $texto; // si usas __toString();
 

De esta manera te asegurás que todos los objetos Indice tengan al menos un ítem, y podés seguir agregandoles opcionalmente una vez creado el objeto.

Saludos!


Sergio Gabriel Rodriguez
Corrientes - Argentina