quisiera saber si mi formulario que hize en PHP esta bien hecho

sabrina_avella · #1 (**permalink**) 21/11/2011, 14:05

<title>Formulario</title>
</head>

<body>
<table width="343" border="0" align="center">
<tr> <td><form name="form1" method="post" action="enviar.php">
<table width="338" border="0">
<tr>
<td>Nombre:</td>
<td><input name="nombre" type="text" id="nombre"></td>
</tr>
<tr>
<td>Email:</td>
<td><input name="email" type="text" id="email"></td>
</tr>
<tr>
<td>Mensaje:</td>
<td><textarea name="msg" rows="10" id="msg"></textarea></td>
</tr>
<tr>
<td><input type="submit" name="Submit" value="Enviar" /></td>
<td> </td>
</tr>
</table>
</form></td>
</tr> </table>
<p><?php

$mail='[email protected]';

$nombre = $_POST['nombre'];
$email = $_POST['email'];
$msg = $_POST['msg'];

$thank="index.html";

$message = "
nombre:".$nombre."
email:".$email."
msg:".$msg."";

if (mail($mail,"consulta",$message))
Header ("Location: $thank");

?>  </p>
</body>
</html>

Carlangueitor · #2 (**permalink**) 21/11/2011, 15:16

Funciona?

Pues en teoría debería estar bien. Solo habría que poner el código PHP hasta arriba y cambiar el nombre del archivo a enviar.php

Saludos

emprear · #3 (**permalink**) 21/11/2011, 16:09

sabrina_avella:

Eso asi como está no te va a funcionar, el php para enviar el email, está correcto, aunque es muy básico, yo vería de hacerle algunas mejoras, pero el problema es otro ya que pusiste el form y el php que procesa en una misma página y no hay ningún tipo de condición de cuando debe procesarce el form. vale decir que cualquier persona que entre a enviar.php automáticamente provocará que tu email sea despachado y se redirija al index.html, y te va a llegar esto

asunto: consulta
cuerpo del mensaje
nombre:
email:
msg:

es más ni siquiera van a ver el form, por la redirección
lo minimo que tendrias que hacer es encerrar el envio del email entre una condición como

Código PHP:

Ver originalif(isset($_POST['Submit'])){
 
// aqui tu código
 
}

Pero como ya te señalé, el procesamiento es bastante precario y necesita mejoras, una validación minima, pasar headers a mail(), por citar algunas.

Te dejo 2 tutoriales

http://www.w3schools.com/php/php_mail.asp
http://www.w3schools.com/php/php_secure_mail.asp

Saludos

quedecosas · #4 (**permalink**) 21/11/2011, 23:57

Código PHP:

Ver original<html>
<head>
<title>Formulario</title>
</head>
 
<body>
<table width="343" border="0" align="center"> 
<tr> <td><form name="form1" method="post" action="enviar.php">
<table width="338" border="0">
<tr>
<td>Nombre:</td>
<td><input name="nombre" type="text" id="nombre"></td>
</tr>
<tr>
<td>Email:</td>
<td><input name="email" type="text" id="email"></td>
</tr>
<tr>
<td>Mensaje:</td>
<td><textarea name="msg" rows="10" id="msg"></textarea></td>
</tr>
<tr>
<td><input type="submit" name="Submit" value="Enviar" /></td>
<td>&nbsp;</td>
</tr>
</table> 
</form></td> 
</tr> </table>
<p><?php
if(isset($_POST['Submit'])){
$mail='[email protected]';
 
 
$nombre = $_POST['nombre'];
$email = $_POST['email'];
$msg = $_POST['msg'];
 
$thank="index.html";
 
$message = "
nombre:".$nombre."
email:".$email."
msg:".$msg."";
 
if (mail($mail,"consulta",$message)) 
Header ("Location: $thank");
}
?> &nbsp;</p>
</body>
</html>

asi deberia quedar tu formulario para que funcione, sin embargo como dice emprear necesita muchas mejoras, saludos

Carlangueitor · #5 (**permalink**) 22/11/2011, 00:23

@quedecosas, Creo ese código te daría error, ya que no puedes poner un header() después de cualquier código html. Por eso le dije que debería ponerlo al principio.

Saludos

emprear · #6 (**permalink**) 22/11/2011, 01:11

@carlangueitor

una aclaración respecto de los headers() de php, no siempre una salida html previo al envío del header genera un error

manual php

Cita:

Nota:

Se puede usar búfer de salida para evitar este problema, procesando en el servidor todo el contenido a mostrar en el navegador antes de ser enviado. Se puede hacer esto usando ob_start() y ob_end_flush() en el script, o definiendo la directiva output_buffering en el php.ini o en los ficheros de configuración del servidor.

Y en muchos casos los servidores tienen configurado
output_buffering = 4096 en php.ini, con lo que el error no se genera.

De todas formas, la práctica correcta para evitar cualquier tipo de problema, es pasarlos al inicio del php, previos a cualquier salida html

Saludos