Error en Sentencia SQL

[rossijona]

Buenas muchachos, bueno tengo un problema al intentar comparar datos de un formulario (login) con una DB de MySql, e buscado solución y e encontrado respuestas, pero no doy con mi problema especificamente por esa razon posteo, Muchas gracias desde ya!.

El error muy famoso:
"Problemas al tomar los datosYou have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near ' contrasena='asdadasd'' at line 1"

ahora el código completo desde la conexión a la DB:

Código:

<?php 
$usuario=$_REQUEST['usuario'];
$contrasena=$_REQUEST['contrasena'];

//contecto a DB
$conexion=mysql_connect("xxxxxxxxx","xxxxxxxxx","xxxxxxxxxxx") or die ("Problemas para conectar a la DB");

//selecciono tabla
mysql_select_db("xxxxxxxxxxx",$conexion) or die ("Problemas al seleccionar la Tabla");

//tomo los datos Usuario y Contraseña en este caso
$datos=mysql_query("SELECT usuario, contrasena FROM usuarios WHERE usuario='$_REQUEST[usuario]', contrasena='$_REQUEST[contrasena]'", $conexion) or die ("Problemas al tomar los datos".mysql_error());

if ($reg=mysql_fetch_array($datos))
{if ($usuario!=$reg) {echo "El usuario no existe.";}
else {if ($contrasena==$reg){
setcookie("usuario",$usuario,0,"/"); header("location: index.php?pag=principal");}
else {echo "Esa no es tu contraseña.";}
}};
mysql_close($conexion);
?>

Un fuerte abrazo!

[pateketrueke]

las múltiples comparaciones de un WHERE deben ser unidas con los operadores lógicos AND/OR y no con comas....

eso deberías estudiarlo en una referencia de SQL, porque de PHP nada que ver!!

[Nano_]

Saludos

Cuando tu indicas condición WHERE en el SELECT, no debes separarlo por comas sino por un operador lógico ya sea AND o OR

[rossijona]

Iniciado por Nano_

Saludos

Cuando tu indicas condición WHERE en el SELECT, no debes separarlo por comas sino por un operador lógico ya sea AND o OR

Buenas,

quieres decir que sería de este modo:

Código:

SELECT usuario, contrasena FROM usuarios WHERE usuario='$_REQUEST[usuario]' AND contrasena='$_REQUEST[contrasena]'

?

espero la respuesta, gracias!

[pateketrueke]

Iniciado por rossijona

espero la respuesta, gracias!

¡intentalo y nos dices!

[rossijona]

Iniciado por pateketrueke

¡intentalo y nos dices!

Justito estaba editando el tema y al enviar la edición el foro no me permitio porque estabas posteando

escribí esto:

...espero la respuesta, gracias!

Muchas gracias, ya lo pude resolver (tal cual, cambie la coma por el AND), ahora me queda la parte de abajo de comparar los datos ingresados por el usuario (usuario y pass) y si estos son verdaderos crear la cookie para luego levantar en el index. (en caso de que no me salga postero en este mismo tema).


Gracias!.

Bueno, no me quiere quedar la cuestion es que si el usuario y la pass son correctas crear una cookie, en el caso contrario mostrar un mensaje con un enlace.

Mi cód es:

Código:

//tomo los datos Usuario y Contraseña en este caso
$datos=mysql_query("SELECT usuario, pass FROM usuarios WHERE usuario='$_REQUEST[usuario]' AND pass='$_REQUEST[pass]'", $conexion) or die ("Problemas al tomar los datos".mysql_error());
//comparo los datos
if ($reg=mysql_fetch_array($datos))
if ($usuario==$reg('usuario'))
if ($pass==$reg('pass'))
setcookie ("usuario",$usuario,0,"/"); header ("location: http://laboratorio.host22.com/index.php?pag=principal");
else {echo "Usuario o Contraseña incorrectos <a href='http://laboratorio.host22.com/index.php?pag=principal'>Volver</a>";};
mysql_close($conexion);
?>

El mensaje de error es:
Parse error: syntax error, unexpected T_ELSE in /home/a4177777/public_html/bienvenido.php on line 24

Ya esta resuelto, volvi a escribir "else" de nuevo

[JoseTejada]

Después de la llave de cierre del else has puesto un punto y coma... quítala...
además pon esta línea dentro de uno de los if

Código PHP:

  header ("location: http://laboratorio.host22.com/index.php?pag=principal");