Discussion :

[florent]

Bonjour,

j'ai un soucis avec mon expression régulière :

Code :

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

1
2
 
^(.*)\.s?(\d{1,2})[ex]?(\d{2})\.

sur une valeur telle "How.I.Met.Your.Mother.8x16.FR.CaR.zip"

cela me sort
=> How.I.Met.Your.Mother
=> 8
=> 16

C'est parfait !
Mais probleme avec "How.I.Met.Your.Mother.8x12.8x13.FR.CaR.zip"
=> How.I.Met.Your.Mother.8x12
=> 8
=> 13

j'ai donc fait évoluer ma regex :

Code :

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

^(.*)\.s?(\d{1,2})[ex]?(\d{2})\.(?:\d{1}x\d{2}\.)?

Mais le problème persiste ... et je sèche

Un petit lien si vous voulez vous y essayer
http://regexr.com?33dt1

Merci

[florent]

Arf j'ai finalement trouvé une regex qui fonctionne

Code :

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

^(.*[^s?\d{1,2}(x|e)?\d{2}])\.s?(\d{1,2})[ex]?(\d{2})\.(?:s?(\d{1,2})[ex]?(\d{2})\.)?

[mont29]

Désolé, mais cette regex ne fait certainement pas ce que tu veux (pas en python, en tout cas*!). Le contenu des classes de caractères (entre crochets []) a sa propre syntaxe, la syntaxe regex “normale” n’y a pas cours… Autrement dit, "[^s?\d{1,2}(x|e)?\d{2}]" signifie «*tout sauf un s, un ?, un \, un d, un {, etc.*»*!

Ce que tu cherches à faire, c’est un lookahead négatif, mais tu n’en a pas besoin ici, amha un simple opérateur non-glouton devrait suffire*:

Code :

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

"^(.*?)\.s?(\d{1,2})[ex]?(\d{2})\.(?:\d{1}x\d{2}\.)?"

Alors que "(.*)" signifie «*capturer autant de caractère que possible tant que le reste de la regex est satisfaite*» (opérateur glouton), "(.*?)" signifie «*capturer le minimum de caractères nécessaires afin de satisfaire le reste de la regex*» (opérateur non-glouton, obtenu par l’ajout du “?” supplémentaire)*!

[florent]

Tu as complétement raison Bastien !

Comment j'ai pu passé à côté de ça et me compliquer autant la vie

Passé une certaine heure, il vaut mieux reporter au lendemain

Merci

[ripat]

Ce que tu cherches à faire, c’est un lookahead négatif, mais tu n’en a pas besoin ici, amha un simple opérateur non-glouton devrait suffire*:

Code :

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

"^(.*?)\.s?(\d{1,2})[ex]?(\d{2})\.(?:\d{1}x\d{2}\.)?"

Un multiplicateur non gourmand peut toujours être avantageusement remplacé par une classe négative. Souvent plus véloce.

Code :

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

^([^\d]+)\.(\d{1,2})[ex]?(\d{2})

Dans ce cas-ci et avec l'exemple donné: de 60% à deux fois plus rapide d'après timeit().

[mont29]

Certes… mais cette regex est moins universelle, elle bloquera complètement (pas de match) sur "How.I.Met.Your.Mother.At.6PM.8x16.FR.CaR.zip", par exemple. Donc, il n’y a que deux possibilités, amha*: opérateur non-glouton, ou lookahead négatif (ce qui doit, je pense, revenir au même…). Pas testé les performances, mais l’opérateur non-glouton est bien plus simple à écrire (et à comprendre*!).