[Py-MAD] Mi primer programa en python

[Jesus Cea]

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

On 22/02/13 14:03, Oldzhay Saroolu wrote:
> Igual vosotros hacéis esto en 4 minutos, pero yo tarde 4 horas :-)
> así que ha sido un buen curro... y si, se aceptan criticas :-) (por
> favor)

Para temas concretos de programación, yo te recomendaría que te
suscribieses a <http://mail.python.org/mailman/listinfo/python-es>.

Un par de comentarios:

1. ¿Por qué usas un string "raw" para LOGF?

2. Pero luego LOGF parece que no lo usas para nada.

3. DEBUG=1 debería ser "DEBUG=True".

4. Te recomendaría compilar la expresión regular fuera del bucle, en
vez de computarla en cada ciclo del bucle (aunque creo que las
versiones recientes de python "cachean" la versión compilada
"internamente").

5. El "INSERT INTO" lo repites dos veces. Mejor asigna la cadena a una
variable, y luego usa esa variable en el EXECUTE y en el print.

6. El split lo haces varias veces. Tal vez puedas hacerlo con una
expresión regular más compleja, o con algo tipo

"date, dummy, dummy, email = line.split(' ')"

7. En vez del "debug==1" parece más intuitivo poner "if debug".

Pero, vaya, no hay nada incorrecto. Son ganas de tocar las narices :)

- -- 
Jesús Cea Avión                         _/_/      _/_/_/        _/_/_/
jcea at jcea.es - http://www.jcea.es/     _/_/    _/_/  _/_/    _/_/  _/_/
jabber / xmpp:jcea at jabber.org         _/_/    _/_/          _/_/_/_/_/
.                              _/_/  _/_/    _/_/          _/_/  _/_/
"Things are not so easy"      _/_/  _/_/    _/_/  _/_/    _/_/  _/_/
"My name is Dump, Core Dump"   _/_/_/        _/_/_/      _/_/  _/_/
"El amor es poner tu felicidad en la felicidad de otro" - Leibniz
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.10 (GNU/Linux)
Comment: Using GnuPG with Thunderbird - http://www.enigmail.net/

iQCVAwUBUSdvfZlgi5GaxT1NAQKlEwP/aVz42EhAty3qiuwQ932H+jWb39TD7GxT
o3jDIseDRiWuThdZRGdUcF8uY+f3npjL/Cg5ivXHnJC+jg/Asx0xfOSCssS8SRvW
+hcC2A2x9svcaZ5awI4rA7/92Hi0vgu6xCByeyh9YP4CK2BqDrxoYf2uv6ea8h9S
mlmx5L1EHps=
=oCeO
-----END PGP SIGNATURE-----