[SOLUCIONADO] Base de datos no guarda

[jappier]

Hola amigos, estoy haciendo un formulario de registro pero a la hora de insertar los datos en la BD pasa algo raro que hace que no se introduzcan los datos. Me da un error tipo:
Warning: mysql_num_rows() expects parameter 1 to be resource, boolean given in C:\wamp\www

Os pongo el código a ver si me podeis ayudar:

Código PHP:

  <?php
 
    $nombre=$_POST['nombre'];
    $apellidos=$_POST['apellidos'];
    $DNI=$_POST['DNI'];
    $direccion=$_POST['direccion'];
    $telefono=$_POST['telefono'];
    $email=$_POST['email'];
    $provincia=$_POST['provincia'];
    $contraseña= $_POST['contraseña'];
    $drpass=$_POST['drpass'];
 
    require("connect_db.php");
    $checkdni = mysql_query("SELECT * FROM tb-clientes WHERE DNI='$DNI' ");
    $check_dni = mysql_num_rows($checkdni);
        if($contraseña==$drpass){
            if($check_dni>0){
                echo ' <script language="javascript">alert("Atencion, ya existe el mail designado para un usuario, verifique sus datos");</script> ';
            }else{
                
                //require("connect_db.php");
                mysql_query("INSERT INTO tb-clientes VALUES('','$nombre','$apellidos','$DNI','$direccion','$telefono','$email','$provincia','$contraseña''')");
                //echo 'Se ha registrado con exito';
                echo ' <script language="javascript">alert("Usuario registrado con éxito");</script> ';
                mysql_close($link);
            }
            
        }else{
            echo 'Las contraseñas son incorrectas';
        }
 
    
?>   

[gnzsoloyo]

Lo más probable es que la consulta SLEECT y el INSERT te estén devolviendo un error de sintaxis, y tu ni te enteras porque no estás validando nada ni recuperando los mensajes de error.
Estás usando un símbolo reservado en el nombre de la tabla...

[Nedned]

Si solo fuera mysql debería ser así:


SELECT * FROM 'tb-clientes' WHERE DNI=

entre comillas simples

[gnzsoloyo]

Nop...
Eso daría un error de sintaxis, porque estás confundiendo los apóstrofos ('), con los acentos agudos (`).
Los primero se usan para cadenas de texto, y el nombre de una tabla no es una cadena de texto. Los segundos si se usan para objetos de bases de datos.

La idea sería:

Código MySQL:

SELECT * FROM `tb-clientes` WHERE DNI =
 

Más alla de eso, no está validando nada. Está asumiendo que las cosas siembre funcionan bien, y ningún programador asume eso jamás...

[Nedned]

Ostia si, totalmente cierto, no se por qué lo pense en comillas simples y mysql usa "acentos agudos" que no sabía que se llamase así...

De todas maneras debería ir validando todos los pasos que va dando, como tu bien dices.
Asumir que todo funciona es muy arriesgado, además que tardas más en ver de que falla que en poner código que te vaya mostrando los errores por pantalla en el caso d que existan.

Saludos

[jappier]

Lo más probable es que la consulta SLEECT y el INSERT te estén devolviendo un error de sintaxis, y tu ni te enteras porque no estás validando nada ni recuperando los mensajes de error.

Efectivamente, gracias a la función:
or die(mysql_error()) he podido ver cual era el fallo:
column count doesn't match value count at row 1 mysql,
habia 2 espacios en blanco a la hora de insertar los datos.

Gracias por vuestra ayuda.

Saludos!!