Discussion :

[cpalperou]

Bonjour,
j'ai une liste de nom de fichiers avec les paths complets. Ci-desous un exemple avec seulement quelques fichiers dans la liste (dans mon cas réel, il y a bien plus de fichiers!).

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

File_list = ['/DATA/Lines/file0398374.nc', '/DATA/Lirens3/file03988484374.nc', '/DATA/Livrens22/file0398339393374.nc', '/DATA/Litres/file03983339474.nc' ]

Je voudrais copier ces fichiers dans un répertoire wrk_tmp en utilisant la librairie shutil donc je fais:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

shutil.copy(File_list, wrk_tmp)

et là, une erreur : expected str, bytes or os.PathLike object, not list

Je fais donc une boucle:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
for i in range(len(File_list)):
    shutil.copy(File_list[i], wrk_tmp)

Ca fonctionne bien mais je me dis que cette boucle va ralentir mon code et je ne trouve pas ça très élégant.

Y a t'il un moyen plus rapide et plus élégant ?
Merci

[lg_53]

si la liste des fichiers que vous déplacez est tous simplement l'ensemble des fichiers (placé plus ou moins loin dans l'arborescence) d'un dossier précis, alors vous pouvez utiliser shutil.copytree

[MPython Alaplancha]

Je fais donc une boucle:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
for i in range(len(File_list)):
    shutil.copy(File_list[i], wrk_tmp)

Ca fonctionne bien mais je me dis que cette boucle va ralentir mon code et je ne trouve pas ça très élégant.

Y a t'il un moyen plus rapide et plus élégant ?
Merci

Bonjour,
Ce n'est pas une boucle qui va plomber ton code...
Sinon je préfère l'écriture (et le PEP aussi):

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
for file in File_list:
    shutil.copy(file, wrk_tmp)

[lg_53]

Bonjour,
Ce n'est pas une boucle qui va plomber ton code...

Si le PO veut vraiment le savoir, il faut le mesurer, et ne pas se fier à son intuition.
D'intuition j'aurais dis que boucle ou pas boucle, ce qui va brider ici c'est l'écriture sur disque, donc j'aurais tendance aussi à dire que la boucle n'a que peu d'impact sur le cout, mais bon, à vérifier.

Sinon je préfère l'écriture (et le PEP aussi):

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
for i in File_list:
    shutil.copy(File_list, wrk_tmp)

Euh... édite ton code, car là ce n'est pas bon ce que tu proposes.

[MPython Alaplancha]

Euh... édite ton code, car là ce n'est pas bon ce que tu proposes.

Fait.

[wiztricks]

Salut,

Y a t'il un moyen plus rapide et plus élégant ?

Tout le problème est dans ce qui constitue la collection de fichiers à copier. Si c'était le contenu d'un répertoire, shutil.move ou shutil.copytree partiraient de la collections constituée par ce qu'il y a sur le disque...

Pour ce qui est de la rapidité, ce sont des lectures/écriture sur disques et çà va 1000 fois moins vite que les lectures/écriture mémoire. S'il y a beaucoup de fichiers, vous pouvez essayer de paralléliser jusqu'à la saturation du disque, histoire de le faire "plus rapidement".

- W