Question

I have used this code which shows an error:ie if my meters is 175m but when i convert it does’nt show that thing

<?php
function metersToFeetInches($meters, $echo = true)
{
    $m = $meters;
    $valInFeet = $m*3.2808399;
    $valFeet = (int)$valInFeet;
    $valInches = round(($valInFeet-$valFeet)*12);
    $data = $valFeet."&prime;".$valInches."&Prime;";
    if($echo == true)
    {
            echo $data;
    } else {
            return $data;
    }
}
?>
<?php
$feetInches = metersToFeetInches(1.75,false);
echo $feetInches;
?>

Answer 1

This is one of those things you can easily find on Google, but I guess you didn’t look further than the first link.

Anyways, you should do it like this.

<?php

function metersToFeet($meters) {
    return floatval($meters) * 3.2808399;
}

?>

But why is this better than the code you posted? Well, functions are not supposed to do everything. You should write a function for a certain action, not one big function with everything you can every need. Because if you need to add something to that function or change the order of actions, it’s a hell of a lot of work.

Furthermore, your function has an option $echo. But why would you need that? You can add such an option to every function, but PHP has a nice commando for that: echo. So instead, it’s way better to write echo metersToFeet(10). Or $value = metersToFeet(10) if you need to save the result in a variable.