Ayuda con "Validacion de un formulario"

[Xilver266]

Hola!, estoy haciendo un formulario, y le quiero poner unas validaciones. Ya le he añadido un codigo, pero no me funciona, así que confío en vosotros para que me ayudeis xD

Aquí os dejo el codigo PHP:

Código PHP:

  <?php 
 
    if (empty($_POST['nombre'])) $error .= "<p class=\"error\">No has introducido tu nombre</p>"; } 
    if (empty($_POST['mail'])) $error .= "<p class=\"error\">No has introducido tu dirección de correo</p>"; } 
    if (empty($_POST['tipo'])) $error .= "<p class=\"error\">No has introducido el tipo de reporte</p>"; } 
    if (empty($_POST['mensaje'])) $error .= "<p class=\"error\">No has escrito nada en el cuerpo del mensaje</p>"; } 
    $text_len = strlen($_POST['mensaje']);  
    if($text_len > 800) { $error .= "<p class=\"error\">El comentario debe ser de menor a 800, se ingresaron $text_len </p>"; } 
    if(isset($_POST['mail'])) 
      { 
        if (!comprobar_email($_POST['mail'])) 
        { $error .= "<p class=\"error\">La dirección de correo es incorrecta</p>"; } 
      }// end of email check 
    if($error) { 
        echo $error; 
 
 
$nombre = $_POST['nombre']; 
$mail = $_POST['mail']; 
$tipo = $_POST['tipo']; 
$contenido = $_POST['mensaje']; 
 
$header = 'From: ' . $mail . " \r\n"; 
$header .= "X-Mailer: PHP/" . phpversion() . " \r\n"; 
$header .= "Mime-Version: 1.0 \r\n"; 
$header .= "Content-Type: text/plain"; 
 
$mensaje = "Este mensaje fue enviado por " . $nombre . ", Usuario de NeverFall.co.cc \r\n"; 
$mensaje .= "Su e-mail es: " . $mail . " \r\n"; 
$mensaje .= "Tipo: " . $tipo . " \r\n"; 
$mensaje .= "Mensaje: " . $contenido . " \r\n"; 
$mensaje .= "Enviado el " . date('d/m/Y', time()); 
 
$para = "[email protected]"; 
$asunto = "Bug Reportado en NeverFall"; 
 
$descodificado = utf8_decode($mensaje); 
 
mail($para, $asunto, $descodificado, $header); 
 
header("Location: http://neverfall.co.cc/bugreport/mensajecorrecto.html"); 
 
?>   

Si necesitais algun dato más, no teneis más que pedirlo.

Saludos y gracias anticipadas

[dggluz]

Me parece que te faltó cerrar un if ("}"). ¿No te funciona nada o sólo no hace lo que esperas?, ¿tira algún error? Indenta el código y ponlo más prolijo, sino siempre tendrás errores de sintaxis. ¡Suerte!

[Xilver266]

Sigue sin funcionarme, ahora tengo este código:

Código PHP:

  if (empty($_POST['nombre'])) $error .= "<p class=\"error\">No has introducido tu nombre</p>"; } 
    if (empty($_POST['mail'])) $error .= "<p class=\"error\">No has introducido tu dirección de correo</p>"; } 
    if (empty($_POST['tipo'])) $error .= "<p class=\"error\">No has introducido el tipo de reporte</p>"; } 
    if (empty($_POST['mensaje'])) $error .= "<p class=\"error\">No has escrito nada en el cuerpo del mensaje</p>"; } 
    $text_len = strlen($_POST['mensaje']);  
    if($text_len > 800) { $error .= "<p class=\"error\">El comentario debe ser de menor a 800, se ingresaron $text_len </p>"; } 
    if(isset($_POST['mail'])) } 
      { 
        if (!comprobar_email($_POST['mail'])) } 
        { $error .= "<p class=\"error\">La dirección de correo es incorrecta</p>"; } 
      }// end of email check 
    if($error)}  
      { 
        echo $error; 
 
 
$nombre = $_POST['nombre']; 
$mail = $_POST['mail']; 
$tipo = $_POST['tipo']; 
$contenido = $_POST['mensaje']; 
 
$header = 'From: ' . $mail . " \r\n"; 
$header .= "X-Mailer: PHP/" . phpversion() . " \r\n"; 
$header .= "Mime-Version: 1.0 \r\n"; 
$header .= "Content-Type: text/plain"; 
 
$mensaje = "Este mensaje fue enviado por " . $nombre . ", Usuario de NeverFall.co.cc \r\n"; 
$mensaje .= "Su e-mail es: " . $mail . " \r\n"; 
$mensaje .= "Tipo: " . $tipo . " \r\n"; 
$mensaje .= "Mensaje: " . $contenido . " \r\n"; 
$mensaje .= "Enviado el " . date('d/m/Y', time()); 
 
$para = "[email protected]"; 
$asunto = "Bug Reportado en NeverFall"; 
 
$descodificado = utf8_decode($mensaje); 
 
mail($para, $asunto, $descodificado, $header); 
 
header("Location: http://neverfall.co.cc/bugreport/mensajecorrecto.html"); 
 
?>  
  

Si solo hay un fallo o 2, que alguien copie el codigo, lo reparte y me lo vuelva a oegar

Saludos y gracias

[juamd]

Que tal !!!

Hay muchos errores en los if y según el codigo veo que envias el mensaje si hay error, no se si esa es la lógica de la aplicación pero de todas formas revisa esto:

Código PHP:

  <?php 
 
if (empty($_POST['nombre'])) {$error .= "<p class=\"error\">No has introducido tu nombre</p>"; 
} 
if (empty($_POST['mail'])) {$error .= "<p class=\"error\">No has introducido tu dirección de correo</p>"; 
} 
if (empty($_POST['tipo'])) {$error .= "<p class=\"error\">No has introducido el tipo de reporte</p>"; 
} 
if (empty($_POST['mensaje'])) {$error .= "<p class=\"error\">No has escrito nada en el cuerpo del mensaje</p>"; 
} 
$text_len = strlen($_POST['mensaje']); 
if($text_len > 800) { $error .= "<p class=\"error\">El comentario debe ser de menor a 800, se ingresaron $text_len </p>"; 
} 
if(isset($_POST['mail'])){ 
    if (!comprobar_email($_POST['mail'])) { 
        $error .= "<p class=\"error\">La dirección de correo es incorrecta</p>"; 
    } 
}// end of email check 
if($error) 
{ 
echo $error; 
 
 
$nombre = $_POST['nombre']; 
$mail = $_POST['mail']; 
$tipo = $_POST['tipo']; 
$contenido = $_POST['mensaje']; 
 
$header = 'From: ' . $mail . " \r\n"; 
$header .= "X-Mailer: PHP/" . phpversion() . " \r\n"; 
$header .= "Mime-Version: 1.0 \r\n"; 
$header .= "Content-Type: text/plain"; 
 
$mensaje = "Este mensaje fue enviado por " . $nombre . ", Usuario de NeverFall.co.cc \r\n"; 
$mensaje .= "Su e-mail es: " . $mail . " \r\n"; 
$mensaje .= "Tipo: " . $tipo . " \r\n"; 
$mensaje .= "Mensaje: " . $contenido . " \r\n"; 
$mensaje .= "Enviado el " . date('d/m/Y', time()); 
 
$para = "[email protected]"; 
$asunto = "Bug Reportado en NeverFall"; 
 
$descodificado = utf8_decode($mensaje); 
} 
 
mail($para, $asunto, $descodificado, $header); 
 
header("Location: http://neverfall.co.cc/bugreport/mensajecorrecto.html"); 
?>   

Lo que te acabo de pegar no tiene errores de sintaxis pero creo que de lógica si, y otra cosa deberias usas un ide, puedes mirar netbeans o eclipse estos te ayudan a escribir un codigo mas ordenado y fácil de entender.

Saludos.

[Xilver266]

He puesto ese código y sigue sin ir, y esa no es la logica,

A ver si alguien puede arreglar ese codigo porfaplis

Saludos

[juamd]

No puedes pretender que la gente aca te arregle todo, la idea de esto es aclarar las dudas y ayudarnos mutuamente pero tambien tienes que poner de tu parte, tienes que tomar el codigo y correrlo por bloques, utilizar die() para ver donde entra y donde no entra, imprimir las variables o usar var_dump() para que vacie el resultado y puedas ver que estas haciendo bien o mal. Además debes verificar si la función mail si te esta enviando el mensaje y si no lo esta enviando cual es la razón.

Saludos.

[Xilver266]

Iniciado por juamd

No puedes pretender que la gente aca te arregle todo, la idea de esto es aclarar las dudas y ayudarnos mutuamente pero tambien tienes que poner de tu parte, tienes que tomar el codigo y correrlo por bloques, utilizar die() para ver donde entra y donde no entra, imprimir las variables o usar var_dump() para que vacie el resultado y puedas ver que estas haciendo bien o mal. Además debes verificar si la función mail si te esta enviando el mensaje y si no lo esta enviando cual es la razón.

Saludos.

El problema es que no tengo ni zorra de PHP, pero dentro de poco empezaré a aprenderlo, por eso recurro al foro para que me ayuden a solucionar el bug que hay en el codigo.

Es por eso que necesito el codigo FULL, y no que me den unos datos que ni siquiera se lo que son

[dggluz]

Iniciado por Xilver266

El problema es que no tengo ni zorra de PHP, pero dentro de poco empezaré a aprenderlo, por eso recurro al foro para que me ayuden a solucionar el bug que hay en el codigo.

Es por eso que necesito el codigo FULL, y no que me den unos datos que ni siquiera se lo que son

Xilver266: lo siento, pero eso no forma parte del espíritu de FDW. La idea aquí es ayudar a los demás a aprender, no realizar trabajo gratis. Muchos de nosotros trabajamos en programación y nos pagan, ¿para qué hacerlo gratis? Sólo que sentimos la vocación docente y disfrutamos del placer de enseñar y de ayudar. Si quieres aprender PHP, éste foro pone a disposición un excelente manual para que lo hagas. ¡Suerte!