[pyar] Opinión sobre proyecto

[Haitike]

>
>
>        criterio =
> self.lvPalabras.GetColumn(event.GetColumn()).GetText() # Cojo el
> nombre de la columna
>
> Ojo con la barrera lingüal con los argentos :P
>
>
xDDDDDDDDDDDD

La verdad es que hicimos este proyecto antes de entrar a la lista de correo,
así que claro, uno no pensó en esas cosas. Pero debe sonar raro xD



>        if self.criterio == criterio:   # Compruebo que el anterior
> criterio y el nuevo sea el mismo.
>            if self.orden == "DESC":    # En caso de serlo
> intercambiamos entre ASC y DESC
>                self.orden = "ASC"
>            else:
>                self.orden = "DESC"
>
> En vez de comentar línea por línea, yo generalmente prefiero hacer
> funciones cortas (como veo que vos también haces) y darle un docstring
> más general[0]. Algo como
> """
> Ordenar la ListView lvPalabras por la columna seleccionada.
> Si la columna de criterio de orden no cambia, se cambia el tipo de
> ordenamiento por ascendente o descendente.
> """
>
> y no comentaria línea por línea, ya que en
>
>  if self.orden == "DESC":    # En caso de serlo intercambiamos entre ASC y
> DESC
>    self.orden = "ASC"
>
> es más largo el comentario que el código, y el código es entendible ya
> de por sí. Ademas, los comentarios a lo largo del tiempo mienten, el
> código no ;)
>
> Lo que estoy tratando de decir es, el código dice que se hace, los
> comentarios por qué se hace.
>
>        else:                           # Guardamos el criterio para
> que sirva para las demás funciones (filtrar, nueva_palabra, etc.)
>            self.criterio = criterio
>            self.orden = "ASC"
>        array_ordenado =
> self.deutschDB.extraer(self.criterio,self.orden)  # consulta SQL ORDER
> BY ese nombre de columna
>        self.lvPalabras.OnRellenar(array_ordenado)    # Relleno el
> listview con el array ordenado
>
> Ves, éste es otro caso del comentario repitiendo el código.
>
> Me la agarré con ese archivo porque es el que leí :P, pero no lo tomes
> a mal, son simplemente sugerencias. Tu código se ve muy limpio y fácil
> de leer. Otro detalle que te diria que mires es la separación de
> elementos en listas de parámetros y tuplas[1], como en
>
>
>  self.lvPalabras.OnRellenar(self.deutschDB.extraer(self.criterio,self.orden))
>
> Pylint[2] te puede ayudar a revisar esas cositas menores.
>
> Notá que estoy nit picking y tu código ya es bastante limpio.
>
>
Gracias por los comentarios. Realmente yo no suelo abusar tanto de los
comentarios. En este caso es un poco por ser un ejemplo de aprendizaje y nos
sobrepasamos un poco. Pero en algo más serio los comentarios serían más
adecuados xD. De todas formas tienes razón, limpiaré un poco el código
quitando comentarios innecesarios y pasando algunos a las docstrings.

Un saludo.
------------ próxima parte ------------
Se ha borrado un adjunto en formato HTML...
URL: <http://listas.python.org.ar/pipermail/pyar/attachments/20100924/f2075feb/attachment-0001.html>