Question

How could I improve next function?

function treatFormLinks () {
    if (!$('#One input').valid() ) {
        $('#LinkOne').addClass('hasErrors');    
    }
    if (!$('#Two input').valid() ) {
        $('#LinkTwo').addClass('hasErrors');    
    }
    if (!$('#Three input').valid() ) {
        $('#LinkThree').addClass('hasErrors');  
    }
};

$("#Submit").click(function () {
    treatFormLinks();
});

For example, if I’d had hundred (#One, #Two .etc) containers it would be stupid to create conditions for each case. So how could I make it less “id selector dependent”.

Thank you!

Answer 1

You can try using each and toggleClass methods.

function treatFormLinks () {
   $('#One, #Two, #Three').each(function(){
      $('#Link'+this.id).toggleClass('hasErrors', !$('input', this).valid())
   })
};