Как написать скрипт $.each()?

Question

[leni_m]

есть button onclick="checkForm();", где checkForm() должна возвращать "return false", если поля не заполнены
и поля с классом .checkText, которые должны быть заполнены.
написал

function checkForm() {
    $('.checkText').each(function(i,elem) {
        if ($(this).val().length == 0) {
            return false;
        }
    });
}

но не работает

Answer 1

[Stalker_RED]

У вас же return только внутри вложенной функции, а сама checkForm() ничего не возвращает.

function checkForm() {
    var isValid = true;
    $('.checkText').each(function(i,elem) {
        if ($(this).val().length == 0) {
            isValid = false
        }
    });
    return isValid;
}

Еще можно воспользоваться встроенной в html5 валидацией. Атрибут required, и всё такое.

Comments

[leni_m]

не помогло, да я и пробовал onclick="return checkForm();"

[leni_m]

а валидация не подходит, т.к. я ajax-ом отправляю данные

[leni_m]

а вот return checkForm(); сейчас помогло, спасибо! понял ошибку

[leni_m]

хотя из за return не работают следующие функции в onclick="...", не подскажете как решить?

[leni_m]

может функция может вернуть return false, а в случае если все ок, ничего, и все последующие функции будут выплнены

[Stalker_RED]

leni_m: Перестать мучаться с onclick и написать нормальный обработчик, например.

$('#myForm').on('submit', function(){
  checkForm();
  esheChtoTo();
})

[Владимир]

leni_m: Валидация (точнее использование атрибута required) производится браузером и со способом отправки данных формы никак не связана.

[leni_m]

Stalker_RED: так тоже самое ведь, из checkForm() можно вернуть false или true
а запись

$('3myForm').on('submit', function(){
 return checkForm();
  esheChtoTo();
})

не даст выполнения esheChtoTo();

[Stalker_RED]

leni_m: Ну конечно, строки после return выполняться не будут. Вернитесь к началу учебника, там где про if и другие полезные штуки.

[leni_m]

Владимир: я ставил required="required" и не сработало. у меня скрипт

var params = {
1: $(requiredField).val(),
}
    $.post("...", params);

и он прекрасно передает пустые значения

[Stalker_RED]

leni_m: или просто переставьте свое "что-то" повыше.

[leni_m]

Stalker_RED: onclick="if (checkForm()){ esheChtoTo();} else {return false;}" а вот так и в onclicke работает)

[Stalker_RED]

leni_m: а с чего-бы ему не передавать, если вы явным образом прописываете поля?
Кстати, не думали ли вы о использовании $(form).serialize()?

[Stalker_RED]

leni_m: Ура! Даешь больше ада в шаблонах \0/

[leni_m]

Stalker_RED: так я и делаю через .serialize() , просто для примера так привел

Answer 2

[Александр Владимирович]

function checkForm() {
    $('.checkText').each(function(i,elem) {
        if ($(elem).val().length == 0) {
            return false;
        }
    });
}

Comments

[leni_m]

не помогло