Fix deprecated eregi function in php?

Question

I'm currently using a function to figure out if a string consists only of numbers. The function is_numeric usually works, but if the string contains a letter, it will break. I am trying this instead:

function is_signedint($val)
{
    $val = str_replace(" ", "", trim($val));
    //line below is deprecated
    $bool = eregi("^-?([0-9])+$",$val);
    if($bool == 1)
        return true;
    else
        return false;
}

Anyways, I was wondering how I could replace the eregi line to comply with PHP6

possible duplicate of [Converting ereg expressions to preg](http://stackoverflow.com/questions/6270004/converting-ereg-expressions-to-preg) — mario, Aug 09 '11 at 23:24

score 2 · Accepted Answer · answered Aug 09 '11 at 23:06

2

preg_match("~^-?([0-9])+$~", $val);

just add delimiters

answered Aug 09 '11 at 23:06

genesis

50,477
20
96
125

score 0 · Answer 2 · answered Aug 09 '11 at 23:05

0

preg_match("/^-?([0-9])+$/i",$val);

case-insensitive i as modifier at the end

answered Aug 09 '11 at 23:05

Jacek Kaniuk

5,229
26
28

case-insensitive as general eregi replacement. wont hurt either – Jacek Kaniuk Aug 09 '11 at 23:29

score 0 · Answer 3 · answered Aug 09 '11 at 23:08

Here's a couple of options

if ($val === strval(intval($val)))

if ($val == intval($val))

I would use the first method as it checks the value and type. You shouldn't use regular expressions unless it is really necessary (or it can squash 10+ lines of code into 1).

Fix deprecated eregi function in php?

3 Answers3