Пахнет так, как будто вам нужен какой-то рефакторинг и извлекать код, который все время повторяется, в свою собственную функцию:
function validateElement(element, condition) {
if(condition){
element.removeClass("error");
}
else {
element.addClass("error");
}
return condition;
}
Используйте это так (я также рекомендую превратить ваши переменные в параметры, чтобывы можете использовать валидаторы в разных полях):
function validateEmail(field) {
//testing regular expression
var a = field.val(),
filter = /^[a-zA-Z0-9]+[a-zA-Z0-9_.-]+[a-zA-Z0-9_-]+@[a-zA-Z0-9]+[a-zA-Z0-9.-]+[a-zA-Z0-9]+.[a-z]{2,4}$/;
return validateElement(field, filter.test(a));
}
validateEmail($('#email'));