Problema con registro de usuario

makeinn · #1 (**permalink**) 09/05/2009, 08:58

Buenas este es mi primer post en forosdelweb. Estoy intentando hacer un script para agregar usuarios a una base de datos, cabe destacar que tengo un nivel de principiante en php y a lo mejor es un fallo tonto o algo asi.

Bueno os dejo el codigo para ver si me podeis echar una mano, el problema es que no se ejecuta y no me da ningun error mas.

Código PHP:

  $link=mysql_connect($server,$dbuser,$dbpass); 
//declaro variables 
$nombre = $_POST['nombre']; 
$apellidos = $_POST['apellidos']; 
$login = $_POST['login']; 
$pass = $_POST['pass1']; 
$email = $_POST['email']; 
//compruebo que no haya ningun campo vacio 
if($nombre == '' or $apellidos == '' or $login == '' or $pass == '' or $email == ''){ 
echo "*Tienes que rellenar todos los campos"; 
} 
$sql = "SELECT login FROM usuarios WHERE login='$login' "; 
$result = mysql_query($sql,$link); 
//si no estan vacios compruebo que el nick no este usado 
else{$row = mysql_fetch_array($result) or die (mysql_error()); 
echo "El login ya esta siendo usado por otra personaje, por favor elige otro."; 
} 
//si el nick no esta usado inserto el registro 
else{mysql_query("INSERT INTO usuarios (nombre,apellidos,login,password,email) VALUES ('$nombre','$apellidos','$login','$pass','$email')"); 
echo "Ya estas registrado"; 
} 
 
?>

the_web_saint · #2 (**permalink**) 09/05/2009, 09:07

Código php:

Ver original$link=mysql_connect($server,$dbuser,$dbpass);
//declaro variables
$nombre = $_POST['nombre'];
$apellidos = $_POST['apellidos'];
$login = $_POST['login'];
$pass = $_POST['pass1'];
$email = $_POST['email'];
//compruebo que no haya ningun campo vacio
if($nombre == '' or $apellidos == '' or $login == '' or $pass == '' or $email == ''){
echo "*Tienes que rellenar todos los campos";
}
$sql = "SELECT login FROM usuarios WHERE login='$login' ";
$result = mysql_query($sql,$link);
//si no estan vacios compruebo que el nick no este usado
if{$row = mysql_num_rows($result)>0){
echo "El login ya esta siendo usado por otra personaje, por favor elige otro.";
}else{
mysql_query("INSERT INTO usuarios (nombre,apellidos,login,password,email) VALUES ('$nombre','$apellidos','$login','$pass','$email')");
echo "Ya estas registrado";
}
 
?>

Intenta de esta forma.

Saludos

makeinn · #3 (**permalink**) 09/05/2009, 09:14

Nada, sigue sin funcionar, ademas es que no me imprime ningun error en la pantalla.

el-eles · #4 (**permalink**) 09/05/2009, 09:24

Hola

prueba esta versión:

Código PHP:

  <?php  
$link=mysql_connect($server,$dbuser,$dbpass); 
mysql_select_db("nombre_de_tu_base_de_datos"); 
//declaro variables 
$nombre = $_POST['nombre']; 
$apellidos = $_POST['apellidos']; 
$login = $_POST['login']; 
$pass = $_POST['pass1']; 
$email = $_POST['email']; 
 
//compruebo que no haya ningun campo vacio 
if($nombre == '' or $apellidos == '' or $login == '' or $pass == '' or $email == '') 
{ 
    echo "*Tienes que rellenar todos los campos"; 
} 
 
$sql = "SELECT login FROM usuarios WHERE login='$login' "; 
$result = mysql_query($sql,$link); 
 
//si no estan vacios compruebo que el nick no este usado 
if(mysql_num_rows($result) > 0) 
{ 
    echo "El login ya esta siendo usado por otra personaje, por favor elige otro."; 
} 
else 
{ 
    mysql_query("INSERT INTO usuarios (nombre,apellidos,login,password,email) VALUES ('$nombre','$apellidos','$login','$pass','$email')"); 
    echo "Ya estas registrado"; 
} 
?>

Así debería de funcionar

fcdragons · #5 (**permalink**) 09/05/2009, 09:24

vale emm podrias decirnos si te sale algun error, o solo no te va y ya esta, y por lo que veo, no pones el nombre de la base de datos, tan solo pones el host, el usuario y la contraseña, y ningun include a ningun archivo de conexion donde se encuentren las variables $server,$dbuser,$dbpass

podrias poner el codigo completo?

Código php:

Ver original<?php  
$link=mysql_connect('host', 'user', 'contraseña'); 
$db_select = mysql_select_db('nombre de la DB', $link);
//declaro variables 
$nombre = $_POST['nombre']; 
$apellidos = $_POST['apellidos']; 
$login = $_POST['login']; 
$pass = $_POST['pass1']; 
$email = $_POST['email']; 
 
//compruebo que no haya ningun campo vacio 
if($nombre == '' or $apellidos == '' or $login == '' or $pass == '' or $email == '') 
{ 
    echo "*Tienes que rellenar todos los campos"; 
} 
 
$sql = "SELECT login FROM usuarios WHERE login='$login' "; 
$result = mysql_query($sql,$link); 
 
//si no estan vacios compruebo que el nick no este usado 
if(mysql_num_rows($result) > 0) 
{ 
    echo "El login ya esta siendo usado por otra personaje, por favor elige otro."; 
} 
else 
{ 
    mysql_query("INSERT INTO usuarios (nombre,apellidos,login,password,email) VALUES ('$nombre','$apellidos','$login','$pass','$email')"); 
    echo "Ya estas registrado"; 
} 
?>

Prueba aver si asi te va

makeinn · #6 (**permalink**) 09/05/2009, 10:32

Pues la conexion la hago mediante la variable $link que es la que almacena los datos de la db, cuando hago mysql_query($result,$link)

Y antes de declarar variables tambien selecciono la base de datos, perdon por no ponerlo se me olvido al pegarlo: mysql_select_db($database,$link);.

Veis algo que pueda estar mal ? El error no me da ningun error, simplemente me sale la pagina vacia, en blanco.

Gracias por vuestra ayuda y saludos.

argy · #7 (**permalink**) 09/05/2009, 10:33

Código PHP:

  <?php 
$link=mysql_connect($server,$dbuser,$dbpass);  
mysql_select_db('db', $link); 
//declaro variables  
$nombre = $_POST['nombre'];  
$apellidos = $_POST['apellidos'];  
$login = $_POST['login'];  
$pass = $_POST['pass1'];  
$email = $_POST['email'];  
//compruebo que no haya ningun campo vacio  
if(empty($nombre) or empty($apellidos) or empty($login) or empty($pass) or empty($email )){  
echo "*Tienes que rellenar todos los campos";  
}  
else 
{ 
$sql = "SELECT login FROM usuarios WHERE login='$login' ";  
$result = mysql_query($sql,$link); 
if(mysql_num_rows($result)>0) 
{ 
         echo "usuario ya esta rtegistrado 
}  
else 
{ 
       mysql_query("INSERT INTO usuarios (nombre,apellidos,login,password,email) VALUES ('$nombre','$apellidos','$login','$pass','$email')");  
       if(mysql_affected_rows()>0) 
      { 
        echo "se ha registrado correctamente"; 
      } 
      else 
     { 
       echo "no se pudo registrar 
    }   
} 
}  
?>

además debes de encriptar el password con alguna de las funciones como md5,sha1

fcdragons · #8 (**permalink**) 09/05/2009, 12:06

lo que yo dije, se te olvido poner la db.

Código php:

Ver original<?php  
include ('config.php');
//declaro variables 
$nombre = $_POST['nombre']; 
$apellidos = $_POST['apellidos']; 
$login = $_POST['login']; 
$pass = $_POST['pass1']; 
$email = $_POST['email']; 
 
//compruebo que no haya ningun campo vacio 
if($nombre == '' or $apellidos == '' or $login == '' or $pass == '' or $email == '') 
{ 
    echo "*Tienes que rellenar todos los campos"; 
} 
 
$sql = "SELECT login FROM usuarios WHERE login='$login' "; 
$result = mysql_query($sql,$link); 
 
//si no estan vacios compruebo que el nick no este usado 
if(mysql_num_rows($result) > 0) 
{ 
    echo "El login ya esta siendo usado por otra personaje, por favor elige otro."; 
} 
else 
{ 
    mysql_query("INSERT INTO usuarios (nombre,apellidos,login,password,email) VALUES ('$nombre','$apellidos','$login','$pass','$email')"); 
    echo "Ya estas registrado"; 
} 
?>

Crea un archivo nuevo llamado config.php por ejemplo, y pon el include en el codigo

Ejemplo del archivo config.php

Código php:

Ver original<?
$link=mysql_connect($server, $dbuser, $dbpass); 
$db_select = mysql_select_db('nombre de la DB', $link);
$server = "Host mysql";
$dbuser = "Usuario mysql";
$dbpass = "contraseña mysql";
?>

Asi te deberia de funcionar

argy · #9 (**permalink**) 09/05/2009, 12:18

no importa si lo declaras en una función aparte la conexión, lo importante es que sea de manera correcta.

además no se olviden de utilizar el tag completo de php, ya que en el futuro van a tener serios problemas.

por último después del insert, debes de utilizar mysql_affected_rows()(como en el ejm que te envié) para saber si se insertó o no el registro y no mostrar el mensaje así por así.

fcdragons · #10 (**permalink**) 09/05/2009, 12:21

pero esque argy, las variables $server, $dbuser, $dbpass
no tienen valores, entonces si ponia eso no conectaba con nada

makeinn · #11 (**permalink**) 09/05/2009, 12:44

por partes, las variables de dbuser dbpass etc.. no las puse en el post pero si que las tengo en mi php y el script ahora con la modificacion que hizo argy si que funciona, pero mal. Solo procesa la sentencia de los campos vacios, aun que ponga un login que ya existe o haga un registro nuevo siempre me dice que faltan campos por rellenar, alguna sugerencia ?.

fcdragons · #12 (**permalink**) 09/05/2009, 16:04

Como argy se equivoco, se le olvido cerrar una comilla, te lo posteo:

Código php:

Ver original<?php 
$link=mysql_connect($server,$dbuser,$dbpass);  
mysql_select_db('db', $link); 
//declaro variables  
$nombre = $_POST['nombre'];  
$apellidos = $_POST['apellidos'];  
$login = $_POST['login'];  
$pass = $_POST['pass1'];  
$email = $_POST['email'];  
//compruebo que no haya ningun campo vacio  
if(empty($nombre) or empty($apellidos) or empty($login) or empty($pass) or empty($email )){  
echo "*Tienes que rellenar todos los campos";  
}  
else 
{ 
$sql = "SELECT login FROM usuarios WHERE login='$login' ";  
$result = mysql_query($sql,$link); 
if(mysql_num_rows($result)>0) 
{ 
         echo "usuario ya esta registrado"; 
}  
else 
{ 
       mysql_query("INSERT INTO usuarios (nombre,apellidos,login,password,email) VALUES ('$nombre','$apellidos','$login','$pass','$email')");  
       if(mysql_affected_rows()>0) 
      { 
        echo "se ha registrado correctamente"; 
      } 
      else 
     { 
       echo "no se pudo registrar"; 
    }   
} 
}  
?>

Prueba asi, habia 2 comillas sin cerrar y faltaba un ;
Asi te deberia funcionar, claro que poniendo el nombre de la db

makeinn · #13 (**permalink**) 10/05/2009, 08:49

Si las comillas y el ; ya se lo habia puesto yo pero aun asi sigue sin funcionar, solo hace la funcion de validar los campos vacios, ni registra, ni comprueba el login.

fcdragons · #14 (**permalink**) 10/05/2009, 09:58

Prueba a cambiar lo que te mira los campos vacios por esto

Código php:

Ver originalif(!empty($_POST['nombre']) && $_POST['apellidos'] && $_POST['login'] && $_POST['pass'] && $_POST['email']){

makeinn · #15 (**permalink**) 10/05/2009, 10:31

fcdragons, Nada :( sigue sin funcionar, igual es por que lo estoy haciendo mal, me explico. Tengo una carpeta a la que llamo registro osea seria misitio.com/registro/ entonces en el index de /registro/ meto el formulario y el php, todo junto, de tal forma que al acceder a la carpeta de /registro/ ves el formulario y cuando envias el formulario los datos del form van a esa misma pagina que es donde tambien esta metido el php que procesa el registro, puede que este mal o algo asi ?

fcdragons · #16 (**permalink**) 10/05/2009, 10:54

aver, enseña eñ formulario, y como conectas el formulario con el codigo PHP

makeinn · #17 (**permalink**) 10/05/2009, 11:18

perdon, post repetido

makeinn · #18 (**permalink**) 10/05/2009, 11:18

Código PHP:

 
            <h1>Registro</h1> 
            <?php  
 
$link=mysql_connect($server,$dbuser,$dbpass);   
mysql_select_db('db', $link);  
//declaro variables   
$nombre = $_POST['nombre'];   
$apellidos = $_POST['apellidos'];   
$login = $_POST['login'];   
$pass = $_POST['pass1'];   
$email = $_POST['email'];   
//compruebo que no haya ningun campo vacio   
if(!empty($_POST['nombre']) && $_POST['apellidos'] && $_POST['login'] && $_POST['pass'] && $_POST['email']){ 
echo "*Tienes que rellenar todos los campos";   
}   
else  
{  
$sql = "SELECT login FROM usuarios WHERE login='$login' ";   
$result = mysql_query($sql,$link);  
if(mysql_num_rows($result)>0)  
{  
         echo "usuario ya esta registrado";  
}   
else  
{  
       mysql_query("INSERT INTO usuarios (nombre,apellidos,login,password,email) VALUES ('$nombre','$apellidos','$login','$pass','$email')");   
       if(mysql_affected_rows()>0)  
      {  
        echo "se ha registrado correctamente";  
      }  
      else  
     {  
       echo "no se pudo registrar";  
    }    
}  
}   
?>

Código HTML:

<p>
			<form name="registro" method="post" action="index.php">		
				<p>
				<label>Login</label>
				<input name="login" type="text" size="30" />
				<label>Nombre</label>
				<input name="nombre"  type="text" size="30" />
				<label>Apellidos</label>
				<input name="apellidos" type="text" size="30">
				<label>Contraseña</label>
				<input name="pass1"  type="password" size="30" />
				<label>Repite contraseña</label>
				<input name="pass2"  type="password" size="30" />
				<label>Email</label>
				<input name="email" type="text" size="30" />
				<p>	
				<input class="button" type="submit" />		
				</p>		
			</form>

Pongo el php justo encima del form para que los mensajes de error o campos vacios salgan justo encima del formulario

newboy_master · #19 (**permalink**) 10/05/2009, 11:19

makeinn edita los datos de acceso a tu base de datos para mayor seguridad. No te solucionara el problema del registro de usuarios pero sí uno mayor.

Salu2.

fcdragons · #20 (**permalink**) 10/05/2009, 11:29

una cosa...
<form name="registro" method="post" action="index.php">

no se supone que tiene que ponerse ahi el nombre del archivo que hace la aciion de registrar...

Bueno, me dijiste que el index.php es la pagina del formulario, en action, pon el nombre del archivo que tiene los codigos php de insertar las cosas en la bd

makeinn · #21 (**permalink**) 10/05/2009, 11:41

fcdragons el index.php tambien tiene el script para agregar los datos, tengo formulario y el script en un mismo archivo para poder tenerlos todos en una url que seria misitio.com/registro/

no se si me explico bien

fcdragons · #22 (**permalink**) 10/05/2009, 12:08

emm vale, encontre otra cosa xP
mysql_select_db('db', $link);

ahi estas cambiando db por el nombre real de tu base de datos no¿?
porque si no es asi, es normal que no te inserte las cosas, despues de cada conexion, de cada select agrega esto:
or die(mysql_error)

Ejemplo
mysql_query("INSERT INTO usuarios (nombre,apellidos,login,password,email) VALUES ('$nombre','$apellidos','$login','$pass','$email') ")or die(mysql_error);

y si es un error de mySql te lo dira.

makeinn · #23 (**permalink**) 10/05/2009, 12:30

Ahora lo cambie, y lo puse asi, mysql_select_db('$database', $link);. Es que al copiar tu script lo cambie y no me di cuenta, pero nada esto sigue sin funcionar, no hace las funciones de validar campos vacios ni nada, simplemente pone que no se pudo registrar.

Edito:

Le he agregado lo de or die(mysql_error) como me has dicho y ahora si me lanza un error, pone mysql_error. Me imagino que no podra conectar con la base de datos pero es raro por que tengo todos los datos bien puestos o eso creo, la password la he cambiado por que no me di cuenta antes al postear los datos pero la cambie y sigue sin funcionar, y la base de datos no esta caida.

fcdragons · #24 (**permalink**) 10/05/2009, 12:44

vale, prueba a quitar las variables y ponr los datos directamente.
mysql_connect('host', 'usuario', 'pass');

makeinn · #25 (**permalink**) 10/05/2009, 12:48

Vale, ya esta. No habia echo el mysql_select_db. Pero sigo teniendo un problema, no me valida los campos vacios de tal forma que cuando accedes al script puedes registrate en la bd con todos los campos en vacio.

fcdragons · #26 (**permalink**) 10/05/2009, 12:59

Código php:

Ver originalif(!empty($_POST['nombre'] && $_POST['apellidos'] && $_POST['login'] && $_POST['pass'] && $_POST['email'])) {

pon esto en vez del if que tienes

makeinn · #27 (**permalink**) 10/05/2009, 13:18

fcdragons eso es lo que tenia que ya me lo habias puesto pero no funcionaba asi que lo sustitui por if($nombre == '' or $apellidos etc.. y funciona ya, ahora ya esta todo perfecto, de momento no tengo ningun problema mas, a medida que vaya teniendo problemas resubire el hilo si no lo cierran, gracias a todos.

fcdragons · #28 (**permalink**) 10/05/2009, 13:28

ya pero no funcionaba porque cerre el ) antes , lo puse al final y como te puse deberia funcionar

Da igual, espero haberte ayudado... si es asi dame karma ;)

argy · #29 (**permalink**) 11/05/2009, 08:12

el empty debes de usarlo para cada variable.

suerte.

fcdragons · #30 (**permalink**) 11/05/2009, 09:41

No hace falta porque yo tengo el que puse y me funciona de maravilla