Discussion :

[Stuff90]

Bonjour à tous,

Je me trouve devant un petit problème lié à la fonction re.sub et aux expressions régulières.

Je souhaiterai passer la chaîne capturer par re.sub en paramètre de la fonction eval().

Ce qui donnerai un truc comme ça :

Code :

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

1
2
3
4
maVar = '<p>test <% coucou %><p>'
coucou = 'hello'
 
re.sub(r'<%\s*(.+)\s%*>', eval('\\1'), maVar)

Malheureusement j'ai une jolie erreur :

SyntaxError: unexpected character after line continuation character

Et je ne comprend pas bien pourquoi ça ne passerai pas.
Y'a t'il un moyen pour que ça marche comme ça ou ce n'est pas du tout la bonne manière de procéder ?

Merci d'avance

[mont29]

Je crois pas que tu puisses utiliser eval ici, pas de cette façon en tout cas. Le problème est que le code \1 est géré par sub() (ou un autre élément de re), donc après l’appel à eval(), qui se retrouve donc à évaluer '\\1', ce qui ne veut pythoniquement rien dire

Si tu passes une fonction en second argument de sub(), elle doit accepter en unique argument un objet match, donc tu dois définir une mini-fonction, voire même une lambda:

Code :

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

1
2
3
4
maVar = '<p>test <% coucou %><p>'
coucou = 'hello'
 
re.sub(r'<%\s*(.+?)\s*%>', lambda match: eval(match.group(1)), maVar)

(Petite correction d’une faute de frappe dans ta regex, au passage… Et j’ai aussi passé .+ en non-glouton, sinon tous tes espaces à la fin se retrouveraient dans ce que tu captures, ce qui n’est visiblement pas le but !)

Mais c’est une manière bien risquée de procéder – si quelqu’un a accès au contenu de maVar, il peut y mettre ce qu’il veut, et le faire exécuter par eval() ! Pourquoi ne pas plutôt utiliser un dict, comme ceci ?

Code :

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

1
2
3
4
maVar = '<p>test <% coucou %><p>'
rep_codes = {'coucou' : 'hello'}
 
re.sub(r'<%\s*(.+?)\s*%>', lambda match: rep_codes.get(match.group(1), ""), maVar)

[Stuff90]

Merci pour cette réponse très claire !

Ce pendant, j'ai essayé ta première suggestion mais il me dis que 'coucou' n'est pas déclaré (vraiment pourri comme nom de variable je sais...)

NameError: name 'coucou' is not defined

Je vais préciser un peu pourquoi j'ai besoin de ça, ça sera sûrement facile pour vous de m'aider.
En fait le contenu de 'maVar' sera le contenu d'un fichier (html ou xml) et le contenu de ce fichier devra pouvoir être interprété (expression régulière + eval() me paraissaient une bonne solution) avant de passer en affichage.
Au niveau de la sécurité, en aucun cas, l'utilisateur n'aura accès au contenu du fichier et chaque remplissage de formulaire par l'utilisateur passera par une fonction qui supprimera les occurrences '<%' et '%>' (si je décide de prendre ces balises définitivement bien sûr).

Donc voilà l'objectif final. La deuxième solution que tu me proposais ne colle donc pas car les valeurs que contiendrai le dictionnaire ne seront pas connues lors de l'écriture du fichier source.

Voilà, je cherche encore la meilleure façon de faire, si vous avez une idée ?

Je suis aussi preneur de conseil sur la sécurité du système !

[mont29]

Pour le “coucou” pas déclaré (t’inquiète, c’est de saison ), à mon avis, tu as oublié de le définir… En tout cas, chez moi, ce bout de code marche, en 3.2 comme en 2.6…

De toute façon, je persiste à penser que la solution avec dictionnaire est bien plus élégante et sûre, car elle est bornée (seul le contenu du dictionnaire est accessible). Pour le contenu, tu peux parfaitement mettre à jour les valeurs d’un dictionnaire quand tu veux, voire même en ajouter ou en supprimer, par exemple :

Code :

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

1
2
3
4
5
rep_codes = {'coucou' : "hello"}
rep_codes['err_404'] = open("my/404/error.txt", "r").readall()
rep_codes['usr_nick'] = get_user_nickname()
del rep_codes['coucou']
#etc.

De plus, si tu as besoin de plus de souplesse et de puissance, au lieu d’utiliser eval(), assigne des fonctions à ton dict (qui peuvent même récupérer des chaînes capturées par ta regex), par exemple :

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
import datetime, re
 
rep_codes = {'coucou' : lambda : "hello", # une simple chaîne statique
             'date' : lambda : str(datetime.date.today()), # la date du jour
            }
 
def get_user_nickname():
    # ici tu fais ta requête à la BD, par ex.
    return "einstein"
 
rep_codes['usr_nick'] = get_user_nickname # pas de (), on assigne la fonction, pas son résultat !
 
maVar = '<p><% coucou %> <% usr_nick %> (<% date %>)<p>'
 
re.sub(r'<%\s*(.+?)\s*%>', lambda match: rep_codes.get(match.group(1), lambda : "")(), maVar)

Ce qui produira :

Code :

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

'<p>hello einstein (2011-04-24)<p>'

PS : Je me doutais bien que c’était un genre de template qui était derrière tout ça