[pyar] Tiny Backup

[Martin Cerdeira]

2010/10/21 Juan Manuel Santos <vicariousdm en gmail.com>:
>
> From: Ivan Alejandro <ivanalejandro0 en gmail.com>
> To: Python Argentina <pyar en python.org.ar>
> Date: Thursday 21 October 2010
>> Estuve mirando el codigo y pienso que seria mas adecuado pasarle a tu
>> libreria que hace los backups una lista con los directorios en lugar
>> de que hagas un bucle para cada carpeta que agregues desde la interfaz
>> grafica.
>> Otro pro de dejar el bucle del lado del thread que se encarga de
>> empaquetar/comprimir es que si a tu programa le agregas un boton
>> 'Cancelar', no lo podrias presionar hasta salir del metodo en el cual
>> entra al presionar 'Start Backup', es decir, hasta terminar el backup.
>
> En realidad esto es por usar un método iterativo en vez de recursivo (al menos
> cuando yo tuve que crawlear, me salió naturalmente usar un método recursivo.
>
> Bah ahora que lo pienso en un método iterativo se puede hacer igual, es
> cuestión de tener un flag y chequearlo cada vez que empieza el bucle. Ese flag
> es el que vos seteás si apretás en cancel, entonces en la próxima iteración,
> el método no corre y sale.

El crawler en si, ES recursivo. Fijate lo que hace cuando encuentra
una carpeta (se llama a sí mismo)
Lo que habría que mejorar es, como dijo Ivan, que la funcion zipi,
reciba la lista de las carpetas, en vez de llamarla N veces, siendo N
= cantidad de carpetas (de la lista)
Es decir, ahora llama, iterativamente, por cada carpeta EN la lista, a
un crawler recursivo. Lo unico que hay que cambiar es, llamar una sola
vez al crawler recursivo, mandandole todos los items juntos. No se si
soy claro, releo lo que puse y cada vez me parece menos claro =)
-------------------------------------
Martín Cerdeira - Software Developer
[web] http://www.codmacs.blogspot.com/
()  ascii ribbon campaign
/\  www.asciiribbon.org