Comprobar si los campos están rellenos

[keco]

Hola, mu buenas he cogido este script de esta misma web, y me funciona todo menos que cuando hay algun campo sin rellenar sigue enviando el formulario y eso es lo que no kiero.

El script es este:

<script>
function comprobar(){var mal="";
numero=document.forms[0].elements.length;
for(a=0;a<numero;a++){
if (document.forms[0].elements[a].value==""){document.forms[0].elements[a].style.backgroundColor="#ffffcc";mal=true;}
else{document.forms[0].elements[a].style.backgroundColor="#D9D9D9";}
}
if(mal){alert("Por favor, rellene los campos vacíos y coloreados.");}
else{document.forms[0].submit()}
}
</script>

Yo no entiendo mucho de javascript xo en la segunda condición no deberia haber algo que parase el formulario?? igual que en el else se envia en el if se deberia parar.
Bueno yo no entiendo mucho por eso pido ayuda.

Venga muchas gracias.
Dew.