Discussion :

[midomarc]

J'ai besoin de Copier des fichiers dans un répertoire A ayant des extensions spécifiques vers un autre répertoire B
Mon script ça ne marche pas au niveau extensions

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
from pathlib import Path
import os
import shutil
 
src = '/Users/Administrator/Desktop/A'
dst = '/Users/Administrator/Desktop/B'
 
files=os.listdir(src)
 
for fname in files:
    if os.path.exists(dst) or files.endswitch(".txt") (dst) :
        shutil.copy2(os.path.join(src,fname), dst)

[papajoker]

bonjour

original , une personne qui demande de l'aide ici

Ta ligne 11 est plus que "particulière"

- pourquoi ne pas tester une seule fois si la destination existe, mais pourquoi pas si la destination disparait en cours d'exécution...
- pas compris ce que tu comptes faire avec ton or. "or" veut dire "ou" en français.
- ne teste même pas l'extension du fichier mais d'une liste
- encore moins compris ton dernier bloc (dst) à la fin de cette ligne


----------
Note: existe glob en python

[midomarc]

Bonjour,

j'ai essayé, mais ça marche pas

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
from pathlib import Path
import glob
import os
import shutil
 
src = '/Users/Administrator/Desktop/A'
dst = '/Users/Administrator/Desktop/B'
 
files =os.listdir(dst)
files =glob.iglob(os.path.join(src, ".txt"))
 
for fname in files:
    if os.path.exists(dst) and os.path.isfile(files): 
        shutil.copy2(os.path.join(src,fname), dst)

[tyrtamos]

Bonjour,

Plusieurs remarques concernant ton code.

- l'appel à files=os.listdir(dst) est inutile puisque files est affecté par iglob tout de suite après

- iglob renvoie des adresses complètes et pas seulement le nom des fichiers. Il te suffisait d'un "print(files)" pour le voir.

- iglob demande un motif wildcard valable pour ta recherche: "*.txt" et non ".txt"

- tester si le répertoire destination existe pour chacun des fichiers n'est pas logique: il ne faut le faire qu'une seule fois avant la copie. Tu pourrais aussi décider de créer le répertoire s'il n'existe pas.

- ce n'est pas utile de lire tous les fichiers de la source avant d'être sûr que le répertoire destination existe.

- on peut se passer de créer la variable "files".

- tu importes pathlib sans l'utiliser

Voilà ton code corrigé (à ma sauce):

Code :

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

1
2
3
4
5
6
7
8
9
10
11
import glob
import os
import shutil
 
src = '/Users/Administrator/Desktop/A'
dst = '/Users/Administrator/Desktop/B'
 
if os.path.exists(dst):
    for file in glob.iglob(os.path.join(src, "*.txt")):
        if os.path.isfile(file): 
            shutil.copy2(file, dst)

[MPython Alaplancha]

Bonjour,
avec pathlib:

Code :

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

1
2
3
4
5
6
7
8
from pathlib import Path
import shutil
 
src = '/Users/Administrator/Desktop/A'
dst = '/Users/Administrator/Desktop/B'
path = Path(scr)
for file in path.glob('*.txt'):
    shutil.copy2(file, dst)

[midomarc]

Merci de m'avoir aidé et conseillé.