Pequeño problema en formlario

[idro]

Hola, estoy haciendo una web, y tengo un problema con el formulario de contacto. Funciona correctamente, pero si no se rellenan los "campos" y se "envia" se recibe un mensaje en blanco.

Como lo puedo hacer para que esto no ocurra? que funcion hace esto?

Mi codigo es este:

function enviar(){

name = document.form2.Nombre.value;
ape = document.form2.Apellidos.value;
mail = document.form2.Email.value;
com = document.form2.Comentario.value;

if(name == ""){
alert("El campo 'Nombre' es obligatorio");
document.form2.Nombre.focus();
return;
}
if(ape == ""){
alert("El campo 'Apellidos' es obligatorio");
document.form2.Apellidos.focus();
return;
}
if(Email == ""){
alert("El campo 'E-mail' es obligatorio");
document.form2.Email.focus();
return;
}
else{
if(Email.indexOf("@") == -1 || Email.indexOf("@") != Email.lastIndexOf("@")){
alert("Direccion de 'E-mail' incorrecto");
document.form2.Email.value="";
document.form2.Email.focus();
return;
}
}
if(com == ""){
alert("Introduzca su comentario");
document.form2.Comentario.focus();
}
else{
alert("Formulario enviado correctamente");
document.form2.Nombre.value="";
document.form2.Apellidos.value="";
document.form2.Email.value="";
document.form2.Comentario.value="";

}

}

Que os parece? como puedo solucionar que no se envie el formulario si esta en blanco?

[David]

No entiendo lo que quieres decir. ¿A qué te refieres con que recibe un mensaje en blanco?

Prueba a poner return false en lugar de return y lo llamas desde el evento onsubmit.

[idro]

haer, si tu envez de rellenar el formulario, clicas directamente a al boton enviar "submit", sin rellenar nada, yo recibo el mensaje pero sin nada...

Nombre:"..........."

Apellidos: "..........."

email: "..........."

Comentario: "..........."

asi me sale, en blanco, que tengo que hacer para que no ejecute esta funcion si no me escriben nada en el formulario?

Saludos

[David]

Como te dije, sustituye todos los return; por return false;

Además, llama a la función en el onsubmit de esta manera:

Código html:

onsubmit="return enviar();"
 

[idro]

He puesto el return false; i nada... me hace lo mismo. Y donde tengo que poner "onsubmit="return enviar();" en el function ....() {}


function onsubmit= return enviar(){

}

O como?

No hay ninguna manera de que si no cumple el primer "if" termine la funcion? poniendo "else" end o fin...

[David]

No, el onsubmit debes ponerlo en el tag <form>.

P.S.: ¿Cómo estás llamando a la función en este momento?

[idro]

function enviar(){
----------------
----------------
}

Voy a provar, haver...

[David]

No, esa es la definición de la función. Lo que quiero saber es en qué momento llamas a la función, en qué evento y en qué elemento.

[idro]

Pues cuando le das asl boton de enviar:

<div id="enviar">
<input type="submit" name="boton" value="Enviar" onclick="enviar()" method="post" action="/cgibin/FormMail.pl" />

<input type="reset" Value="Borrar" />
</div>

Hora he puesto en el <form> lo del "onsubmit= return enviar(){"... y funciona, si no pones nada te sale la ventana diciendote que es obligatorio poner el nombre y no havance y no se envia el formulario en "blanco".

Ire provando con cada 1, haver si funciona correctamente.

Saludos

[idro]

me falla a partir de apellidos, tendre que revisar el codigo... haver.

[idro]

Ves algo mal apartir de aqui¿? aqui me falla, nombre y apellidos si funciona, si no pones nada no continua, pero en email y comentario, si no pones nada si continua y se envia el formulario con los campos nombrados en "blanco".

if(Email == ""){
alert("El campo 'E-mail' es obligatorio");
document.form2.Email.focus();
return false;
}
else{
if(Email.indexOf("@") == -1 || Email.indexOf("@") != Email.lastIndexOf("@")){
alert("Direccion de 'E-mail' incorrecto");
document.form2.Email.value="";
document.form2.Email.focus();
return false;
}
}
if(com == ""){
alert("Introduzca su comentario");
document.form2.Comentario.focus();
return false;
}
else{
alert("Formulario enviado correctamente");
document.form2.Nombre.value="";
document.form2.Apellidos.value="";
document.form2.Email.value="";
document.form2.Comentario.value="";

}

Saludos

[David]

Sí, y es que tu variable se llama mail, no Email.

[idro]

Ostras, es cierto, esque como lo cambie, para que el nombre me saliera Email, envez de mail, y sin querer cambi el nombre de la variable.

Madre mia... jajejajja, sorry y gracias por tu tiempo compañero!

Saludos

[David]

Me alegra que lo soluciones:

P.S.: ¿Por qué pusiste el method y el action en el input submit cuando deberían ir en el form?

[idro]

Pues la verdad, esque el form lo ya lo tenia hecho de cuando hice el curso de disseño web, creo recordar que nos los enseño el profe, nose... tonces lo quito.

PD: ahora me pasa lo mismo pero con el "comentario", haver si haveriguo lo que tengo mal.

Saludos

[idro]

Joe, tu ves algo? porqe yo no...

if(com == ""){
alert("Introduzca su comentario");
document.form2.Comentario.focus();
return false;
}
else{
alert("Formulario enviado correctamente");
document.form2.Nombre.value="";
document.form2.Apellidos.value="";
document.form2.Email.value="";
document.form2.Comentario.value="";

}

Saludos

[David]

En principio no parece haber error. Pero, me pregunto, ¿por qué vacías los valores de los campos del formulario? Si lo haces, el formulario se enviará vacío.

[idro]

Ok, ya lo he quetado, pero sigue igual...

Saludos

[David]

¿Puedes poner el código del formulario tal como lo tienes ahora?

[idro]

<form method="POST" action="/cgibin/FormMail.pl" name="form2" onsubmit="return enviar()">

<input type="hidden" name="recipient" value="[email protected]" />

<input type="hidden" name="subject" value="Tiene 1 nuevo mensaje: Formulario de contacto"/>

<input type="hidden" name="redirect" value="http://www.jjfemar.es" />

<input type="hidden" name="realname" value="Mensaje web" />

<div class="bloque2">
<div class="titulo">
Nombre:
</div>
<div class="campo">
<input type="text" name="Nombre" value="" />
</div>
</div>

<div class="bloque2">
<div class="titulo">
Apellidos:
</div>
<div class="campo">
<input type="text" name="Apellidos" value="" />
</div>
</div>

<div class="bloque2">
<div class="titulo">
E-mail:
</div>
<div class="campo">
<input type="text" name="Email" value="" />
</div>
</div>

<div class="bloque2">
<div class="titulo">
Deje su comentario:
</div>
<div class="campo">
<textarea name="Comentario" value="" rows="8" cols="35" maxlength="30" size="10" ></textarea>
</div>
</div>

esto es...



<div class="bloque2">
<div id="enviar">
<input type="submit" name="boton" value="Enviar" onclick="enviar()" method="post" action="/cgibin/FormMail.pl" />
<input type="reset" Value="Borrar" />
</div>
</div>
</form>

[David]

Para empezar deja tu input submit así, quitando todo lo innecesario (lo que pertenece al <form>):

Código html:

<input type="submit" name="boton" value="Enviar" />
 

P.S.: Tu formulario tiene una divitis aguda (exceso de div, recuerda que puedes usar el tag <label> para mostrar el texto relacionado con el campo).

[idro]

Pero los "div" es para que el formulario este ordenado... asi todo esta en la misma posicion.

Saludos

[David]

Créeme, puedes lograr el mismo efecto sin toda esa cantidad de divs que sólo ensucian el código.

Pero bueno, esa era sólo una observación y no el foco del problema, ¿ya corregiste el input submit? ¿Cuál es el resultado en este momento?

[idro]

Si... el mismo, sigue pasando igual.

Saludos

[David]

Revisa la Consola de Errores de tu navegador para ver si está dando algún error.

Si no logras encontrar el error, copia la función de validación como lo tienes ahora, para que podamos ver el error.

[idro]

Ok, me daba error de la varieble mail... y es que solo cambie la primera:

if(mail.indexOf("@") == -1 || Email.indexOf("@") != Email.lastIndexOf("@")){

Bien, despues de estas provando y provando, parece que ya funciona bien... si todo perfecto.

Saludos y gracias!