está bien escrito este codigo?

[brunob23]

Buenas, tengo un formulario para enviar por correo con php y quiero saber si el codigo que programé esta correcto.

Codigo del formulario:
<form name="form1" method="post" action="enviarfe.php">
<label>

<div align="center">Nombre:

<input type="text" name="nombre" id="textfield">
<br>
<br>
E-mail:<span class="Estilo8">--</span>
<input type="text" name="e-mail" id="textfield2">
<br>
<br>
Telefono:
<input type="text" name="telefono" id="textfield3">
<br>
<br>
Empresa:
<input type="text" name="empresa" id="textfield4">
<br>
<br>
Mensaje:<span class="Estilo8">-------------------------</span><br>
<span class="Estilo8">------------</span>
<textarea name="mensaje" cols="33" rows="8" id="textarea"></textarea>



Codigo php:

<?
$MailToAddress = "mi_mail";
$MailSubject = "contacto";
$desde = 'From: ' .$_POST['e-mail']. "\r\n";
$nombre = 'Nombre:' .$_POST['nombre']. "\r\n";
$Telefono = 'Telefono:' $_POST['telefono'] "\r\n";
$empresa = 'Empresa:' $_POST['empresa'] "\r\n";

$Message = $_POST['mensaje']. "\r\n";

$Header = $_POST['mensaje']. "\r\n"; 'Nombre:' .$_POST['nombre']. "\r\n"; 'Telefono:' $_POST['telefono'] "\r\n"; 'Empresa:' $_POST['empresa'] "\r\n";


if ($Header) {
$Message = $Header."\n\n".$Message."\n\n";
}

mail( "$MailToAddress", "$MailSubject", "$desde", "$Message", "$nombre", "$telefono", "$empresa" );
header ("Location: contacto.html");
?>


gracias

[bacdavi]

Iniciado por brunob23

Buenas, tengo un formulario para enviar por correo con php y quiero saber si el codigo que programé esta correcto.

Codigo del formulario:
<form name="form1" method="post" action="enviarfe.php">
<label>

<div align="center">Nombre:

<input type="text" name="nombre" id="textfield">
<br>
<br>
E-mail:<span class="Estilo8">--</span>
<input type="text" name="e-mail" id="textfield2">
<br>
<br>
Telefono:
<input type="text" name="telefono" id="textfield3">
<br>
<br>
Empresa:
<input type="text" name="empresa" id="textfield4">
<br>
<br>
Mensaje:<span class="Estilo8">-------------------------</span><br>
<span class="Estilo8">------------</span>
<textarea name="mensaje" cols="33" rows="8" id="textarea"></textarea>



Codigo php:

<?
$MailToAddress = "mi_mail";
$MailSubject = "contacto";
$desde = 'From: ' .$_POST['e-mail']. "\r\n";
$nombre = 'Nombre:' .$_POST['nombre']. "\r\n";
$Telefono = 'Telefono:' $_POST['telefono'] "\r\n";
$empresa = 'Empresa:' $_POST['empresa'] "\r\n";

$Message = $_POST['mensaje']. "\r\n";

$Header = $_POST['mensaje']. "\r\n"; 'Nombre:' .$_POST['nombre']. "\r\n"; 'Telefono:' $_POST['telefono'] "\r\n"; 'Empresa:' $_POST['empresa'] "\r\n";


if ($Header) {
$Message = $Header."\n\n".$Message."\n\n";
}

mail( "$MailToAddress", "$MailSubject", "$desde", "$Message", "$nombre", "$telefono", "$empresa" );
header ("Location: contacto.html");
?>


gracias

Excelente chavo!

se nota que lo copiaste de otro site!

no le veo ningun problema, pero por que preguntas, si tu mismo lo puedes probar en tu localhost... la verdad no le veo sentido, lo que podrias preguntar y encuentro factible es algun error que ese script te genere!

caso contrario, ¿PARA QUE ESTA LOCALHOST?

[pateketrueke]

y porque no lo ejecutas??

aunque... para que asignas variables si no las vas a usar???

para que las comillas dobles alrededor de variables sencillas???

[maycolalvarez]

siempre es importante mantener el código ordenado, pero lo importante es que funcione, mantenerlo ordenado es para facilitar el proceso de depuración entre otras cosas, en mis principios mi código era un desastre, poco a poco vas perfeccionando ;)

[brunob23]

no lo probé porque no tengo el hosting contratado aun, simplemente por eso :D

bacdavi: ese codigo me lo pasaron, yo lo que hice fue agregarle cosas, como es la primera vez que hago algo en php queria sentirme bien y decir programé :D

Muchas gracias por las respuestas

pateketrueke: no entendi lo que quisiste decir de las variables y comillas

[Triby]

brunob23, supongo que pateketrueke se refiere a las comillas usadas en:

Código php:

// Esto:
mail( "$MailToAddress", "$MailSubject", "$desde", "$Message", "$nombre", "$telefono", "$empresa" );
 
// Es lo mismo que esto:
mail($MailToAddress, $MailSubject, $desde, $Message, $nombre, $telefono, $empresa);
 

Por otra parte, te recomiendo ver el manual para la funcion mail(), porque asi como lo tienes es casi seguro que obtengas un error, una advertencia o simplemente no funcione.

[brunob23]

Como deberia ser el codigo entonces? tendria que poner lo de "nombre", "telefono", "empresa" adentro de la variable $message? o como deberia hacer?

[KrowMx]

Código PHP:

  
<?php 
// mensaje 
$message = "Line 1\nLine 2\nLine 3"; 
 
 
// envio 
mail('[email protected]', 'TEMA DEL MAIL', $message); 
?>   

Efetivamente amigo en la variable $message, envias, anexas, concatenas todo tu contenido del cuerpo del email

[brunob23]

Muchas gracias, lo probare y les contare que tal.