Ayuda con Clase y Funciones

cellularcd · #1 (**permalink**) 15/06/2010, 00:01

estoy haciendo este ejercicio y me salen varios errores

Código PHP:

  <?php 
 
class producto{ 
    var $trapeador; //variable donde se captura el # de trapeadores 
    var $escoba; // variable donde se captura el # de escobas 
    var $total_escoba; // totaliza las escobas 
    var $total_trapeador; //totaliza los trapeadores 
    var $factura; // se crea la factura 
    var $precio_trapeador; //precio unitario x trapeador 
    var $precio_escoba; //precio unitario x escoba 
    var $iva_escoba; //calcula el iva 
    var $iva_trapeador; //calcula el iva 
     
 
    /*function prod($trapeador,$escoba) { 
       $this->trapeador=$trapeador; 
       $this->escoba=$escoba; 
       //$this->total_escoba=array();  
       //$this->total_trapeador=array();*/ 
    } 
    function tottrap($total_trapeador) { 
         
        if ($trapeador < 500) 
        { 
        $this->precio_trapeador=$precio_trapeador=2500 ;  
//si es menor a 500 cuesta 2500 
        } else { 
        $this->precio_trapeador=$precio_trapeador=2300 ; 
//si es mayor a 500 cuesta 2300 
        } 
        $this->total_trapeador= $total_trapeador=$trapeador * $precio_trapeador; 
        return $total_trapeador; 
    } 
     
    function ivatrap($iva_trapeador) { 
         
        $this->$iva_trapeador = $total_trapeador * 1.14; //calculamos el iva de los trapeadores sin importar la cantidad 
        return $iva_trapeador; 
    } 
 
    function totescb($total_escoba) { 
         
        if ($escoba < 500) 
        { 
        $this->precio_escoba=$precio_escoba = 2200; 
        } else { 
        $this->precio_escoba=$precio_escoba = 1950; 
        } 
        $this->total_escoba=$total_escoba= $escoba * $precio_escoba; 
    } 
 
    function ivaescb($iva_escoba) { 
         
        $this->iva_escoba=$iva_escoba = $total_escoba * 1.16; //calculamos en iva de las escobas sin importar la cantidad 
         
    } 
     
    function fact($factura) { 
     
        $this->factura=$factura = $iva_trapeador + $iva_escoba; //totalizamos los productos 
    } 
     
     
  
//se instancian los objetos y se vuelven a llamar las variables con los calculos ya hechos 
$producto1 = new producto ("$trapeador","$escoba");  
$trapeador = $_POST['trapeador']; 
$escoba = $_POST['escoba']; 
$total_trapeador = $_POST['total_trapeador']; 
$total_escoba = $_POST['total_escoba']; 
$iva_trapeador = $_POST['iva_trapeador']; 
$iva_escoba = $_POST['iva_escoba']; 
$factura = $_POST['factura']; 
 
echo "La cantidad total de trapeadores sin IVA es de: " . $producto1->tottrap(); 
echo "<br>"; 
echo "El total de los trapeadores con IVA es de: " . $_POST['iva_trapeador']; 
echo "<br>"; 
echo "La Cantidad total de escobas sin IVA es de: " . $_POST['total_escoba']; 
echo "<br>"; 
echo "El total de las escobas con IVA es de: " . $_POST['iva_escoba']; 
echo"<br>"; 
echo "El total a pagar es de: " . $_POST['factura']; 
echo "<br>";  
?> 
<?php 
//creamos el formulario para capturar los datos 
?> 
<html> 
<body> 
<form  name= "xyz" method="post" action="">  
Digite el numero de trapeadores:  
<input type="text" name="trapeador" size="80"> 
<br> 
Digite el numero de escobas:  
<input type="text" name="escoba" size="80"> 
<br> 
<input type="submit" value="enviar">  
<input type="reset" value="borrar">  
<br> 
</form> 
<br> 
<br> 
<br> 
..::Resultado::.. 
<br> 
<br> 
<br> 
</body> 
</html> 
<?php 
 
?>

si me pudieran colabrar, c los agradecerua enormemente

pateketrueke · #2 (**permalink**) 15/06/2010, 06:41

es posible que nos digas los mensajes de error completos... o hay que adivinar??

deivinsontejeda · #3 (**permalink**) 15/06/2010, 07:54

Importante que nos mencione los errores que te muestras y aprovechando veo que estas usando POO pero PHP4, pudieras empezar con el pie derecho e ir bien con PHP5...

http://www.php.net/manual/en/language.oop5.php

cellularcd · #4 (**permalink**) 15/06/2010, 12:29

Cita:

Iniciado por pateketrueke

es posible que nos digas los mensajes de error completos... o hay que adivinar??

ok, disculpa, el error que me sale es que quiero que me haga el calculo al capturar en dos cajas de texto llamadas trapeador y escoba.
el error me sale en la linea 73, especificamente esto:

Código PHP:

  echo "La cantidad total de trapeadores sin IVA es de: " . $producto1->tottrap();

y cuando lo ejecuto este es el error:
"Fatal error: Call to undefined method producto::tottrap() in C:\wamp\www\Ejercicios\2.php on line 73"

y del comentario de deivinsontejeda te comento que antes las funciones las tenia con"function __construct()" y las cambie de nombre pero ahi si me disculpas la ignorancia de php4 a php5. antes me guie con un ejm en la web porque la verdad me estaba complicando la vida al hacer el ejercicio.

a todos, les agradezco el interes por ayudarme.

pateketrueke · #5 (**permalink**) 15/06/2010, 13:19

lo que sucede es que estás cerrando tu clase antes de la función -que debería ser método- tottrap()

si, es enserio... pon atención y date cuenta de que cierras la clase, y las funciones las dejas fuera de la misma... cuando deberían ir dentro, para comportarse tal cual métodos!!

cellularcd · #6 (**permalink**) 15/06/2010, 13:55

Cita:

Iniciado por pateketrueke

lo que sucede es que estás cerrando tu clase antes de la función -que debería ser método- tottrap()

si, es enserio... pon atención y date cuenta de que cierras la clase, y las funciones las dejas fuera de la misma... cuando deberían ir dentro, para comportarse tal cual métodos!!

efectivamente tenias razon. vi varios ejemplos y las funciones estaban dentro de la clase. pero... ni fu ni fa. no me quiere mostrar el valor.
para la muestra de un boton:

Código PHP:

  class producto{
    var $trapeador; //variable donde se captura el # de trapeadores
    var $escoba; // variable donde se captura el # de escobas
    var $total_escoba; // totaliza las escobas
    var $total_trapeador; //totaliza los trapeadores
    var $factura; // se crea la factura
    var $precio_trapeador; //precio unitario x trapeador
    var $precio_escoba; //precio unitario x escoba
    var $iva_escoba; //calcula el iva
    var $iva_trapeador; //calcula el iva
function tottrap() {
        
        if ($trapeador < 500)
        {
        $this->precio_trapeador=$precio_trapeador=2500 ;
        } else {
        $this->precio_trapeador=$precio_trapeador=2300 ;
        }
        $this->total_trapeador= $total_trapeador=$trapeador * $precio_trapeador;
        return $total_trapeador;
    }
}
$producto1 = new producto("$trapeador","$escoba"); 
echo "La cantidad total de trapeadores sin IVA es de: ".$producto1->tottrap();

no incluyo el html del formulario porque es exactamente el mismo del 1º mensaje.
y aun asi, m deja ingresar todo, pero no el resultado.

GatorV · #7 (**permalink**) 15/06/2010, 13:58

Tu función tottrap() no regresa nada, es por eso que no ves nada en el echo.

Saludos.

dots3r · #8 (**permalink**) 15/06/2010, 14:01

O.o!
Supongo que return $total_trapeador esta comentado por accidente al poner aquí el codigo no?

cellularcd · #9 (**permalink**) 15/06/2010, 14:53

Cita:

Iniciado por dots3r

O.o!
Supongo que return $total_trapeador esta comentado por accidente al poner aquí el codigo no?

claro, aunque le quite el comentario, muestra 0 en la pagina como resultado. al parecer (desde mi punto de vista) que los trapeadores se estan multiplicando por cero y ni esta tomando los 2500 o 2300 para hacer el calculo

GatorV · #10 (**permalink**) 15/06/2010, 15:46

Pues viendo tu clase no veo de donde obtienes las variables $trapeador, $precio_trapeador, $total_trapeador.

Recuerda que si quieres usar los valores tienes que asignarlos y usarlos con $this:

Código PHP:

Ver originalclass foo
{
       private $_num1;
       private $_num2;
 
       public function __construct($num1, $num2)
       {
              $this->_num1 = $num1;
              $this->_num2 = $num2;
       }
 
       public function suma()
       {
              return $this->_num1 + $this->_num2;
       }
}
 
$foo = new foo(1, 2);
echo $foo->add();

Saludos.

cellularcd · #11 (**permalink**) 15/06/2010, 16:33

Cita:

Iniciado por GatorV

Pues viendo tu clase no veo de donde obtienes las variables $trapeador, $precio_trapeador, $total_trapeador.

Recuerda que si quieres usar los valores tienes que asignarlos y usarlos con $this
Saludos.

el formulario html esta en el primer mensaje y solo hago la captura de cuantas unidades de escobas y trapeadores.
el precio del trapeador es bajo una condicional es decir que son dos valores distintos porque tiene que determinar si son mas de 500 o menos de 500.
y el total vendria siendo del resultado de la condicional * lo que capturo de # de trapeadores.
tu aporte es logico, pero el minusculo problema es que ya esos valores estan ingresados, lo que necesito es capturarlos y hacer el calculo correspondiente ya que me sale 0.

Código PHP:

  <?php

class producto{
    var $trapeador; //variable donde se captura el # de trapeadores
    var $escoba; // variable donde se captura el # de escobas
    var $total_escoba; // totaliza las escobas
    var $total_trapeador; //totaliza los trapeadores
    var $factura; // se crea la factura
    var $precio_trapeador; //precio unitario x trapeador
    var $precio_escoba; //precio unitario x escoba
    var $iva_escoba; //calcula el iva
    var $iva_trapeador; //calcula el iva
function tottrap($trapeador, $precio_trapeador) {
        
        if ($trapeador < 500)
        {
        $this->precio_trapeador=2500;
        } else {
        $this->precio_trapeador=2300;
        }
        $this->total_trapeador= $total_trapeador=$trapeador * $precio_trapeador;
        return $total_trapeador;
    }
}
//se instancian los objetos y se vuelven a llamar las variables con los calculos ya hechos
$producto1 = new producto("$total_trapeador"); 
echo "La cantidad total de trapeadores sin IVA es de: ".$producto1->tottrap($trapeador, $precio_trapeador);
echo "<br>";
?>
<?php
//creamos el formulario para capturar los datos
?>
<html>
<body>
<form  name= "xyz" method="post" action=""> 
Digite el numero de trapeadores: 
<input type="text" name="trapeador" size="80">
<br>
Digite el numero de escobas: 
<input type="text" name="escoba" size="80">
<br>
<input type="submit" value="enviar"> 
<input type="reset" value="borrar"> 
<br>
</form>
<br>
<br>
<br>
..::Resultado::..
<br>
<br>
<br>
</body>
</html>

si no me hago entender, me avisan para ser mas explicito.
gracias por sus recomendaciones

enriqueplace · #12 (**permalink**) 15/06/2010, 20:10

Que tal

Como mínimo deberías iniciar simpre las clases en Mayúsculas, usar camelcase en atributos y métodos, y, por sobre todas las cosas, usar sintaxis PHP5!

Saludos!

GatorV · #13 (**permalink**) 16/06/2010, 09:01

Ok entiendo lo que requieres pero en tu código que expones no muestras de donde estas trayendo esas variables, y si estas descansando sobre register_globals, creo vas a tener un grave problema más tarde cuando migres a un servidor de producción.

Por otro lado nunca defines el constructor de tu clase por lo que tu clase nunca se entera del total, dejo aqui tu código sin pasar las variables del formulario pero eso ya es algo que debes de hacer tú:

Código PHP:

Ver originalclass Producto
{
         private $_totalProducto;
 
         public function __construct($total)
         {
                  $this->_totalProducto = $total;
         }
 
         public function getPrecioTotal()
         {
                  if ($this->_totalProducto < 500) {
                         $precioUnitario = 2500;
                  } else {
                         $precioUnitario = 2300;
                  }
 
                  return $this->_totalProducto * $precioUnitario;
         }
}
 
$Trapeadores = new Producto(599);
echo "El total sin IVA es: $" . number_format($Trapeadores->getPrecioTotal());

Saludos.