adding larger number = wrong total

[f8ball]

Not sure why this is happening. I've tried ceil, round, format_number and I still can't figure this out. When I try to total up 1,000 and 125 I'm getting the total of 126.

If I print the variables on their own, they show correctly.

Can someone tell me what I'm doing wrong? Thanks.

$subT = $repT + $litT;
$o_total = $HTTP_SESSION_VARS["FOD"]->col("Price");
$final_total = round(number_format($subT) + number_format($o_total)); 
print $o_total; 
print $subT; 
print $final_total;

[BuzzLY]

number_format returns a string. What you end up with is

"1,000" + "125"

That's fine, usually. PHP is pretty good at converting strings to numbers before adding them up. However, PHP doesn't know what to do with the comma, so I think it just adds the 1 before the comma to 125.

I'm assuming you are looking to format the number with commas, right? Try this instead:

$subT = $repT + $litT; 
$o_total = $HTTP_SESSION_VARS["FOD"]->col("Price"); 
$final_total = number_format(round($subT + $o_total));  

echo "FOD: $o_total<br/>";  

echo "Subtotal: $subT<br/>";  

echo "Final Total: $final_total";

[f8ball]

Thanks, but that still produces 126 instead of 1,126.00. The figures are 125 + 1,000. I am using commas, but I don't need to. For some reason the script thinks 1,000 (or 1000) is 1.

$subT = $repT + $litT;
$o_total = $HTTP_SESSION_VARS["FOD"]->col("Price");
$final_total = round($subT + $o_total); 
print "Brick Price:". $o_total."<br>"; 
print "Subtotal:". $subT."<br>"; 
print "TOTA:". $final_total;

[drawmack]

I just ran this code on my server:

<?php
$first = array(100,25,"1,000");
$second = array(100,25,"1000");
$third = array(100,25,1000);
$inputs = array($first,$second,$third);

foreach($inputs as $cur_input) {
    echo "Function Input:<br>";
    echo "&nbsp;&nbsp;&nbsp;&nbsp;\$repT = " . $cur_input[0] . "<br>";
    echo "&nbsp;&nbsp;&nbsp;&nbsp;\$litT = " . $cur_input[1] . "<br>";
    echo "&nbsp;&nbsp;&nbsp;&nbsp;\$price = " . $cur_input[2] . "<br><br>";

$output = totals($cur_input[0],$cur_input[1],$cur_input[2]);

echo "Function Output:<br>";
echo "&nbsp;&nbsp;&nbsp;&nbsp;Brick Price = " . $output['brick_price'] . "<br>";
echo "&nbsp;&nbsp;&nbsp;&nbsp;Subtotal = " . $output['subtotal'] . "<br>";
echo "&nbsp;&nbsp;&nbsp;&nbsp;Total = " . $output['total'] . "<br><br>";

}

function totals($repT,$litT,$price) {
    $subT = $repT + $litT; 
    $o_total = $price; 
    $final_total = round($subT + $o_total);
    return array('brick_price' => $o_total,'subtotal' => $subT,'total'=> $final_total);
}  

?>

And it yielded this result set:

Function Input:
    $repT = 100
    $litT = 25
    $price = 1,000

Function Output:
    Brick Price = 1,000
    Subtotal = 125
    Total = 126

Function Input:
    $repT = 100
    $litT = 25
    $price = 1000

Function Output:
    Brick Price = 1000
    Subtotal = 125
    Total = 1125

Function Input:
    $repT = 100
    $litT = 25
    $price = 1000

Function Output:
    Brick Price = 1000
    Subtotal = 125
    Total = 1125

I don't know what you're doing wrong. But my data shows that if you use a comma you get 126 and it you do not use a comma you donot get 126. So it seems that you should take the comma out.

[BuzzLY]

I don't know which of your variables contains 1000, but drawmack's demo shows you that my assertion was correct. When you are adding the numbers up, PHP thinks it has a string delimited with commas.

Why don't you give us the rest of the code so we can test it?

[f8ball]

$HTTP_SESSION_VARS["FOD"]->col("Price") should equal 1000. Posting it would do you no good. Basically, the user selects this item and the price is stored in the db as 1000.00 (decimal(6,2)).

The only thing I can think of is the session is screwing it up somewhere. I only have four products so I think for now I'll set it up manually. If I comment out the session variable and use 1000 it totals correctly.

[f8ball]

In my cart's class I changed:

$FOD_currencyFormat = "9,999.99";

to:

$FOD_currencyFormat = "9999.99";

and that solved it. Thanks.

[BuzzLY]

Sweet. Make sure you mark the thread resolved.