jQuery checked radio button value

Question

<form id="form1" name="form1" method="post" action="">
  <input type="checkbox" id="check1" value="radio" />
  <label>left</label>
  <input type="checkbox" id="check2" value="radio2" />
  <label>right</label>
</form>

I would like to know if the checkbox is checked or not everytime someone clicks on it and then set a variable to the value yes.

if ($('#check1').attr('checked'); ){
var check1 = 'yes'; 
else 
var check1 ='no'
}

this is what I have so far and it doesn't work How do I return the value

Christofer Eliasson · Accepted Answer · 2012-03-14 13:13:48Z

7

You don't need an if-statement to do that. You can use .is(":checked") which will return true if checked and false if not. Assign that value directly to your variable instead of using an if-statement:

var check1;
$("#check1").click(function(){
   check1 = $(this).is(":checked");
});

answered Mar 14, 2012 at 13:13

Christofer Eliasson

34k7 gold badges77 silver badges103 bronze badges

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

Comments

KutePHP · Accepted Answer · 2012-03-14 13:14:13Z

2

Try this :

$("#check1").is(':checked')

answered Mar 14, 2012 at 13:14

KutePHP

2,2476 gold badges39 silver badges58 bronze badges

Comments

JohnColvin · Accepted Answer · 2012-03-14 13:21:24Z

1

Check out this jsfiddle. http://jsfiddle.net/ReF4X/ Let me know if what I'm doing doesn't make sense to you.

In the future, when you have a JS problem you should make a JSfiddle and post it here. People will then revise it with a working solution.

answered Mar 14, 2012 at 13:21

JohnColvin

2,5091 gold badge15 silver badges8 bronze badges

Comments

jbabey · Accepted Answer · 2012-03-14 13:23:11Z

1

in javascript, scope is only specific to a function. unlike most other languages, this means you should not define variables inside of if or else blocks.

you should be using prop instead of attr for the checked property (assuming the latest version of jquery)

lastly, you have an erroneous semicolon which is probably breaking the if statement.

try this instead:

var check1;
if ($('#check1').prop('checked')) {
    check1 = 'yes'; 
} else { 
    check1 ='no'
}

edited Mar 14, 2012 at 13:23

answered Mar 14, 2012 at 13:13

jbabey

46.7k12 gold badges73 silver badges94 bronze badges

1 Comment

jbabey Over a year ago

@Matt correct. javascript does NOT scope blocks like most other languages do, only functions.

Collectives™ on Stack Overflow

jQuery checked radio button value

4 Answers 4

Comments

Comments

Comments

1 Comment

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

4 Answers 4

Comments

Comments

Comments

1 Comment

Your Answer

Sign up or log in

Post as a guest

Related