AYUDA como restringir caracteres en un registro

[spider_boy]

¿Quieres validar el nombre de usuario al hacer la consulta, o al devolver el dato de la BD? Supongo que es la primera, y sería algo así :

Código PHP:

if(!preg_match("/^[a-zA-Z0-9áéíóúàèìòùÁÉÍÓÚÀÈÌÒÙÑñ\s]{2,30}+$/", $login_usuario)) {
     echo "Nombre de usuario inválido";
} else {
    $sql="SELECT id FROM usuarios_datos WHERE login = '$login_usuario'";
    $result = mysql_query($sql);
    if($row_usuario = mysql_fetch_array($result)){
        $error_login = "Ya se encuentra un usuario registrado con ese login.<br>Intenta otra vez!<br>";
        print_r($row_usuario);
        $todo_ok = 1;
    }
}
 

[xosgon]

ok, el problema que tengo es que el print_r me da problemas, parece que cuando el usuario existe todo funciona bien y da el error correspondiente pero en la parte visible de la web se rompe y sale un código (Estilo los errores apache) que pone "Array ([0]=>4 [id]=.....", y si borro el print_r todo funciona bien pero desaparece ese error. Puedes decirme la función que ese print_r tiene en mi código y que pasa si lo quito.

Te pongo el código tal cual.

Código PHP:

<?
include("...");
mbanner($titulo_sitio,$nombre_usuario_session,$apellido_usuario_session,$usuario_usuario_session);
$incluir = "";
if($procesa == "si"){
    $todo_ok = 0;
    $sql="SELECT id FROM usuarios_datos WHERE email = '$email'";
    $result = mysql_query($sql);
    if($row_usuario = mysql_fetch_array($result)){
        $error_email = "Ya se encuentra un usuario registrado con ese email.<br>Intenta otra vez!<br>";
        $todo_ok = 1;
    }
            
    $sql="SELECT id FROM usuarios_datos WHERE login = '$login_usuario'";
    $result = mysql_query($sql);
    if($row_usuario = mysql_fetch_array($result)){
        $error_login = "Ya se encuentra un usuario registrado con ese login.<br>Intenta otra vez!<br>";
        print_r($row_usuario);
        $todo_ok = 1;
    
    }
 
    if(!preg_match("/^[a-zA-Z0-9áéíóúàèìòùÁÉÍÓÚÀÈÌÒÙÑñ\s]{2,30}+$/",$nombre)){
        $error_nombre = "Debe escribir un nombre válido.";
        $todo_ok = 1;
    }
 
    if(!preg_match("/^[a-zA-Z0-9áéíóúàèìòùÁÉÍÓÚÀÈÌÒÙÑñ\s]{2,30}+$/",$apellido)){
        $error_apellido = "HOLAAA.";
        $todo_ok = 1;
    }
 

Como de costumbre spider_boy eres el mejor.

[eulloa]

A ver si te he entendido algo. Pasa que no estás cerrando con un else las condiciones, así que las validará todas como es lógico.
¿Porqué repites la consulta a la BD preguntando si el usuario existe?
Lo de la validación ¿no habíamos quedado en que lo hicieras primero y luego la consulta?
Me parece que lo que te falta es algo de estudio

[xosgon]

donde repito la consulta...?, en PHP estoy empezando y se lo justo, este código lo tenia ya hecho, si eres tan amable puedes decirme según tu cual es el problema basandote en mi código.

[eulloa]

Sin verificar mucho la lógica de tu code

Código PHP:

include("...");
mbanner($titulo_sitio,$nombre_usuario_session,$apellido_usuario_session,$usuario_usuario_session);
$incluir = "";
    if(!preg_match("/^[a-zA-Z0-9áéíóúàèìòùÁÉÍÓÚÀÈÌÒÙÑñ\s]{2,30}+$/",$nombre)){
        $error_nombre = "Debe escribir un nombre válido.";
        $todo_ok = 1;
    } else if {
        if(!preg_match("/^[a-zA-Z0-9áéíóúàèìòùÁÉÍÓÚÀÈÌÒÙÑñ\s]{2,30}+$/",$apellido)){
            $error_apellido = "HOLAAA.";
            $todo_ok = 1;
        }
    } else {
        if($procesa == "si"){
        $todo_ok = 0;
        $sql="SELECT id FROM usuarios_datos WHERE email = '$email'";
        $result = mysql_query($sql);
        if($row_usuario = mysql_fetch_array($result)){
            $error_email = "Ya se encuentra un usuario registrado con ese email.<br>Intenta otra vez!<br>";
            $todo_ok = 1;
        } else if {
            $sql="SELECT id FROM usuarios_datos WHERE login = '$login_usuario'";
            $result = mysql_query($sql);
                if($row_usuario = mysql_fetch_array($result)){
                    $error_login = "Ya se encuentra un usuario registrado con ese login.<br>Intenta otra vez!<br>";
                    print_r($row_usuario);
                    $todo_ok = 1;
                }
            }
 
        }
    }
 

[xosgon]

Ahora si que me mataste, lo que pones ai esta al reves de como lo tengo yo. Y te lo agradezco muchisimo pero no comprendo nada de nada. Lo que menos entiendo es pq subes las comprobaciones de nombre y apellidos (esas son solo dos pero siguen, hay muchisimas más, solo puse esas), y no me queda claro donde repito la consulta del usuario y en que afecta eso a que si tengo el print_r se rompa la web y si no lo tengo no.

[eulloa]

Debes hacer TODAS las comprobaciones antes de consultar la BD, porq sino correrías el riesgo de consultar en bano. ¿no? Aunque ahora lo miro y no consultas contra nombre y apellido, ummmm ¿para q comprobarlos entonces? Me apoyaré en lo que dices que no pusiste todas las comprobaciones.......
Nop, no la repites la consulta, por eso la puse arriba así. Error mío.........
Si el error es el q posteaste arriba (unexpected $end) era una llave q te faltaba de cierre. Si es otro pues lo posteas
salu2

[xosgon]

Buenas Tardes,

Veamos, el problema que tengo es el siguiente, tengo un formulario de registro de usuario que almacena datos en una SQL. El campo del "Nombre de Usuario" tiene una validación para ver si el nombre ya lo tiene otro usuario y de ser asi mostrar un mensaje de error.

el problema radica en que cuando el nombre esta usado o ya existe, efectivamente genera el mensaje de error bajo la celda pero tambien genera un código que se cuela entre el cuerpo de la web que pone (Array ( [0] => 4 [id] => 4 ), concretamente lo mete entre el logotipo de la web y el cuerpo del formulario, muy similar a los errores tipicos de apache que muestra cuando falta un include o algo similar.

El caso es que al borrar el "print_r" el problema desaparece, mi duda es, pq muestra ese código en el medio de la web...? y Como lo soluciono?, os dejo el código. El print_r esta en la linea 18:

Código PHP:

<?
include("...");
mbanner($titulo_sitio,$nombre_usuario_session,$apellido_usuario_session,$usuario_usuario_session);
$incluir = "";
if($procesa == "si"){
    $todo_ok = 0;
    $sql="SELECT id FROM usuarios_datos WHERE email = '$email'";
    $result = mysql_query($sql);
    if($row_usuario = mysql_fetch_array($result)){
        $error_email = "Ya se encuentra un usuario registrado con ese email.<br>Intenta otra vez!<br>";
        $todo_ok = 1;
    }
            
    $sql="SELECT id FROM usuarios_datos WHERE login = '$login_usuario'";
    $result = mysql_query($sql);
    if($row_usuario = mysql_fetch_array($result)){
        $error_login = "Ya se encuentra un usuario registrado con ese login.<br>Intenta otra vez!<br>";
        print_r($row_usuario);
        $todo_ok = 1;
    
    }
 
    if(!preg_match("/^[a-zA-Z0-9áéíóúàèìòùÁÉÍÓÚÀÈÌÒÙÑñ\s]{2,30}+$/",$nombre)){
        $error_nombre = "Debe escribir un nombre válido.";
        $todo_ok = 1;
    }
 
    if(!preg_match("/^[a-zA-Z0-9áéíóúàèìòùÁÉÍÓÚÀÈÌÒÙÑñ\s]{2,30}+$/",$apellido)){
        $error_apellido = "HOLAAA.";
        $todo_ok = 1;
    }
 

El código continua, pero el que pego es el relevante.

[Marvin]

Seria bueno que revisaras esto:

print_r()

Tal vez necesitas print "texto"; o echo "texto";

Suerte!

[xosgon]

si en este caso el print_r lo tengo asi print_r($row_usuario) y esta bien en base a lo que tu me pasas pero el problema esta en que me muestra el resultado del array en medio de la web, cuando ese resultado no deberia mostrarlo no..?

[spider_boy]

A ver...

Código PHP:

$sql="SELECT id FROM usuarios_datos WHERE login = '$login_usuario'";
    $result = mysql_query($sql);
    if($row_usuario = mysql_fetch_array($result)){
        $error_login = "Ya se encuentra un usuario registrado con ese login.<br>Intenta otra vez!<br>";
        print_r($row_usuario);
        $todo_ok = 1;
    }
 

Eso, de por si, en parte, está malo xD.

Esta línea es incorrecta :

Código PHP:

if($row_usuario = mysql_fetch_array($result)){
 

Cuando usas solo un '=', se le llama asignación, para lo cual no te sirve, porque tú deseas comparar, así que debería ser un doble '=', por otro lado, esa comparación estaría mal de todas formas, porque $row_usuario jamás será igual al valor devuelto por mysql_fetch_array. Lo que pretendes hacer es lo siguiente :

Código PHP:

$sql="SELECT id FROM usuarios_datos WHERE login = '$login_usuario'";
    $result = mysql_query($sql);
    $row_usuario = mysql_fetch_array($result);
    // Preguntamos si hay valor alguno, si lo hay, es porque el usuario ya existe.
    if(mysql_num_rows($result) > 0){
        $error_login = "Ya se encuentra un usuario registrado con ese login.<br>Intenta otra vez!<br>";
        #print_r($row_usuario);
        $todo_ok = 1;
    }
 

Con respecto al print_r, sería mejor que nos mostraras que es lo que imprime. Por cierto... ¿para qué pones print_r dentro de ese if? No le veo sentido.

[eulloa]

¿Y si no debería mostrarlo para q el print_r?
¿Q ya no había iniciado otro post cone ste MISMO tema?
¿No resolviste con lo q allí se te dijo?

[spider_boy]

print_r no tiene sentido dentro de ese if. Simplemente bórralo, está sobrando.