Ver Mensaje Individual
  #5 (permalink)  
Antiguo 04/03/2003, 15:54
NeBeRuS
 
Fecha de Ingreso: febrero-2002
Ubicación: ¿donde vivo? pues en mi casa, ¿donde voy a vivir?...
Mensajes: 214
Antigüedad: 22 años, 1 mes
Puntos: 0
Parcheando, y explicando pq

Bien, ya q se pastea código "funcional", me tomo la libertad de parchearlo, y comentarlo


Cita:
Mensaje Original por camargo
<?

if($accion=="enviar"){

; //El e-mail a donde iran los datos del formulario
$asunto ="Asunto del mail"; //El tema o asunto del mail que se enviara

mail("$email", "$asunto", "$info");
header("Location: gracias.php");

} else {

echo "<form method=post action=PAGINA.php?accion=enviar>
<input type=text name=info><br>
<input type=submit name=enviar value=enviar>
</form>";
}
?>


GooD
Made By Me

Bien, mi correccion, mi versión, o como kerais decirlo:

(lo voy a programar aki, sin más, osea q no se si será funcional, = me como alguna coma o algo) xDD

Código PHP:
switch ($_POST['enviar']) {
case 
'enviado':
procesa_form();
break;

default:
procesa_form();
break;
}

function 
muestra_form() {
echo <<< HTML
<form method=post. action=file.php>
<input type=text name=para>
<input t<textarea name=texto>
</textarea>ype=text name=asunto>
<input type=submit name=enviar value=enviado>

</form>
HTML;
}

function 
procesa_form() {
$para=$_POST['mail'];
$asunto=$_POST['asunto']:
$texto=$_POST['texto'];
$de=$_POST['de'];
esto es solo para q kede más claro.

mail ($para$asunto$testo"From:".$de);

A ver, las correcciones, son el uso de $_POST, para evitar problemas con globals, y esas cosas..

y luego el uso de switch y un par de funcioncillas, por simple estructuracion de codigo. así keda más legible y facilmente modificable. Aunq en este caso es un tanto absurdo.
otra correcion, es el mail(); q le faltaban parametros en el caso anterior.
__________________
Linux, es para los que odian Microsoft/Windows, BSD, es para los que amamos *IX.

Última edición por NeBeRuS; 04/03/2003 a las 15:56