Nested Jquery selector

Question

I want to write a selector which will select
1. Input elements of type radio OR checkbox
2. Which are selected ie. wich have is(':checked') as true
3. and have a some class applied to it.
I come up with following code

 $([':radio'],[':checkbox']).each(function() {
    if($(this).is(':checked'))
    {
        className = $(this).attr('class'); 
        if(className!= "")
        {   
            $('#'+className +' :input').attr('disabled', false);
        }
    }   
  });

Its workign fine but I am looking more efficient solution as I am selecting all radio and checkbox and filtering over it.
How can I combine all conditions ?

Thanks!

EDIT:
The radio or checkbox can have ANY class ie I want to select all radio or checkboxes which are selected and have atleast one class applied it.

jfriend00 · Accepted Answer · 2012-02-08 09:18:17Z

4

This selector will find any radio or checkbox control that is checked and has some class on it.

$(":radio:checked[class], :checkbox:checked[class]")

You can see it working here: http://jsfiddle.net/jfriend00/bYbnH/

If you need to avoid empty class names (where the attribute exists, but it set to an empty string, then you can use this:

$(":radio:checked[class], :checkbox:checked[class]").filter(function() {
    return(this.className && this.className.length != 0);
})

edited Feb 8, 2012 at 9:18

answered Feb 8, 2012 at 8:30

jfriend00

711k104 gold badges1.1k silver badges1k bronze badges

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

4 Comments

Yoshi Over a year ago

is the :checkbox/:radio actually needed? I guess :checked[class] would be enough, or is there an element that can be checked, I'm forgetting? And wouldn't it match elements with empty class attributes also?

jfriend00 Over a year ago

@Yoshi - it will match an empty class. Don't know if that's a concern of the OP or not as that doesn't occur in normal markup, though it could happen programmatically. It is possible to have a checked attribute on other types of input controls (not sure why it would be there, but it can be) which will match the selector :checked so this limits it to only radio and checkboxes types. I added a new option that filters out empty class names.

Ajinkya Over a year ago

@jfriend00: Thanks for updating on empty class, I didnt thought about it

yunzen Over a year ago

@jfriend00 it could be possible to have an empty class with just spaces. Would be better to use regexp in filter function: return ! this.className.match(/^\s*$/) . See my answer

Rafay · Accepted Answer · 2012-02-08 08:34:28Z

2

you can try

$(":radio:checked ,:checkbox:checked").hasClass("someclass")

after the question edit

$(":radio:checked[class] ,:checkbox:checked[class]")

edited Feb 8, 2012 at 8:34

answered Feb 8, 2012 at 8:15

Rafay

31.1k5 gold badges71 silver badges106 bronze badges

3 Comments

Ajinkya Over a year ago

Sorry for putting ambiguous question. I want to select all radio or checkboxes which are selected and have atleast one class applied it.

Barry Chapman Over a year ago

@Ajinkya - the response I provided should accomplish that for you.

Ajinkya Over a year ago

@aro: The input element can have ANY class applied to it. It think your answer is specific to class with name someClass.

Hadas · Accepted Answer · 2012-02-08 08:30:07Z

1

$(':radio:checked,:checkbox:checked').filter(function(){
     return $(this).attr('class').length>0;
});

answered Feb 8, 2012 at 8:30

Hadas

10.4k1 gold badge25 silver badges31 bronze badges

Comments

Barry Chapman · Accepted Answer · 2012-02-08 08:37:35Z

0

$(':radio:checked,:checkbox:checked').filter(function (index) {

    if ( typeof $(this).attr('class') != 'undefined' )
        return true;
    else
        return false;   

}).each(function() {

   alert('i am checked and i am checkbox or radio and i have a class!');

});

UPDATE:

You want to use $.filter():

http://jsfiddle.net/grGHa/1/

HTH

edited Feb 8, 2012 at 8:37

answered Feb 8, 2012 at 8:15

Barry Chapman

6,7904 gold badges40 silver badges71 bronze badges

Comments

yunzen · Accepted Answer · 2012-02-08 09:14:08Z

0

I'd go for

jQuery(":radio:checked ,:checkbox:checked")
  .filter(function() {
    // return true if className is not empty or just whitespace
    return ! this.className.match(/^\s*$/) 
  })

answered Feb 8, 2012 at 9:14

yunzen

33.5k13 gold badges79 silver badges114 bronze badges

Collectives™ on Stack Overflow

Nested Jquery selector

5 Answers 5

4 Comments

3 Comments

Comments

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

5 Answers 5

4 Comments

3 Comments

Comments

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related