[SOLUCIONADO] ¿Está bien hecha esta consulta? Tengo Dudas

[basura1973]

Foreros hola, espero que se encuentren muy bien. Mi pregunta es la siguiente:

Resulta que estoy estudiando acerca de las sesiones en PHP y estoy haciendo un sistema de logueo en el cual el usuario pueda entrar con su nombre de usuario o su email, obviamente ingresando su password. El campo de texto para ese usuario o email lo llame "login" y en el script de logueo lo almacene en una variable $_POST con el mismo nombre. Yo lo que deseo hacer es que al consultar la base de datos se me verifique si el usuario o email existen ahi y la contraseña es correcta, entonces se me incie la sesión. La consulta que hice fue esta. ¿Está correcta? O cómo la debo hacer

Código:

Código:

$sql = "SELECT * FROM prusuarios WHERE (usuario='$login' XOR email='$login') AND pass='$pass'";

Muchísimas gracias por su ayuda.

[leonardo_josue]

Hola basura1973:

En realidad la consulta es correcta, sin embargo hay algunos detalles que debes considerar.

1. Como buena práctica, trata de evitar los SELECT * en tus consultas, en lugar de esto lista todos los campos que realmente vas a necesitar. Incluso si fuera el caso y en realidad necesitas TODOS los campos de tu tabla (cosa que muy rara vez ocurre) es mejor listar la totalidad de los campos a poner el comodín asterisco (*).

2. Mucho ojo con el operador XOR, ya que no es lo mismo que el operador OR... no sé como tienes organizados tus datos, pero si fuera el caso y tanto tu campo USUARIO como tu campo LOGIN tuvieran el mismo valor, el resultado de XOR es FALSO (Sería algo como en el primer caso de este ejemplo):

Código MySQL:

mysql> SELECT 1 XOR 1;
        -> 0
mysql> SELECT 1 XOR 0;
        -> 1
mysql> SELECT 1 XOR NULL;
        -> NULL
mysql> SELECT 1 XOR 1 XOR 1;
        -> 1
 

Observa la diferencia con el operador OR:

Código MySQL:

mysql> SELECT 1 OR 1;
        -> 1
mysql> SELECT 1 OR 0;
        -> 1
mysql> SELECT 0 OR 1;
        -> 1
mysql> SELECT 0 OR 0;
        -> 0
 

3. Finalmente, dado que estás utilizando variables para construir tu sentencia SQL, debes verificar antes que estas tengan un valor, de lo contrario el resultado puede ser incierto.

Saludos
Leo.

[basura1973]

Leo excelente la explicación mi hermano, te cuento que había acabado de hacer un script de prueba y me funcionó perfectamente. Pero no sabía acerca de las aclaraciones que me hiciste y me pareció demasiado importante e interesante. De hecho me pusiste a consultar y encontré que OR, significa "y/o" y XOR, significa "o" lo que quiere decir que a pesar de ser muy similares pueden llevarlo a uno a cometer un error en la cconsulta si no la usamos de forma adecuada. De verdad mil y mil gracias mi hermano por tu colaboración, ya te dejé mmi voto en agradecimiento. Un abrazo para vos.