Funcion retorna resultado adverso con operadores logicos

[sumawiseguy2]

Hola todos!

Tengo una funcion para determinar las condiciones de venta (credito o contado) y se supone que verifica si las variables $efectivo, $tarjeta y $cheque tienen algun valor, si lo tienen entonces la venta es de contado, si el valor de todos estos campos es cero, entonces es una venta a credito, el problema con mi funcion es que no me esta funcionando y la he revisado pero no veo que tiene de malo, podrian echarle un vistazo y decirme que estoy pasando por alto? Gracias!

Código PHP:

   
function condiciones($condiciones) {
if ( $efectivo != 0 || $tarjeta != 0 || $cheque != 0 ) {
    $condiciones = "CONTADO";
    }
    else {
    $condiciones = "CREDITO";
        }
    return $condiciones;
}
 
$condicionVentas = condiciones(); 
   

[Ueki]

De entrada veo un pequeño error, es que al crear la función la defines con un parámetro llamado $condiciones, pero luego al llamarla abajo no defines ese parámetro.

Además, ¿dónde están definidas las variables efectivo, tarjeta y cheque?

[pzin]

¿De dónde vienen $efectivo, $tarjeta y $cheque?

¿No debería de ser function condiciones($efectivo,$tarjeta,$cheque)? ¿O ($efecto=0,....?

[sumawiseguy2]

Gracias UEKI

Ante todo debo advertirlos que soy un novato tratando de aprender sobre programacion y mucho de lo que hago es guiandome de los ejemplos que voy encontrando.

Primero, asi estan definidas las variables (vienen de un formulario)

Código PHP:

   
 
$efectivo = number_format($_POST['efectivo'],2);
$tarjeta = number_format($_POST['tarjeta'],2);
$cheque = number_format($_POST['cheque'],2);
 
function condiciones($condiciones) {
if ( $efectivo != 0 || $tarjeta != 0 || $cheque != 0 ) {
    $condiciones = "CONTADO";
    }
    else {
    $condiciones = "CREDITO";
        }
    return $condiciones;
}
 
$condicionVentas = condiciones(); 
   

En cuanto al llamado de la funcion, corrigeme, pues mi principal tranque era llevar el resultado de la funcion a una variable que luego se mostrara al imprimir la factura, o sea, que la factura imprima $condicionVentas (CONTADO o CREDITO)

Gracias nuevamente!

[pzin]

No le estás pasando nada a la función.

Prueba así:

Código PHP:

  function condiciones($var1,$var2,$var3) { 
if ( $var1 !== 0 || $var2 !== 0 || $var3 !== 0 ) { 
        $condiciones = "CONTADO"; 
    } 
    else { 
        $condiciones = "CREDITO"; 
    } 
    return $condiciones; 
} 

$condicionVentas = condiciones($efectivo,$tarjeta,$cheque); 
   

Puedes nombrar $var1, $var2 y var3 como quieras. Así te va a funcionar. Lo he puesto así para que vieses cómo funciona.

[jossss]

Claro es por el ámbito de las variables. También te sobra el parámetro de la función.

Código PHP:

<?php
$efectivo = number_format($_POST['efectivo'],2);
$tarjeta = number_format($_POST['tarjeta'],2);
$cheque = number_format($_POST['cheque'],2);
 
function condiciones() {
    global $efectivo,$tarjeta,$cheque;
    if ( $efectivo != 0 || $tarjeta != 0 ||  $cheque != 0 ) {
            $condiciones = "CONTADO";
        }else {
            $condiciones = "CREDITO";
        }
        return $condiciones;
    }
 
$condicionVentas = condiciones();
echo $condicionVentas;
?>
 

[sumawiseguy2]

Gracias a todos! Probando es que se sabe! Ambas soluciones funcionan bien.