Ayuda para pulir una función para validar

[tampon]

Hola, eh dejado por un tiempo el javascript y ahora no puedo echar a andar esta validacion, alguien me puede ayudar para que encuentre el error? gracias.

esta es la función

Código Javascript:

function validar(formulario) { 
 
var nombre = /(^([a-zA-Z]{1,4}\.\s)?[a-zA-Z\s\u00D1\u00F1\u00E1\u00E9\u00ED\u00F3\u00FA\u00C1\u00C9\u00CD\u00D3\u00DA]{10,400}$)/;
if (!nombre.test(author)) { 
    formulario.alerta.value='Se requiere el nombre del remitente.';
    formulario.author.focus(); 
    return (false); 
} 
 
var mail  = /^([a-zA-Z0-9_.-])+@(([a-zA-Z0-9-])+.)+([a-zA-Z0-9]{2,4})+$/; 
if (!mail.test(email)) { 
    formulario.alerta.value='Email vacio o con formato no valido';
    formulario.email.focus(); 
    return (false); 
}
 
var cadena = /(^[a-zA-Z0-9\s\u00D1\u00F1\u00E1\u00E9\u00ED\u00F3\u00FA\u00C1\u00C9\u00CD\u00D3\u00DAu0021\u0022\u0023\u0024\u0025\u0026\u0027\u0028\u0029\u002B\u002C\u002D\u002E\u002F\u003A\u003B\u003F\u0040\u00A1\u00BF]{10,400}$)/; 
if (!cadena.test(comment)) { 
    formulario.alerta.value='El comentario es muy corto o contiene caracteres invalidos.';
    formulario.comment.focus(); 
    return (false); 
} 
 
return (true); 
}
 

y esto ya es el formulario:

Código HTML:

<form name="formulario" onsubmit="return validar(this);">
 

de antemano gracias.

[caricatos]

Hola:

Por lo pronto parece que te faltan los atributos value de los controles (author, email y comment) en los test...

Saludos

[IsaBelM]

aparte de también faltarle en esos mismos controles el parámetro que le pasa a la función

[tampon]

Gracias a ambos por el dato, saludos.