Problema simple: Validar campos de formulario(Error de código).

[ras_chalo]

Hola, bueno resulta que estuve practicando haciendo un script para un formulario...algo simple, solo validar que los campos que 2 campos no esten vacios y que avise mediante un mensaje de alerta...
OK..el script valida que no este vacio con su respectivo mensaje

pero mis problemas son 2:

1ro: una vez que me manda el mensaje de alerta, luego de aceptarlo, me envia el formulario. Es decir me sale el mensaje pero envia igual el formulario.

2do: Me sale dos veces el mensaje de "GRACIAS"....


Dejo el script:

Código Javascript:

<SCRIPT LANGUAGE="JavaScript">
                function valida_envia(){ 
    //valido el nombre 
    if (document.fvalida.nombre.value.length==0){ 
         alert("Tiene que escribir su nombre") 
         document.fvalida.nombre.focus() 
         return 0; 
    } 
 
    
 
    //valido el inter�s 
    if (document.fvalida.asunto.selectedIndex==0){ 
         alert("Debe seleccionar un asunto de su contacto.") 
         document.fvalida.interes.focus() 
         return 0; 
    } 
    
        //el formulario se envia y da las GRACIAS
    alert("Muchas gracias por enviar el formulario");
    document.fvalida.submit();
    
} 
</SCRIPT>
 
[B]Dejo el formulario:[/B]
 

Código HTML:

<table>
<form id="form" name="fvalida" method="post" action="valida_contacto.php" onsubmit="return valida_envia(this);">
    <tr><td></td><td>Empresa: </td> <td><input name="empresa" type="text" id="empresa" size="35"  /></td><tr>
    <tr><td></td><td>Cargo: </td> <td><input name="cargo" type="text" id="cargo" size="35"  /></td></tr>
    <tr><td>*</td><td>Nombre Contacto: </td> <td><input name="nombre" type="text" id="nombre" size="35"  /></td></tr>    
    <tr><td>*</td><td>Apellidos: </td> <td><input name="apellidos" type="text" id="apellidos" size="35"  /></td></tr>  
    <tr><td>*</td><td>Tel&eacute;fono: </td> <td><input name="telefono" type="text" id="telefono" size="15" /></td></tr>   
    <tr><td>*</td><td>E-mail: </td> <td><input name="email" type="text" id="email" size="25" /></td></tr>   
 
<tr><td> </td><td> </td><td><br/><input type="submit"  value=" Enviar Formulario " onclick="valida_envia()"/>
            </td></tr>
            
</table>
 

Muchas gracias por el tiempo y espero sus respuestas!!

Saludos!

[MK2]

EDITO:

Varias cosas:

1 - El form está dentro del <table> entre el tag table y tr. Eso está mal.... debería estar fuera.
2 - Hay un tr sin cerrar (el primero).
3 - No tiene sentido utilizar el evento onclick en el botón submit llamando a valida_envia si después haces los mismo en el action del form, por eso te aparecen 2 veces los mensajes. Yo haría onsubmit="valida_envia(this);return false;" y borraría el evento onclick del input submit.


Saludos

[caricatos]

Iniciado por MK2

3 - No tiene sentido utilizar el evento onclick en el botón submit llamando a valida_envia si después haces los mismo en el action del form, por eso te aparecen 2 veces los mensajes. Yo haría onsubmit="valida_envia(this);return false;" y borraría el evento onclick del input submit.

Aparte de cambiar lo del return 0 por return false...

Y es mejor así:

onsubmit="return valida_envia(this)"

Otra cosilla, lo de this es un parámetro que representa el mismo formulario, así que el código se simplificaría cambiando:
document.fvalida por el parámetro que pongas en la declaración de la función, que no has puesto, por ejemplo:

function valida_envia(f)

y dentro...

if (f.nombre.value.length==0){ ...

No me he fijado con detenimiento, pero puedes empezar con esas cosillas...

Saludos

[ras_chalo]

Muchas gracias perfecto funciono a la perfección!!

Saludos =D !!!