Hola:
Has puesto bien los paréntesis y operadores...
De todos modos, puedes orientarte con este artículo:
El abc de los formularios
La forma que yo uso para validar es más o menos:
function validar(formulario, campo1, campo2, campo3) {
var errores = [];
if (formulario[campo1].value == "") errores.push("falta el campo 1");
if (formulario[campo2].value == "") errores.push("falta el campo 2");
if (formulario[campo3].value == "") errores.push("falta el campo 3");
if (errores.length > 0) alert(errores.join("\n"));
return (errores.length == 0)
}
Saludos