Formulario y procesos en un mismo archivo

Erythnul · #1 (**permalink**) 06/07/2010, 23:22

Hola amigos, quiero hacer un registro en el que en el mismo archivo donde el usuario llena los campos procese los datos.
Ej.

http://localhost/registro.php?accion=procesar
http://localhost/registro.php?accion=confirmar

Ademas de eso quiero que los errores salten en la misma pagina, no se por ejemplo que si se envió el formulario correctamente el formulario desaparezca y aparezca un campo de color con el mensaje y si el formulario esta mal que el formulario no desaparezca y ademas que salte un error arriba con un campo de color también. Asi como cuando te registras en un foro; de antemano se agradece cualquier ayuda. ¡Saludos!

abimaelrc · #2 (**permalink**) 07/07/2010, 07:50

Si no sabes como obtener los valores de las peticiones, te recomiendo que leas este material http://php.net/manual/en/language.va...s.external.php

GatorV · #3 (**permalink**) 07/07/2010, 08:51

Tema movido desde Configuración PHP a PHP

Erythnul · #4 (**permalink**) 07/07/2010, 14:39

Gracias, pero eso no es lo que quiero hacer. Resulta que tengo un formulario y quiero que en ese mismo archivo se lleve acabo todas las operaciones del registro, si hay un error en algun campo que ese dicho error aparezca dentro de un cartel como cuando te equivocas al registrar una cuenta en este mismo foro que te salta un cartel arriba del formulario con los errores enumerados y todo. Por ahi me explico mal pero es que soy muy novato en esto y les pido una mano para hacer esto. ¡Saludos!

shakaran · #5 (**permalink**) 07/07/2010, 14:54

¿Puedes poner algo de código de lo que tienes?

PD: Por favor, no pongas todo tu texto en negritas, se lee bien sin ellas.

Erythnul · #6 (**permalink**) 07/07/2010, 15:22

registro.php

Código HTML:

Ver original<!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 http-equiv="Content-Type" content="text/html; charset=iso-8859-1" />
<title>Registro</title>
<link href="css.css" rel="stylesheet" type="text/css" />
</head>
<body>
<div align="center">
<table width="800" height="100%" border="0" cellspacing="0" cellpadding="10" bgcolor="#FFFFFF">
<tr>
<td style="border: solid 1px #000000">
  <form name="form1" method="post" action="proceso.php">
  <table width="100%" border="0" cellspacing="0" cellpadding="5">
  <tr>
      <td style="background: #000000; color: #FFFFFF; border: 1px solid #000000; font-size: 12px"><strong>Registro de cuentas; Ingresa los datos de la cuenta</strong></td>
    </tr>
    <tr>
      <td class="formadesc" >Nombre de usuario</td>
    </tr>
    <tr>
      <td class="formacampo"><input name="usuario" type="text" id="usuario" size="20" maxlength="16" /></td>
    </tr>
    <tr>
      <td class="formadesc" >Contrase&ntilde;a</td>
    </tr>
    <tr>
      <td class="formacampo"><input name="clave1" type="password" id="clave1" size="20" maxlength="16" /></td>
    </tr>
    <tr>
      <td class="formadesc" >Confirma contrase&ntilde;a</td>
    </tr>
    <tr>
      <td class="formacampo"><input name="clave2" type="password" id="clave2" size="20" maxlength="16" /></td>
    </tr>
    <tr>
      <td class="formadesc" >Correo electr&oacute;nico</td>
    </tr>
    <tr>
      <td class="formacampo"><input name="correo" type="text" id="correo" size="20" maxlength="50" /></td>
    </tr>
    <tr>
      <td class="formaboton"><div align="right"><input name="boton" type="submit" id="boton" value="Registrar" />
      </div></td>
    </tr>
  </table>
</form>
</td>
</tr>
</table>
</div>
</body>
</html>

proceso.php

Código PHP:

Ver original<?php
 
$usuario = $_POST['usuario'];
$clave1 = $_POST['clave1'];
$clave2= $_POST['clave2'];
$correo = $_POST['correo'];
 
if ($usuario == "")
{
echo "Debes ingresar un nombre de usuario para registrarte.";
}
elseif ($clave1 == "")
{
echo "Debes ingresar una contrase&ntilde;a para registrarte.";
}
elseif ($clave1 != $clave2)
{
echo "Por favor confirma la contrase&ntilde;a ingresada.";
}
// Aqui una nueva forma para validar el email, usando la funcion "strpos":
elseif (!strpos($correo,"@hotmail.") && !strpos($correo,"@gmail.") && !strpos($correo,"@yahoo.") && !strpos($correo,"live.com."))
{
echo "El correo ingresado es incorrecto.";
}
else
{
$conexion = mysql_connect("localhost","root","");
mysql_select_db("prueba", $conexion);
$codigo = rand(0000000000,9999999999); // Conseguimos un codigo aleatorio de 10 digitos.
if (!mysql_query("INSERT INTO pendientes (usuario,clave,correo,codigo) values ('".$usuario."','".$clave1."','".$correo."','".$codigo."')")) die (mysql_error());
$headers = "From: UO Daery <[email protected]>";
$mensaje = "Hola $usuario: \n
Te enviamos este correo para completar el proceso de registro. \n
Para activar tu cuenta visita el siguiente enlace: \n
http://localhost/confirmar.php?codigo=".$codigo;
if (!@mail("$correo","Confirmacion de registro","$mensaje","$headers")) die ("No se pudo enviar el email de confirmacion.");
echo "Tu cuenta ha sido registrada, sin embargo, esta requiere que la confirmes desde el email que ingresaste en el registro.";
}
?>

Lo del texto en negrita era mas bien para tener mi estilo en el foro pero si molesta pido perdón y no lo hago mas. ¡Saludos!

shakaran · #7 (**permalink**) 07/07/2010, 15:34

Veo varias cosas mejorables.

1 - No filtras/validas tus variables $_POST
2 - En los if comparas con ="" pero puede ser 0 o otros valores, mejor usar empty()
3 - Cuando validas el email, hay mas proveedores que los que pones, mas bien infinitos, porque
cualquiera puede hacerse una cuenta de su correo en su dominio, luego no es buena forma de validar. Te recomiendo usar filter_var de php y checkdomain()
4 - El charset mejor en utf-8:
<meta http-equiv="Content-Type" content="text/html; charset=iso-8859-1" />
5- Crea clases, no pongas css inline para esto y demas:
style="border: solid 1px #000000"
6- Los formularios contra menos campos mejor, repetir la contraseña el algo viejuno ya, la gente sabe lo pone, es como tambien tuvieras que poner otro para el email
7 - Identa mas el codigo html (sobre todo la zona de head)
8 - No es buena practica conectar a mysql con un usuario root, crea uno especifico
9 - llamando a rand sin parametros, te generara un numero aleatorio suficiente.

Para lo de accion podrias poner

Código PHP:

Ver originalswitch($_REQUEST['accion'])
{
    case 'confirmar':
    # Codigo aqui
    break;
    case 'procesar':
    # Codigo aqui
    break;
}

PD: lo de poner el tamaño de letra en mayor tambien igual xD

Erythnul · #8 (**permalink**) 07/07/2010, 16:20

Le cambie el código a las lineas 8 y 12

Código PHP:

Ver original<?php
 
$usuario = $_POST['usuario'];
$clave1 = $_POST['clave1'];
$clave2= $_POST['clave2'];
$correo = $_POST['correo'];
 
if (empty($usuario))
{
echo "Debes ingresar un nombre de usuario para registrarte.";
}
elseif (empty($clave1))
{
echo "Debes ingresar una contrase&ntilde;a para registrarte.";
}

Ahora como puedo validar correctamente el usuario? ¡Saludos!

shakaran · #9 (**permalink**) 07/07/2010, 16:25

El skype no me va muy bien con la conexión actual, puedes agregarme a gmail y hablar conmigo por ahi (es mi mismo nick shakaran at gmail dot com).

Para validar los datos depende de la variable, si es un correo tiene unas restricciones, si es una cadena normal otras, si es un correo otras, la longitud que permitas, etc.

Erythnul · #10 (**permalink**) 08/07/2010, 00:20

Por ahora quiero seguir esa serie de pasos, lo siguiente seria verificar el usuario en la db y si esta registrado salte error. Tengo el siguiente código pero no se como adaptarlo o modificarlo para que entre en mi proceso.

Código PHP:

Ver original$query = mysql_query("SELECT * FROM accounts WHERE user='$usuario' ");
if ($usuario_existe = mysql_fetch_array($query))
{
echo 'El usuario ya esta registrado';
mysql_free_result($query);
}

Entonces como le hago? ¡Saludos!