как поместить следующее в цикл for

Question

Я хотел бы поместить следующее в цикл for, но у меня возникли трудности. Любая помощь будет оценена

 $("input:submit").click(function(){  
         if (!$("input[name=attendance1]").is(":checked")) {
            alert('Please select preference');
            return false;
            }
       else if (!$("input[name=attendance2]").is(":checked")) {
            alert('Please select preference');
            return false;
                }
        else if (!$("input[name=attendance3]").is(":checked")) {
            alert('Please select preference');
            return false;
        }

        });


});

Я пробовал:

for($i=1; $i<=3; $i++)
                 {
$("input:submit").click(function(){  
             if (!$("#food" + $i).is(":checked")) {
                alert('Please select preference');
                return false;
                }

            });         

    });

Answer 1

Первое исправление:

alert('Please select preference);

с

alert('Please select preference');

Затем, если вы хотите выполнить цикл:

for (var i = 0; i < 3; i++) {
    if (!$("input[name=attendance" + i + "]").is(":checked")) {
            alert('Please select preference');
            return false;
    }
}

Или, что еще лучше, используйте jQuery setsWith селектор:

if (!$('input[name^="attendance"]').is(":checked")) {
            alert('Please select preference');
            return false;
}

Пример

Answer 2

Обычно я использую имя класса в своих элементах DOM, когда я хочу сделать что-то подобное.Это облегчает итерацию по элементам с помощью .each ().Я не знал о селекторе стартов с указанным выше, но он выглядит немного чище, чем мой метод.

<!-- DO STUFF -->

    <input name="attendance1" class="my-unique-class-name" />
    <input name="attendance2" class="my-unique-class-name" />
    <input name="attendance3" class="my-unique-class-name" />

<!-- DO STUFF -->

<script type="text/javascript">
    $("input:submit").click(function(){
        var valid = true;
        $("input.my-unique-class-name").each(function (el) {
            if ( ! $(el).is(":checked") ) {
                valid = false;
            }
        });
        if ( ! valid ) {
            alert('Please select preference');
            return false;
        }
    });
</script>

Answer 3

Вот мое мнение.он имеет преимущество в том, что вы перечисляете вниз в массиве, какие имена вы хотите проверить, независимо от того, какие имена они являются

//jQuery '.on()' for versions 1.7+
$("input:submit").on('click', function() {

    //assume valid unless found otherwise
    var valid = true;

    //add the input names you want to verify
    var nameList = [
        'attendance1',
        'attendance2',
        'attendance3'
    ];

    //loop through names
    for (var i = 0; i < nameList.length; i++) {
        var checked = $('input[name="'+nameList[i]+'"]').is(':checked');
        if (!checked) {
            alert('Please select a preference');

            //mark false when something wrong found
            valid = false;
        }
    }

    //check if validity persisted
    if(valid){
        //do something
    }

    //prevent default actions
    return false;
});​

Answer 4

Вы пропускаете заключительную одинарную кавычку во всех трех операторах оповещения.