[Buoh-dev] Fechas de publicación

[carlosgc at gnome.org (Carlos Garcia Campos)]

El jue, 29-09-2005 a las 15:50 +0200, Esteban S?nchez escribi?:
> El jue, 29-09-2005 a las 11:54 +0200, Carlos Garcia Campos escribi?:
> <snip />
> > > > > +               if (!comic_manager->priv->restrictions[i]) {
> > > > 
> > > > nunca he entendido esto de las restricciones. Si son los dias de
> > > > publicaci?n, por que no se llama simplemente publication_days y tiene
> > > > una l?gica directa en vez de indirecta? Vamos que cada vez que veo esto
> > > > de las restricciones me tengo que acordar que corresponde a los dias de
> > > > pucblicaci?n, pero teniendo en cuenta que FALSE significa que si se
> > > > publica ese dia. 
> > > 
> > > Bueno, en principio se da por hecho que los comics salen todos los d?as,
> > > as? que los que no son as? tienen restricciones. Esto parece l?gico,
> > > pero quiz?s al implementarlo segu? la misma l?gica (sin hacerla
> > > positiva) y sali? el l?o este :) Si me animo lo cambio porque
> > > evidentemente es m?s ofuscado, pero como lo hice yo pues no cai en la
> > > cuenta.
> > 
> > ok, si en el XML es mas c?modo as?, se le puede dar la vuelta a la
> > l?gica al crear el objeto a partir del XML y dejar el XML tal y como
> > est?
> 
> Venga, pues lo hago esta tarde.
> 
> > > Esto fue un desliz producto de reescribir el algoritmo (antes era un
> > > gint). Prefiero no usar ni TRUE ni FALSE, si no directamente prev.
> > > 
> > > > > +                               if (aux->len) {
> > > > 
> > > > es necesario comprobar siempre esto antes de un append? en este punto
> > > > nos hemos metido ya en un 4 nivel   for { if { if { if { Esto es una
> > > > locura, si hay manera de mejorarlo bien sino pues na.
> > > 
> > > Es la ?nica manera que he visto de hacerlo sin repetir c?digo. Si lo
> > > prefieres lo pongo as? (no se me ocurren m?s ahora a bote pronto):
> > > 
> > > if (prev && aux->len) {
> > > 	g_string_append (aux, ", ");
> > > 	g_string_append (aux, get_dayweek (i));
> > > 	last_printed = i;
> > > } else if (prev) {
> > > 	g_string_append (aux, get_dayweek (i));
> > > 	last_printed = i;
> > > }
> > 
> > La verdad es que a mi tampoco se me ocurre como hacerlo, as? que de
> > momento as? evitamos un nivel.
> 
> Pues no s?, pero despu?s de cambiarlo me he dado cuenta de que es peor
> el remedio que la enfermedad, pues en un caso se convierte en esto:
> 
> if (prev && (last_printed != i - 1) && aux->len) {
> 	...
> } else if (prev && (last_printed != i - 1)) {
> 	...
> }
> 
> Y personalmente me parece m?s ofuscado, pues me resulta m?s dificil una
> condici?n tocha en la que adem?s se repiten cosas que un nuevo nivel de
> anidaci?n.

pues si, tienes raz?n

> De momento lo que puedo hacer es comentarlo m?s.
> 
> > > > Salu2
> > > 
> > > Ma?ana mando el parche con m?s calma.
> > 
> > guay!!
> > 
> > > Saludos!
> > 
> > Salu2
> > _______________________________________________
> > Buoh-dev mailing list
> > Buoh-dev forge novell com
> > http://forge.novell.com/mailman/listinfo/buoh-dev
> _______________________________________________
> Buoh-dev mailing list
> Buoh-dev forge novell com
> http://forge.novell.com/mailman/listinfo/buoh-dev
-- 
=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=
 Carlos Garcia Campos a.k.a. KaL
   elkalmail yahoo es
   carlosgc gnome org
   http://carlosgc.linups.org
=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=             
PGP key: http://pgp.mit.edu:11371/pks/lookup?op=get&search=0x523E6462
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Esta parte del mensaje =?ISO-8859-1?Q?est=E1?= firmada
	digitalmente
Url : http://forge.novell.com/pipermail/buoh-dev/attachments/20050929/d7b86a9d/attachment.pgp