Formulario no funciona

[fackius]

Hola amigos.

Me dirian que parte esta mal en este script?

No puedo saber porque no funcionan los alert...
Tengo javascipt activado..y probe un alert aparte sin ninguna otra line ade codigo y funciono.

Creo que estoy llamando mal a la funcion o algo.

Pero uds me diran.

Código HTML:

<script type="text/javascript">
 
function formCheck(){
    var ok=true;
    if (document.formulario.nombre.value==""){      
        ok=false;
        alert("Falta Nombre");
    }
    
    if ((document.formulario.email.value=="")&& (ok)){
        ok=false;
        alert("Falta email");
    }
    if ((document.formulario.asunto.value=="")&& (ok)){
        ok=false;
        alert("Falta Asunto");
    }
    if ((document.formulario.texto.value=="")and (ok)){
        ok=false;
        alert("Falta Mensaje");
    }
    
    if (ok){
        document.formulario.submit();
    }
 
}
 
    </script>
                        
<form name="formulario"  action="envio.php" method="POST" onSubmit="javascript:formCheck();">
    <input type="text" name="nombre" size="30"><br><br>
    <input type="text" name="email" size="30" ><br><br>
    <input type="text" name="asunto" size="30"><br><br>
    <textarea name="texto" rows="7" cols="30"></textarea><br><br>
<input type="submit" value="Enviar" name="enviar" >                 
</form>
 

Lo unico que quiero es que me funcione el javascript para que no se envien campos vacios.

Espero su respuesta!.

Saludos!

[ClubIce]

ya esta corregido:

Código Javascript:

<script type="text/javascript">
 
function formCheck(){
    var ok=true;
    if (document.formulario.nombre.value==""){      
        ok=false;
        alert("Falta Nombre");
    }
    
    if ((document.formulario.email.value=="")&& (ok)){
        ok=false;
        alert("Falta email");
    }
    if ((document.formulario.asunto.value=="")&& (ok)){
        ok=false;
        alert("Falta Asunto");
    }
    if ((document.formulario.texto.value=="")&& (ok)){
        ok=false;
        alert("Falta Mensaje");
    }
    
    if (ok){
        document.formulario.submit();
    }
 
}
 
    </script>
                        
<form name="formulario"  action="envio.php" method="POST">
    <input type="text" name="nombre" size="30"><br><br>
    <input type="text" name="email" size="30" ><br><br>
    <input type="text" name="asunto" size="30"><br><br>
    <textarea name="texto" rows="7" cols="30"></textarea><br><br>
<input type="button" value="Enviar" name="enviar" OnClick="formCheck()">                 
</form>
 

1º) no se usa and se usa && (linea #18)
2º) la llamada a la funcion de validacion se debe colorcar en el OnClick del boton de enviar por que aunque el formulario no sea valido se enviara.

PD: para hacer scripts te recomindo que veas la consola de errores (de Firefox y Opera) asi puedes corregir errores tan simples como este

[fackius]

Gracias ClubIce.

No me di cuenta de ese and. Me olvide de editarlo.

Pero Gracias por las explicaciones y el codigo,

Saludos!

[caricatos]

¡No, no!

La estructura de una validación javascript debe ser así:

<form onsubmit="return validar(this)"...

... y el envío con un botón submit sin más...
<button type="submit">enviar formulario</button>

Luego en la validación:

function validar(f) {
// pasando el formulario como parámetro, se accede a los elementos por su nombre...
vale = f.dato.value != "";
// luego para la cancelación, solo basta con devolver false
// en el ejemplo en la variable vale, si está vacío...
return vale;
}

Si se usa el método submit(), no se enviará nunca si se tiene desactivado javascript desactivado...

Y si quieres hacer una validación buena... también debe hacerse en el servidor.

Saludos