jQuery: checkbox

Question

what's wrong with my code, I don't get any values:

<script>
    $(document).ready(function()
    {
        $("input[type=checkbox][checked]").each(function(event){
            var get = $("input[@name=\'checkbox_pref\']:checked").val();
            $("#result").html("&id=" + get);
        });
    });
</script>
</head>                                                                 
<body>                                                                
<input type="checkbox" name="checkbox_pref" value = "1"/>
<input type="checkbox" name="checkbox_pref" value = "2"/>   
<input type="checkbox" name="checkbox_pref" value = "3"/>   
<div id="result">result ...</div>

the way I see it, you're doing that function for the checkboxes that are checked on loading. are any of them automatically checked or something? — Eton B.
– Eton B., Commented Aug 10, 2010 at 15:03

Tim Rogers · Accepted Answer · 2010-08-10 15:08:44Z

6

Depends what you're trying to do, but it should be more along the lines of:

$(document).ready(function()
{
    $("input[type=checkbox]").change(function(event){
        $("#result").html("&id=" + this.value);
    });
});

answered Aug 10, 2010 at 15:08

Tim Rogers

21.8k8 gold badges55 silver badges69 bronze badges

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

2 Comments

David Yell Over a year ago

I agree, although I would do $('input:checkbox')

user319854 Over a year ago

Thanks, but how get all selected values, no't just last?

Matthew Jones · Accepted Answer · 2010-08-10 15:03:42Z

2

You don't need to escape the single quotes or the @ symbol. Use this line:

$(document).ready(function()
{
    $("input[type=checkbox][checked]").each(function(event){
        var get = $("input[name='checkbox_pref']:checked").val();
        $("#result").html("&id=" + get);
    });
});

answered Aug 10, 2010 at 15:03

Matthew Jones

26.3k17 gold badges105 silver badges156 bronze badges

2 Comments

user113716 Over a year ago

There's nothing wrong with escaping the single quotes. EDIT: I see you updated your answer to remove the @ symbol.

Hristo Over a year ago

.. I just checked your solution on jsFiddle.net, and it doesn't seem to be working. jsfiddle.net/cxhVV

David Yell · Accepted Answer · 2010-08-10 15:34:49Z

0

Having just been checking a very similar type of functionality myself, here is my test. Just as a footnote FYI ;)

http://jsfiddle.net/7Tdgw/

answered Aug 10, 2010 at 15:34

David Yell

11.9k13 gold badges64 silver badges102 bronze badges

Collectives™ on Stack Overflow

jQuery: checkbox

3 Answers 3

2 Comments

2 Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

2 Comments

2 Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related