Error imprimiendo datos desde una clase con objetos

[kahlito]

Hola.

Estoy intentando hacer un ejercicio de clases, funciones y objetos, pero no termino de entender por que en el primer caso no me imprimen los datos que llamo si no pongo los "echo" dentro de la función:

Código PHP:

  <?php
 
class Taxi {
    
    var $estado;
    var $pasajeros;
    var $subir_pasajeros;    
    
    function estado($pasajeros=0) { // Esto se ejecutará al instanciar la clase
        if ($pasajeros<=0){
        $this->estado = "libre <br />";    
        $this->subir_pasajeros = $pasajeros;
    }
    elseif ($pasajeros>0){
        $this->estado = "ocupado <br />";
        $this->subir_pasajeros = $pasajeros;
        }
    }
    
    function subir_pasajeros($pasajeros){
        if ($pasajeros<=0){        
        $this->estado = "libre <br />";
        $this->subir_pasajeros = $pasajeros;
    }
    elseif ($pasajeros>0){
        $this->estado = "ocupado <br />";
        $this->subir_pasajeros = $pasajeros;
        }
        
        }
}
$taxi = new Taxi();
 
echo $taxi->estado(); // Mostrará "libre"
 
$taxi->subir_pasajeros(3); // Suben 3 pasajeros al taxi
 
echo $taxi->estado(); // Mostrará "ocupado"*/
 
?>   

La manera de imprimirlo tiene que ser así, pero si no pongo los "echo" no imprime nada.


Si los pongo en todos si imprime pero a la vez no le cambia el valor a estado en la segunda función cambiándolo a ocupado:

Código PHP:

  <?php
 
class Taxi {
    
    var $estado;
    var $pasajeros;
    var $subir_pasajeros;    
    
    function estado($pasajeros=0) { // Esto se ejecutará al instanciar la clase
        if ($pasajeros<=0){
        echo  $this->estado = "libre <br />";    
        $this->subir_pasajeros = $pasajeros;
    }
    elseif ($pasajeros>0){
        echo $this->estado = "ocupado <br />";
        $this->subir_pasajeros = $pasajeros;
        }
    }
    
    function subir_pasajeros($pasajeros){
        if ($pasajeros<=0){
        echo $this->estado = "libre <br />";
        $this->estado = "libre <br />";
        $this->subir_pasajeros = $pasajeros;
    }
    elseif ($pasajeros>0){
        echo $this->estado = "ocupado <br />";
        $this->estado = "ocupado <br />";
        $this->subir_pasajeros = $pasajeros;
        }
        
        }
}
$taxi = new Taxi();
 
echo $taxi->estado(); // Mostrará "libre"
 
$taxi->subir_pasajeros(3); // Suben 3 pasajeros al taxi
 
echo $taxi->estado(); // Mostrará "ocupado"*/
 
?>   

imprime: libre, ocupado y nuevamente libre (que debería de salir ocupado) es decir el último estado a imprimir siempre coge el primero.

¿Estoy haciendo algo mal o me falta algo?

[pateketrueke]

el punto es que php no imprime por arte de magia, en algún punto debes programar que imprima...

para dicho caso lo mejor es que los métodos devuelvan algún resultado con return para así imprimirlo por fuera del objeto y no dentro de el...

[Hidek1]

bueno es simple.. porque los IF dentro del método estado().. se refieren al parámetro del método y tu no le pasas ningún parámetro -.-!

verifica lo que digo con esto
$taxi->estado(0); // Mostrará "libre"

saludos.

[HiToGoRoShi]

crea metodos get and set de tus variables, y luego funciones que hagan return, jamas imprimir ahi mismo ... y sobre todo hacer logica fuera de una funcion

ejem ..

Código PHP:

public function subirPasajeros($pasajeros)
{
if()
return true; //Si este se subio
else
return false; //Si no esta ocupado
}
 
Y fuera de cla clase, instancias el objeto
 
$obj = new Taxi();
 
if($obj->subirTaxi($pasajero))
print "EL PASAJERO SE SUBIOo";
else
print "NO HAY MAS ESPACIO";
 

Es una idea tio, espero que lo entiendas

[GatorV]

Tema movido desde PHP a Frameworks y PHP orientado a objetos

[kahlito]

Iniciado por pateketrueke

el punto es que php no imprime por arte de magia, en algún punto debes programar que imprima...

para dicho caso lo mejor es que los métodos devuelvan algún resultado con return para así imprimirlo por fuera del objeto y no dentro de el...

Iniciado por Hidek1

bueno es simple.. porque los IF dentro del método estado().. se refieren al parámetro del método y tu no le pasas ningún parámetro -.-!

verifica lo que digo con esto
$taxi->estado(0); // Mostrará "libre"

saludos.

hola de nuevo, he probado hacinedo que me devuelva el resultado con "return" para no poner los echos dentro y si se imprimen, incluso si pongo dentro de estado fuera un valor mayor a 0 ( echo $taxi->estado(5); por ejemplo) si me da ocupado:

Código PHP:

  <?php
 
class Taxi {
    
    var $estado;
    var $pasajeros;
    var $subir_pasajeros;    
    
    function estado($pasajeros=0) { // Esto se ejecutará al instanciar la clase
        if ($pasajeros<=0){
        return $this->estado = "libre <br />";    
        $this->subir_pasajeros = $pasajeros;
    }
    elseif ($pasajeros>0){
        return $this->estado = "ocupado <br />";
        $this->subir_pasajeros = $pasajeros;
        }
    }
    
 
}
$taxi = new Taxi();
 
echo $taxi->estado(); // Mostrará "libre"
//$taxi->estado(0);
 
?>   

Sin embargo en la segunda parte intento hacer aparte lo siguiente para probar y me da error ya que el método "estado" no está definido en este caso

Código PHP:

  <?php
 
class Taxi {
    
    var $estado;
    var $pasajeros;
    var $subir_pasajeros;    
    
    function subir_pasajeros() {
        if ($pasajeros<=0){
        return $this->estado = "libre <br />";    
        $this->subir_pasajeros = $pasajeros;
    }
    elseif ($pasajeros>0){
        return $this->estado = "ocupado <br />";
        $this->subir_pasajeros = $pasajeros;
        }
        
        }
 
}
$taxi = new Taxi();
 
$taxi->subir_pasajeros(3); // Suben 3 pasajeros al taxi
echo $taxi->estado(); // Mostrará "ocupado"*/
 
?>   

No entiendo como puedo llamarlo de esa forma sin que me de error.

Iniciado por HiToGoRoShi

crea metodos get and set de tus variables, y luego funciones que hagan return, jamas imprimir ahi mismo ... y sobre todo hacer logica fuera de una funcion

ejem ..

Código PHP:

public function subirPasajeros($pasajeros)
{
if()
return true; //Si este se subio
else
return false; //Si no esta ocupado
}
 
Y fuera de cla clase, instancias el objeto
 
$obj = new Taxi();
 
if($obj->subirTaxi($pasajero))
print "EL PASAJERO SE SUBIOo";
else
print "NO HAY MAS ESPACIO";
 

Es una idea tio, espero que lo entiendas

El problema es que tengo que hacer el ejercicio a partir de esta llamada:

Código:

$taxi = new Taxi();

echo $taxi->estado(); // Mostrará "libre"

$taxi->subir_pasajeros(3); // Suben 3 pasajeros al taxi

echo $taxi->estado(); // Mostrará "ocupado"*/

Saludos.

[kahlito]

Igualmente uniendo lo anterior que había separado con las nuevas llamadas utilizando return me sigue sin coger el último valor de "subir_pasajeros" y me deja libre el estado por segunda vez:

Código PHP:

  <?php
 
class Taxi {
    
    var $estado;
    var $pasajeros;
    var $subir_pasajeros;
    
    function estado($pasajeros=0) { // Esto se ejecutará al instanciar la clase
        if ($pasajeros<=0){
        return $this->estado = "libre <br />";    
        $this->subir_pasajeros = $pasajeros;
    }
    elseif ($pasajeros>0){
        return $this->estado = "ocupado <br />";
        $this->subir_pasajeros = $pasajeros;
        }
    }
    
    function subir_pasajeros() {
        if ($pasajeros<=0){
        return $this->estado = "libre <br />";    
        $this->subir_pasajeros = $pasajeros;
    }
    elseif ($pasajeros>0){
        return $this->estado = "ocupado <br />";
        $this->subir_pasajeros = $pasajeros;
        }
        
        }
 
}
$taxi = new Taxi();
echo $taxi->estado(); // Mostrará "libre" 
$taxi->subir_pasajeros(3); // Suben 3 pasajeros al taxi
echo $taxi->estado(); // Mostrará "ocupado"*/
 
?>   

¿Sigo teniendo mal los parámetros?

[GatorV]

El problema es que al llegar al return, la función sale de contexto y nunca ejecuta tus siguientes operaciones, deberías de leer un poco el manual de PHP, para que entiendas como es que funciona, si tu pones un return hasta ahí llega la función:

Código PHP:

function foo()
{
         return true;
         echo "hi"; // nunca se va a ejecutar
}
 
foo();
 

Saludos.

[GatorV]

Aqui tienes un ejemplo simple corregido:

Código PHP:

class Taxi
{
    protected $_pasajeros = 0;
    
    public function getEstado()
    {
        if ($this->getPasajeros() <= 0) {
            return 'libre';
        } else {
            return 'ocupado';
        }
    }
    
    public function subirPasajeros($nPasajeros)
    {
        $this->_pasajeros = $nPasajeros;
    }
    
    public function getPasajeros()
    {
        return $this->_pasajeros;
    }
}
 
$taxi = new Taxi();
echo $taxi->getEstado(); // libre
$taxi->subirPasajeros(3);
echo $taxi->getEstado(); // ocupado
 

Saludos.

[kahlito]

Gracias GatorV, muy buen ejemplo.

Tengo que revisar a fondo las explicaciones ya que entre otras cosas veo parte del código que no conocía como public y protected. Estoy haciendo ejercicios de un manual de php 4 creo que del año 2004 así que tengo que aplicarme a muchas cosas nuevas.

Gracias a todos por la ayuda

[GatorV]

Es la visibilidad de las variables, si estas leyendo uno del 2004, te recomiendo te actualices un poco, ha cambiado mucho el manejo de PHP Orientado a Objetos...

[kahlito]

Gracias de nuevo GatorV, me actualizo desde ya