Question

My validation function looks like that.

var fname = $("#fname").val();
var lname = $("#lname").val();

function validate() {
    var isValid = true;
     if (!fname) {
            $("#fname").attr('class', 'invalid');
            isValid=false;
        }

    if (!lname) {
        $("#lname").attr('class', 'invalid');
        isValid=false;
    }

It simply changes the class of unfilled input box.

I know that i can write else for every if and change back to default (class="valid") if user fills some of inputs. But how can i create something universal for all inputs to change back to default class the input that user has filled after first validation error?

Answer 1

That was good Tural! HOWEVER, why the excess processing in your code? That will add unecessary stress. Since you, for what you “solved”, will add the “valid” class to ALL the input type text or password, just add that to the actual input element in the straight code:

<input class='valid' ..... />

Now, back to your original validation: why not make it universal?:

function validate(formField) {
    if !formField $('#'+formField).removeClass('valid').addClass('invalid');
}

Or something in that vein …