<span style="color:rgb(0,153,0)">Hola Jesus,<br>Gracias por los comentarios,</span><br><br><blockquote style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex" class="gmail_quote">Para temas concretos de programación, yo te recomendaría que te<br>
suscribieses a <<a href="http://mail.python.org/mailman/listinfo/python-es" target="_blank">http://mail.python.org/mailman/listinfo/python-es</a>>. - <span style="color:rgb(0,153,0)"> ya estoy :-). Aparte, solo quería compartir mis primeras lineas de Python con gente Python. Créeme, no es lo mismo que compartirlas con peachepistas. </span><br>
<br>
Un par de comentarios:<span style="color:rgb(0,153,0)"> (realmente son 3.5 pares :-) )</span><br>
<br>
1. ¿Por qué usas un string "raw" para LOGF? <span style="color:rgb(0,153,0)">(aquí me mas pillado)</span><br>
<br>
2. Pero luego LOGF parece que no lo usas para nada. <span style="color:rgb(0,153,0)">(en realidad la idea era usarlo abajo donde abro el fichero, para dividir esto en 2 - arriba defines variables, abajo no tocas nada. A</span><span style="color:rgb(0,153,0)"><span style="color:rgb(0,153,0)">l final resulta que se me pasó.</span>)</span><br>
<br>
3. DEBUG=1 debería ser "DEBUG=True". <span style="color:rgb(0,153,0)">(vengo de Basic, Pascal y sh, imagino que te imaginas :-))</span><br>
<br>
4. Te recomendaría compilar la expresión regular fuera del bucle, en<br>
vez de computarla en cada ciclo del bucle (aunque creo que las<br>
versiones recientes de python "cachean" la versión compilada<br>
"internamente"). <span style="color:rgb(0,153,0)">(tendría que estudiar un poco sobre esto)</span><br>
<br>
5. El "INSERT INTO" lo repites dos veces. Mejor asigna la cadena a una<br>
variable, y luego usa esa variable en el EXECUTE y en el print. <span style="color:rgb(0,153,0)">(No se me había ocurrido. 10x, aparte el print lo puse a ultima hora porque no se me ocurrió manera de emular el set -x de sh)</span><br>
<br>
6. El split lo haces varias veces. Tal vez puedas hacerlo con una<br>
expresión regular más compleja, o con algo tipo<br>
<br>
"date, dummy, dummy, email = line.split(' ')" - <span style="color:rgb(0,153,0)">(Intenté extraer la primera y la quinta columna pero no me funcionó. habrá que leer mas).</span><br>
<br>
7. En vez del "debug==1" parece más intuitivo poner "if debug". <span style="color:rgb(0,153,0)">(lo mismo que en 3)</span><br>
<br>
Pero, vaya, no hay nada incorrecto. Son ganas de tocar las narices :) <span style="color:rgb(0,153,0)">(Me has enseñado cosas nuevas tocando las narices, por mi bien)</span><br>
<br></blockquote><br><span style="color:rgb(0,153,0)">Gracias por tu correo! Se agradece mucho!<br><br>Saludos</span><br><br><div class="gmail_quote">2013/2/22 Jesus Cea <span dir="ltr"><<a href="mailto:jcea@jcea.es" target="_blank">jcea@jcea.es</a>></span><br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">-----BEGIN PGP SIGNED MESSAGE-----<br>
Hash: SHA1<br>
<div><br>
On 22/02/13 14:03, Oldzhay Saroolu wrote:<br>
> Igual vosotros hacéis esto en 4 minutos, pero yo tarde 4 horas :-)<br>
> así que ha sido un buen curro... y si, se aceptan criticas :-) (por<br>
> favor)<br>
<br>
</div>Para temas concretos de programación, yo te recomendaría que te<br>
suscribieses a <<a href="http://mail.python.org/mailman/listinfo/python-es" target="_blank">http://mail.python.org/mailman/listinfo/python-es</a>>.<br>
<br>
Un par de comentarios:<br>
<br>
1. ¿Por qué usas un string "raw" para LOGF?<br>
<br>
2. Pero luego LOGF parece que no lo usas para nada.<br>
<br>
3. DEBUG=1 debería ser "DEBUG=True".<br>
<br>
4. Te recomendaría compilar la expresión regular fuera del bucle, en<br>
vez de computarla en cada ciclo del bucle (aunque creo que las<br>
versiones recientes de python "cachean" la versión compilada<br>
"internamente").<br>
<br>
5. El "INSERT INTO" lo repites dos veces. Mejor asigna la cadena a una<br>
variable, y luego usa esa variable en el EXECUTE y en el print.<br>
<br>
6. El split lo haces varias veces. Tal vez puedas hacerlo con una<br>
expresión regular más compleja, o con algo tipo<br>
<br>
"date, dummy, dummy, email = line.split(' ')"<br>
<br>
7. En vez del "debug==1" parece más intuitivo poner "if debug".<br>
<br>
Pero, vaya, no hay nada incorrecto. Son ganas de tocar las narices :)<br>
<br>
- --<br>
Jesús Cea Avión _/_/ _/_/_/ _/_/_/<br>
<a href="mailto:jcea@jcea.es" target="_blank">jcea@jcea.es</a> - <a href="http://www.jcea.es/" target="_blank">http://www.jcea.es/</a> _/_/ _/_/ _/_/ _/_/ _/_/<br>
jabber / <a href="mailto:xmpp%3Ajcea@jabber.org" target="_blank">xmpp:jcea@jabber.org</a> _/_/ _/_/ _/_/_/_/_/<br>
. _/_/ _/_/ _/_/ _/_/ _/_/<br>
"Things are not so easy" _/_/ _/_/ _/_/ _/_/ _/_/ _/_/<br>
"My name is Dump, Core Dump" _/_/_/ _/_/_/ _/_/ _/_/<br>
"El amor es poner tu felicidad en la felicidad de otro" - Leibniz<br>
-----BEGIN PGP SIGNATURE-----<br>
Version: GnuPG v1.4.10 (GNU/Linux)<br>
Comment: Using GnuPG with Thunderbird - <a href="http://www.enigmail.net/" target="_blank">http://www.enigmail.net/</a><br>
<br>
iQCVAwUBUSdvfZlgi5GaxT1NAQKlEwP/aVz42EhAty3qiuwQ932H+jWb39TD7GxT<br>
o3jDIseDRiWuThdZRGdUcF8uY+f3npjL/Cg5ivXHnJC+jg/Asx0xfOSCssS8SRvW<br>
+hcC2A2x9svcaZ5awI4rA7/92Hi0vgu6xCByeyh9YP4CK2BqDrxoYf2uv6ea8h9S<br>
mlmx5L1EHps=<br>
=oCeO<br>
-----END PGP SIGNATURE-----<br>
_______________________________________________<br>
Madrid mailing list<br>
<a href="mailto:Madrid@lists.es.python.org" target="_blank">Madrid@lists.es.python.org</a><br>
<a href="https://lists.es.python.org/listinfo/madrid" target="_blank">https://lists.es.python.org/listinfo/madrid</a></blockquote></div><br><br clear="all"><br>-- <br>Best Regards<br>Oldzhay Saroolu<br>tel: <a href="tel:%2B%20%2834%29%206%20444%207%20333%201" value="+34644473331" target="_blank">+ (34) 6 444 7 333 1</a><br>
<span style="background-color:rgb(51,102,255)"> <font color="#FFFFFF">in</font> </span>: <a href="http://es.linkedin.com/in/oldzhaysaroolu" target="_blank">oldzhaysaroolu</a><br><br><div><p style="color:rgb(34,34,34);font-size:13px;font-family:arial,sans-serif;margin:0px">
<span style="font-size:10pt;font-family:Arial,sans-serif" lang="EN-IE">This communication contains information which is confidential and may also be privileged. It is for the exclusive use of the intended recipient(s).</span><span style="font-size:12pt;font-family:Arial,sans-serif" lang="EN-IE"><u></u><u></u></span></p>
<p style="color:rgb(34,34,34);font-size:13px;font-family:arial,sans-serif;margin:0px"><span style="font-size:10pt;font-family:Arial,sans-serif" lang="EN-IE">If you are not the intended recipient(s), please note that any distribution, copying or use of this communication or the information in it is strictly prohibited.</span><span style="font-size:12pt;font-family:Arial,sans-serif" lang="EN-IE"><u></u><u></u></span></p>
<p style="color:rgb(34,34,34);font-size:13px;font-family:arial,sans-serif;margin:0px"><span style="font-size:10pt;font-family:Arial,sans-serif" lang="EN-IE">If you have received this communication in error, please notify the sender immediately and then destroy any copies of it.</span></p>
</div>