Tengo intención de reescribir el código bajo TDD. Si alguien desea contribuir de alguna forma, que se ponga en contacto conmigo. Gracias.
El 4 de marzo de 2013 17:35, Miguel Ángel Alarcos Torrecillas < [email protected]> escribió: > Muchas gracias Chema, no esperaba tanto. Agradezco todas las > recomendaciones que me das. > > En principio sólo esperaba críticas respecto del propósito de la librería, > de la API, y no que se buceara en el código que está un poco patas arriba. > Una vez más muchas gracias. > > > El 3 de marzo de 2013 14:51, Chema Cortes <[email protected]> escribió: > > El día 1 de marzo de 2013 19:11, Miguel Ángel Alarcos Torrecillas >> <[email protected]> escribió: >> > Hola a todos, me gustaría dar a conocer un proyecto al que he dado el >> nombre >> > de PyViewPredicate. Se trata de crear GUIs de manera declarativa, y >> además >> > reactiva. Con esto quiero decir que la GUI recibe los nuevos datos a >> mostrar >> > después de que se hayan modificado en el mismo u otro computador. >> > >> > De momento he implementado un visualizador de objetos BasicView >> (visualiza >> > una fila de una tabla de BDD) y un visualizador Table que visualiza un >> > predicado más complejo que es realmente una SQL. >> > >> > La wiki del proyecto tiene un ejemplo explicativo: >> > >> > https://bitbucket.org/miguel_alarcos/pyviewpredicate/wiki/Home >> > >> > Admito todo tipo de críticas, pero especialmente las constructivas. >> > Responderé a cuantas dudas o cuestiones me planteéis. >> > >> > Espero no robaros mucho tiempo, saludos pythónicos desde Murcia :) >> >> He estado revisando el código un rato hasta que me he cansado, por lo >> que solo puedo darte una impresión superficial. >> >> Lo primero, es recomendarte que siguas un mismo estilo para nombrar >> las clases. No resulta cómodo ver mezclados los estilos "CamelCase" y >> unix. >> >> Se ve que has invertido muchas horas en crear este código. Pero >> necesita bastante refactorización, sobre todo deberías hacer un >> paquete con el monolítico 'wxpython_gui' y, de paso, darle otro nombre >> que lo diferencie mejor del paquete wxpython general. >> >> Por otro lado, deberías limpiar el código. Al repositorio puedes >> indicarle que no versione los ficheros .pyc (sobra designable.pyc), >> quitar ficheros de pruebas (wxApp1.py, wxApp2.py,...) y que no >> aparezcan otras versiones que no tienen nada qué ver ahora >> (tkinter_app.py, tkinter_gui.py). Así mismo, asegúrate que estás >> versionados todos los ficheros que usas (falta 'pubsub.py', que >> supongo es una dependencia externa). >> >> Siguiendo con cuestiones de estilo, nunca deberías usar 'from modulo >> import *'. Está totalmente desaconsejado fuera de su uso en la línea >> de comandos. En un código normal deberías siempre indicar >> explícitamente qué objetos estás importando. >> >> En cuanto a mejorar la compresión, creo, por lo que he visto, que >> intentas implementar algo similar al patrón Actor. Localizar todo el >> código relativo a la creación de actores en su propio módulo puede >> ayudar más a entender lo que hace el código, así como localizar más >> rápido los mensajes a los que responden los actores (por ejemplo, en >> con un diccionario). >> >> >> -- >> Hyperreals *R: http://ch3m4.org/blog >> Quarks, bits y otras criaturas infinitesimales >> _______________________________________________ >> Python-es mailing list >> [email protected] >> http://mail.python.org/mailman/listinfo/python-es >> FAQ: http://python-es-faq.wikidot.com/ >> > >
_______________________________________________ Python-es mailing list [email protected] http://mail.python.org/mailman/listinfo/python-es FAQ: http://python-es-faq.wikidot.com/
