[pyar] Cuestión de elegancia (?)

Claudio claudiobidau en gmail.com
Sab Mayo 26 21:16:47 ART 2012


El "len" lo necesito porque algún chabon podría meter un string tipo:
"autos, motos, , lanchas". Modifiqué lo que me pasaste y quedó así:

for tag_name in set([tag.strip() for tag in
form.habilidades.data.split(",") if len(tag.strip()) > 0]):

Slds y gracias nuevamente

Claudio

2012/5/26 Claudio <claudiobidau en gmail.com>

> Muchas gracias Martin!
> Ahora voy a probar esto a ver que onda. Con respecto a meter la lógica en
> el modelo como lo ves?
>
> Saludos
>
> Claudio
>
> 2012/5/26 Martin Alderete <malderete en gmail.com>
>
>> Hola!
>> Como va?
>>
>> 2012/5/26 Claudio <claudiobidau en gmail.com>
>>
>>> Hola!
>>> Tengo un código que funciona pero es horrible (sobre todo la linea del
>>> 'for') y me gustaría saber si existe alguna forma mas elegante de
>>> escribirlo.
>>>
>>> En la vista tengo lo siguiente (es el post de un form):
>>>
>>>     post = Post()
>>>
>>>     for tag_name in set(map(lambda tag:tag.strip(),
>>> form.tags.data.split(','))):
>>>
>>
>> No es necesario usar map si podes usar un list comprehension, esto puede
>> solucionarse asi:
>>
>> for tag_name in set([tag.strip() for tag in form.tags.data.split(",")]):
>>
>>
>>
>> No es necesario que mires la longitud, ya que vas a iterar sobre los
>> elementos del conjunto y el strip te elimina los "blancos"
>>
>>>       if len(tag_name) > 0:
>>>
>>
>>  Luego viene tu logica la cual supongo bien :)
>>
>>
>>         t = Tag.query.filter_by(name=tag_name).first() or Tag(tag_name)
>>>         post.tags.append(t)
>>>
>>>     db.session.add(post)
>>>     db.session.commit()
>>>
>>>
>>>
>> Bueno espeor que sirva!
>>
>> saludos
>>
>> --
>> Alderete, Martin Nicolas
>> Python Developer
>>
>> _______________________________________________
>> pyar mailing list pyar en python.org.ar
>> http://listas.python.org.ar/listinfo/pyar
>>
>> PyAr - Python Argentina - Sitio web: http://www.python.org.ar/
>>
>> La lista de PyAr esta Hosteada en USLA - Usuarios de Software Libre de
>> Argentina - http://www.usla.org.ar
>>
>
>
------------ próxima parte ------------
Se ha borrado un adjunto en formato HTML...
URL: <http://listas.python.org.ar/pipermail/pyar/attachments/20120526/283f6e5e/attachment.html>


More information about the pyar mailing list