Jquery checkbox selection

Question

I have checkbox with the same name ,I need to get the selected value of checkbox in a array.(It should exclude non selected value). I have following code.

$(document).ready(function() {
    $(".t").change(function() {
        var allVals = [];
        $("input[name='checkbox1']:checked").each(function() {
            // alert($(this).val());
            allVals.push($(this).val());
        });
        $('#click').click(function() {
            $.each(allVals, function() {
                alert(this); // It alerts old past value also
            });
        });
    });
});


<input name="checkbox1" class="t" type="checkbox" value="1"/>a <br/>
<input name="checkbox1" class="t" type="checkbox" value="2"/>b <br/>
<input name="checkbox1" class="t" type="checkbox" value="3"/>c <br/>
<input name="checkbox1" class="t" type="checkbox" value="4"/>d <br/>
<input id="click" id="button" type="button" value="click" />

Here is a JSFiddle with my code -> http://jsfiddle.net/manseuk/VHSrZ/

Can anyone suggest how to solve it ?

What error value? We can't all read minds, even if you can. — Bojangles
– Bojangles, Commented Nov 25, 2011 at 14:29
I should get value of checkbox current selection. In my case It get printed all previous non selection also — venkat
– venkat, Commented Nov 25, 2011 at 14:31

Nicola Peluchetti · Accepted Answer · 2011-11-25 14:30:14Z

2

You could do:

$(document).ready(function() {
    var allVals;
    $(".t").change(function() {
        allVals = [];
        $("input[name='checkbox1']:checked").each(
        function() {
            allVals.push($(this).val());
        });

    });
});

In this way allVals is an array that contains only checked values

Fiddle here http://jsfiddle.net/K6dTF/ (use firebug to check the results)

answered Nov 25, 2011 at 14:30

Nicola Peluchetti

77k32 gold badges150 silver badges197 bronze badges

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

Comments

Niels · Accepted Answer · 2011-11-25 14:31:15Z

1

You are binding a new click function each time you click a checkbox. You need to put this out of this scope, or unbind the function and rebind.

Something like:

$(document).ready(function() {
    var allVals = [];
    $(".t").change(function() {
        allVals = [];
        $("input[name='checkbox1']:checked").each(function() {
            // alert($(this).val());
            allVals.push($(this).val());
        });
    });
    $('#click').click(function() {
        $.each(allVals, function() {
            alert(this); // It alerts old past value also
        });
    });
});

See fiddle: http://jsfiddle.net/VHSrZ/3/

answered Nov 25, 2011 at 14:31

Niels

50.1k4 gold badges68 silver badges86 bronze badges

Comments

Manse · Accepted Answer · 2011-11-25 14:33:11Z

1

Change your JS to the following :

It will alert an array of selected values

$('#click').click(function() {
    var allVals = [];
    $("input[name='checkbox1']:checked").each(function() {
        // alert($(this).val());
        allVals.push($(this).val());
    });
    alert(allVals);
});

Working example : http://jsfiddle.net/manseuk/VHSrZ/2/

answered Nov 25, 2011 at 14:33

Manse

38.1k11 gold badges88 silver badges112 bronze badges

Collectives™ on Stack Overflow

Jquery checkbox selection

3 Answers 3

Comments

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

Comments

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related