jquery change text once on change function

Question

The problem is that when i change the option it keeps summing, and i would like for it to only sum one time the value of 10.00, and subtract only once if the value is AR .

LIVE LINK: http://jsfiddle.net/A3qLG/1/

JQUERY:

$(document).ready( function() {

    $('#country').on('keyup change', function(e){

        var subt_value = $('#subt0').text();

        if($(this).val() == 'AR' || $(this).val() == ''){           
        if(subt_value != '0.00' ){$('#subt0').text(parseFloat(subt_value-10).toFixed(2));}      
        }else{          
        var add_sub = parseFloat(subt_value)+parseFloat('10.00');
        $('#subt0').text(parseFloat(add_sub).toFixed(2));
        }

    });

});

HTML:

<label id="subt0">8.90</label>

<select id="country">
  <option value="US">USA</option>
  <option value="BR">Brasil</option>
  <option value="AR">Argentina</option>
  <option value="CU">Cuba</option>
</select>

score 1 · Accepted Answer · 2014-06-12 00:11:15Z

1

I would change the HTML to this to store the original value:

<label id="subt0" data-original="8.90">8.90</label>

And then adjust the Javascript to this:

$(document).ready( function() {

    $('#country').on('keyup change', function(e){

        //Changed the following line to get the original value      
        var subt_value = $('#subt0').attr('data-original');

        //Changed this to say, NOT AR and has a choice per the comments.            
        if($(this).val() != 'AR' && $(this).val().length > 0) {
          var add_sub = parseFloat(subt_value)+parseFloat('10.00');
          $('#subt0').text(parseFloat(add_sub).toFixed(2));
        } else {
          //Otherwise put it back to the original value
          $('#subt0').text(subt_value);
        }

    });

});

edited Jun 12, 2014 at 0:11

answered Jun 11, 2014 at 23:59

user1932079

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

5 Comments

Ered Over a year ago

here is ur code online jsfiddle.net/A3qLG/4 but there is a problem when i go back to the Argentina option i get -1.10

user1932079 Over a year ago

That's because 8.9 - 10 = -1.10. You wanted it to subtract once, right? If not, then you'll need to give some details about how your code should display results.

Ered Over a year ago

i made it subtract because country Argentina does not have a recharge all other country's must add $10 from the original price...so lets say the total order is $8.90 if i select any other country besides Argentina it should add $10 to the total... any thoughts of what can i do to get that?

user1932079 Over a year ago

Yep, just convert what you typed into code. I have edited that section in my answer.

user1932079 Over a year ago

Just made a correction by adding an else clause to restore the value in case it was changed.

asdf · Accepted Answer · 2014-06-12 00:06:26Z

1

try this code

$(document).ready( function() {

    $('#country').on('keypress change', function(e){

        var subt_value = $('#subt0').text();

        if($(this).val() == 'AR' || $(this).val() == ''){           
        if(subt_value != '0.00' ){$('#subt0').text(parseFloat(subt_value-10).toFixed(2));}      
        }else{          
        var add_sub = parseFloat(subt_value)+parseFloat('10.00');
        $('#subt0').text(parseFloat(add_sub).toFixed(2));
        }

    });

});

answered Jun 12, 2014 at 0:06

asdf

4591 gold badge11 silver badges31 bronze badges

1 Comment

user1932079 Over a year ago

When submitting an answer, code by itself may solve the problem, but it doesn't give an answer. Please explain your changes.

Bogdan Savluk · Accepted Answer · 2014-06-12 00:07:21Z

1

You need to remember previous #country select value, and use it when making decision:

$(document).ready(function () {
    var prevValue = '';
    $('#country').on('keyup change', function (e) {
        var newValue = $(this).val();
        var subt_value = parseFloat($('#subt0').text());

        if ((newValue == 'AR' || newValue == '')) {
            if (prevValue != '' && prevValue != 'AR') {
                $('#subt0').text((subt_value - 10).toFixed(2));
            }
        } else {
            if (prevValue == 'AR' || prevValue == '') {
                $('#subt0').text((subt_value + 10).toFixed(2));
            }
        }
        prevValue = newValue;
    });

});

http://jsfiddle.net/A3qLG/5/

answered Jun 12, 2014 at 0:07

Bogdan Savluk

6,3121 gold badge32 silver badges37 bronze badges

Comments

Sesertin · Accepted Answer · 2014-06-11 23:53:43Z

0

Simply remove the onkeyup and onchange event handler functions after they have done their work.

$('#country').off('keyup change');

answered Jun 11, 2014 at 23:53

Sesertin

4622 silver badges12 bronze badges

2 Comments

user1932079 Over a year ago

This was my first impression, too, but from the post saying to subtract only once... or add only once... I take it to be asking "each time the value is changed".

Sesertin Over a year ago

than instead of using an anonymous function in the handler, declare it as a named function.Wait a sec, I'll edit my post

jhyap · Accepted Answer · 2014-06-12 00:27:01Z

0

Jquery provide a .change() specifically for <select>

You want to sum only one time and substract only one time base on the condition of the selected value AR. Therefore you only need to check the selected value if there is change and do the math within the if-else statement. Finally attach the result with .toFixed(2)

var subt_value = $('#subt0').text(); 
var add_sub = 0, f=10.00;
$('#country').change(function(){
    if ($(this).val() != 'AR'){
       add_sub = parseFloat(subt_value)+f;
    }else{
        add_sub = parseFloat(subt_value)-f;
    }

    $('#subt0').text(add_sub.toFixed(2));

});

JSFiddle Demo

answered Jun 12, 2014 at 0:27

jhyap

3,8496 gold badges30 silver badges49 bronze badges

Collectives™ on Stack Overflow

jquery change text once on change function

5 Answers 5

5 Comments

1 Comment

Comments

2 Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

5 Answers 5

5 Comments

1 Comment

Comments

2 Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related