Jquery using map to sum array

Question

My values array is returning correctly in the form of [100,200,300,500] I just wish to sum up these values, what am I doing wrong?

Jquery

$('select').focus(function () {
    previous = parseInt($(this).val());
}).change(function() {

    var item_cost = parseInt($(this).attr('cost'));
    
    values = $.map($('select[name="cookies"]'), function (e) {
            
        return $(e).val()* item_cost;
                     
        for (var i = 0; i < values.length; i++) {
            total += parseInt(values[i]);
            console.log(total);                              
        }
    });

    alert(values);

Dunno, what are you seeing in the console? Please fix your code, it's hard to read as-is. — Madbreaks
– Madbreaks, Commented Nov 21, 2012 at 17:41
You have a return statement in the first line of your map callback function, before the for loop — Madbreaks
– Madbreaks, Commented Nov 21, 2012 at 17:43
return statement must be in the end of function if you ofcourse don`t want to exit function — Epsil0neR
– Epsil0neR, Commented Nov 21, 2012 at 17:46

Joe · Accepted Answer · 2012-11-21 17:44:48Z

2

Your code needs formatting. You missed a closing }); You need to declare var total = 0; as well. I'm not sure if that is elsewhere in your code.

$('select').focus(function() {
    previous = parseInt($(this).val());
}).change(function() {
    var item_cost = parseInt($(this).attr('cost'));

    values = $.map($('select[name="cookies"]'), function(e) {
        return $(e).val() * item_cost;
    }); // <-- RIGHT THERE

    for (var i = 0; i < values.length; i++) {
        total += parseInt(values[i]);
        console.log(total);
    }
});
alert(values);

answered Nov 21, 2012 at 17:44

Joe

82.8k18 gold badges130 silver badges147 bronze badges

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

Collectives™ on Stack Overflow

Jquery using map to sum array

Jquery

1 Answer 1

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

Jquery

1 Answer 1

Comments

Your Answer

Sign up or log in

Post as a guest

Related