Uso se &&

[majo_83]

Hola gente, tengo la siguiente función, la cual necesito que devuelva 1 si $total es mayor a 0 y $dni es distinto de 11111111; no se que esta mal, porque no esta funcionando de esta manera..

Código PHP:

<?php require_once('conectar.php'); 
$dni=$_POST['dni'];
mysql_select_db($base, $conectar);
$query = "SELECT * FROM control where dni='$dni'";
$Resultado = mysql_query($query, $conectar) or die(mysql_error());
$total = mysql_num_rows($Resultado);
if($total > 0 && $dni != 11111111)
{echo"1";}
?>
 

Alguna corrección???

[jeixuxspn]

Iniciado por majo_83

Hola gente, tengo la siguiente función, la cual necesito que devuelva 1 si $total es mayor a 0 y $dni es distinto de 11111111; no se que esta mal, porque no esta funcionando de esta manera..

Código PHP:

<?php require_once('conectar.php'); 
$dni=$_POST['dni'];
mysql_select_db($base, $conectar);
$query = "SELECT * FROM control where dni='$dni'";
$Resultado = mysql_query($query, $conectar) or die(mysql_error());
$total = mysql_num_rows($Resultado);
if($total > 0 && $dni != 11111111)
{echo"1";}
?>
 

Alguna corrección???

Prueba esto:

Código PHP:

  $query = "SELECT * FROM control where dni='".$dni."'"; 
   

Saludos,

[maycolalvarez]

usa intval($_POST['dni']) para convertir explícitamente la cadena a número.

consulta la tabla de precedencia de operadores para no tener dudas al respecto de como hacer la expresión: http://www.php.net/manual/es/languag...precedence.php

[jeixuxspn]

Iniciado por jeixuxspn

Prueba esto:

Código PHP:

  $query = "SELECT * FROM control where dni='".$dni."'"; 
   

Por que lo que está haciendo tu consulta, es buscar un dni que sea igual a '$dni' (como si fuera una cadena) y no al valor que contiene la variable.

Saludos,

[majo_83]

OK.. muchas gracias...

[maycolalvarez]

@jeixuxspn, PHP puede analizar variables dentro de cadenas con comillas dobles, por lo tanto:

Código PHP:

$query = "SELECT * FROM control where dni='$dni'";
//es igual a:
$query = "SELECT * FROM control where dni='".$dni."'";
 

de igual forma MySQL el error puede deberse a que trata de comparar un string con un integer, por lo que debería eliminar las comillas simples del SQL:

Código PHP:

$query = "SELECT * FROM control where dni=$dni";
 

y con la conversión explícita que le indique:

Código PHP:

$dni = intval($_POST['dni']);
 

debería funcionar, además de que lo recomendable seria validar si dicho $_POST['dni'] es realmente un número, por el cual podría usar is_numeric().

No hace falta que repitas el mensaje si el usuario quien pregunta no ha respondido, puede que su problema sea otro.

[topo_bionico]

Código PHP:

  $query = "SELECT * FROM control where dni='$dni'"; 
   

faltan los apostrofes en los nombres de tablas y campos, sin eso mysql los toma como valores en vez de nombres.

Código PHP:

  $query = "SELECT * FROM `control` where `dni`='$dni'"; 
   

[maycolalvarez]

@topo_bionico: realmente no es necesario a no ser que se usen espacios o nombres reservados en las tablas o campos, cada SDBMS tiene sus peculiaridades, por ejemplo en PostgreSQL se usan las comillas dobles

[topo_bionico]

Iniciado por maycolalvarez

@topo_bionico: realmente no es necesario a no ser que se usen espacios o nombres reservados en las tablas o campos, cada SDBMS tiene sus peculiaridades, por ejemplo en PostgreSQL se usan las comillas dobles

Pero vos estas usando mysql, o no?

Hace la prueba, no cuesta nada

[gjx2]

Ok y por que no hacer esto

Código PHP:

$query = "SELECT * FROM control where dni='$dni' and dni <> '11111111' ";
$Resultado = mysql_query($query,  $conectar) or die(mysql_error());
if (mysql_num_rows($Resultado)) { 
/* ... */
}
 

Saludos!

[jeixuxspn]

Bueno si vamos a un caso real un dni no es ni muchos menos un entero, ya que este se compone de 8 números y una letra, por lo que estará obligado a hacer una comparación de cadenas.

Tienes razón en esto:

Código PHP:

      $query = "SELECT * FROM control where dni='$dni'";
    //es igual a:
    $query = "SELECT * FROM control where dni='".$dni."'"; 
   

[topo_bionico]

Iniciado por jeixuxspn

Bueno si vamos a un caso real un dni no es ni muchos menos un entero, ya que este se compone de 8 números y una letra, por lo que estará obligado a hacer una comparación de cadenas.

Código PHP:

      $query = "SELECT * FROM control where dni='$dni'"; // ---> Pues esto a veces falla
    //es igual a:
    $query = "SELECT * FROM control where dni='".$dni."'"; 
   

No tiene sentido concatenar variables usando comillas dobles, ya que PHP sabe que adentro de la cadena entre comillas dobles tiene que buscar los nombres de las variables declaradas.

Si el código es muy extenso se traduce en alta carga para el servidor.

Lo correcto seria usar comillas simples y concatenar las variables

Código PHP:

  $query = 'SELECT * FROM `control` where `dni`=\''.$dni.'\''; 
   

[maycolalvarez]

bueno, pasamos de una simple pregunta a una discusión sobre rendimiento .
en el caso del tipo de datos del $dni como indica @jeixuxspn que se compone de letras con números depende de la lógica y/o reglas de negocio por que bien podría un DNI (sea lo que sea) ser diferente de un país a otro, por ejemplo en Venezuela tenemos entre otros el RIF y no se si es lo mismo o sea otra cosa .

@majo_83 es por eso lo importante de aclarar esos puntos para ofrecerte la respuesta precisa, el tema de rendimiento es importante, como no, pero lo fundamental es que se cumplan las reglas de negocio sobre los datos que manipules.
saludos