Ayuda

berto1989 · #1 (**permalink**) 23/08/2010, 04:10

pues sigo con el sistema de registro, y no se dnd tengo el fallo, o si me e liado con los else :P

Código PHP:

  <?php
include("archivos/conexion.php");
?>
<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">
<html xmlns="http://www.w3.org/1999/xhtml">

<head>
<meta content="text/html; charset=utf-8" http-equiv="Content-Type" />
<title>Berto1989 Registro</title>
<link href='css/logeo.css' rel='stylesheet' type='text/css' />
<link rel='shortcut icon' href='archivos/imagenes/favicon.png' type='image/x-icon'/>
</head>

<body>
<br/>
<br/>
<br/>
<br/>
<br/>
<br/>
<br/>
<br/>
<br/>
<form action='?registro=1' method='post' enctype='multipart/form-data' name='formulario'>
<table class='table' align='center' style='width: 490px; height: 390px'>
<tr>
<td>
<p align='center' style='height: 73px'>
<label>Usuario</label> &nbsp; &nbsp;&nbsp;&nbsp;&nbsp;<input name='usuario' type='text' style='width: 230px'/><br/>
<label>email</label> &nbsp; &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;<input name='email' type='text' style='width: 230px'>
<br/>
<label>password</label> &nbsp; &nbsp;<input name='password' type='password' style='width: 230px'/><br/>
<input name='id' type='hidden' value='$id1'/></p>
<p class='error' align='center'>
<?php 
$campo1 = $_POST['usuario'];
$campo2 = $_POST['email'];
$campo3 = $_POST['password'];

$id1 = $_GET['id'];
$query = mysql_query("SELECT * FROM  registros WHERE id = '$id1'") or die(mysql_error());
$data = mysql_fetch_array($query);

if (isset($_GET["registro"]))
{
if($data['id'] == "")
{
echo "Lo sentimos, el registro es por invitacion";
}
else
{
if($campo1;$campo2;$campo3 == ""){
echo "Debe cumplimentar todos los campos";
}
}
else
{
$user = $_POST['usuario'];
$query = mysql_query("SELECT * FROM  usuarios WHERE usuario= '$user'") or die(mysql_error());
$data = mysql_fetch_array($query);

if($data['usuario'] == ($user)){

echo "Lo sentimos, este usuario ya esta en uso";
}
}
else
{
$usuario1 = $_POST['usuario'];
$pass = $_POST['password'];
$pass_md5 = md5($pass);
$email1 = $_POST['email'];
$id2 = $_POST['id'];

@mysql_query("INSERT INTO `usuarios` (`usuario`, `password`, `password2`,  `email`) VALUES ('$usuario1', '$pass_md5', '$pass', '$email1')");
@mysql_query("DELETE FROM registros WHERE id = '$id2'") or die(mysql_error());

header("location: index.php");
exit();
}
}
?>


</p>
<p align='center' style='height: 26px; width: 484px'>
<input type='submit' value='Registrar'/>
<br/>
</p>
</td>
</tr>
</table>
</form>
</body>
</html>

geq · #2 (**permalink**) 23/08/2010, 04:15

Hola,

El primer error que veo es el else. En las condiciones, no puedes usar ;, poner algo como eso. Tienes que validar cada variable por separado y armar la condición lógica con AND (&&) o OR (||). A demás, usa elseif en lugar de else y dentro otro if!

Ejemplo:

Código PHP:

  } elseif($campo1==""||$campo2==""||$campo3=="") {

Saludos!

berto1989 · #3 (**permalink**) 23/08/2010, 05:08

buenas, lo e puesto asi y sigue sin ir

geq · #4 (**permalink**) 23/08/2010, 05:14

¿Revisaste todo el if completo? ¿Qué problema tiene, tira algún error?

Luego envía cómo te ha quedado el código.

Saludos

berto1989 · #5 (**permalink**) 23/08/2010, 05:26

pues ahora me salta seguido k el registro es por invitacion

Código PHP:

  <?php  
$campo1 = $_POST['usuario']; 
$campo2 = $_POST['email']; 
$campo3 = $_POST['password']; 
 
$id1 = $_POST['id']; 
$query = mysql_query("SELECT * FROM  registros WHERE id = '$id1'") or die(mysql_error()); 
$data = mysql_fetch_array($query); 
 
$user = $_POST['usuario']; 
$query1 = mysql_query("SELECT * FROM  usuarios WHERE usuario= '$user'") or die(mysql_error()); 
$data1 = mysql_fetch_array($query1); 
 
 
if (isset($_GET["registro"])) 
{ 
if($data['id'] == "") 
{ 
echo "Lo sentimos el registro es por invitacion o esta<br/> invitacion ya a sido utilizada"; 
} 
elseif($campo1==""||$campo2==""||$campo3=="") 
{ 
echo "Debe cumplimentar todos los campos"; 
} 
elseif($data1['usuario'] == ($user)) 
{ 
echo "Lo sentimos, este usuario ya esta en uso"; 
} 
else 
{ 
 
$usuario1 = $_POST['usuario']; 
$pass = $_POST['password']; 
$pass_md5 = md5($pass); 
$email1 = $_POST['email']; 
$id2 = $_POST['id']; 
 
@mysql_query("INSERT INTO `usuarios` (`usuario`, `password`, `password2`,  `email`) VALUES ('$usuario1', '$pass_md5', '$pass', '$email1')"); 
@mysql_query("DELETE FROM registros WHERE id = '$id2'") or die(mysql_error()); 
 
header("location: index.php"); 
exit(); 
} 
} 
?>

geq · #6 (**permalink**) 23/08/2010, 05:32

¿Has revisado en el código fuente que el campo id tenga algún valor?

Saludos

berto1989 · #7 (**permalink**) 23/08/2010, 05:57

bueno creo que ya esta solucionado

codigo php

Código PHP:

  <?php 
$campo1 = $_POST['usuario'];
$campo2 = $_POST['email'];
$campo3 = $_POST['password'];

$id1 = $_POST['id'];
$query = mysql_query("SELECT * FROM  registros WHERE id = '$id1'") or die(mysql_error());
$data = mysql_fetch_array($query);

$user = $_POST['usuario'];
$query1 = mysql_query("SELECT * FROM  usuarios WHERE usuario= '$user'") or die(mysql_error());
$data1 = mysql_fetch_array($query1);


if (isset($_GET["registro"]))
{
if($data['id'] == "")
{
echo ""Lo sentimos el registro es por invitacion o esta<br/> invitacion ya a sido utilizada"";
}
elseif($campo1==""||$campo2==""||$campo3=="")
{
echo "Debe cumplimentar todos los campos";
}
elseif($data1['usuario'] == ($user))
{
echo "Lo sentimos, este usuario ya esta en uso";
}
else
{

$usuario1 = $_POST['usuario'];
$pass = $_POST['password'];
$pass_md5 = md5($pass);
$email1 = $_POST['email'];
$id2 = $_POST['id'];

@mysql_query("INSERT INTO `usuarios` (`usuario`, `password`, `password2`,  `email`) VALUES ('$usuario1', '$pass_md5', '$pass', '$email1')");
@mysql_query("DELETE FROM registros WHERE id = '$id2'") or die(mysql_error());

header("location: index.php");
exit();
}
}
?>

Ahora el problema que tengo es que si el usuario existe, me lo comprueba bien, pero en la url me pone ?registro=1, y ya si pone bien los datos y le da a registrar, ya no me lo registra

markmb · #8 (**permalink**) 23/08/2010, 06:09

Código PHP:

  <?php
include("archivos/conexion.php");
?>
<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">
<html xmlns="http://www.w3.org/1999/xhtml">

<head>
<meta content="text/html; charset=utf-8" http-equiv="Content-Type" />
<title>Berto1989 Registro</title>
<link href='css/logeo.css' rel='stylesheet' type='text/css' />
<link rel='shortcut icon' href='archivos/imagenes/favicon.png' type='image/x-icon'/>
</head>

<body>
<br/>
<br/>
<br/>
<br/>
<br/>
<br/>
<br/>
<br/>
<br/>
<?php//En el action de aquí abajo, prueba a poner la ruta "entera" ('registro.php?registro=1') ?>
<form action='?registro=1' method='post' enctype='multipart/form-data' name='formulario'>
<table class='table' align='center' style='width: 490px; height: 390px'>
<tr>
<td>
<p align='center' style='height: 73px'>
<label>Usuario</label> &nbsp; &nbsp;&nbsp;&nbsp;&nbsp;<input name='usuario' type='text' style='width: 230px'/><br/>
<label>email</label> &nbsp; &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;<input name='email' type='text' style='width: 230px'>
<br/>
<label>password</label> &nbsp; &nbsp;<input name='password' type='password' style='width: 230px'/><br/>
<?php //Este $id1 debería estar: entre php, y tener un valor, porque creo que el id1 se muestra en el asterisco de más abajo (*). Por tanto, aquí pues sería <input name='id' type='hidden' value='<?php echo $id1; ?>'/></p>. En el caso de que el asterisco de abajo sea el valor que debe mostrar, deberías subir esto hasta aquí ?>
<input name='id' type='hidden' value='$id1'/></p>
<p class='error' align='center'>
<?php 
$campo1 = $_POST['usuario'];
$campo2 = $_POST['email'];
$campo3 = $_POST['password'];
//(*) Este es el asterisco, además, no estoy seguro, pero creo que debería ser por post, ¿o no?
$id1 = $_GET['id'];
$query = mysql_query("SELECT * FROM  registros WHERE id = '$id1'") or die(mysql_error());
$data = mysql_fetch_array($query);
//esto, como han dicho, puede ser un if{}else if{}else{}
if (isset($_GET["registro"])){
       if($data['id'] == ""){
             echo "Lo sentimos, el registro es por invitacion";
       }
       //como han dicho, esto no puede ir con comas, tienen que ser condiciones separadas: else if($campo1 == "" or $campo2 =="" or $campo3==""){   . Ten en cuenta también la función empty
       else if ($campo1;$campo2;$campo3 == ""){
              echo "Debe cumplimentar todos los campos";
      }
      else{
$user = $_POST['usuario'];
$query = mysql_query("SELECT * FROM  usuarios WHERE usuario= '$user'") or die(mysql_error());
$data = mysql_fetch_array($query);
//aqui no hacen falta parentesis en user
if($data['usuario'] == $user){
       echo "Lo sentimos, este usuario ya esta en uso";
}
}
else{
$usuario1 = $_POST['usuario'];
$pass = $_POST['password'];
$pass_md5 = md5($pass);
$email1 = $_POST['email'];
$id2 = $_POST['id'];
@mysql_query("INSERT INTO `usuarios` (`usuario`, `password`, `password2`,  `email`) VALUES ('$usuario1', '$pass_md5', '$pass', '$email1')");
@mysql_query("DELETE FROM registros WHERE id = '$id2'") or die(mysql_error());

header("location: index.php");
exit();
}
}
?>


</p>
<p align='center' style='height: 26px; width: 484px'>
<input type='submit' value='Registrar'/>
<br/>
</p>
</td>
</tr>
</table>
</form>
</body>
</html>

Suerte!

Edito: no habia visto que habias respondido!

berto1989 · #9 (**permalink**) 23/08/2010, 06:16

si pongo como dices de registro.php?registro=1, me salta un error en otra pagina de custom404error

markmb · #10 (**permalink**) 23/08/2010, 06:18

Me he expresado mal, quería decir:

(la pagina actual)?registro=1

geq · #11 (**permalink**) 23/08/2010, 06:25

Yo te recomendaría pasar registro como otro campo hidden en lugar de ponerlo en el action del formulario, pruébalo.

Verifica también el valor del campo id!

Saludos

berto1989 · #12 (**permalink**) 23/08/2010, 06:43

el campo id me fufa bien, pero el problema es el siguiente:

Si pongo un usuario que esta registrado me pone "Este usuario ya esta en uso", y sigue en el registro, pero la url cambio de registro.php?id=xxxxx a registro.php?registro=1, por eso no me debe ir bien.

markmb · #13 (**permalink**) 23/08/2010, 06:53

Te hace eso al enviar, porque cuando lo envías, se queda solo el registro=1.

Quizás me equivoqué y en el action, con ?registro=1 había suficiente.

Pruébalo, a ver si me me he equivocado

berto1989 · #14 (**permalink**) 24/08/2010, 13:04

wenas siento haber tardado tanto en contestar, pero tuve k instalar windows 7 en un ordenador, y me daba problemas la tarjeta wifi.

Cuando le doy a enviar me cambia lo de ?id=xxx por ?registro=1, pero el id ya no me lo tiene, asi k no me puede validar el id y no me lo registra. alguna manera de solucionarlo?

markmb · #15 (**permalink**) 24/08/2010, 13:08

Hola!

prueba así en el form:

action="<?php echo $_SERVER['PHP_SELF']; ?>&registro=1"

No se si PHP_SELF te guardará también el get

EDITO: la he cagado, no va con interrogante, sino con &

berto1989 · #16 (**permalink**) 29/08/2010, 06:04

siento haber tardado tanto de nuevo, al ejecutarlo, me pone en la url los datos, pero no registra.

markmb · #17 (**permalink**) 29/08/2010, 07:59

Hola!

Pero los gets de arriba estan bien puestos?

berto1989 · #18 (**permalink**) 29/08/2010, 14:16

si no recuerdo mal si, puesto k registrar me registra si pongo un usuario k no este en la bd

markmb · #19 (**permalink**) 30/08/2010, 07:40

pues si registra, cúal es el problema?

berto1989 · #20 (**permalink**) 30/08/2010, 12:22

pues k si mete un usuario k esta registrado, le salta el mensaje de "Este usuario esta registrado", y cambia la url de ?id=xxxxx a ?registro=1, y asi luego de volver a escribir otro usuario, no me deja registrarlo, por k no tiene id que validar

markmb · #21 (**permalink**) 30/08/2010, 13:26

Esto último no entiendo demasiado bien por qué es, pero se podría probar así:

Código PHP:

  <?php
$pagina = "la pagina actual.php";
if($_GET['id']){
            $pagina .= '?id='.$_GET['id'].'&registro=1';
}else{
             $pagina .= '?registro=1';
}

<form action="<?php echo $pagina; ?>"....

berto1989 · #22 (**permalink**) 31/08/2010, 02:01

ese codigo no lo entiendo, dnd ai k ponerlo?

markmb · #23 (**permalink**) 31/08/2010, 03:58

<form action='?registro=1' method='post' enctype='multipart/form-data' name='formulario'>

Esto de aquí (no se ahora mismo como está en tu código), prueba a cambiarlo por esto:

<?php
$pagina = "la pagina actual.php";
if($_GET['id']){
$pagina .= '?id='.$_GET['id'].'&registro=1';
}else{
$pagina .= '?registro=1';
}
?>
<form action='<?php echo $pagina; ?>' method='post' enctype='multipart/form-data' name='formulario'>

Espero que funcione bien de una p*** vez

berto1989 · #24 (**permalink**) 31/08/2010, 05:36

pos no me funciona :S, me sale siempre el registro y no me da ni error ni me registra

edito: muchas gracias markmb ya ta solucionao asi k si tal cerrar el post