PHP calculating with decimals

Question

I get some rewards from a XMl and want to calculate the sum. Why isn't the output for $M_commission 166.63?

$i = 0;
$commission_trans = 0;
foreach($responseXml->lead as $lead)
{
    if ($i == 0) {
        $last_trans = $lead->leadTime;
    }
    $commission_trans = $commission_trans + $lead->reward; 
    echo $lead->reward;
    echo "\n";
    $i++;
}

$M_lastclick = $last_trans;
$M_commission = number_format($commission_trans, 2);

echo $M_commission;
echo "\n";
echo $M_lastclick;

Output: 100.0 0.63 3.0 30.0 3.0 30.0 166.00 2013-05-10T13:42:01.058+02:00

I tried

    $commission_trans = number_format($commission_trans + $lead->reward, 2);

but same output.

Tnx a lot!

The expected output is 166.63 and not 166.00: 100.0 + 0.63 + 3.0 + 30.0 + 3.0 + 30.0 = 166.00 ?? - so its not working like above since there are some type caveats with xml strings as summands. — Axel Amthor
– Axel Amthor, Commented May 14, 2013 at 5:24

Axel Amthor · Accepted Answer · 2013-05-14 05:21:35Z

1

The "XML" indicates, this is coming from some XML stream (simple_xml?):

foreach($responseXml->lead as $lead)

and thus, the object members ($lead->reward) are Strings, not Floats.

The $commission_trans is initialized with a plain zero, so its an integer.

This line $commission_trans = $commission_trans + $lead->reward; gives:

(int) = (int) + (string);

I would try to explicitly convert the XML string to a float:

$i = 0;
$commission_trans = 0.0;
foreach($responseXml->lead as $lead)
{
    if ($i == 0) {
        $last_trans = $lead->leadTime;
    }
    $commission_trans += floatval($lead->reward); 
                         // or (float)$lead->reward;
    echo $lead->reward;
    echo "\n";
    $i++;
}

answered May 14, 2013 at 5:21

Axel Amthor

11.1k1 gold badge28 silver badges45 bronze badges

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

1 Comment

BlitZ Over a year ago

@Axel Amthor removed mine. There is someone else.

mpyw · Accepted Answer · 2013-05-14 05:33:13Z

1

I think flag $first is better than $i++.

$first = true;
$commission_trans = 0.0;
foreach($responseXml->lead as $lead) {
    if ($first) {
        $last_trans = (float)$lead->leadTime;
        $first = false;   
    }
    $commission_trans += (float)$lead->reward; 
    echo $lead->reward.PHP_EOL;
}

$M_lastclick = $last_trans;
$M_commission = number_format($commission_trans, 2);

echo $M_commission.PHP_EOL;
echo $M_lastclick;

Test on Ideone: http://ideone.com/1pi7mQ

answered May 14, 2013 at 5:33

mpyw

5,7744 gold badges35 silver badges36 bronze badges

Collectives™ on Stack Overflow

PHP calculating with decimals

2 Answers 2

1 Comment

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

1 Comment

Comments

Your Answer

Sign up or log in

Post as a guest

Related