-1

I have a piece of code, which formats a number to INR Price (₹1.00/- , ₹10,00,000.00/- , etc.):

function price_format($num,$type = 1){
    $num_full = number_format($num,2);
    if (strpos($num, '.') !== false) {
    $num = substr($num_full, 0, strpos($num_full, "."));
    }
    
    if($type == 1){ // '₹10,00,000.00/-'
        $explrestunits = "" ;
    if(strlen($num)>3) {
        $lastthree = substr($num, strlen($num)-3, strlen($num));
        $restunits = substr($num, 0, strlen($num)-3); // extracts the last three digits
        $restunits = (strlen($restunits)%2 == 1)?"0".$restunits:$restunits; // explodes the remaining digits in 2's formats, adds a zero in the beginning to maintain the 2's grouping.
        $expunit = str_split($restunits, 2);
        for($i=0; $i<sizeof($expunit); $i++) {
            // creates each of the 2's group and adds a comma to the end
            if($i==0) {
                $explrestunits .= (int)$expunit[$i].","; // if is first value , convert into integer
            } else {
                $explrestunits .= $expunit[$i].",";
            }
        }
        $thecash = "₹".$explrestunits.$lastthree.substr($num_full, -3, strpos($num_full, "."))."/-";
    } else {
        $thecash = "₹".$num.substr($num_full, -3, strpos($num_full, "."))."/-";
    }
    return $thecash; // writes the final format where $currency is the currency symbol.
    }
}

For the below test, it is not giving as expected result.

echo price_format(1)."<br />";
echo price_format(10)."<br />";
echo price_format(84289.35);

Live Demo

Also, the code is open and I am happy if someone wants to have this code more clear and short

Alive to die - Anant
  • 70,531
  • 10
  • 51
  • 98
kayos
  • 31
  • 5

1 Answers1

2

There is no use of strpos while setting the value to $thecash. strpos doc

Use number_format() as: number_format((float)$num,2,'.','');

Working Code:-

function price_format($num,$type = 1){
    $num_full = number_format((float)$num,2,'.','');
    if (strpos($num, '.') !== false) {
    $num = substr($num_full, 0, strpos($num_full, "."));
    }
    
    if($type == 1){ // '₹10,00,000.00/-'
        $explrestunits = "" ;
    if(strlen($num)>3) {
        $lastthree = substr($num, strlen($num)-3, strlen($num));
        $restunits = substr($num, 0, strlen($num)-3); // extracts the last three digits
        $restunits = (strlen($restunits)%2 == 1)?"0".$restunits:$restunits; // explodes the remaining digits in 2's formats, adds a zero in the beginning to maintain the 2's grouping.
        $expunit = str_split($restunits, 2);
        for($i=0; $i<sizeof($expunit); $i++) {
            // creates each of the 2's group and adds a comma to the end
            if($i==0) {
                $explrestunits .= (int)$expunit[$i].","; // if is first value , convert into integer
            } else {
                $explrestunits .= $expunit[$i].",";
            }
        }
        $thecash = "₹".$explrestunits.$lastthree.substr($num_full, -3)."/-";
    } else {
        $thecash = "₹".$num.substr($num_full, -3)."/-";
    }
    return $thecash; // writes the final format where $currency is the currency symbol.
    }
}

echo price_format(1).PHP_EOL;
echo price_format(10).PHP_EOL;
echo price_format(84289.35);
Chirag Jain
  • 1,367
  • 1
  • 11
  • 27