Discussion :

[Balbuzard]

Bonjour à tous!

Encore débutant sur Python, j'essaie de détruire tous les fichiers d'un répertoire dont le nom termine par '.cfg'.

J'ai testé ceci en m'inspirant de la doc python:

Code :

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

1
2
3
4
5
6
7
8
9
 
>>> import os
>>> import re
>>> prog = re.compile("cfg")
>>> for f in os.listdir('/home/moi/Desktop'):
...     if prog.match(f):
...             print f,"yes"
...     else:
...             print f,"non"

J'obtiens ceci en sortie:

Code :

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

1
2
3
4
5
6
 
hosts-paris.cfg non
config-paris.conf non
sixth.py non
hosts-reims.cfg non
hostgroup.cfg non

Je ne comprends pas pourquoi le nom du fichier ne matche jamais la regex? Est-ce que je me suis trompé dans la regex à compiler?

Merci de votre aide!!

[wiztricks]

Je ne comprends pas pourquoi le nom du fichier ne matche jamais la regex? Est-ce que je me suis trompé dans la regex à compiler?

Utilisez plutôt rg = re.compile('.*.cfg') et si vous êtes faché avec les regexp regardez le module glob
- W

[tyrtamos]

Bonjour,

Moi, je préfère le module fnmatch:

Code :

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

1
2
3
4
5
6
7
8
 
import os
import fnmatch
for f in os.listdir('/home/moi/Desktop'):
    if fnmatch.fnmatch(f, "*.cfg"):
        print f,"yes"
    else:
        print f,"non"

Tyrtamos

[kango]

Utilisez plutôt rg = re.compile('.*.cfg') et si vous êtes faché avec les regexp regardez le module glob
- W

je pense qu'il y a une toute petite erreur, je me permets de la corriger:

Code :

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

rg = re.compile('.*\.cfg')

[PauseKawa]

Bonsoir,

Je ne comprend pas, vous avez tout sous la main :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
import os
rep = os.path.join(os.path.expanduser('~'), 'Bureau')
# 'Bureau' pour moi > a remplacer par 'Desktop'.
# os.path.expanduser('~') pour le home de l'utilisateur : Universel.
for f in os.listdir(rep):
    if os.path.splitext(f)[1].lower() == '.cfg':
    # tutu.cfg > ('tutu', '.cfg')
    # .lower() pour les minuscules
        print(os.path.join(rep, f), 'yes')
        # Vu que c'est pour un delete... os.path.join(rep, f) au cas ou...
        # print() pour Python 3 > compatible 2.x.
    else:
        print(os.path.join(rep, f), 'non')
# Soit :
#for f in os.listdir(rep):
#    if os.path.splitext(f)[1].lower() == '.cfg':
#        os.remove(os.path.join(rep, f))
# Ou :
#for f in (f for f in os.listdir(rep) if os.path.splitext(f)[1].lower() == '.cfg'):
#    os.remove(os.path.join(rep, f))

Pourquoi chercher plus loin ? Inutile non ?

Bon code

[PauseKawa]

Bonjour,

Pour ce qui est de glob ce n'est qu'un 'outils' utilisant os/yield/re/fnmatch.
Du moment ou l'on connais les modules en question et que l'on a compris yield je n'en vois pas l'utilité.
Mieux : du moment ou l'on sais ce que l'on veux faire il est largement préférable de s'en passer pour aller directement au but.
Ceci dit l'étude du code de glob.py peux être un bon apprentissage à la découverte de l'exploration d'un système de fichier.

@+

[wiztricks]

Salut,

Il y a toujours plusieurs façons de réaliser une fonction.

Mais quand la fonctionnalité est déjà réalisée par un module standard, soit on l'ignore et c'est dommage, soit on explique pourquoi il est intéressant, dans ce cas, d'écrire plus de lignes de code.
Cela donne un critère pour mesurer pourquoi "on préfère" plutôt que de rester sur du subjectif.

- W
pour "os.path.splitext(f)[1].lower() == '.cfg'"
Désolé, mais je trouve cela très "beurk"

[wiztricks]

je pense qu'il y a une toute petite erreur, je me permets de la corriger:

Code :

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

rg = re.compile('.*\.cfg')

Oops, merci
- W

[Balbuzard]

Merci à tous pour vos réponses!

Elles m'ont permis de résoudre mon soucis.

La réponse de wiztricks m'a est cele que je cherchais en premier, mais j'ai finalement préféré celle de PauseKawa qui évite d'utiliser le module re.

Merci encore

Bonne journée!

[PauseKawa]

Bonsoir,

Salut,

Il y a toujours plusieurs façons de réaliser une fonction.

Mais quand la fonctionnalité est déjà réalisée par un module standard, soit on l'ignore et c'est dommage, soit on explique pourquoi il est intéressant, dans ce cas, d'écrire plus de lignes de code.
Cela donne un critère pour mesurer pourquoi "on préfère" plutôt que de rester sur du subjectif.

- W
pour "os.path.splitext(f)[1].lower() == '.cfg'"
Désolé, mais je trouve cela très "beurk"

La démonstration viens d'être faite wiztricks.

j'essaie de détruire tous les fichiers d'un répertoire dont le nom termine par '.cfg'.

et si vous êtes faché avec les regexp regardez le module glob

Du moment ou l'on connais les modules en question ...
Mieux : du moment ou l'on sais ce que l'on veux faire il est largement préférable de s'en passer pour aller directement au but.

glob (module + ses imports) ou os.path.splitext ?

Pour le beurk :
C'est vrais que cela arrache la rétine.
Toutefois comme f viens d'un os.listdir :

os.path.splitext(path)

Split the pathname path into a pair (root, ext) such that root + ext == path, and ext is empty or begins with a period and contains at most one period. Leading periods on the basename are ignored; splitext('.cshrc') returns ('.cshrc', '').

Donc if os.path.splitext(f)[1] == ... Ok.
Pour le lower c'est de dernière minute.
Dans l'attente de vous lire à ce sujet.

@+

[PauseKawa]

Bonjour Balbuzard,

J'ai fais des tests pour un autre sujet et il s'avère qu'il faut savoir rester simple.
Soit:

Code :

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

1
2
3
4
5
6
7
8
import os
rep = os.path.join(os.path.expanduser('~'), 'Bureau')
delfic = os.remove
joinpath = os.path.join
splitname = os.path.splitext
for f in os.listdir(rep):
    if splitname(f)[1] == '.cfg':
        delfic(joinpath(rep, f))

@+