[SOLUCIONADO] ¿Qué falla en este sistema de login?

diegoguerrero · #1 (**permalink**) 30/07/2015, 15:47

Código PHP:

  <?php 
$conn = mysql_connect('localhost', 'root', '') 
    or die('No se pudo conectar: ' . mysql_error()); 
mysql_select_db('educamos') or die('No se pudo seleccionar la base de datos'); 
?> 
<html> 
<head> 
<link href="css/css.css" rel="stylesheet"/> 
</head> 
<body> 
<h1 align="center" style="color:#2491C9">Plataforma Educamos</h1> 
<table width="30%" align="center" border="0"> 
<form name="form" method="get" action="index.php" enctype="text/plain"> 
<tr id="tr1-h"> 
<td align="center"><b>Introduce tus datos a continuación</b></td> 
</tr> 
<tr id="tr1"> 
<td align="center" valign="middle"><input type="text" placeholder="Usuario" id="CampoTexto" name="user"/></td> 
</tr> 
<tr id="tr1"> 
<td align="center" valign="middle"><input type="password" placeholder="Contraseña" id="CampoTexto" name="clave"/></td> 
</tr> 
<tr id="tr1"> 
<td align="center" valign="middle"><input type="submit" id="boton" value="Acceder"></td> 
</tr> 
</form> 
</table> 
<?php 
 
if(isset($_GET['enter'])){ 
    $user = $_GET['user']; 
    $clave = $_GET['clave']; 
 
    $query = mysql_query ("SELECT * FROM usuarios WHERE user = '$user' AND clave = '$clave'",$conn); 
    $conta = mysql_num_rows($query); 
 
    if($conta == '0'){ 
        echo "Mal"; 
    }else{ 
        echo "Bien"; 
    } 
?> 
</body> 
</html>

Los echos del final no trabajan. Por cierto, ¿cómo se podría hacer que cuando los datos sean erróneos se muestre un mensaje en otra parte de la página? Gracias

NueveReinas · #2 (**permalink**) 30/07/2015, 15:53

El error es este:

Código PHP:

Ver originalif($conta == '0'){

Debería ser

Código PHP:

Ver originalif($conta === 0){

diegoguerrero · #3 (**permalink**) 30/07/2015, 16:01

Pues la verdad es que no funciona. Aquí adjunto el SQL.

Código SQL:

Ver originalCREATE TABLE IF NOT EXISTS `usuarios` (
  `id` INT(3) NOT NULL,
  `nombre` VARCHAR(1000) NOT NULL,
  `user` VARCHAR(255) NOT NULL,
  `clave` VARCHAR(255) NOT NULL
) ENGINE=InnoDB AUTO_INCREMENT=3 DEFAULT CHARSET=latin1;

pateketrueke · #4 (**permalink**) 30/07/2015, 16:38

Tu código espera la existencia de una variable $_GET['enter'] pero no existe por ningún lado, no la definiste en el formulario y por ende aunque lo envíes jamás se ejecutará dicho bloque.

chronos682 · #5 (**permalink**) 30/07/2015, 18:46

Y deberias pasar los datos por post o sino cualquiera podrá ver en la url el usuario y la contraseña de la persona que está haciendo el login.

diegoguerrero · #6 (**permalink**) 31/07/2015, 06:12

Gracias. Por cierto, ¿este código funcionaría para lo que quiero?

Código PHP:

Ver originalif(isset($_GET['boton'])){
    $user = $_GET['user'];
    $clave = $_GET['clave'];
        $rol = mysql_query ("SELECT rol FROM usuarios WHERE user = '$user' AND clave = '$clave'",$conn);
    $query = mysql_query ("SELECT * FROM usuarios WHERE user = '$user' AND clave = '$clave'",$conn);
    if (!$query) {
echo 'Usuario no existente'
    }else{
        if ($rol == 'A') {
            header('Location: localhost/educamos/alumno');
        }else{
            header('Location: localhost/educamos/profesor');
        }
    }

Lo que quiero es que el usuario se meta y, si su rol es A, que vaya a alumno y, si no, que será P, que vaya a profesor.

gnzsoloyo · #7 (**permalink**) 31/07/2015, 06:18

Seré curioso, pero ¿para qué pones la primera, si la segunda obtiene todo?
Estás consultando la misma tabla, con las mismas condiciones, ergo, la segunda obtiene el campo "rol" de neuvo...:

Código PHP:

Ver original$rol = mysql_query ("SELECT rol FROM usuarios WHERE user = '$user' AND clave = '$clave'",$conn);
$query = mysql_query ("SELECT * FROM usuarios WHERE user = '$user' AND clave = '$clave'",$conn);

Por otro lado, esto:

Código PHP:

Ver originalif (!$query)

Sólo evalúa que la ejecución de esa consulta no haya devuelto un error, no que haya devuelto registros, que no es lo mismo.
Esa validación daría TRUE si la query vuelve vacía...

Código PHP:

Ver originalif ($rol == 'A')

$rol no es un valor, es un result. Eso no te dará el resultado que quieres.

Deberías consultar el manual para ver como se recuperan los valores devueltos en un result. Es un error demasiado básico.

diegoguerrero · #8 (**permalink**) 31/07/2015, 06:38

¿Y cómo podría expresar lo que he comentado antes? Gracias.

Cita:

Lo que quiero es que el usuario se meta y, si su rol es A, que vaya a alumno y, si no, que será P, que vaya a profesor.

Uncontroled_Duck · #9 (**permalink**) 31/07/2015, 06:51

Cita:

Iniciado por diegoguerrero

¿Y cómo podría expresar lo que he comentado antes? Gracias.

Cita:

Lo que quiero es que el usuario se meta y, si su rol es A, que vaya a alumno y, si no, que será P, que vaya a profesor.

Por ejemplo con un if...

Código PHP:

  if($rol == 'A') 
{ 
    // ir a alumno 
} 
elseif($rol == 'P') 
{ 
   // ir a profesor 
} 
else 
{ 
   // ir a login 
}

También se pordría hacer con un switch

Código PHP:

  switch ($rol) { 
    case 'A': 
        // ir a alumno; 
        break; 
    case 'P': 
        // ir a profesor 
        break; 
    default: 
        // ir a login; 
}

Saludos,

gnzsoloyo · #10 (**permalink**) 31/07/2015, 06:56

Pues es medio obvio... Eso lo evalúas a partir de lo que obtienes en esta consulta:

Código MySQL:

Ver originalSELECT * 
FROM usuarios 
WHERE user = '$user' AND clave = '$clave'

Pero partiendo de la base que los campos que describes:

Código MySQL:

Ver originalCREATE TABLE IF NOT EXISTS `usuarios` (
  `id` INT(3) NOT NULL,
  `nombre` VARCHAR(1000) NOT NULL,
  `user` VARCHAR(255) NOT NULL,
  `clave` VARCHAR(255) NOT NULL
) ENGINE=InnoDB;

Puede verse que no tienes ningun campo que determine qué rol tiene el usuario.
¿No te parece que te está faltando o un campo o una tabla? Claro, a menos que nos estés escondiendo información de la tabla, caso en el que no podremos ayudarte...

Por lo pronto, lo primero que debes entender es cómo validar si una consulta devuelve registros.
PHP manual: mysql_num_rows()

Además, entender cómo leer los valores del registro obtenidos en la query:
PHP manual: mysql_query()

diegoguerrero · #11 (**permalink**) 31/07/2015, 11:06

Puede que suene básico, pero me explicaré con un ejemplo.

Quiero mostrar el nombre de un usuario cuando se loguea (por ejemplo Fernández, Carlos). Ese usuario está en el campo name de una tabla de MySQL. Para mostrarlo arriba como 'Identificado como Fernández, Carlos', ¿cómo lo haría? Es decir, quiero ver un campo de un registro de MySQL en PHP.

xfxstudios · #12 (**permalink**) 31/07/2015, 11:19

bueno realizas la consulta basado en la sesion y lo muestras, seria algo asi:

consulta:

Código PHP:

Ver original<?php
 
$ver = $db->query("SELECT * FROM usuarios WHERE usuario = '$_SESSION[user]'");
$row_ver = $ver->fetch_assoc();
//este es solo un ejemeplo
?>

donde quieras que se vea:

Código PHP:

Ver original<?php echo $row_ver['nombre']; ?>

claro hay muchas maneras de hacerlo, pero esta seria una de ellas

gnzsoloyo · #13 (**permalink**) 31/07/2015, 11:37

Podrías haber continuado la pregunta en el otro tema, y no abrir un hilo para básicamente lo mismo

http://www.forosdelweb.com/f18/que-f...6/#post4731104