Problema al pasar desde "ereg()" a preg_match()

[sesebas]

Hola a todos,

Necesito de su ayuda para salir de este paso:

Tengo la siguiente funci♀n en php para identificar el navegador que usa el cliente

Código PHP:

function GetContentSel3($sel3) {
 
            if ((ereg("Nav", $_SERVER["HTTP_USER_AGENT"])) || (ereg("Gold", $_SERVER["HTTP_USER_AGENT"])) || (ereg("X11", $_SERVER["HTTP_USER_AGENT"])) || (ereg("Mozilla", $_SERVER["HTTP_USER_AGENT"])) || (ereg("Netscape", $_SERVER["HTTP_USER_AGENT"])) AND (!ereg("MSIE", $_SERVER["HTTP_USER_AGENT"])) AND (!ereg("Konqueror", $_SERVER["HTTP_USER_AGENT"])) AND (!ereg("Firefox", $_SERVER["HTTP_USER_AGENT"]))) $browser = "Netscape"; 
            elseif(ereg("Firefox", $_SERVER["HTTP_USER_AGENT"])) $browser = "FireFox"; 
            elseif(ereg("MSIE", $_SERVER["HTTP_USER_AGENT"])) $browser = "MSIE"; 
            elseif(ereg("Lynx", $_SERVER["HTTP_USER_AGENT"])) $browser = "Lynx"; 
            elseif(ereg("Opera", $_SERVER["HTTP_USER_AGENT"])) $browser = "Opera"; 
            elseif(ereg("WebTV", $_SERVER["HTTP_USER_AGENT"])) $browser = "WebTV"; 
            elseif(ereg("Konqueror", $_SERVER["HTTP_USER_AGENT"])) $browser = "Konqueror"; 
            elseif((eregi("bot", $_SERVER["HTTP_USER_AGENT"])) || (ereg("Google", $_SERVER["HTTP_USER_AGENT"])) || (ereg("Slurp", $_SERVER["HTTP_USER_AGENT"])) || (ereg("Scooter", $_SERVER["HTTP_USER_AGENT"])) || (eregi("Spider", $_SERVER["HTTP_USER_AGENT"])) || (eregi("Infoseek", $_SERVER["HTTP_USER_AGENT"]))) $browser = "Bot"; 
            else $browser = "Other";
 
              $result = "";
              $found = FALSE;
            
            if ($browser == "FireFox"){
 
/*
 Continua el Script...
*/
 

Todo bien hasta que tuve que correr la aplicacion con PHP 5 x. Aqui la funcion "eregi()" es obsoleta y debo utilizar "preg_match()". Después de buscar un buen rato llegué a probar con los cambios siguientes: (puse delimitadores solo en la linea de inicio del "if" ya que si el error me lo marcaba a partir de la suiente linea ya sabria que cambios hacer en las siguientes, pero....

Código PHP:

function GetContentSel3($sel3) {
            if ((preg_match('/Nav/', $_SERVER["HTTP_USER_AGENT"])) || (preg_match('/Gold/' $_SERVER["HTTP_USER_AGENT"])) || (preg_match('/X11/' $_SERVER["HTTP_USER_AGENT"])) || (preg_match('/Mozilla/' $_SERVER["HTTP_USER_AGENT"])) || (preg_match('/Netscape/' $_SERVER["HTTP_USER_AGENT"])) AND (!preg_match('/MSIE/' $_SERVER["HTTP_USER_AGENT"])) AND (!ereg('/Konqueror/' $_SERVER["HTTP_USER_AGENT"])) AND (!ereg('/Firefox/' $_SERVER["HTTP_USER_AGENT"]))) $browser = "Netscape"; 
            elseif(preg_match("Firefox", $_SERVER["HTTP_USER_AGENT"])) $browser = "FireFox"; 
            elseif(preg_match("MSIE", $_SERVER["HTTP_USER_AGENT"])) $browser = "MSIE"; 
            elseif(preg_match("Lynx", $_SERVER["HTTP_USER_AGENT"])) $browser = "Lynx"; 
            elseif(preg_match("Opera", $_SERVER["HTTP_USER_AGENT"])) $browser = "Opera"; 
            elseif(preg_match("WebTV", $_SERVER["HTTP_USER_AGENT"])) $browser = "WebTV"; 
            elseif(preg_match("Konqueror", $_SERVER["HTTP_USER_AGENT"])) $browser = "Konqueror"; 
            elseif((preg_match("bot", $_SERVER["HTTP_USER_AGENT"])) || (preg_match("Google", $_SERVER["HTTP_USER_AGENT"])) || (preg_match("Slurp", $_SERVER["HTTP_USER_AGENT"])) || (preg_match("Scooter", $_SERVER["HTTP_USER_AGENT"])) || (preg_match("Spider", $_SERVER["HTTP_USER_AGENT"])) || (preg_match("Infoseek", $_SERVER["HTTP_USER_AGENT"]))) $browser = "Bot"; 
            else $browser = "Other";
 
              $result = "";
              $found = FALSE;
            
            if ($browser == "FireFox"){
 

pero obtengo errores. El primero es

Código HTML:

Parse error: parse error in ....\SelectS.php on line 39
 
o sea ya en el inicio:
 
            if ((preg_match('/Nav/', $_SERVER["HTTP_USER_AGENT"])) || (preg_match('/Gold/' $_SERVER["HTTP_USER_AGENT"])) || (preg_match('/X11/' $_SERVER["HTTP_USER_AGENT"])) || (preg_match('/Mozilla/' $_SERVER["HTTP_USER_AGENT"])) || (preg_match('/Netscape/' $_SERVER["HTTP_USER_AGENT"])) AND (!preg_match('/MSIE/' $_SERVER["HTTP_USER_AGENT"])) AND (!ereg('/Konqueror/' $_SERVER["HTTP_USER_AGENT"])) AND (!ereg('/Firefox/' $_SERVER["HTTP_USER_AGENT"]))) $browser = "Netscape";
 

Bueno, he buscado manera de resolverlo, pero no he podido.

Ojala alguien me pueda dar una mano.

Gracias.

[CloudFFVII]

podes usar directamente strpos para eso o no??

[fran86]

Hola.. como va?
El primer reemplazo lo hiciste bien, pero en el resto te olvidaste la coma para separar los parametros:
if ((preg_match('/Nav/', $_SERVER["HTTP_USER_AGENT"])) || (preg_match('/Gold/', $_SERVER["HTTP_USER_AGENT"])) || ....

Ademas, para imitar el eregi(), que si mal no recuerdo no es sensible a minusculas/mayusculas, vas a tener que usar "i" como modificador del patron: "/bot/i"

Saludos!

[sesebas]

Hola fran86,

Gracias por darte el tiempo para buscar el problema, efectivamente creo que diste en el punto. Lamentablemente no podré probarlo ahora. Apenas lo haga te contaré cómo resultó. También tienes razón en el uso de la "i".

Gracias de nuevo

[CloudFFVII]

repito que la forma correcta de hacerlo creo que es strpos...

de php.net/preg_match

Notas
Tip

No use preg_match() si solo quiere verificar si una cadena está contenida en otra cadena. Use strpos() o strstr() en su lugar ya que éstas serán más rápidas.

[fran86]

Iniciado por sesebas

Hola fran86,

Gracias por darte el tiempo para buscar el problema, efectivamente creo que diste en el punto. Lamentablemente no podré probarlo ahora. Apenas lo haga te contaré cómo resultó. También tienes razón en el uso de la "i".

Gracias de nuevo

Por nada!

Igual me sumo a lo que te decia CloudFFVII sobre strpos/stripos, si es que tu server tiene mucha carga, o tenes TOC

Saludos!

[sesebas]

Gracias a los dos amigos que respondieron mi consulta.

Después de lidiar un par de horas conseguí habilitar toda mi aplicación en mi laptop personal y pude llegar al punto de probar sus sugerencias... en realidad sólo la de usar "preg_match()" y funcionó sin problemas!!!

Mañana probaré la de CloudFFVII y volveré a contarles el resultado, mientras tanto podemos dar por solucionado el problema.

Gracias.