[Buoh-dev] Fechas de publicación

[Esteban Sánchez]

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.

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
-- 
Esteban Sánchez
 esteban at steve-0.com
 http://steve-o.org
 http://subanales.com/
 ------------------------------------------------
 PGP key: http://pgp.mit.edu:11371/pks/lookup?op=get&search=0xB6E0F8AF
-------------- 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/2cebe4eb/attachment.pgp