[Buoh-dev] Fechas de publicación

[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 at forge.novell.com
> > http://forge.novell.com/mailman/listinfo/buoh-dev
> _______________________________________________
> Buoh-dev mailing list
> Buoh-dev at forge.novell.com
> http://forge.novell.com/mailman/listinfo/buoh-dev
-- 
=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=
 Carlos Garcia Campos a.k.a. KaL
   elkalmail at yahoo.es
   carlosgc at 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