foreach y query en PHP

[JuninhoP]

Muy buenas,

Tengo un problemilla con mi código, y es que me da un error en la linea del ejecución del foreach.

Antes de nada, para situaros un poco, os comento que este codigo pretende comprobar que el usuario y password introducidos en un formulario anteriormente son correctos para así llevarnos al panel de control.

El error es este:
Warning: Invalid argument supplied for foreach() in /Applications/XAMPP/xamppfiles/htdocs/conquers/actions/comprobar-login.php on line 17

Y el código es el siguiente:

Código PHP:

  <?php 
 
    //Incluir 
 
    include('../includes/mysql.php'); 
    //include('../includes/segur.php'); 
 
    //Recibe datos del formulario de login 
    $usuario = $_POST['usuario']; 
    $pass = $_POST['pass']; 
 
    //Comprueba si existe el usuario 
 
    $sql = "SELECT * FROM Cuentas WHERE Usuario = '".$usuario."' AND Contrasena = '".$pass."'"; 
    $query = mysql_query($sql); 
 
    foreach (mysql_fetch_array($query) as $fila) { 
        if ($fila[0] != ""){ 
            session_star(); 
            session_register("autentificado"); 
            session_register("nusuario"); 
            $autentificado = "SI"; 
            $nusuario = $fila[0]; 
            header ("Location: ../panel-control.php"); 
        } else { 
            header("Location: ../login.php"); 
        } 
 
    } 
?>   

Muchas gracias y saludos.

[ocp001a]

No estoy seguro si el problema es que tratas de meter el array de la consulta directo en el foreach o si es por que el array está vacío (no es array).

Imprime lo que te devuelva la consulta para verificar que hay datos.

[JuninhoP]

Iniciado por ocp001a

No estoy seguro si el problema es que tratas de meter el array de la consulta directo en el foreach o si es por que el array está vacío (no es array).

Imprime lo que te devuelva la consulta para verificar que hay datos.

He probado a sustituir el contenido a ejecutar del if y del else por mostrar un text pero nada, el mismo error.

[gjx2]

El error es por que foreach espera que el valor devuelto por mysql_fetch_array sea un array
y si en tu consulta no existe ningún usuario con ese password la funcion mysql_fetch_array en vez de retornate un array devolvera falso y por eso se produce el error.

Pero también mas allá del error hay algo mas que es asunto de logica.

Si andas buscando solo el usuario que coincida con el nombre y el password que le estas pasando, por que tienes que meter el resultado en un bucle ?

Puedes hacer uso de mysql_num_rows para comprobar si la consulta arrojo resultados

eje.

Código PHP:

$sql = "SELECT * FROM Cuentas WHERE Usuario = '".$usuario."' AND Contrasena = '".$pass."'"; 
    $query = mysql_query($sql); 
 
# Verifico si arrojo datos la consulta.
if (mysql_num_rows($query)){
 
# El usuario y password son validos  
$fila = mysql_fetch_array($query);      
 
} else {
 
# El usuario y password no son validos.
 
}
 

Saludos.

[ocp001a]

¿Pero no verificaste el contenido de la consulta?

Antes del foreach trata de hacer lo siguiente:
$resultado=mysql_fetch_array($query);
var_dump($resultado);

Si $resultado contiene algo, quizá el problema sea que en la versión de php que usas no puedes pasar el resultado directamente al foreach.

[JuninhoP]

Iniciado por gjx2

El error es por que foreach espera que el valor devuelto por [URL="http://www.php.net/manual/es/function.mysql-fetch-array.php"]mysql_fetch_array [/URL] sea un array
y si en tu consulta no existe ningún usuario con ese password la funcion [URL="http://www.php.net/manual/es/function.mysql-fetch-array.php"]mysql_fetch_array [/URL] en vez de retornate un array devolvera falso y por eso se produce el error.

Pero también mas allá del error hay algo mas que es asunto de logica.

Si andas buscando solo el usuario que coincida con el nombre y el password que le estas pasando, por que tienes que meter el resultado en un bucle ?

Puedes hacer uso de [URL="http://www.php.net/manual/es/function.mysql-fetch-row.php"]mysql_num_rows [/URL]para comprobar si la consulta arrojo resultados

eje.

Código PHP:

$sql = "SELECT * FROM Cuentas WHERE Usuario = '".$usuario."' AND Contrasena = '".$pass."'"; 
    $query = mysql_query($sql); 
 
# Verifico si arrojo datos la consulta.
if (mysql_num_rows($query)){
 
# El usuario y password son validos  
$fila = mysql_fetch_array($query);      
 
} else {
 
# El usuario y password no son validos.
 
}
 

Saludos.

Tienes toda la razón, el problema era de que no había ningún array, de ahí el error. De todas maneras, no se por qué me sale de qué no hay ninguno cuando en realidad si debe haberlo...

Muchas gracias por vuestras respuestas.

[JuninhoP]

Para no recargar el comentario anterior os comento el error que tengo.

He simplificado de momento el código de comprobación de usuario para que me diga con un comentario si existe en la tabla de Cuentas de la base de datos el usuario y contraseña enviados.

He aquí el código del formulario:

Código HTML:

<html xmlns="http://www.w3.org/1999/xhtml" xml:lang="es" lang="es"> 
 
    <head>
        <title>Login</title>
    </head>
 
    <body>
        <h1> Formulario de Acceso</h1>
        <form name ="login" enctype="multipart/form-data" method="POST" action="actions/comprobar-login.php">
            <label for="usuario"> Usuario: </label> <br />
            <input type="text" name="usuaro" id="usuario"> <br />
            <label for="pass">Password: </label> <br />
            <input type="text" name="pass" id="pass"> <br />
            <input type="submit" value= "Entrar">
 
        </form>
    </body>
 
</html>
 

Y aquí el codigo de comprobación de usuario:

Código PHP:

<?php
 
    //Incluir
 
    include('../includes/mysql.php');
    //include('../includes/segur.php');
 
    //Recibe datos del formulario de login
    $usuario = $_POST['usuario'];
    $pass = $_POST['pass'];
 
    //Comprueba si existe el usuario
 
    $sql = "SELECT * FROM Cuentas WHERE Usuario = '".$usuario."' AND Contrasena = '".$pass."'";
    $query = mysql_query($sql);
 
    if (mysql_num_rows($query)){
        $fila = mysql_fetch_array($query); 
        echo 'Hay datos';
        } else {
            echo 'No hay datos';
        }
?>
 

El caso es que en la base de datos si que tengo ese usuario y esa contraseña, pero siempre me sale de que no hay datos, por qué puede ser? Gracias de nuevo.

[gjx2]

Verifica por que el campo de usuario se llama usuaro y lo estas recibiendo como $_POST['usuario']

Saludos.

[JuninhoP]

Iniciado por gjx2

Verifica por que el campo de usuario se llama usuaro y lo estas recibiendo como $_POST['usuario']

Saludos.

Mira que he revisado el código mil veces para que no fuera un fallo de ese tipo...
Muchas gracias!