[John]

Either way - to figure out percentages, you divide the small number by the big number, and multiple by 100.

Looking at your code, I think you're doing it the other way round. (Presuming that $numbermembers is the total number, and $newuserid is the fraction of that.) If $newuserid is the bigger number, ignore the variable switch I made in the vb_number_format function below.

I've also removed the if statement, as I can't see what the difference is between the code in each branch.

PHP Code:


// this gives us a value in the $statscache['activepercent'] variable
$statscache['activepercent'] = vb_number_format(($newuserid / $numbermembers) * 100, 2 ); 

// You can lob off the last two digits: 
$lasttwo = substr($statscache['activepercent'], -2); 

// And then see if that equals 00 
if ($lasttwo == '00') 
{ 
     // intval returns an integer value 
     $statscache['activepercent']= intval($statscache['activepercent']); 
} 

// if you want it in the variable, we can add the trailing % from before
$statscache['activepercent'] .= '%';