🚀 go-pugleaf

En el cap�tulo anterior (news:87ac67i43m.fsf@guti.cl), Cristian Gutierrez 
<crgutier@dcc.uchile.cl> dijo:

> Hoy en la tarde, T. Q. LEO dijo:
> 
> [...]
> 
>> select * from lider where cliente='$empresa' and
>> MONTH(FROM_UNIXTIME(fecha))='$mescons' and
>> YEAR(FROM_UNIXTIME(fecha))='$anocons' order by fecha DESC LIMIT
>> ".$st.",".$pp;
> 
> Una sugerencia de estilo: es m�s legible (y hasta dir�a que un poco m�s
> mantenible) usar `sprintf' para componer sentencias SQL:
> 
> ,----
>| $template =<<<ENDOFTEMPLATE;
>| 
>| SELECT *
>| FROM lider
>| WHERE cliente = '%s'
>|   AND MONTH(FROM_UNIXTIME(fecha)) = '%s'
>|   AND YEAR(FROM_UNIXTIME(fecha))  = '%s'
>| ORDER BY fecha DESC
>| LIMIT %d,%d
>| 
>| ENDOFTEMPLATE
>| 
>| $query = sprintf($template,
>|                  $empresa, $mescons, $anocons, $st, $pp);
> `----
> 
> Lo ideal ser�a usar �prepared statements�, pero no siempre est�n
> disponibles (entiendo que para PHP5 + MySQL 4.1 s�).
> 

Yo no se cuan "ideal" es realmente eso...

Sobre todo, si se analiza la forma que tom� esta pregunta: hab�a un query 
que funcionaba y despu�s se dio cuenta que le faltaba un par�metro 
importante (el filtro por cliente)...

No quiero sonar pedante, pero ante un nivel tan bajo de dise�o previo, 
pretender usar templates o prepared statments es un salto importante...

Por supuesto que, en beneficio de la legibilidad es bueno gastar un par de 
"espacios" para identar el sql...

Algo as� como:
 // preparar par�metros:
 $cliente = $empresa;
 $mescons = ... ;
 $anocons = ... ;
 $limit1  = $st ;
 $limit2  = $pp ;

 $query = select * 
            from lider 
           where cliente = '$cliente'
             and MONTH(FROM_UNIXTIME(fecha)) = '$mescons'
             and YEAR(FROM_UNIXTIME(fecha)) = '$anocons'
           order by fecha DESC 
           LIMIT $limit1,$limit2";

Eso permite manejar adecuadamente el query en caso de necesitar nuevos 
filtros posteriormente...

Saludos!

-- 
Apoye a la campa�a de alfabetizaci�n y  |       Jose M. Santiba�ez A.
ordenaci�n de las news, cada post en su |          <jms@nivel5.cl>
lugar y el lugar exacto para cada post  |        <http://caos.cl/jms>