Clase no muestra datos

[claudio_hbl]

Hola Que tal?

Estoy empezando con POO PHP5, y encontré en la web un tuto, que estoy siguiendo y del cual he sacado este ejemplo y lo modifiqué a mis necesidades, pero no me muestra los datos en la página, que le indico, este es el código.

Código PHP:

  <?php 
class Indice { 
        private $lista=array(); 
        private $titulo 
 
 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>'; 
        echo '<ul>'; 
        echo '<li>'$this->lista[$f]'</a></li>'; 
      echo "</ul>"; 
    } 
  } 
} 
} 
 
$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.'); 
?>   

posteriormente en otra página lo llamo de la siguiente forma.

Código PHP:

  <?php 
       require('../clases/indice.class'); 
       $texto1->mostrar();   

Agradezco cualquier ayuda que me puedan brindar

Un Cordial saludo

[GatorV]

Debes de saber (uses PHP OOP o no) que PHP es un lenguaje stateless, es decir los datos no persisten entre las paginas.

Por otro lado donde tu estas o más bien lo que estas haciendo te falta el segundo parámetro a tu clase, ya que el segundo parámetro es la lista que quieres mostrar. Es por eso que no te muestra nada.

Saludos.

[chilakilez]

te falta mandar llamar la clase.

Código:

$texto1 = new Indice();
 $texto1->mostrar();

eso deberia funcionar...

[claudio_hbl]

puedo reemplazarlo por '' si no lo necesito?

Reemplace en las partes que no iba lista o titulo con '','blablabla'

pero tampoco+ los muestra

[claudio_hbl]

Iniciado por chilakilez

te falta mandar llamar la clase.

Código:

$texto1 = new Indice();
 $texto1->mostrar();

eso deberia funcionar...

no, tampoco funciona

[eulloa]

Iniciado por claudio_hbl

puedo reemplazarlo por '' si no lo necesito?

Reemplace en las partes que no iba lista o titulo con '','blablabla'

pero tampoco+ los muestra

Pasa, como ya te dijo GatorV, que no estás pasando el segundo parámetro al constructor. Esto debería lanzarte un Warning........
Pásale el segundo parámetro al __construct() o le pones en el mismo __construct el parámetro por defecto, tipo ($param1, $param2='blablabla') y ejecuta nuevamente

[enriqueplace]

Que tal Claudio

Por lo que vi del ejemplo, te recomendaría cambiar de tutorial

Te lo ajusto:

• Le cambio el nombre por algo más nemotécnico
• La clase está mal indentada
• tiene errores de sintaxis (falta un ; en el atributo, concatenas y te faltan los .), también de lógica (el título se repite tantas vaces como ítems tiene la lista, tienes llaves de más, etc.
• agrego estándar de codificación Zend
• Los objetos por defecto no deberían imprimir por pantalla, nunca tendrías el control de tu interfaz

Código PHP:

  // Archivo Lista.php

class Lista 
{
   private $_items = array();
   private $_titulo;

   public function __construct($titulo, $items)
   {
        $this->_titulo = $titulo;
        $this->_items = $items;
   }
   public function mostrar()
   {
      $html = '<h1>' . $this->_titulo . '</h1>';

      $html .= '<ul>';

      foreach ( $this->_items as $item){

         $html .= '<li>' . $item . '</li>';

      }
      $html .= '</ul>';

      return $html;
   }
   public function __toString()
   {
       return $this->mostrar();
   }
}

// Archivo index.php

require_once 'Lista.php';

$lista = new Lista('Mi título', array('item 1','item 2','item 3') );

echo $lista;  
   

A ver si este ejemplo te queda mucho más claro.

Saludos!

[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

[claudio_hbl]

Hola Enrique.

Gracias por tu respuesta, seguí tu código y funcionó perfecto, solo que queda una duda, la siguiente linea,

Código PHP:

$lista = new Lista('Mi título', array('item 1','item 2','item 3') );
 

, no puede ir en un archivo aparte, te explico:

La idea es que esa parte es como un encabezado de páginas donde el $titulo y los $items son diferente para cada una, es por ello que estoy tratando de hacerlo asi, para que en una segunda página, los pueda llamar indistintamente, como por ejemplo

Código PHP:

$lista2 = new Lista('Segundo Titulo', array('item 1','item 2','item 3')
 

.

Espero se haya entendido.

Nuevamente Gracias

Un Cordial saludo.

[enriqueplace]

Estimado, el que no está entendiendo eres tú , las clases sirven para reusarse, por lo tanto la clase se encontrará en un único lugar y tú la llamarás todas las veces que necesites, desde archivos distintos, con los valores que necesites.

El ejemplo es claro, ahora tienes que razonar y armar lo que necesitas, no solicitar que nosotros terminemos lo que necesitas hacer.

¿La idea es aprender, no?

Saludos!

[claudio_hbl]

si, esta claro, solo tenia esa duda, ya que estoy recién introduciéndome a la POO. con tu ejemplo me las voy a arreglar.
Yo uso Linux, y las pruebas las hago en local, es por ello que alomejor no muestra los errores, eso de ZEND lo puedo usar en mi pc?, me puedes indicar alguna página de algun tutorial bueno.

Gracias