Pequeño cambio en la API

[cristianrasch]

Qué te parece si en lugar de usarlo así:

MontoEscrito::Conversor.convertir(10)

lo cambiamos por:

MontoEscrito::Conversor.new(10).to_s

Así es fácil de interpolar:

total = MontoEscrito::Conversor.new(10)
"El total es: #{total}"

[delucas]

No está mal... me gusta. Lo que me empieza a hacer ruido en ese caso es el namespace y el nombre de la clase. Quizás en una próxima iteración podría hacerse así, y de paso cambiar ese nombre, ¿no te parece?

[cristianrasch]

El namespace siempre es bueno dejarlo, así no contaminás el global
namespace. El nombre de la clase sí lo cambiaría por algo como
MontoEscrito::Decorador o algo así :S

On Sun 05 Jan 2014 12:13:04 PM ART, delucas wrote:

No está mal... me gusta. Lo que me empieza a hacer ruido en ese caso
es el namespace y el nombre de la clase. Quizás en una próxima
iteración podría hacerse así, y de paso cambiar ese nombre, ¿no te parece?

—
Reply to this email directly or view it on GitHub
#5 (comment).

Ing. Cristian Rasch http://www.cristianrasch.com.ar/
@cristianrasch https://twitter.com/cristianrasch

[cristianrasch]

Por supuesto, antes de hacer cambios a la API chequeá semver por si no
lo conocés: http://semver.org/

On Sun 05 Jan 2014 12:15:06 PM ART, Cristian Rasch wrote:

El namespace siempre es bueno dejarlo, así no contaminás el global
namespace. El nombre de la clase sí lo cambiaría por algo como
MontoEscrito::Decorador o algo así :S

On Sun 05 Jan 2014 12:13:04 PM ART, delucas wrote:

No está mal... me gusta. Lo que me empieza a hacer ruido en ese caso
es el namespace y el nombre de la clase. Quizás en una próxima
iteración podría hacerse así, y de paso cambiar ese nombre, ¿no te
parece?

—
Reply to this email directly or view it on GitHub
#5 (comment).

Ing. Cristian Rasch http://www.cristianrasch.com.ar/
@cristianrasch https://twitter.com/cristianrasch

Ing. Cristian Rasch http://www.cristianrasch.com.ar/
@cristianrasch https://twitter.com/cristianrasch

[delucas]

De acuerdo a eso, la única forma en que esto sería un cambio menor es si
diéramos ambas posibilidades (como helper o como método de instancia).
Dado que es tan nueva, yo no cambiaría la versión, sino que directamente
cambiaría la API.

Otra cosa: no pondía Decorador :D

[cristianrasch]

También podés liberar la versión 1.0 y ya ;)

On Sun 05 Jan 2014 12:32:18 PM ART, delucas wrote:

De acuerdo a eso, la única forma en que esto sería un cambio menor es si
diéramos ambas posibilidades (como helper o como método de instancia).
Dado que es tan nueva, yo no cambiaría la versión, sino que directamente
cambiaría la API.

Otra cosa: no pondía |Decorador| :D

—
Reply to this email directly or view it on GitHub
#5 (comment).

Ing. Cristian Rasch http://www.cristianrasch.com.ar/
@cristianrasch https://twitter.com/cristianrasch