[ APORTE ] parsear URLs de referals como Google y Yahoo

[Italico76]

A veces uno quiere saber con que palabras se llegó a nuestro sitio para tener una estadistica o tomar alguna desición.

Aca un codigo que hice para ver con que palabras se llegó

Código PHP:

  <? 
Function parse_referal(){ 
  $referer=$_SERVER['HTTP_REFERER']; 
  if (strstr($referer,'yahoo.')){ 
    preg_match('/\?p=([0-9a-zA-Z.\+%]{1,})/', $referer, $matches);       
  }else{ 
    preg_match('/&q=([0-9a-zA-Z.\+%]{1,})/', $referer, $matches); 
  }   
return ($matches[1]);   
} 
 
echo parse_referal(); 
?>   

Vean la version actualizada...... mas abajo que reconoce tambien el patron de MSN

[okram]

En vez de usar if...else para decidir el patrón a utilizar, podrías emplear un array, donde el índice sea el buscador y el valor sea el patrón. Con un foreach identificas el buscador y utilizas el patrón correspondiente.

Gracias por el aporte

[Italico76]

okram: amigo, si uso un foreach seria ineficiente porque recoreria todos los elementos del array y aunque sean 2 o 3...... va en contra de lo que he aprendido.

Usar un break para salir del ciclo..... tampoco es algo que quiero hacer pero gracias igual porque me hicistes reflexionar un rato.

Aca un codigo que incluye al buscador MSN:

Código PHP:

  <? 
Function parse_referal(){ 
  $referer= $_SERVER['HTTP_REFERER']; 
   
  $pattern = array ('google'=>'/&q=([0-9a-zA-Z.\+%]{1,})/','yahoo'=>'/\?p=([0-9a-zA-Z.\+%]{1,})/','msn'=>'/\?q=([0-9a-zA-Z.\+%]{1,})/'); 
   
  $matches=array(); // vacio 
   
  if (strstr($referer,'yahoo.')) 
    preg_match($pattern['yahoo'], $referer, $matches);       
  else     
      if (strstr($referer,'google.')) 
        preg_match($pattern['google'], $referer, $matches);       
      else 
        if (strstr($referer,'msn.')) 
          preg_match($pattern['msn'], $referer, $matches);       
           
return ($matches[1]);   
} 
 
echo parse_referal(); 
?>   

Las expresiones tienen una parte en comun pero creo que no hace a la cuestion....asi que asi lo dejo :)

Valoro otras opiniones

[GatorV]

Es más limpio y legible si usas un switch().

Saludos

[Italico76]

GatorV: no veo como pueda hacerlo con switch porque $referer no me seriviria creo..... fijate que la proceso con strstr() en cada anidamiento del IF

Un abrazo

[okram]

Como ya te mencione en otro post, hablar de eficiencia es discutir quien fue primero, si el huevo o el gallo Ineficiente dices porque recorre el array, pero al hacer if... elseif... elseif... igual estamos evaluando condiciones una y otra vez. Por otro lado, no veo cuál es el problema en usar break; para romper el ciclo o continue; para saltar un loop. Además, usando un foreach el código se vuelve más flexible, pues para agregar buscadores sólo hace falta agregar el patrón al array. Incluso, en algún caso, podría usarse una base de datos para extraer los patrones y buscar. Es eficiente cuando uno mismo afirma que lo es.

Ahora, claro que se puede usar switch. Como te darás cuenta en el siguiente código, hay pequeños trucos que buscan explotar al máximo las cualidades de php

Código php:

function parse_referal() {
 
    // Le modifiqué esta línea para probar
    # $referer = $_SERVER['HTTP_REFERER'];
    $referer = 'http://search.msn.com/results.aspx?q=okram&FORM=MSNTST';
  
    $pattern = array ('google' => '/&q=([0-9a-zA-Z.\+%]{1,})/',
                      'yahoo' => '/\?p=([0-9a-zA-Z.\+%]{1,})/',
                      'msn' => '/\?q=([0-9a-zA-Z.\+%]{1,})/' );
  
    $matches = array(); // vacio
  
    switch(true):
        case strstr($referer,'yahoo.'):
            preg_match($pattern['yahoo'], $referer, $matches);      
            break;
        case strstr($referer,'msn.'):
            preg_match($pattern['msn'], $referer, $matches);
            break;
        case strstr($referer,'google.'):
            preg_match($pattern['google'], $referer, $matches);      
            break;
        default:
            return false;
            break;
    endswitch;
          
    return ($matches[1]);  
 
}
 

Por último, como también ya te lo dije en otro mensaje, debes hacer el código lo más "readable" posible. Quiero decir, de fácil lectura. Para ello un poco de orden estaría bien. El servidor no te reclamará nada si ve tu código en más o menos líneas. Hay un artículo... dale una ojeada

[Italico76]

switch(true):

Esa es muyyy buena! no la sabia

al hacer if... elseif... elseif... igual estamos evaluando condiciones una y otra vez

Si se cumple la primera se deja de evaluar.......si se cumple la segunda se deja de evaluar,... si se cumple la i-esima no se evaluará la (i+1)-ésima. Eso si, es poco claro

Me gusto lo del switch..... GatorV decia.....pero yo no lo veia posible

[okram]

En un foreach, si se cumple en un loop N, dejas de ciclar (break;), si no, te lo saltas avanzas al siguiente (continue;). Es lo mismo. Como te dije, cada uno maneja sus códigos según su propio concepto de eficiencia.

He notado que siempre evitas usar ciertas estructuras de control como break;. Puedo preguntar por qué?

[Italico76]

He notado que siempre evitas usar ciertas estructuras de control como break;. Puedo preguntar por qué

Si amigo. Antes usaba mucho break con for y otras estructuras de control con repeticion pero en la Universidad me prohibieron su uso y entonces no quiero volver a acostumbrarme.

Supuestamente es una mala-practica de programacion porque existen estructuras de control como while() donde el numero de ciclos no esta pre-determinado.

El curso era con Pascal, se que en otros lenguajes como C podes evaluar la condicion de lazo como en el while de de Pascal.........en fin..... [no necesariamente habira que usar while]

Esa es la razon

[okram]

El punto es que muchas veces realmente necesitas escapar de ciclos u otras estructuras. El switch es un ejemplo. Si no colocas break; allí no hay forma de hacer que funcione.

while, foreach, for, etc. crean ciclos en base a sus propias condiciones, pero hay ocasiones en las que tú debes imponer las tuyas. Por ejemplo. Necesitas extraer unas noticias de una base de datos. Pero quieres mostrar 3 con un formato especial, y las siguientes con otro tipo de formato. Harías dos consultas? Lo más práctico (al menos para mí) sería pedir todas las noticias (digamos que quiero mostrar 10) a la base de datos, y usar un primer ciclo, del cuál escaparé usando break; al llegar a la tercera noticia. Luego, al hacer un segundo ciclo, éste empezará en la 4ta noticia pues el puntero del array de resultados allí se encuentra.

Se puede prescindir de él, y generalmente hay otras opciones, pero muchas veces el no querer usarlo conlleva más código del que en realidad se necesitaría.

[Italico76]

El punto es que muchas veces realmente necesitas escapar de ciclos u otras estructuras. El switch es un ejemplo. Si no colocas break; allí no hay forma de hacer que funcione.

Eso es porque break es la palabra reservada de la estructura switch-case pero no tiene 'nada que ver'*, digo en otros lenguajes es CASE OF por ejemplo y no aparece la palabra "break" obligadamente en ninguna parte de la estructura.

En lo demas......puede que tengas razon! de hecho....creo que lo tienes! el FOREACH es un caso paradigmatico creo porque tiene una potencia que no la tenes con otras estructuras de control..... y lo unico "malo" es que no se puede especificar una condicion adiciona al numero de elementos de un arreglo pero usando break........

Gracias por compartir tu punto de vista!

(*) En realidad SI tiene que ver........porque implicitamente esta el CONCEPTO del break..... pero como decia...... no tiene porque llamarse break puede ser una llave un end u otro delimitador en la estructura de control