Discussion :

[neyloe]

Bonjour,

A mon travail, j'ai repris un script sur lequel je m'appuie et que je modifie. Dans celui-ci, je suis tombé sur l'expression suivante dont je me demande le sens et si elle n'est pas incomplète:

if '%s' in chaine:
[...]

Dans cette expression, chaine est une variable de type str. Je connais par exemple l'utilisation de %s dans une chaine de caractère pour signifier la présence d'une variable de type str (ex : print ("le résultat vaut : %s" %res)). Mais dans l'expression ci-dessus, la chaîne de caractère à laquelle %s devrait faire référence semble à priori absente, cette ligne n'a donc t elle pas de sens telle quelle? Je me demandais si le sens était celui de : if chaine : [...], mais en faisant un test dans un interpréteur sur une chaine de caractères, le résultat n'est pas le même.

Cette expression est elle incomplète? Quel sens aurait elle pu avoir?

[papajoker]

bonjour

tu te compliques la vie
ici le code teste si une (sous-)chaine est dans une autre

Code :

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

1
2
3
chaine = "bonjour"
if "on" in chaine:
     ok

Que la sous-chaine soit "on" ou "%x" ne change strictement rien

Code :

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

1
2
3
4
chaine = "bonjour %s et les autres"
if "%s" in chaine:
     # code bidon mais pourquoi pas
     print(chaine.replace('%s', 'toi'))

La raison de ce test est justement dans le bloc qui suis

[Sve@r]

Bonjour

Je connais par exemple l'utilisation de %s dans une chaine de caractère pour signifier la présence d'une variable de type str (ex : print ("le résultat vaut : %s" %res)).

Cette règle ne s'applique que si la chaine de gauche (ici "le résultat vaut : %s") est associée, via l'opérateur modulo, à une expression à droite (ici la variable res).
Si pas d'association, alors "%s" devient une simple string. Donc "toto_%s_titi" % "tata" vaudra "toto_tata_titi" mais "toto_%s_titi" vaut textuellement ce qui est écrit sans substitution.
Là où ça devient difficile, c'est quand on veut à la fois du pourcent en tant que caractère, et à la fois utiliser le formatage "%s" ou autre. Dans ce cas il faut le doubler => tva=20; print("la tva est de %d%%" % tva). Mais seulement dans ce cas. Si on supprime le formatage "%s/%d/%f" alors il ne faut plus doubler le caractère "%" sous peine de le voir alors apparaitre deux fois => print("la tva est de 20%").
Et enfin là où ça devient amusant, c'est si on associe à "%s" une chaine elle-aussi présente dans "res". Si "res" contient par exemple "123%s456", dans ce cas, if "%s" in res renvoie True, et if "%s" % "123" in res renvoie aussi True mais pas pour la même raison.

[neyloe]

merci à vous pour vos réponses.

@papajoker: en effet, ça pourrait être une possibilité, mais dans le contexte du programme (ce qui est supposé pouvoir se trouver dans la variable chaine) et du bloc d'instruction de la condition, un test de présence de ces deux caractères n'a pas de sens/intérêt, donc je ne pense pas que c'était l'intention ici.

A priori, je penche plus pour une mauvaise codification d'un test de chaîne non nulle (if chaine : ...), qui lui pourrait avoir du sens, ou un oubli de référence au motif de caractères concerné par le test...

[wiztricks]

A priori, je penche plus pour une mauvaise codification d'un test de chaîne non nulle (if chaine : ...), qui lui pourrait avoir du sens, ou un oubli de référence au motif de caractères concerné par le test...

Comme vous ne montrez pas ce qu'on fait lorsque la condition est vraie/fausse, difficile de vous raconter quoi que ce soit...
Et sans un minimum de factuel... pensez un peu ce que vous voulez...

- W

[papajoker]

mais dans le contexte du programme

Oui, la partie que tu ne nous donnes pas Alors pourquoi nous poser la question ?
puisque tu as repris un script, je suppose qu'il fonctionne...

Code :

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

1
2
if "%s" in chaine:
     raise Exception("Nom de fichier invalide!")

Code :

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

1
2
if "%s" in chaine:
     raise Exception("code SQL non sécurisé!")

Code :

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

1
2
if "%s" in chaine:
    chaine = chaine % str(valeur)

Trois contextes peut-être "valide" parmi 50 000

[neyloe]

@wiztricks, papajoker: ma question portait surtout sur la syntaxe de l'expression, à savoir si celle-ci pouvait avoir un sens telle quelle. Une réponse à cette question m'était suffisante et mon idée n'était pas d'entrer dans le détail du code pour une discussion sur celui-ci pour laquelle je manquerais de temps tout simplement.

Le script fonctionne pour certaines choses, d'autres non. En l'occurrence, c'est une base qu'il est nécessaire de modifier.

[wiztricks]

@wiztricks, papajoker: ma question portait surtout sur la syntaxe de l'expression, à savoir si celle-ci pouvait avoir un sens telle quelle.

C'est le genre de question à laquelle l'interpréteur Python sait répondre (avec un éventuel SyntaxError).

- W

[neyloe]

C'est le genre de question à laquelle l'interpréteur Python sait répondre (avec un éventuel SyntaxError).

- W

pour erreur de syntaxe oui (merci mais j'ai bien vu que l'exécution du code ne produisait pas d'erreur... ) , pour un sens non.