Mala condicion?

[AdrianSeg]

Estoy intentando hacer que compruebe si el usuario existe en la base de datos, pero nunca lo comprueba, siempre lo agrega, sé que el fallo tiene que estar en el if pero no me doy cuenta.

Código PHP:

  <?php
 
// Boton
 
if (isset($_POST["boton"])) {  
 
        $nombre = $_POST['nombre'];
        $apellidos = $_POST['apellidos'];
        $email = $_POST['email'];
        $departamento = $_POST['departamento'];
        $login = $_POST['login'];
        $pass = $_POST['pass'];
        $acceso = $_POST['acceso'];
        $titulo = $_POST['titulo'];
 
 
$sql="SELECT login FROM usuarios";
 
    $result=mysql_query($sql);
 
$fila=mysql_fetch_assoc($result);
 
        if ($login==$fila["login"]){
 
            echo "El usuario ya existe";
 
            echo "<a href='./index.php'>Regresar</a>";
 
            }
 
        else{    
        
$sSQL="INSERT INTO usuarios (login,nombre,apellidos,password,email,rango,titulo,departamento) VALUES ('$login','$nombre','$apellidos','$pass','$email','$acceso','$titulo','$departamento')";
 
$result=mysql_query($sSQL) or die("<b>Error. El servidor dijo: </b> " . mysql_error());  
    }
}
?>   

[cesarpunk]

creo que sería más rápido si pusieras

Código PHP:

   
$sql="SELECT login FROM usuarios where login=".$login; 
$rs=mysql_query($sql,$c);
$existe=mysql_num_rows($rs);
 
if($existe>0){
 echo "El usuario ya existe";
}
else{
//lo que quieras;
} 
   

[AdrianSeg]

Warning: mysql_query(): supplied argument is not a valid MySQL-Link resource in D:\AppServ\www\proyectougr2\includes\Submenus\agre garu.php on line 209

Warning: mysql_num_rows(): supplied argument is not a valid MySQL result resource in D:\AppServ\www\proyectougr2\includes\Submenus\agre garu.php on line 210

[Nemutagk]

Estas copiando tal cual el codigo que te dio cesarpunk, minimo adaptalo a tus necesidades/variables, no quieras todo hecho...

el primer error dice que la conexion indicada es erronea, porque la variable $C no tiene nada, la segunda es causada por la primera

Código PHP:

$c = mysql_connect($servidor,$usuario,$password);
mysql_select_db($base_de_datos);
 
$sql='SELECT login FROM usuarios where login="'.$login'"'; 
$rs=mysql_query($sql,$c);
$existe=mysql_num_rows($rs);
 
if($existe>0){
 echo "El usuario ya existe";
}
else{
//lo que quieras;
}
 

[camsworksinc]

cesarpunk y nemutagk te dieron la solucion.
Nada mas para aclararte un poco, la causa de que siempre se agregue el usuario, es porque en el query

$sql="SELECT login FROM usuarios";

estas trayendo TODA la columna login de la tabla usuarios, y luego estas usando solo EL PRIMER renglon, que es el que comparas con tu variable $login. Por lo tanto, a menos que $login coincida con el valor del primer renglon de la base de datos, la condicion nunca se cumplira, y siempre ira a agregar al usuario.
Para traer solamente el renglon en el que coincide el usuario con el valor de $login, se usa la clausula WHERE, que implica una condicion que debe de cumplirse para que nos regrese solamente lo que necesitamos.

Espero que te quede mas claro!

Suerte!

[AdrianSeg]

esto sigue sin andar borre la variable c ya que la conexión a hago más atras pero nada ni sand el where ni nada detecta el user....

Ahroa tira esto:

Parse error: syntax error, unexpected T_CONSTANT_ENCAPSED_STRING in


Cambio esto:

$sql='SELECT login FROM usuarios where login="'.$login'"';

por esto:

$sql="SELECT login FROM usuarios where login=$login'';

Y nada de nada.

[Nemutagk]

Porque quitaste las dobles comillas, si lo estas comparando contra un string, si las quitas nunca te va a funcionar y el error es porque faltaba un '.' en la sentencia sql,

Código PHP:

$sql='SELECT login FROM usuarios where login="'.$login.'"';
 

Por lo que veo no tienes idea de PHP, mejor lee un buen tutorial =/

[CHuLoSoY]

Edito: no he dicho nada XD

[AdrianSeg]

Arreglado, lo hice asi:

Código PHP:

  <?php
 
// Boton
 
if (isset($_POST["boton"])) {  
 
        $nombre = $_POST['nombre'];
        $apellidos = $_POST['apellidos'];
        $email = $_POST['email'];
        $departamento = $_POST['departamento'];
        $login = $_POST['login'];
        $pass = $_POST['pass'];
        $acceso = $_POST['acceso'];
        $titulo = $_POST['titulo'];
 
 
 
$sql="SELECT login FROM usuarios where login='$login'";
 
$result=mysql_query($sql);
 
$fila=mysql_fetch_assoc($result);
 
        if ($login==$fila["login"]){
 
            echo "El usuario ya existe";
            }
            
            else{
        
$sSQL="INSERT INTO usuarios (login,nombre,apellidos,password,email,rango,titulo,departamento) VALUES ('$login','$nombre','$apellidos','$pass','$email','$acceso','$titulo','$departamento')";
 
$result=mysql_query($sSQL) or die("<b>Error. El servidor dijo: </b> " . mysql_error());  
    }
}
?>   

[MaBoRaK]

loading............


Siempre lo he dicho, si no esta mal la programación est'a mal el comportamiento! DEPUREN DEPUREN y DEPURENNNNN.

podrias poner.

print_r($fila);

para empezar para ver como se est'a validando el usuario.


connection closed.