jQuery foreach array object

Question

I have made an array of element ID's for radio buttons like so:

var buttons = [$('#first'), $('#second'), $('#third'), $('#fourth')];

What I am now trying to do is, check to see if each one is checked and if so, add a class to something, else add a class to something else. I know how do this separately for each individual id:

if($('#first').is(':checked')) { 
    $('.cb-enable').addClass('selected');
}
else {
    $('.cb-disable').addClass('selected');
}

I tried using if($('buttons').is(':checked')) { but it doesn't work.

Can someone please helps me on this one?

That's an array of jquery objects, not an array of element id's. also, what does $('buttons') have to do with your buttons array? None of your code ever uses the buttons array. — user400654
– user400654, Commented Aug 20, 2013 at 19:08
buttons.each(function(index, value){value.is(':checked')}); — abc123
– abc123, Commented Aug 20, 2013 at 19:12
Are there no satisfactory answers to your question here? You have not accepted or voted them. If anything is unclear, I will be glad to further assist you. — m59
– m59, Commented Sep 7, 2013 at 16:17

m59 · Accepted Answer · 2013-08-20 19:30:39Z

1

Here is a working demo: http://jsbin.com/UpUk/1/edit

  <button class="check">Tell me what is checked</button>
  <input type="radio" class="checkable">
  <input type="radio" class="checkable">
  <input type="radio" class="checkable">
  <input type="radio" class="checkable">
  <input type="radio" class="checkable">

var $radios = $('.checkable');

$('.check').click(function() {
  $radios.each(function(index, item) {
    if ($(item).is(':checked')) {
      alert('item: '+index+' is checked!');
    }
  });
});

It is generally better to use classes, and you don't need to deal with each item individually to accomplish what you are trying to do.

Here's a better version that deals with switching classes. It may be more than what you're looking for, but I don't know quite what you're using this for.

http://jsbin.com/UpUk/3/edit

That should be plenty to help you understand how to approach your project.

edited Aug 20, 2013 at 19:30

answered Aug 20, 2013 at 19:15

m59

43.9k14 gold badges122 silver badges139 bronze badges

Sign up to request clarification or add additional context in comments.

Comments

Robert McKee · Accepted Answer · 2013-08-20 19:13:00Z

1

Try:

var buttons = $('#first:checked,#second:checked,#third:checked,#fourth:checked');
if (buttons.length>0)
{
...
} else {
...
}

answered Aug 20, 2013 at 19:13

Robert McKee

21.5k1 gold badge48 silver badges64 bronze badges

Comments

tormex · Accepted Answer · 2013-08-20 19:13:07Z

1

You should use

$(elem).prop("checked")

to get true or false value if the radio button is checked or is not checked.

answered Aug 20, 2013 at 19:13

tormex

12710 bronze badges

Comments

Steve Snow · Accepted Answer · 2013-08-20 19:15:17Z

1

Maybe you can try something like this:

        $('input[type=radio]').each(function (index) {
            if ($(this).is(':checked')) 
               $('.cb-disable').addClass('selected');
            else 
               $('.cb-enable').addClass('selected');
        }

You may not need the index but it will be incremental in order it is found in the dom like what you attempted depending on the details of what your trying to do.

answered Aug 20, 2013 at 19:15

Steve Snow

612 silver badges5 bronze badges

Comments

Roko C. Buljan · Accepted Answer · 2013-08-20 19:17:06Z

1

var buttons = [$('#first'), $('#second'), $('#third'), $('#fourth')];

$.each(buttons, function(i,el){
  alert( el[0].checked );
});

DEMO

answered Aug 20, 2013 at 19:17

Roko C. Buljan

209k41 gold badges335 silver badges346 bronze badges

Collectives™ on Stack Overflow

jQuery foreach array object

5 Answers 5

Comments

Comments

Comments

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

5 Answers 5

Comments

Comments

Comments

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related