php foreach calculating sum from row

Question

I have search and implements from many question in stackoverflow to my code but it always returning only the last row value. Here my code :

$totalpayment = 0;
foreach($respon2 as $key => $row) {
    $name = $row['title'];
    $quantity = $row['quantity'];
    $price =  $row['price']*$row['quantity'];
    $totalpayment += $price;
    $price  = '$ ' . number_format($price) ;
    $totalpayment  = '$ ' . number_format($totalpayment) ;
    $mytable .= '<tr width="100%">';
    $mytable .= '<td  width="55%" align="left">' . $name . "</td>";
    $mytable .= '<td  width="20%" align="right">' . $quantity . "</td>";
    $mytable .= '<td  width="25%" align="right">' . $price . "</td>";
    $mytable .= '</tr>';
} 
$mytable .= '</table>';
$mytable .= '<table border="0" cellpadding="10" cellspacing="0" width="100%%" style="padding: 0px 25px 0px 25px;">';
$mytable .= '<tr><td width="45%"></td><td width="30%" class="table-title" align="right">Total Payment:</td><td width="25%" align="right">'. $totalpayment ."</td></tr>";
$mytable .= '</table>';

How to fix this ?

Simply typo, you are overwriting your variable.

Script47
– Script47

2018-06-08 04:14:13 +00:00
Commented Jun 8, 2018 at 4:14 — Script47
– Script47, Commented Jun 8, 2018 at 4:14

Eddie · Accepted Answer · 2018-06-08 04:12:07Z

2

You are overding $totalpayment on every loop by this code:

$totalpayment = '$ ' . number_format($totalpayment) ;

$totalpayment = 0;
foreach($respon2 as $key => $row) {
    $name = $row['title'];
    $quantity = $row['quantity'];
    $price =  $row['price']*$row['quantity'];
    $totalpayment += $price;
    $price  = '$ ' . number_format($price) ;
    $mytable .= '<tr width="100%">';
    $mytable .= '<td  width="55%" align="left">' . $name . "</td>";
    $mytable .= '<td  width="20%" align="right">' . $quantity . "</td>";
    $mytable .= '<td  width="25%" align="right">' . $price . "</td>";
    $mytable .= '</tr>';
} 

$totalpayment  = '$ ' . number_format($totalpayment); //Try transferring this code outside the loop

$mytable .= '</table>';
$mytable .= '<table border="0" cellpadding="10" cellspacing="0" width="100%%" style="padding: 0px 25px 0px 25px;">';
$mytable .= '<tr><td width="45%"></td><td width="30%" class="table-title" align="right">Total Payment:</td><td width="25%" align="right">'. $totalpayment ."</td></tr>";
$mytable .= '</table>';

answered Jun 8, 2018 at 4:12

Eddie

26.8k6 gold badges39 silver badges59 bronze badges

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

2 Comments

BИ Develop Over a year ago

Thanks, it's my fault

Eddie Over a year ago

Happy to help :)

Sandeep Singh · Accepted Answer · 2018-06-08 06:03:43Z

0

It is not so efficient to do a sum in loop, better change your code as follows and to be done out side the loop as mentioned in question.

$prices = sum(array_coulmn($respon2, 'price')); $quantity = sum(array_coulmn($respon2, 'quantity')); $totalPayment = $prices*$quantity; $totalPayment = '$ ' . number_format($totalPayment);

answered Jun 8, 2018 at 6:03

Sandeep Singh

414 bronze badges

1 Comment

BИ Develop Over a year ago

Did you mean array_sum(array_column($respon2, 'price ')); ? But your code giving the wrong calculating of total payment

F5 Buddy · Accepted Answer · 2018-06-08 07:34:47Z

0

        $totalpayment = 0;
        $mytable ='<table>';
        $respon2=array(array('title' =>'p1','quantity' =>2,'price' =>10),array('title' =>'p2','quantity' =>2,'price' =>20),array('title' =>'p3','quantity' =>1,'price' =>10));
        foreach($respon2 as $key => $row) {
        $name = $row['title'];
        $quantity = $row['quantity'];
        $price =  $row['price']*$row['quantity'];
        $totalpayment += $price;
        $price  = '$ ' . number_format($price) ;    
        $mytable .= '<tr width="100%">';
        $mytable .= '<td  width="55%" align="left">' . $name . "</td>";
        $mytable .= '<td  width="20%" align="right">' . $quantity . "</td>";
        $mytable .= '<td  width="25%" align="right">' . $price . "</td>";
        $mytable .= '</tr>';
        } 
        $totalpayment  = '$ ' . number_format($totalpayment) ;
        $mytable .= '</table>';
        $mytable .= '<table border="0" cellpadding="10" cellspacing="0" width="100%%" style="padding: 0px 25px 0px 25px;">';
        $mytable .= '<tr><td width="45%"></td><td width="30%" class="table-title" align="right">Total Payment:</td><td width="25%" align="right">'. $totalpayment ."</td></tr>";
        $mytable .= '</table>';

        echo $mytable;

answered Jun 8, 2018 at 7:34

F5 Buddy

4944 silver badges4 bronze badges

1 Comment

BИ Develop Over a year ago

it was already answered bro, i just write some part of my code. the purpose of my full code is for print the html to pdf file.

Collectives™ on Stack Overflow

php foreach calculating sum from row

3 Answers 3

2 Comments

1 Comment

1 Comment

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

2 Comments

1 Comment

1 Comment

Your Answer

Sign up or log in

Post as a guest

Related