Putting values into an array and using array_sum

Question

ok im a little stuck, i know its a simple thing im missing here so hoping fresh eyes will help

I have values in a column stored as 2:7:99 etc each value is seperated by :

Now I can seperate all the values out and query another table to get the price which corresponds to that value.

The issue i'm having is doing a SUM of all the price values See code below I think the easiest way would be to add all the price values into an array and then do array_sum() but for some reason i just cant get it working

** Please DO NOT Mention SQL Injection .. Its on a LOCAL machine with NO outside access and only myself will be using this

    <?php
include('config.php');
// Function for calculation Rough Invoice Total
function basicTotal() {
    $con = mysqli_connect("localhost","USER","PASS","TABLE");
    $wtbdq = mysqli_query($con,"SELECT * FROM `jobs` WHERE wsjid = '18'");
    $wtbdr = mysqli_fetch_assoc($wtbdq);
    do {
    $wtbd = explode(":",$wtbdr['worktobedone']);
    foreach($wtbd as $item) 
        {
            $priceq = mysqli_query($con,"SELECT * FROM `workshop-items` WHERE wsiid = '$item'");
            $pricer = mysqli_fetch_assoc($priceq);

            $price = array($pricer['incvat']);

            echo $item.' - '. $pricer['incvat'].'<br>';

        }

    } while($wtbdr = mysqli_fetch_assoc($wtbdq));

    $total = array_sum($price);
    echo $total;
}
basicTotal();
?>

Thanks in advance

How about $total = 0; foreach(...) {... $total += $pricer['incvat']; ...}? — Ruslan Osmanov
– Ruslan Osmanov, Commented May 17, 2017 at 12:46
Why not use SELECT *, SUM(incvat) AS total FROM workshop-items WHERE wsiid = '$item' ? — Gerard
– Gerard, Commented May 17, 2017 at 12:56

Nithin John · Accepted Answer · 2017-05-17 12:46:08Z

4

just replace  $price = $pricer['incvat']; 

with this in your code

$price[] = $pricer['incvat'];

answered May 17, 2017 at 12:46

Nithin John

9273 gold badges8 silver badges22 bronze badges

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

Comments

matiaslauriti · Accepted Answer · 2017-05-17 12:43:39Z

2

You are all the time overwriting you final price:

$price = array($pricer['incvat']);

Replace that with:

$price[] = $pricer['incvat'];

answered May 17, 2017 at 12:43

matiaslauriti

8,2004 gold badges37 silver badges48 bronze badges

3 Comments

Chris Yates Over a year ago

Any idea why its adding up the values of all the records and not just the current row

matiaslauriti Over a year ago

It should be adding only the ones that comes from SELECT * FROM jobs WHERE wsjid = '18'. Debug that this query is not returning all IDs you are then using to get the prices. But you can explain a little more about what is exactly happening now. I think you should open a new question.

Chris Yates Over a year ago

Hi ive raised a new question showng all code stackoverflow.com/questions/44028136/…

Agam Banga · Accepted Answer · 2017-05-17 12:43:03Z

The issue with your current approach is you are overwriting the $price variable. You need to push the value in the $price array like $price[] = $pricer['incvat']. You need to do this

    <?php
include('config.php');
// Function for calculation Rough Invoice Total
function basicTotal() {
    $con = mysqli_connect("localhost","USER","PASS","TABLE");
    $wtbdq = mysqli_query($con,"SELECT * FROM `jobs` WHERE wsjid = '18'");
    $wtbdr = mysqli_fetch_assoc($wtbdq);
    do {
    $wtbd = explode(":",$wtbdr['worktobedone']);
    foreach($wtbd as $item) 
        {
            $priceq = mysqli_query($con,"SELECT * FROM `workshop-items` WHERE wsiid = '$item'");
            $pricer = mysqli_fetch_assoc($priceq);

            $price[] = $pricer['incvat'];

            echo $item.' - '. $pricer['incvat'].'<br>';

        }

    } while($wtbdr = mysqli_fetch_assoc($wtbdq));

    $total = array_sum($price);
    echo $total;
}
basicTotal();
?>

Collectives™ on Stack Overflow

Putting values into an array and using array_sum

3 Answers 3

Comments

3 Comments

Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

Comments

3 Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related