better rounding function

[Mgccl]

this function supplies the number($n), how many place you want keep after the decimal point($dec) and the function it will use($func, 3 values are 'round', 'floor' and 'ceil'). This is a handy script saves about 3 line of code if you are using the ceil and floor function

function func_round($n, $dec, $func) {
if ($func = 'round'){
  $result = round($n, $dec);
}
if ($func = 'floor){
  $n = $n * (pow(10, $dec));
  $n = floor($n);
  $result = $n/(pow(10, $dec));
}
if ($func = 'ceil){
  $n = $n * (pow(10, $dec));
  $n = ceil($n);
  $result = $n/(pow(10, $dec));
}


returt $result;
}

[halojoy]

sure can be useful
for those that have a lot of math in their php scripts

was a couple of single quotes missing
and 'return' misspelled

function func_round($n, $dec, $func){
    if ($func = 'round'){
        $result = round($n, $dec);
    }
    if ($func = 'floor'){
        $n = $n * (pow(10, $dec));
        $n = floor($n);
        $result = $n/(pow(10, $dec));
    }
    if ($func = 'ceil'){
        $n = $n * (pow(10, $dec));
        $n = ceil($n);
        $result = $n/(pow(10, $dec));
    }
return $result;
}

[halojoy]

even if still there is no checking for bad parameters,
this version makes even less code

function func_round($n, $dec, $func){
    if ($func = 'round')
        return round($n, $dec);
    $n = $n * (pow(10, $dec));
    if ($func = 'floor')
        $n = floor($n);
    if ($func = 'ceil')
        $n = ceil($n);
    return $n/(pow(10, $dec));
}

[laserlight]

Tsk tsk, you forgot to correct the use of operator = instead of operator ==

I would suggest:

function func_round($n, $dec, $func = 'round') {
	switch ($func) {
	case 'round':
		return round($n, $dec);
	case 'floor':
	case 'ceil':
		$multiplier = pow(10, $dec);
		return $func($n * $multiplier) / $multiplier;
	default:
		return $n;
	}
}

In retrospect, a switch may be a little too much:

function func_round($n, $dec, $func = 'round') {
	if ($func == 'round') {
		return round($n, $dec);
	} elseif ($func == 'floor' || $func == 'ceil') {
		$multiplier = pow(10, $dec);
		return $func($n * $multiplier) / $multiplier;
	} else {
		return $n;
	}
}

[halojoy]

laserlight wrote:

Tsk tsk, you forgot to correct the use of operator = instead of operator ==

In retrospect, a switch may be a little too much:

function func_round($n, $dec, $func = 'round') {
	if ($func == 'round') {
		return round($n, $dec);
	} elseif ($func == 'floor' || $func == 'ceil') {
		$multiplier = pow(10, $dec);
		return $func($n * $multiplier) / $multiplier;
	} else {
		return $n;
	}
}

Here is my corrected version, with ==

Even if other versions here, do the job as well.
Still this one would be what I think is most nice.
Note that this version will return $n (and not an error)
if no valid parameter $func is set or $func is not submitted. ( but then is better to do round($n,$dec); )
My version:

function func_round($n, $dec, $func=""){
    if ($func == 'round')
        return round($n, $dec);
    $n = $n * (pow(10, $dec));
    if ($func == 'floor')
        $n = floor($n);
    if ($func == 'ceil')
        $n = ceil($n);
    return $n / (pow(10, $dec));
}

[laserlight]

Still this one would be what I think is most nice.

The unfortunate part is that it calculates pow(10, $dec) twice if the third argument is not 'round'. This is rather inefficient, since pow() is likely to be (relatively) expensive.

Just for the fun of it, I have amended my version to be more like what is used in the PHP manual:

function func_round($value, $precision = 0, $func = 'round') {
	if ($func == 'round') {
		return round($value, $precision);
	} elseif ($func == 'floor' || $func == 'ceil') {
		$multiplier = pow(10, $precision);
		return $func($value * $multiplier) / $multiplier;
	} else {
		return $value;
	}
}