[Buoh-dev] Fechas de publicación

[esteban at steve-0.com (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 forge novell com
> http://forge.novell.com/mailman/listinfo/buoh-dev
-- 
Esteban S?nchez
 esteban 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