Grand Total Function Returns the Wrong Grand Total

Maureen Moore

This function returns the wrong grand total. When I add items to the cart, it returns a much higher grand total than is actually there. How can I fix it?

You can see the app in action at cart-64d3f.firebaseapp.com

In the component:

totals = [];
get grandTotal() {
let i;
let sub_total = 0;
let grand_total = 0;
if (this.isSubmitted == true) {
if (typeof this.product_price !== "undefined" && typeof this.quantity !== "undefined") {
sub_total = this.product_price * this.quantity;
this.totals.push(sub_total);
}
}
for (i = 0; i < this.totals.length; i++) {
grand_total += this.totals[i];
}
return grand_total;
}

In the HTML:

<td>{{grandTotal | currency:'USD':true }}td>

9 Replies

Know the answer? Post it — somebody with the same question will find it here.

Maureen Moore6y ago
With the help of Scott Logsdon, I was able to fix my code. You can see the solution at https://angular-ivy-at2fdp.stackblitz.io/
0
Maureen Moore6y ago
Thanks, Laxmidhar Sahoo, what should yourfield be set to?
0
Laxmidhar Sahoo6y ago
Hi
The bellow field not assign in the function deleteItem
this.grandTotal = item.yourfield;
Check the bellow if cluse .I think not return true
if (typeof this.product_price !== "undefined" && typeof this.quantity !== "undefined") {
}
Thanks and regards
0
Maureen Moore6y ago
The grand totals are updating correctly except for when I delete an item. When I delete an item, the grand total doesn't decrease. So I just need a way to update the grand totals when the item is deleted.
0
Maureen Moore6y ago
I'm trying to force the grand total to re-evaluate after the item is deleted but what I have here isn't working:
deleteItem(i){
this.items.splice(i,1);
this.setStorageItems(this.items);
this.isSubmitted = true;
this.grandTotal;
}
After the item is deleted, it is the same grand total as before.
<tr *ngFor="let item of items; let i = index">
<td>
<button type="button" (click)="deleteItem(i)" class="deletebtn">Xbutton>td>
tr>
0
Maureen Moore6y ago
When I console log this.totals, it shows totals that aren't in localStorage any more.
0
Maureen Moore6y ago
If I initialize totals as an empty array like this var totals = []; inside of the function, it resets the totals to an empty array every time the function is called and the grand total is just the quantity times price subtotal every time the function is called.
0
Varun Setia6y ago
Make sure totals = []; is called inside function so calculation does not do dirty read
0
Guest User6y ago
Check this.totals, probably contains too many items.
+1