Comprobar si usuario existe en bbdd

[frmfernando]

Hola buenas tardes.

Se que tengo algún fallo pero no lo detecto.

Les comento lo que pretendo es que cuado dé de alta un usuario si ya existe que me lo diga y no me lo deje meter de nuevo.

Si no introdusco datos si que me avisa y no me introduce datos en la base de datos, pero cuando introduzco algún nombre de usuario que ya existe me lo introduce directamente.

Les dejo el codigo, seguro que es que lo tengo mal enfocado el problema.

Código PHP:

  session_start();
if(isset($_SESSION["validado"])){
 
    $usuario=$_POST["usuario"];
    $password=$_POST["password"];
    
    if ($usuario==""){
        echo "No ha introducido ningún usuario";
        echo "<a href='./usualta.php'>Regresar</a>";
        }else{
    
    $sql="SELECT usuario FROM usuarios";
    $r=mysql_query($sql, $link);
    
    while ($fila=mysql_fetch_assoc($r)){
        if ($usuario==$fila["usuario"]){
            echo "El usuario ya existe";
            echo "<a href='./usualta.php'>Regresar</a>";
            }else{
                $sql="INSERT INTO usuarios (usuario, password) VALUES ('$usuario', '$password')";
                $r=mysql_query($sql, $link);
                header("location:./usualta.php");
                }
        }
        }
    }else{
        header ("location:./usualta.php");
} 
   

Muchas gracias una vez más.

[maycolalvarez]

es mejor que apliques un indice unico (unique key) al campo y así evitas la duplicacion en base de datos del nombre del usuario

[TKZeXe]

a simple vista te falta una condicion:

ahi va.

Código PHP:

session_start();
if(isset($_SESSION["validado"])){
 
    $usuario=$_POST["usuario"];
    $password=$_POST["password"];
    
    if ($usuario==""){
        echo "No ha introducido ningún usuario";
        echo "<a href='./usualta.php'>Regresar</a>";
        }else{
    
    $sql="SELECT usuario FROM usuarios where usuario='$usuario'";
    $r=mysql_query($sql, $link);
    
$fila=mysql_fetch_assoc($r);
        if ($usuario==$fila["usuario"]){
            echo "El usuario ya existe";
            echo "<a href='./usualta.php'>Regresar</a>";
            }
        else{
             $sql="INSERT INTO usuarios (usuario, password) VALUES ('$usuario', '$password')";
             $r=mysql_query($sql, $link);
             header("location:./usualta.php");
             }
    }
    }else{
        header ("location:./usualta.php");
}
 

[darkasecas]

Ese codigo hace un ciclo recorriendo todos los registros de usuario, y por cada uno que no sea igual al introducido en $_POST, inserta este dato.

Seria mejor hacer la consulta buscando un registro donde el campo usuario sea igual al introducido en $_POST, si la consulta devuelve un registro, mostrar el mensaje de que ya existe, y si no, insertarlo.

[frmfernando]

Ya lo habia pensado, en la bbdd ya tengo tres campos, id, usuario y password, en mi caso el campo con indice unico se lo tengo puesto al id, lo quería controlar por codigo para que me saltase un mensaje de aviso y no de error.

De todas formas muchas gracias.

[frmfernando]

Muchas gracias a todos.

Con el código que ha publicado TKZeXe me funciona perfectamente.

El error por lo que he podido comprobar lo tenía con el While.

Muchas gracias una vez más y ya van unas cuantas.

Fernando

[TKZeXe]

tenias un enredo de identacion de locos, asi que borre el end del while al azar casi ajja

^^!

[jkano8]

Buenas tardes, yo quiero hacer lo mismo con mi bbdd pero no me sale.
Dispongo del siguiente codigo:

Código PHP:

session_start();
        
        include("usuario_validado.php");
        
        $accion=$_GET['accion'];
        if ($accion=="enviar"){
            $cadena_sql="INSERT INTO usuario (nombre, email, login, password, tipo_usuario, activo)";
            $cadena_sql .=" VALUES (";
            $cadena_sql .="'".$_POST['NOMBRE']."', ";
            $cadena_sql .="'".$_POST['EMAIL']."', ";
            $cadena_sql .="'".$_POST['LOGIN']."', ";
            $cadena_sql .="'".md5($_POST['PASSWORD'])."', ";
            $cadena_sql .="'web', ";
            $cadena_sql .="'t' ";
            $cadena_sql .=")";
            
            $conector->ejecutarConsultaActualizacion($cadena_sql);
            //$contenido="El formulario ha sido enviado correctamente";
            $contenido=$smarty->fetch('registro_usuarios_fin.html');
        }
        else
        {
            $contenido=$smarty->fetch('registro_usuarios.html');
        }
 

Gracias de antemano