CODEWITHSUNDEEP
phpdivisiondivide-by-zero
Seth-77
Seth-77

Reputation: 254

Division by zero error

I have this code throwing up the error:

<?php

    $val1 = $totalsum;
    $res = ( $val1 / $val2) * 100;
    $val2 = count($allcontent);
    $res = ( $val1 / $val2) * 100;
    // 1 digit after the decimal point
    $res = round($res, 1); // 66.7

    echo "Success: ";
    echo $res;
    echo "%";

?>

I have tried adding this line: 

if ($res === 0) 
{ 
   echo "not eligible"; 
}

but it still gives the error. any ideas?

Upvotes: 6

Views: 36161

Answers (5)

Zest
Zest

Reputation: 1

Make sure $val2 is NOT zero before trying to divide $val1 with it. Try this:

<?php
    $val1 = $totalsum;
    $res = ( $val1 / $val2) * 100;
    $val2 = count($allcontent);

    if( $val2 != 0 ){
        $res = ( $val1 / $val2) * 100;
        // 1 digit after the decimal point
        $res = round($res, 1); // 66.7
        echo "Success: ";
        echo $res;
        echo "%";
    }
?>

Upvotes: 0

Jack B
Jack B

Reputation: 149

I'm guessing $val2 is coming out to 0. Make sure $allcontent is being initialized and filled.

Upvotes: -2

Amal Murali
Amal Murali

Reputation: 76646

You have the following in your code:

$val2 = count($allcontent);

If the $allcontent array is empty, then the value of $val2 will be 0, and you will essentially be doing:

$res = ( $val1 / 0) * 100;

As expected, this will cause PHP to return the 'Division by zero' error.

To make sure this doesn't happen, simply use an if statement:

if ($val2 != 0) {
    $res = ( $val1 / $val2) * 100;
    // 1 digit after the decimal point
    $res = round($res, 1); // 66.7
    echo "Success: ";
    echo $res;
    echo "%";
}

This can be rewritten using sprintf():

if ($val2 > 0) {
    $res = round( ($val1 / $val2) * 100 , 1); // 66.7
    echo sprintf('Success: %d%%', $res); // % is used for escaping the %
}

It does the same thing, but looks a bit more cleaner, in my opinion.

Upvotes: 2

Chris Rasco
Chris Rasco

Reputation: 2731

You'd want to check $val2 before the division occurs:

<?php
$val1 = $totalsum;
$val2 = count($allcontent);
if($val2 != 0)
{
    $res = ( $val1 / $val2) * 100;
    // 1 digit after the decimal point
    $res = round($res, 1); // 66.7
    echo "Success: ".$res."%";
}
else
{
    echo "Count of allcount was 0";
}
?>

Upvotes: 16

undone
undone

Reputation: 7888

if($val2!=0){
  //Do it
}else{
  //Don't
}

Upvotes: 1

Related Questions