Duda validar email, error inexplicable!

AdriMoron · #1 (**permalink**) 18/05/2011, 09:30

Buenas tengo un formulario html y necesito mandar mis datos al correo, pues bien esto lo consigo, pero no tengo claro como validar el campo email...aqui os dejo el codigo....pero me da un error extraño, aver si sabeis que puede ser:

Código PHP:

  <? 
$nombre=$_POST['nombre']; 
$telefono=$_POST['telefono']; 
 
$envalado=$_POST['envalado']; 
$direccion=$_POST['direccion']; 
$poblacion=$_POST['poblacion']; 
$provincia=$_POST['provincia']; 
$direccion1=$_POST['direccion1']; 
$poblacion1=$_POST['poblacion1']; 
$provincia1=$_POST['provincia1']; 
$bultos=$_POST['bultos']; 
$tamanomedio=$_POST['tamanomedio']; 
$dia=$_POST['dia']; 
$mes=$_POST['mes']; 
$ano=$_POST['ano']; 
$asunto="Datos de la Mudanza"; 
$mensaje="Estos son las especificaciones de la mudanza: \n"; 
$mensaje.="Nombre = ".$nombre." \n"; 
$mensaje.="Telefono = $telefono \n"; 
$mensaje.="Email = $email \n"; 
$mensaje.="Envalado = $envalado \n"; 
$mensaje.="La mudanza proviene de: \n"; 
$mensaje.="Dirección = $direccion \n"; 
$mensaje.="Población = $poblacion \n"; 
$mensaje.="Provincia = $provincia \n"; 
$mensaje.="La mudanza irá dirigida a: \n"; 
$mensaje.="Dirección = $direccion1 \n"; 
$mensaje.="Población = $poblacion1 \n"; 
$mensaje.="Provincia = $provincia1 \n"; 
$mensaje.="Bultos = $bultos \n"; 
$mensaje.="Tamaño medio = $tamanomedio \n"; 
$mensaje.="La mudanza se hará en la siguiente fecha: \n"; 
$mensaje.="Dia = $dia \n"; 
$mensaje.="Mes = $mes \n"; 
$mensaje.="Año = $ano \n"; 
 
function valida_email($email){    
  if(eregi("^([_a-z0-9-]+)(\.[_a-z0-9-]+)*@([a-z0-9-]+)(\.[a-z0-9-]+)*(\.[a-z]{2,4})$", $email))    
  return true;    
    else   
  return false; 
} 
$email = $_POST['email']; 
if(empty($_POST['nombre']) || empty($_POST['telefono']) || empty($_POST['email'])) 
    echo "Nombre, Teléfono y Email son campos obligatorios"; 
elseif(strlen($telefono) != 9) 
    echo "El telefono debe contener 9 caracteres"; 
elseif(!is_numeric($telefono)) 
    echo "El telefono solo debe contener numeros"; 
elseif(valida_email($email)) 
    mail("[email protected]",$asunto,$mensaje,$email) 
else 
    echo "El mail NO es valido";  
 
     
 
 
?>

Error que me da:
Parse error: syntax error, unexpected T_ELSE in /homepages/26/d225861722/htdocs/adrian/mudancis/enviar.php on line 53

pateketrueke · #2 (**permalink**) 18/05/2011, 09:41

¿Extraño?

¡Para nada!

Solo estas olvidando colocar el punto-y-coma al final de la instrucción email() de la linea anterior a la del error.

AdriMoron · #3 (**permalink**) 18/05/2011, 10:11

Cita:

Iniciado por pateketrueke

¿Extraño?

¡Para nada!

Solo estas olvidando colocar el punto-y-coma al final de la instrucción email() de la linea anterior a la del error.

Pues lo acabo de poner y ahora me salta el siguiente error:
Parse error: syntax error, unexpected T_ELSEIF in /homepages/26/d225861722/htdocs/mundacis/enviar.php on line 48

Aqui mi nuevo codigo:

Código PHP:

  <? 
$nombre=$_POST['nombre']; 
$telefono=$_POST['telefono']; 
 
$envalado=$_POST['envalado']; 
$direccion=$_POST['direccion']; 
$poblacion=$_POST['poblacion']; 
$provincia=$_POST['provincia']; 
$direccion1=$_POST['direccion1']; 
$poblacion1=$_POST['poblacion1']; 
$provincia1=$_POST['provincia1']; 
$bultos=$_POST['bultos']; 
$tamanomedio=$_POST['tamanomedio']; 
$dia=$_POST['dia']; 
$mes=$_POST['mes']; 
$ano=$_POST['ano']; 
$asunto="Datos de la Mudanza"; 
$mensaje="Estos son las especificaciones de la mudanza: \n"; 
$mensaje.="Nombre = ".$nombre." \n"; 
$mensaje.="Telefono = $telefono \n"; 
$mensaje.="Email = $email \n"; 
$mensaje.="Envalado = $envalado \n"; 
$mensaje.="La mudanza proviene de: \n"; 
$mensaje.="Dirección = $direccion \n"; 
$mensaje.="Población = $poblacion \n"; 
$mensaje.="Provincia = $provincia \n"; 
$mensaje.="La mudanza irá dirigida a: \n"; 
$mensaje.="Dirección = $direccion1 \n"; 
$mensaje.="Población = $poblacion1 \n"; 
$mensaje.="Provincia = $provincia1 \n"; 
$mensaje.="Bultos = $bultos \n"; 
$mensaje.="Tamaño medio = $tamanomedio \n"; 
$mensaje.="La mudanza se hará en la siguiente fecha: \n"; 
$mensaje.="Dia = $dia \n"; 
$mensaje.="Mes = $mes \n"; 
$mensaje.="Año = $ano \n"; 
 
function valida_email($email){    
  if(eregi("^([_a-z0-9-]+)(\.[_a-z0-9-]+)*@([a-z0-9-]+)(\.[a-z0-9-]+)*(\.[a-z]{2,4})$", $email))    
  return true;    
    else   
  return false; 
} 
$email = $_POST['email']; 
if(empty($_POST['nombre']) || empty($_POST['telefono']) || empty($_POST['email'])) 
    echo "Nombre, Teléfono y Email son campos obligatorios"; 
    header ("location:indexerror.html"); 
elseif(strlen($telefono) != 9) 
    echo "El telefono debe contener 9 caracteres"; 
    header ("location:indexerror1.html"); 
elseif(!is_numeric($telefono)) 
    echo "El telefono solo debe contener numeros"; 
    header ("location:indexerror1.html"); 
elseif(valida_email($email)) 
    mail("[email protected]",$asunto,$mensaje,$email); 
else 
    echo "El mail NO es valido";  
 
     
 
 
?>

Gracias por responder.

pep0te · #4 (**permalink**) 18/05/2011, 10:13

Código PHP:

  if(empty($_POST['nombre']) || empty($_POST['telefono']) || empty($_POST['email'])) {
    echo "Nombre, Teléfono y Email son campos obligatorios";
    header ("location:indexerror.html");
}elseif(strlen($telefono) != 9) {
    echo "El telefono debe contener 9 caracteres";
    header ("location:indexerror1.html");
}elseif(!is_numeric($telefono)){
    echo "El telefono solo debe contener numeros";
    header ("location:indexerror1.html");
}elseif(valida_email($email)){
    mail("[email protected]",$asunto,$mensaje,$email);
}else{
    echo "El mail NO es valido"; 
}

Prueba así

AdriMoron · #5 (**permalink**) 18/05/2011, 10:14

Cita:

Iniciado por pep0te

Código PHP:

  if(empty($_POST['nombre']) || empty($_POST['telefono']) || empty($_POST['email'])) { 
    echo "Nombre, Teléfono y Email son campos obligatorios"; 
    header ("location:indexerror.html"); 
}elseif(strlen($telefono) != 9) { 
    echo "El telefono debe contener 9 caracteres"; 
    header ("location:indexerror1.html"); 
}elseif(!is_numeric($telefono)){ 
    echo "El telefono solo debe contener numeros"; 
    header ("location:indexerror1.html"); 
}elseif(valida_email($email)){ 
    mail("[email protected]",$asunto,$mensaje,$email); 
}else{ 
    echo "El mail NO es valido";  
}

Prueba así

Ajam, que pruebes colocando las llaves no?

pep0te · #6 (**permalink**) 18/05/2011, 10:16

Sí, así no da error. En el último elseif // else podrías quitarlas. Al ser 2 sentencias después del elseif, es obligatorio ponerlas (o eso creo).

AdriMoron · #7 (**permalink**) 18/05/2011, 10:19

Cita:

Iniciado por pep0te

Sí, así no da error. En el último elseif // else podrías quitarlas. Al ser 2 sentencias después del elseif, es obligatorio ponerlas (o eso creo).

Cierto es, pero ahora no se lo que pasa que mi navegador no me redirecciona a indexerror.html

Error:Nombre, Teléfono y Email son campos obligatorios
Warning: Cannot modify header information - headers already sent by (output started at /homepages/26/d225861722/htdocs/mundacis/enviar.php:47) in /homepages/26/d225861722/htdocs/mundacis/enviar.php on line 48

codigo php

Código PHP:

  <? 
$nombre=$_POST['nombre']; 
$telefono=$_POST['telefono']; 
 
$envalado=$_POST['envalado']; 
$direccion=$_POST['direccion']; 
$poblacion=$_POST['poblacion']; 
$provincia=$_POST['provincia']; 
$direccion1=$_POST['direccion1']; 
$poblacion1=$_POST['poblacion1']; 
$provincia1=$_POST['provincia1']; 
$bultos=$_POST['bultos']; 
$tamanomedio=$_POST['tamanomedio']; 
$dia=$_POST['dia']; 
$mes=$_POST['mes']; 
$ano=$_POST['ano']; 
$asunto="Datos de la Mudanza"; 
$mensaje="Estos son las especificaciones de la mudanza: \n"; 
$mensaje.="Nombre = ".$nombre." \n"; 
$mensaje.="Telefono = $telefono \n"; 
$mensaje.="Email = $email \n"; 
$mensaje.="Envalado = $envalado \n"; 
$mensaje.="La mudanza proviene de: \n"; 
$mensaje.="Dirección = $direccion \n"; 
$mensaje.="Población = $poblacion \n"; 
$mensaje.="Provincia = $provincia \n"; 
$mensaje.="La mudanza irá dirigida a: \n"; 
$mensaje.="Dirección = $direccion1 \n"; 
$mensaje.="Población = $poblacion1 \n"; 
$mensaje.="Provincia = $provincia1 \n"; 
$mensaje.="Bultos = $bultos \n"; 
$mensaje.="Tamaño medio = $tamanomedio \n"; 
$mensaje.="La mudanza se hará en la siguiente fecha: \n"; 
$mensaje.="Dia = $dia \n"; 
$mensaje.="Mes = $mes \n"; 
$mensaje.="Año = $ano \n"; 
 
function valida_email($email){    
  if(eregi("^([_a-z0-9-]+)(\.[_a-z0-9-]+)*@([a-z0-9-]+)(\.[a-z0-9-]+)*(\.[a-z]{2,4})$", $email))    
  return true;    
    else   
  return false; 
} 
$email = $_POST['email']; 
if(empty($_POST['nombre']) || empty($_POST['telefono']) || empty($_POST['email'])) 
    { 
    echo "Nombre, Teléfono y Email son campos obligatorios"; 
    header ("location:indexerror.html"); 
    } 
elseif(strlen($telefono) != 9) 
    { 
    echo "El telefono debe contener 9 caracteres"; 
    header ("location:indexerror1.html"); 
    } 
elseif(!is_numeric($telefono)) 
    { 
    echo "El telefono solo debe contener numeros"; 
    header ("location:indexerror1.html"); 
    } 
elseif(valida_email($email)) 
    mail("[email protected]",$asunto,$mensaje,$email); 
else 
    echo "El mail NO es valido";  
 
     
 
 
?>

pateketrueke · #8 (**permalink**) 18/05/2011, 10:24

Erro clásico: si vas a redireccionar, ¿para que imprimes?

AdriMoron · #9 (**permalink**) 18/05/2011, 10:27

Cita:

Iniciado por pateketrueke

Erro clásico: si vas a redireccionar, ¿para que imprimes?

Jajaja cierto, muchas gracias, todo resuelto! no hacia falta el echo!

Patriarka · #10 (**permalink**) 18/05/2011, 10:31

siempre llego tarde, pero podrias actualizar la funcion eregi
ya que esta obsoleta, te va a servir

AdriMoron · #11 (**permalink**) 18/05/2011, 10:32

Cita:

Iniciado por Patriarka

siempre llego tarde, pero podrias actualizar la funcion eregi
ya que esta obsoleta, te va a servir

¿Y cual es su actualización?

pep0te · #12 (**permalink**) 18/05/2011, 10:34

Cita:

Iniciado por pateketrueke

Erro clásico: si vas a redireccionar, ¿para que imprimes?

Joder, es verdad xDDD