[SOLUCIONADO] Else no funciona

odek · #1 (**permalink**) 27/08/2014, 17:05

Porque mi else no funciona, ya intente hacer el cambio a else if

Código PHP:

Ver original<?php
session_start();
 
$usuario = $_POST['usuario'];
$contrasena = $_POST['contrasena'];
 
$con = mysqli_connect("localhost","prueba","prueba","lolo");
$sen = "SELECT * FROM personal WHERE usuario = '$usuario' AND contrasena = '$contrasena'";
$res = mysqli_query($con,$sen);
 
while($fila = mysqli_fetch_array($res)){
    $usbase = $fila['usuario'];
    $conbase = $fila['contrasena'];
    
    if($usuario == $usbase && $contrasena == $conbase){
        $_SESSION['usuario'] = $usuario;
        $_SESSION['contrasena'] = $contrasena;
        echo '
        <html>
        <head>
        <meta http-equiv="refresh" content="0;url=../acceso/inicio.php">
        </head>
        </html>
        ';
} else {
    echo '
        <html>
        <head>
        <meta http-equiv="refresh" content="0;url=../index.php">
        </head>
        </html>
        ';
}
}
mysqli_close($con);
?>

pateketrueke · #2 (**permalink**) 27/08/2014, 17:31

Si no entra al else es porque la condición del if() se cumple, ¿no te parece lógico?

Puedes verificar los valores que comparas así:

Código PHP:

Ver originalvar_dump($usuario, $usbase, $contrasena, $conbase);
var_dump($usuario == $usbase, $contrasena == $conbase);

¿Qué resultados obtienes?

bookmaster · #3 (**permalink**) 27/08/2014, 17:38

No te funciona el else porque la consulta te va a devolver siempre un valor correcto por que se cumple con el WHERE, y en el caso de que no te lo devuelva, el while no tendrá efecto porque no tendrá nada que recorrer.
Tendrías que eliminar el while siempre y cuando solo obtengas un registro.

gnzsoloyo · #4 (**permalink**) 27/08/2014, 17:40

Con las condiciones de ese where, jamás entrará en el else...

odek · #5 (**permalink**) 28/08/2014, 10:30

Ya cambie el where pero ahora el problema es que todo es negativo, no importa si el usuario y contraseña sean correctos

Código PHP:

Ver original<?php
session_start();
$con = mysqli_connect("localhost","prueba","prueba","lolo");
 
$usuario = mysqli_real_escape_string($con,$_POST['usuario']);
$contrasena = mysqli_real_escape_string($con,$_POST['contrasena']);
 
$sen = "SELECT * FROM personal ";
$query = htmlentities($sen);
$res = mysqli_query($con,$query);
 
while($fila = mysqli_fetch_array($res)){
    $usbase = $fila['usuario'];
    $conbase = $fila['contrasena'];
    
    if($usuario == $usbase & $contrasena == $conbase){
        $_SESSION['usuario'] = $usuario;
        $_SESSION['contrasena'] = $contrasena;
        echo '
        funciona
        ';
} else {
    echo '
        No funciona
        ';
}}
mysqli_close($con)
?>

pateketrueke · #6 (**permalink**) 28/08/2014, 11:16

Más grave es que estás extrayendo todos los registros para comparar sólo uno.

Lo correcto es una sola consulta con el WHERE, y es muy simple: ya no necesitas ningún else.

Verificas con mysql_num_rows() si la consulta trae resultados, si no los hay sería lo mismo del else que intentabas hacer.

gnzsoloyo · #7 (**permalink**) 28/08/2014, 11:23

En realidad lo estabas haciendo bastante bien al principio, pero evaluando incorrectamente lo que obtienes de la base, porque si el usuario es único, y su password también, entonces la query sólo devolvería registros si hay coincidencia, por lo que todo el while() y ese if eran innecesarios.
La cosa podría andar, basándome en tu propio código, mas o menos así:

Código PHP:

Ver original<?php
 
session_start();
$usuario = $_POST['usuario'];
$contrasena = $_POST['contrasena'];
$con = mysqli_connect("localhost", "prueba", "prueba", "lolo");
$sen = "SELECT * FROM personal WHERE usuario = '$usuario' AND contrasena = '$contrasena'";
$res = mysqli_query($con, $sen);
if (!res) {
    echo '
        <html>
        <head>
        <meta http-equiv="refresh" content="0;url=../index.php">
        </head>
        </html>
        ';
} else {
    if (mysqli_num_rows($res) > 0) {
        $fila = mysqli_fetch_array($res);
        $usbase = $fila['usuario'];
        $conbase = $fila['contrasena'];
        $_SESSION['usuario'] = $usuario;
        $_SESSION['contrasena'] = $contrasena;
        echo '
        <html>
        <head>
        <meta http-equiv="refresh" content="0;url=../acceso/inicio.php">
        </head>
        </html>
        ';
    } else {
        echo '
        <html>
        <head>
        <meta http-equiv="refresh" content="0;url=../index.php">
        </head>
        </html>
        ';
    }
}
mysqli_close($con);
?>