From: Mariano D. <ch...@si...> - 2005-08-27 20:09:28
|
El s=C3=A1b, 27-08-2005 a las 16:01 -0300, Facundo Batista escribi=C3=B3: > Mariano Draghi wrote: >=20 > > "quantdec" qued=C3=B3 como una property; es posible hacer un get/set de > > quantdec, exactamente igual que antes. Lo hice as=C3=AD porque me parec= i=C3=B3 > > redundante guardar los dos atributos, cuando quantdec se puede obtener = a > > partir de decplaces. >=20 > No me gusta para nada. Eso significa que cada vez que haga quantize, cada= vez=20 > que llame a SgfDecimal, estoy creando un objeto Decimal al re-pedo, ya qu= e no=20 > cambia nunca y lo podr=C3=ADa guardar. >=20 > Yo creo que debemos guardar ambas cosas. >=20 Mmmhh.. dejav=C3=B9. Ahora que lo mencion=C3=A1s por el lado de no crear un Decimal cada vez, creo recordar que tuvimos la misma conversaci=C3=B3n en s= u momento. Originalmente guard=C3=A1bamos la cantidad de decimales (no se si lleg=C3=B3 a commitearse alguna vez as=C3=AD, pero lo *pensamos* as=C3=AD),= y despu=C3=A9s se cambi=C3=B3 para evitar crear un Decimal todo el tiempo. Ten=C3=A9s raz=C3=B3n. >=20 > > =C2=BFOpiniones? >=20 > Dejemos todo igual, lo =C3=BAnico es guardar tambi=C3=A9n el 2. >=20 Ok. Corregido, testeado y commiteado. El "todo igual" incluye a la API? A mi me sigue resultando feo, muy feo ahora que lo v=C3=AD, que una property reciba un valor en el set y devuelva otro en el get... no es _nada_ intuitivo. Hagamos que el setter de quantdec reciba un Decimal (lo cual no me gusta mucho porque no hay una manera obvia de llegar a decplaces a partir del decimal...), o dej=C3=A9mos= la read-only. >=20 > > Commiti=C3=A9 todo esto, junto con algunos toques menores en > > Tools->Currencies (m=C3=A1s sobre esto en otro mail) >=20 > Lo otro (del otro mail incluso), esta superperfecto. >=20 Ok. Sigo por ese lado entonces. Una cosa m=C3=A1s a actualizar en la doc de la GUI... ya lo agregu=C3=A9 en el TODO. --=20 Mariano Draghi / el cHagHi PyAr - Python Argentina [http://python.org/ar] |