Ver Mensaje Individual
  #7 (permalink)  
Antiguo 07/07/2010, 15:34
Avatar de shakaran
shakaran
 
Fecha de Ingreso: agosto-2005
Ubicación: España - Ciudad Real
Mensajes: 374
Antigüedad: 19 años, 6 meses
Puntos: 7
Respuesta: Formulario y procesos en un mismo archivo

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 original
  1. switch($_REQUEST['accion'])
  2. {
  3.     case 'confirmar':
  4.     # Codigo aqui
  5.    break;
  6.     case 'procesar':
  7.     # Codigo aqui
  8.    break;
  9. }
PD: lo de poner el tamaño de letra en mayor tambien igual xD
__________________
Quijost Backend Engineer - www.quijost.com - Hosting rápido, eficiente y profesional
Blog: www.shakaran.net