Trả về giá trị true/false của hàm javascript và sau đó thực hiện các công cụ dựa trên đó

Question

Tôi đang tạo biểu mẫu liên hệ và tôi cần trợ giúp với trình xác thực jQuery.

function contactform() { 
    $("form #submit").on("click", function() { 
     $("form input").removeClass("error"); 
     validator(); 
     // 3rd action goes here 
    }); 
}); 

validator() kiểm tra xem nếu có đầu vào được để trống, và nếu như vậy nó sẽ thêm một lớp lỗi với nó:

function validator() { 
    $("form input").each(function() { 
     var value = $(this).val(); 
     if (value.length <= 0) { 
      $(this).addClass("error"); 
      return false; 
     } 
    }); 
}); 

Bây giờ, đối với các hành động thứ 3 trong contactform() tôi muốn nói rằng nếu validator() = true (nghĩa là không có đầu vào nào trống), sau đó tiếp tục đến mã tiếp theo.

Tôi dường như không thể trả lại giá trị của validator(). Có ai biết cách tốt nhất để làm điều này?

Answer 1

Dưới đây là một giải pháp sử dụng filter phương pháp:

function validator() { 
    return $("form input").filter(function() { 
     return $.trim(this.value).length == 0; 
    }).addClass("error").length == 0; 
}); 

function contactform() { 
    ... 
    if (validator()) { 
     // it's OK 
    } else { 
     // there are errors 
    } 
} 

CẬP NHẬT: awesomely cập nhật với the help của @am_not_i_am. Cảm ơn!

Answer 2

function validator() { 
    var result = true; 
    $("form input").removeClass("error"); 
    $('form input').each(function() { 
      if(!$.trim(this.value)) { 
      $(this).addClass('.error'); 
      result = false; 
      } 
    }); 
    return result; 
} 


function contactform() { 
    $("form #submit").on("click", function() { 
     if(validator()) { // pass the validation 

     } else { // fail validation 

     } 
     // 3rd action goes here 
    }); 
}); 

Answer 3

Vấn đề bạn gặp phải là bạn có chức năng lồng nhau và đóng, ngăn bạn trực tiếp trả lại giá trị.

Something như thế nên làm các trick:

function validator() { 
    var result=true; 
    $("form input").each(function() { 
     var value = $(this).val(); 
     if (value.length <= 0) { 
      $(this).addClass("error"); 
      result = false; 
     } 
    }); 
    return result; 
});