0

How can eregi() function can be replaced here with preg_match() function in below given code?

function getOS($userAgent) {
  // Create list of operating systems with operating system name as array key 
    $oses = array (
        'iPhone' => '(iPhone)',
        'Windows 3.11' => 'Win16',
        'Windows 95' => '(Windows 95)|(Win95)|(Windows_95)', // Use regular expressions as value to identify operating system
        'Windows 98' => '(Windows 98)|(Win98)',
        'Windows 2000' => '(Windows NT 5.0)|(Windows 2000)',
        'Windows XP' => '(Windows NT 5.1)|(Windows XP)',
        'Windows 2003' => '(Windows NT 5.2)',
        'Windows Vista' => '(Windows NT 6.0)|(Windows Vista)',
        'Windows 7' => '(Windows NT 6.1)|(Windows 7)',
        'Windows NT 4.0' => '(Windows NT 4.0)|(WinNT4.0)|(WinNT)|(Windows NT)',
        'Windows ME' => 'Windows ME',
        'Open BSD'=>'OpenBSD',
        'Sun OS'=>'SunOS',
        'Linux'=>'(Linux)|(X11)',
        'Safari' => '(Safari)',
        'Macintosh'=>'(Mac_PowerPC)|(Macintosh)',
        'QNX'=>'QNX',
        'BeOS'=>'BeOS',
        'OS/2'=>'OS/2',
        'Search Bot'=>'(nuhk)|(Googlebot)|(Yammybot)|(Openbot)|(Slurp/cat)|(msnbot)|(ia_archiver)'
    );

    foreach($oses as $os=>$pattern){ // Loop through $oses array
    // Use regular expressions to check operating system type
        if(eregi($pattern, $userAgent)) { // Check if a value in $oses array matches current user agent.
            return $os; // Operating system was matched so return $oses key
        }
    }
    return 'Unknown'; // Cannot find operating system so return Unknown
}
OM The Eternity
  • 15,694
  • 44
  • 120
  • 182

2 Answers2

2

You have to add an "i" after the right pattern delimiter to enable the case insensitive match

so:

a|b 

will be:

/a|b/i

To do it quickly write before preg_match:

$pattern='/'.$pattern.'/i';

Otherwise you have to change all your patterns and substitute like this:

(Linux)|(X11)

becomes

/(Linux)|(X11)/i
kappa
  • 1,559
  • 8
  • 19
0

You can use either

if(preg_match("#".$pattern."#", $userAgent)) {

or

 if(preg_match('`'.$pattern.'`i', $userAgent)) {
Jsdodgers
  • 5,253
  • 2
  • 20
  • 36
warfish
  • 613
  • 5
  • 20