Buenas.
Te paso una función que uso para validar campos obligatorios, que se adpata fácil a cualquier formulario y además te indica cuál es el campo que falta completar.
Código:
function validarObligatorios() {
var msg = "";
var campoNombre= document.frm.nombre_cl.value;
var campoApellido = document.frm.apellido_cl.value;
if (campoNombre== "") {
msg += "Debe ingresar su nombre\n";
}
if (campoApellido == "") {
msg += "Debe ingresar su apellido\n";
}
if (msg != "") {
window.alert(msg);
return false;
}
return true;
}
El funcionamiento es simple. Al principio incializa la variable msg que va a guardar el mensaje de error. Por cada campo que esté vacío, agrega una línea a msg. En el último if, se fija si msg es distinto de vacío. Si es así, significa que hay campos obligatorios que quedaron vacíos. En ese caso, la función devuelve false.
Si msg está vacío, es que el formulario está bien, y entonces devuelve true. no hace falta poner este return en un else, porque sí entra en el if anterior, la función ya va a tener un return, y al llegar a ese punto, va a cortarse su ejecución: return true no va a leerse.
Suerte
Califa