Crear forumulario php para enviar

[djkiku]

Hola a todos! Pido disculpas por molestarles.. pero llevo 1 horita intentando areglar.. pero es la primera vez que trato con php..

Estoy haciendo un formulario de envio.. y el problema es que no me funciona.. sale que hay 6 errores.. y no se ni por donde empesar.. aqui os lo dejo, si alguien tan amable me pudiera decir donde podria estar el fallo? y como areglarlo?
Ahi va todo el .php

Código PHP:

  <?php $nombre = $_POST['nombre'];
$mail = $_POST['mail'];
$dni= $_POST['dni'];
$contrato = $_POST['contrato'];
 
if($nombre == "" || $mail == "" || $dni == ""; ){
$error=1;
 
}
 
elseif(!eregi("^[a-z0-9]+([_.-][a-z0-9]+)*" ."@"."([a-z0-9]+([.-][a-z0-9]+)*)+".".[a-z]{2,}"."$",$correo)){
$error=2; Si el correo electronico no tiene un formato valido, mostrara el error numero 2
}
 
if($error==1){
echo "El siguiente error ha ocurrido!<BR>"; 
echo "No ha llenado todos los campos obligatorios.<BR> Por favor vuelva <A HREF="javascript:history.back()">atras</A>.<BR>";
exit;
}
 
 
 
elseif($error==2){
 
echo "El siguiente error ha ocurrido!<BR>";
echo "El correo electronico es invalido!<BR> Por favor vuelva <A HREF="javascript:history.back()">atras</A>.<BR>";
exit;
}
 
else{
print "<meta http-equiv="refresh" content="0;URL=gracias.html">";
}
 
$header .= "X-Mailer: PHP/" . phpversion() . " rn";
$header .= "Mime-Version: 1.0 rn";
$header .= "Content-Type: text/plain";
 
 
 
$mensaje = "Este mensaje fue enviado por " . $nombre . " rn";
$mensaje .= "Su e-mail es: " . $mail . " rn";
$mensaje .= "Su dni es: " . $dni . " rn";
$mensaje .= "contrato: " . $_POST['contrato'] . " rn";
$mensaje .= "Enviado el " . date('d/m/Y', time());
 
 
$para = '[email protected]';
 
mail($para, $asunto, utf8_decode($mensaje), $header); 
 
?>   

También adjunto el html, aun que me parece que esta bien:

http://mermelada.uphero.com/contrato%20legal.html

Ojala tenga areglo!

[CesarHC]

Seria bueno que pusieras lo errores, por cierto esto esta bien?

"$dni == "";"

[asrael]

No me a dado tiempo de verlo mucho pero asi a primera vista:

if($nombre == "" || $mail == "" || $dni == ""; ){

Esta mal porque sobra el punto y coma, seria asi:

if($nombre == "" || $mail == "" || $dni == ""){

elseif(!eregi("^[a-z0-9]+([_.-][a-z0-9]+)*" ."@"."([a-z0-9]+([.-][a-z0-9]+)*)+".".[a-z]{2,}"."$",$correo)){
$error=2; Si el correo electronico no tiene un formato valido, mostrara el error numero 2
}

Parece que se te a olvidado comentar la ultima frase para que la tome como comentario, seria asi:

elseif(!eregi("^[a-z0-9]+([_.-][a-z0-9]+)*" ."@"."([a-z0-9]+([.-][a-z0-9]+)*)+".".[a-z]{2,}"."$",$correo)){
$error=2; //Si el correo electronico no tiene un formato valido, mostrara el error numero 2
}

Siento no ser de mas ayuda pero no me a dado tiempo a fijarme en nada mas

Un saludo

[CesarHC]

Este eregi esta obsoleta tal vez sea eso http://www.php.net/manual/es/function.eregi.php

[djkiku]

Muchisimas Gracias por las rapidas respuestas! Probando inmediatamente!

[djkiku]

No.. me sigue mostrando mas errores... :(

Seguire probando..

[andresdzphp]

Coloca todos los errores aquí...

[djkiku]

Iniciado por andresdzphp

Coloca todos los errores aquí...

No se como hacerlo

Yo veo los errores por aqui: http://mermelada.uphero.com/enviar.php

[andresdzphp]

yo solo veo esto:

Website Under Review

You are seeing this page because the system administrator of 000webhost.com is currently checking this website for malicious content. This redirect page will be removed once we finish manually checking all files on this account.

Te están investigando la cuenta...

[djkiku]

Iniciado por andresdzphp

yo solo veo esto:



Te están investigando la cuenta...

ups :S

Voy a probar en otro host.. Haber...

Muchas Gracias pro la buena intencion! Ahora vuelvo

[andresdzphp]

Ten en cuenta que muchos hosting gratuitos tienen desactivada las funciones para el envio de emails y lo tienen prohibido.

[CesarHC]

Supongo que te aparecen en el navegador(si no no entiendo ), pues como cualquier texto seleccionalo
ctrl + c y en el post ctrl + v.

Solo veo la paginna de inicio del host en el que te alojas.

[djkiku]

Iniciado por CesarHC

Supongo que te aparecen en el navegador(si no no entiendo ), pues como cualquier texto seleccionalo
ctrl + c y en el post ctrl + v.

Solo veo la paginna de inicio del host en el que te alojas.

jeje que bobo claro! ahi va:


Parse error: syntax error, unexpected T_STRING, expecting ',' or ';' in /home/a2668010/public_html/enviar.php on line 17

[andresdzphp]

En esa línea te sobra el punto y coma

Código PHP:

if($nombre == "" || $mail == "" || $dni == "" ){
 

Además tienes graves problemas con las comillas, te recomiendo que te leas esto urgente:

COMO: usar las comillas bien

[djkiku]

Iniciado por andresdzphp

Ten en cuenta que muchos hosting gratuitos tienen desactivada las funciones para el envio de emails y lo tienen prohibido.

Si tienes toda la razon, mientras lo montava.. pude mandar 2 emails, pero sin los codigos de los errores 1 y error 2,

También adjunto donde saque este formulario: http://www.taringa.net/posts/info/12...lario-php.html

[djkiku]

Iniciado por andresdzphp

En esa línea te sobra el punto y coma

Código PHP:

if($nombre == "" || $mail == "" || $dni == "" ){
 

Además tienes graves problemas con las comillas, te recomiendo que te leas esto urgente:

COMO: usar las comillas bien

jeje Gracias! la verdad es que es mi primer dia con PHP (Deberias ver mi cara) jeje

Dejo el lugar de donde lo saque, contacte con el tio del post pero dudo que conteste ya que su ultima visita al acomunidad fue hace 8 dias http://www.taringa.net/posts/info/12...lario-php.html

[andresdzphp]

Bueno los errores están claros. No puedes usar comillas dobles dentro de comillas dobles... Debes escaparlas o combinar sencillas con dobles. Lee el enlace que te dejé.

Esto está mal

Código PHP:

print "<meta http-equiv="refresh" content="0;URL=gracias.html">";
 

esto está mejor

Código PHP:

print '<meta http-equiv="refresh" content="0;URL=gracias.html">';
 

[djkiku]

Iniciado por andresdzphp

Bueno los errores están claros. No puedes usar comillas dobles dentro de comillas dobles... Debes escaparlas o combinar sencillas con dobles. Lee el enlace que te dejé.

Esto está mal

Código PHP:

print "<meta http-equiv="refresh" content="0;URL=gracias.html">";
 

Voy a leerlo de cabeza! Gracias, pero antes pruebo corregir este error

[djkiku]

Esta claro que el error esta en este codigo... por que sin el me lo manda bien..

Código PHP:

  
if($nombre == "" || $mail == "" || $dni == ""){ 
$error=1;

}

elseif(!eregi("^[a-z0-9]+([_.-][a-z0-9]+)*" ."@"."([a-z0-9]+([.-][a-z0-9]+)*)+".".[a-z]{2,}"."$",$correo)){
$error=2; 
}

if($error==1){
echo "El siguiente error ha ocurrido!<BR>"; 
echo "No ha llenado todos los campos obligatorios.<BR> Por favor vuelva <A HREF="javascript:history.back()">atras</A>.<BR>";
exit;
}



elseif($error==2){

echo "El siguiente error ha ocurrido!<BR>";
echo "El correo electronico es invalido!<BR> Por favor vuelva <A HREF="javascript:history.back()">atras</A>.<BR>";
exit;
} 
   

[andresdzphp]

Como te digo el problema está en las comillas... Además de que debes cambiar eregi por preg_match, en google encuentras bastantes expresiones listas.

[CesarHC]

En tus if creo que es asi:

Código PHP:

if($error==1){
echo "El siguiente error ha ocurrido!<BR>"; 
echo 'No ha llenado todos los campos obligatorios.<BR> Por favor vuelva <A HREF="javascript:history.back()">atras</A>.<BR>';
exit;
}
elseif($error==2){
 
echo "El siguiente error ha ocurrido!<BR>";
echo 'El correo electronico es invalido!<BR> Por favor vuelva <A HREF="javascript:history.back()">atras</A>.<BR>';
exit;
}
 

Y no asi:

Código PHP:

if($error==1){
echo "El siguiente error ha ocurrido!<BR>"; 
echo "No ha llenado todos los campos obligatorios.<BR> Por favor vuelva <A HREF="javascript:history.back()">atras</A>.<BR>";
exit;
}
 
 
 
elseif($error==2){
 
echo "El siguiente error ha ocurrido!<BR>";
echo "El correo electronico es invalido!<BR> Por favor vuelva <A HREF="javascript:history.back()">atras</A>.<BR>";
exit;
}
 

Como te dijeron revisa las comillas.

[djkiku]

Iniciado por andresdzphp

Como te digo el problema está en las comillas... Además de que debes cambiar eregi por preg_match, en google encuentras bastantes expresiones listas.

Genial! Voy a por ellas!

Cerramos el tema!!! Me aveis sido de gran ayuda!! Solo me falta encontrar esos codigos sin fallos y listo! (Y leer el articulo de las comillas)

Un millon de Gracias! Si necesitais cual quier cosa! Avisat porfavor! (Se me da muy bien el SEO si hay algo que pueda ayudar.. dudeis en preguntar por mp)

[djkiku]

Iniciado por CesarHC

En tus if creo que es asi:

Código PHP:

if($error==1){
echo "El siguiente error ha ocurrido!<BR>"; 
echo 'No ha llenado todos los campos obligatorios.<BR> Por favor vuelva <A HREF="javascript:history.back()">atras</A>.<BR>';
exit;
}
elseif($error==2){
 
echo "El siguiente error ha ocurrido!<BR>";
echo 'El correo electronico es invalido!<BR> Por favor vuelva <A HREF="javascript:history.back()">atras</A>.<BR>';
exit;
}
 

Y no asi:

Código PHP:

if($error==1){
echo "El siguiente error ha ocurrido!<BR>"; 
echo "No ha llenado todos los campos obligatorios.<BR> Por favor vuelva <A HREF="javascript:history.back()">atras</A>.<BR>";
exit;
}
 
 
 
elseif($error==2){
 
echo "El siguiente error ha ocurrido!<BR>";
echo "El correo electronico es invalido!<BR> Por favor vuelva <A HREF="javascript:history.back()">atras</A>.<BR>";
exit;
}
 

Como te dijeron revisa las comillas.

Lo he probado y nada... me dice esto:

Parse error: syntax error, unexpected T_STRING, expecting ',' or ';' in /home/a2668010/public_html/enviar.php on line 17

Pero Gracias!!!

[djkiku]

Muchas Gracias a todos!!! Encontre en error !!!! Sin ustedes no lo lograria!!!

Simplemente borre lo que esta en negrita:



if($error==1){
echo "El siguiente error ha ocurrido!<BR>";
echo "No ha llenado todos los campos obligatorios.<BR> Por favor vuelva <A HREF="javascript:history.back()">atras</A>.<BR>";
exit;
}



elseif($error==2){

echo "El siguiente error ha ocurrido!<BR>";
echo "El correo electronico es invalido!<BR> Por favor vuelva <A HREF="javascript:history.back()">atras</A>.<BR>";
exit;
}



Muchisimas Gracias!!!!! Todo areglado! :P