Comprobar nombre de usuario y email.

[nobo]

Hola tengo esto que me genero el dreamweaver para verificar que el email introducido a la hora de registrarse no estaba ya en la base de datos.

// *** Redirect if username exists
$MM_flag="MM_insert";
if (isset($_POST[$MM_flag])) {
$MM_dupKeyRedirect="alta_emailrepetido.php";
$loginUsername = $_POST['strEmail'];
$login = $_POST['strNombre'];
$LoginRS__query = sprintf("SELECT * FROM tblusuario WHERE strEmail=%s OR strNombre=%s", GetSQLValueString($loginUsername, "text"), GetSQLValueString($login, "text"));
mysql_select_db($database_conexionzulo, $conexionzulo);
$LoginRS=mysql_query($LoginRS__query, $conexionzulo) or die(mysql_error());
$loginFoundUser = mysql_num_rows($LoginRS);

//if there is a row in the database, the username was found - can not add the requested username
if($loginFoundUser){
$MM_qsChar = "?";
//append the username to the redirect page
if (substr_count($MM_dupKeyRedirect,"?") >=1) $MM_qsChar = "&";
$MM_dupKeyRedirect = $MM_dupKeyRedirect . $MM_qsChar ."requsername=".$loginUsername;
header ("Location: $MM_dupKeyRedirect");
exit;
}
}

Yo ademas queria que comprobase el nombre de usuario. Asi que le añadi el OR strNombre=%s

Bien ahora lo esta haciendo bien, pero no me diferencia si es el usuario o el email. Me gustaria saber si puedo aprovechar ese codigo para diferenciarlos o tendria que hacer uno nuevo para cada cosa. Para el user y para el email.

Gracias de antemano !

[hasdpk]

Cambia el OR por un AND, aunque no he leído el código que te ha generado.

[snowmanchip]

Hola,
El AND exigirá que ambas condiciones se cumplan..el or está bién..lo malo es que como validas las filas no sabrás cuál de los dos campos está repetido..puedes enviar un mensaje como el nombre de usuario y /o el email está en uso...o hacer dos validaciones 1 para cada variable...para seguir con la lógica de tu script..

[hasdpk]

Lo que te hace el OR es que si el nombre o el email existen, entra, no que coinciden en la misma fila, por eso el AND.

Para que lo entiendas, tenemos varios usuario:

Código:

Nombre      | Correo
--------------------------------------------------
Juan        | [email protected]
Pedro       | [email protected]
Estefanía   | [email protected]

Si nosotros mandamos la consulta como tu la tienes del siguiente modo:

Código SQL:

SELECT * FROM tblusuario WHERE strEmail='[email protected]' OR strNombre='Juan'
 

Nos devolverá 2 filas con todos los datos del usuario:

Código:

Juan        | [email protected]
Estefanía   | [email protected]

Y como el script que te ha generado el Dreamweaver con que simplemente devuelva algo (en este caso dos filas), pues sigue trabajando y por eso te da fallo.

Con el AND eso no pasa, si hacemos la misma consulta no devolverá nada, porque no coincide el nombre Juan con el correo [email protected] en la misma fila.

[snowmanchip]

Aquí lo que nobo quiere es que si existe un nombre en una tabla o un mail en una tabla no se registre, con el and si un nombre de usuario ya existe pero se ingresa con otro mail lo dejaría crear el usuario con el mismo username, yo entiendo que lo que quiere validar es que no existan dos username iguales y que además no existan dos correos electrónicos iguales en distintas filas...por eso el OR:

Esta es la validación que hace:

Código PHP:

//if there is a row in the database, the username was found - can not add the requested username
if($loginFoundUser)
 

si hay una fila con esos parámetros que no se ingrese....

Bien ahora lo esta haciendo bien, pero no me diferencia si es el usuario o el email.

La consulta no tiene ningún error y no arroja error...nobo lo que quiere es que le indique si es el email o el username el que está repetido en la tabla,

Saludos

[nobo]

Si eso es... El AND lo unico que hace es que las 2 condiciones se tengan que cumplir para que reedireccione a la pagina de error. Con el OR lo que hace es que si uno u otro ya estan en la base de datos no lo registre. Lo esta haciendo bien. Pero me gustaria efectivamente como dice snowmanchip que diferencie entre que es lo que esta repetido, si el email o el nombre de usuario...

Ahora bien... Me pregunto si necesito repetir el codigo, exactamente igual y poner strEmail en uno y strNombre en otro. Si debieran ser igual o... Por el contrario para ahorrar codigo si con este que ya tengo puedo hacer que valide los dos ;)

No se si me explique bien :S

snowman si me entendio. Alguna sugerencia de como hacerlo snowman ?

[snowmanchip]

Lo mejor en esto es probar...si usas el código actual no diferenciará la variable que se repite ya que lo que busca es una fila y para cualquiera de los dos valores te arrojará el mismo mensaje...prueba repitiendo cambiando la validación por email y la variable loginfounderuser por $validaemail para que mantengas la estructura de tu script..

Saludos

[Sourcegeek]

Tu condición está mal, debe ir algo como:

Código PHP:

if($loginFoundUser == 0){
    // usuario no existe, continuar registro....
}else{
    // usuario(s) ya existe(n)
}
 

[nobo]

sourcegeek, pues funcionar funciona...

Y eso que pones es para el user y para el mail ?

[snowmanchip]

Nobo prueba este código:

Código PHP:

//Array con mensajes de error
$errores=array();
//después de $loginFoundUser = mysql_num_rows($LoginRS);
$prueba=mysql_fetch_array($LoginRS);
//Después de if($loginFoundUser){
if($$loginUsername == $prueba['strEmail'])
//Mensaje de error si el mai existe
      $error ['username']= "$loginUsername ya está en uso, por favor elige otro email.";
      if($login == $prueba['strNombre'])
      //Mensaje de error si el nombre de usuario existe
      $error ['mail']= "$login ya está en uso, por favor elige un nombre de usuario diferente.";
   
  }
 

en el lugar del html que quieras mostrar los errores pones esto:

Código PHP:

<?php
     if ($errores){
        echo '<ul>';
        foreach($errores as $alerta){
            echo "<li>$alerta<li>\n";
        }
        echo "</ul>";
         
     }
     
     ?>
 

Con esto puedes usar la consulta como la tienes y también te mostrará si es el mail , el nombre de usuario el repetido o ambos..
Saludos

[snowmanchip]

Hasdpk...no comprendo el motivo del karma negativo que me has puesto, realmente desmotiva esto cuando se trata de un usuario con experiencia, lo peor de todo es que tu comentario fué equivocado , ni yo pensé siquiera en ponerte una nota negativa, ya que todos nos equivocamos y además estabas tratando de ayudar como la mayoría aquí...
Saludos

[hasdpk]

Cierto, mi afirmación estaba equivocada en relación a la consulta. Te lo puse porque simplemente comentastes mi errata y te fuistes sin ofrecer una solución, como puse de manifiesto en el karma.

PD: Estas cosas se solucionan por MP.

[snowmanchip]

Por MP estas cosas prefiero que queden publicas..porque sencillamente no me parece...y si es así tu en dos oportunidades no ofreciste ninguna solución, es más empeoraste el tema...
Afortunadamente no vivo del karma, creo que te molestó tu error...
Te dejo una cita:

“No hay necesidad de apagar la luz del prójimo para que la nuestra brille”
— BERNARD BARUCH

Saludos

[Sourcegeek]

Continuando con el tema, cometí un error de no comprender bien lo que querías lograr jeje.
Si lo necesitas hacer con una sola consulta, puedes hacer algo así:

Código MySQL:

SELECT *,
CASE resultado WHEN strEmail = 'el_email' AND strNombre = 'el_nombre' THEN 'ambos'
WHEN strEmail = 'el_email' AND strNombre != 'el_nombre' THEN 'solo_email'
WHEN strEmail != 'el_email' AND strNombre = 'el_nombre' THEN 'solo_nombre'
END
FROM tblusuario WHERE strEmail = 'el_email' OR strNombre = 'el_nombre'
 

Se traduce a:
Devuelve resultado = ambos cuando ambos coinciden
Devuelve resultado = solo_email cuando sólo el email coincide
Devuelve resultado = solo_nombre cuando sólo el nombre coincide
No devuelve filas cuando no hay coincidencias

Saludos!

[hasdpk]

Iniciado por snowmanchip

Por MP estas cosas prefiero que queden publicas..porque sencillamente no me parece...y si es así tu en dos oportunidades no ofreciste ninguna solución, es más empeoraste el tema...
Afortunadamente no vivo del karma, creo que te molestó tu error...
Te dejo una cita:


Saludos

Sí, cierto, empeoré el tema porque entendí mal el concepto, pero eso no descarta que te fueras sin ofrecer una solución, ¿en que mejor eso la comunidad?.

"El sabio no dice lo piensa, sino piensa lo que dice"
Dicho por un sabio.

[snowmanchip]

Según tu lógica entonces tú y Sourcegeek merecen karma negativo por su primera respuesta?Así este foro sería negativismo absoluto...si crees que es correcto lo que hiciste ok,
Bueno no quiero comenzar a empeorar más el tema..pero al menos ahora que discutí un poco el karma negativo ya me lo gané y no fue gratuito...

[hasdpk]

Mira, para no irte la vida en el Karma estás dandole mucho pie a este tema.

A mi me han puesto Karma negativo por decir por ejemplo que una función de PHP devuelve la variable de un modo y en realidad era de otra, o simplemente porque no entendieron lo que expliqué, pero no voy detrás de la gente pidiendo explicaciones, para ello están los comentarios en los Karmas.


PD: Es el último post que escribo en relación a ésto.

[nobo]

Bah! Dejar lo del karma que me interesa mas lo de sourcegeek...

Sourcegeek me gusta tu respuesta, acabo de llegar a casa y voy a cenar en cuanto tenga un rato lo reviso. Pero asi por encima... La primera la podriamos suprimir, no? Es decir para que coincidan los 2 antes ha tenido que coincidir alguno de ellos...

En cuanto tenga 5min la vuelvo a montar a ver si funciona con mis datos y como lo tengo hecho ok? Y pongo los resultados ;)

PD: Que mas da el karma?