guilleiguaran/fakeredis — pull request #78 by ivanetchart (2013-12-04 06:19:25+00:00)
Title: Split commands into different files & add bitcount

Seguro hablas español!

Básicamente me tomé la molestia de separar los comandos en distintos archivos, y por cada archivo que corresponde a categorías de comandos Redis, documentar cuales comandos estaban implementados y cuales no (hasta la versión 2.8.2). Lo hice porque era bastante incomodo trabajar con un archivo tan grande. Si bien los métodos tienen el mismo nombre que su correspondiente comando en Redis, a la hora de editar y organizarlos lógicamente me pareció una buena idea. Espero lo tomes bien.

También implementé el comando bitcount, con specs. Miralo, sobre todo por el tema de los rangos. Si bien me basé en lo que vos ya habías hecho quizás hubo algo que no tuve en cuenta.

Algunas cosas que se podrían tener en cuenta para mejor a futuro:

* Documentar los métodos. Si bien las implementaciones están claras, poder generar un YARD o RDOC creo que le aportaría a la gema.
* Desacoplar los módulos.
* Usar concerns para el manejo de dependencias
* Manejo de constantes. Por todos lados están los 'OK'
* Excepciones para métodos que no están implementados (cuando tuve el problema del bitcount, no fue tan directo darme cuenta que la gema no lo implementaba). Evita la dependencia de la documentación para saber que métodos no están implementados y los placeholders dan una mano.
* Agrupar specs por métodos. Lo hice para el bitcount, si te gusta el ejemplo se puede tomar como referencia. A simple vista es muy difícil ver que métodos están probados y cuales no.

Obviamente todo esto, es para poder mejorar lo que ya existe, nuevamente, espero que lo tomes a bien, y en la medida de lo posible es mi intención irte ayudando si estás de acuerdo.

Muchas gracias por la gema,
Saludos