jQuery if statement syntax check

Question

Whats wrong with this statement.

if (data.bookingtype = 'Corporate') {
    jQuery('#rentalprice').val(data.baserentalprice);

} else if (data.bookingtype = 'Groups') {
    jQuery('#priceperperson').val(data.baserentalprice);

} else if (data.bookingtype = 'Leisure') {
    jQuery('#priceperperson').val(data.baserentalprice);

}

I am trying to populate a field based on the value of the bookingtype variable sent back from my ajax request.

If bookingtype is Corporate then populate the #rentalprice input, else if its Groups or Leisure then populate the #priceperperson input. At the moment it always populate the #rentalprice input.

== or === (equality operator) instead of = (assignment operator) — Satpal
– Satpal, Commented Jul 24, 2015 at 11:14
Duplicate of stackoverflow.com/questions/16250235/… and what not — Salman Arshad
– Salman Arshad, Commented Jul 24, 2015 at 11:20

Tushar · Accepted Answer · 2015-09-01 04:21:56Z

3

You are assigning value to the variable instead of comparing. So, the first if is always evaluated to true and the statement of if block will be executed.

Use equality operator(==) or strict equality operator(===) instead of assignment operator(=).

if (data.bookingtype == 'Corporate') {
    jQuery('#rentalprice').val(data.baserentalprice);
} else if (data.bookingtype == 'Groups') {
    jQuery('#priceperperson').val(data.baserentalprice);
} else if (data.bookingtype == 'Leisure') {
    jQuery('#priceperperson').val(data.baserentalprice);
}

edited Sep 1, 2015 at 4:21

answered Jul 24, 2015 at 11:13

Tushar

87.4k21 gold badges164 silver badges182 bronze badges

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

1 Comment

Tushar Over a year ago

Why Downvote? Please add comments here. Can something be improved?

Rajan Goswami · Accepted Answer · 2015-07-24 11:24:38Z

1

You are using an assignment operator = instead of equality operator == or ===. The code should be as follows:

JQUERY

if (data.bookingtype == 'Corporate') {
    jQuery('#rentalprice').val(data.baserentalprice);
} else if (data.bookingtype == 'Groups') {
    jQuery('#priceperperson').val(data.baserentalprice);
} else if (data.bookingtype == 'Leisure') {
    jQuery('#priceperperson').val(data.baserentalprice);
}

answered Jul 24, 2015 at 11:24

Rajan Goswami

7691 gold badge5 silver badges17 bronze badges

Collectives™ on Stack Overflow

jQuery if statement syntax check

2 Answers 2

1 Comment

Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

1 Comment

Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related