Trouble with arithmetic operators in AngularJS

Question

I have following code:

$scope.showTotal = function() {
    $scope.pT = [];
    var iT = 0;
    for (var i = 0; i < $scope.od.length; i++) {
        console.log($scope.od[i]['bpr']);
        iT += $scope.od[i]['bpr'];
        // also tried this -> iT = iT + $scope.od[i]['bpr']
    }
    $scope.pT.push({iTotal: iT});
    console.log($scope.popupTotals);
    $scope.showPopupNow = true;
}

But I don't know why it's not working. If the bpr is for example 50 and 43.1034, then it logs the output in console something like this, iTotal:"050.000043.1034"

I am new to JavaScript and I started it directly with AngularJS. So please help me with arithmetic operators in JS. Thank You.

What type do you have in the object $scope.order[i]['baseprice'] ? If it string, you must to parseFloat. — Alexander Anikeev
– Alexander Anikeev, Commented Aug 8, 2017 at 6:38
You are welcome! You have many correct answers at bottom. Check one of its! — Alexander Anikeev
– Alexander Anikeev, Commented Aug 8, 2017 at 6:43

Aks1357 · Accepted Answer · 2017-08-08 06:48:27Z

2

$scope.showTotal = function() {
    $scope.popupTotals = [];
    var itemtotal = 0;
    for (var i = 0; i < $scope.order.length; i++) {
        console.log($scope.order[i]['baseprice']);
        itemtotal += parseFloat($scope.order[i]['baseprice']);
        // parseFloat will convert string to number and add the number instead of concatenating the strings
    }
    $scope.popupTotals.push({itembasetotal : itemtotal});
    console.log($scope.popupTotals);
    $scope.showPopupNow = true;
}

edited Aug 8, 2017 at 6:48

answered Aug 8, 2017 at 6:37

Aks1357

1,0721 gold badge9 silver badges19 bronze badges

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

1 Comment

Aks1357 Over a year ago

@RahulJain - Welcome!

Vivz · Accepted Answer · 2017-08-08 06:46:28Z

1

You are incrementing i inside the loop .Remove the duplicate i and I suspect that your $scope.order[i]['baseprice'] is not an integer. So convert it to an integer using parseFloat

$scope.showTotal = function(){
        $scope.popupTotals = [];
        var itemtotal = 0;
        for (var i = 0; i<$scope.order.length; i++){
            console.log($scope.order[i]['baseprice']);
            itemtotal += parseFloat($scope.order[i]['baseprice']);
            //also tried this -> itemtotal = itemtotal + $scope.order[i]['baseprice']
            //i++; No need to increment here
        }
        $scope.popupTotals.push({itembasetotal : itemtotal});
        console.log($scope.popupTotals);
        $scope.showPopupNow = true;
    }

edited Aug 8, 2017 at 6:46

answered Aug 8, 2017 at 6:34

Vivz

6,6282 gold badges19 silver badges34 bronze badges

2 Comments

TheCleverIdiot Over a year ago

Still the same result.

Vivz Over a year ago

Is your $scope.order[i]['baseprice'] a string?. Try updated answer using parseFloat

Collectives™ on Stack Overflow

Trouble with arithmetic operators in AngularJS

2 Answers 2

1 Comment

2 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

1 Comment

2 Comments

Your Answer

Sign up or log in

Post as a guest

Related