Soucis de regex

Version imprimable

Soucis de regex

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:

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:

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!!

13/03/2011, 20h19
wiztricks

Citation:

Envoyé par Balbuzard

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

Bonjour,

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

Code:

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

13/03/2011, 22h01
kango

Citation:

Envoyé par wiztricks

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:

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

regexp ? pourquoi ici ?

Bonsoir,

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

Code:

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

14/03/2011, 08h02
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.

@+
14/03/2011, 09h54
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"
14/03/2011, 10h07
wiztricks

Citation:

Envoyé par kango

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

Code:

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

Oops, merci
- W
14/03/2011, 15h12
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!
14/03/2011, 22h03
PauseKawa

Bonsoir,

Citation:

Envoyé par 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"

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

Citation:

Envoyé par Balbuzard

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

Citation:

Envoyé par wiztricks

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

Citation:

Envoyé par PauseKawa

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 :

Citation:

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.

@+

Bonjour Balbuzard,

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

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))
@+