Discussion :

[mecmec]

Bonjour,

Il y a plusieurs années j'ai trouvé une fonction PHP sur Internet qui est très utile, qui permet par exemple de détecter un mot dans une chaine avec ou sans astérisques et qui permet donc de bannir ce mot détecté.
Voici le code:

Code php :

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
 
function match_string($pattern, $str)
{
  $pattern = preg_replace('/([^*])/e', 'preg_quote("$1", "/")', $pattern);
  $pattern = str_replace('*', '.*', $pattern);
  return (bool) preg_match('/^' . $pattern . '$/i', $str);
}
// (ne fonctionne pas sur PHP 7 ? et il fonctionne sur PHP 5.6 avec des millions d'alertes dans error_log)
/*
Example:
 
match_string("*world*","hello world") // returns true
match_string("world*","hello world") // returns false
match_string("*world","hello world") // returns true
match_string("world*","hello world") // returns false
match_string("*ello*w*","hello world") // returns true
match_string("*w*o*r*l*d*","hello world") // returns true
 
*/

Voici les millions de lignes générés dans "error_log" avec PHP 5.6:

Code txt :

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

[20-Jun-2018 00:54:56 Europe/Paris] PHP Deprecated:  preg_replace(): The /e modifier is deprecated, use preg_replace_callback instead in ..... on line 4

Est-ce que quelqu'un sait comment arranger la ligne N°4 "preg_replace" pour qu'elle soit compatible à la fois sur PHP 5.6 et PHP 7 ?

Merci d'avance.
Salutations.

PS: Le code PHP il vient de : https://stackoverflow.com/a/5622211 (en 2011)

[Dendrite]

Tout est dans le message d'erreur.
Il te faut remplacer preg_replace par preg_replace_callback...

L'étape que tu ne connais pas apparemment, c'est "comment accéder au manuel PHP' ?

http://php.net/manual/fr/function.pr...e-callback.php

Lis tout tranquillement et va voir les exemples, tu trouves toujours sur ces pages, ou quasiment, un truc utile dans les exemples.

PS : tu ne peux pas dire "tout fonctionne bien" sauf que ça "génère des alertes"... un peu comme tu ne peux pas dire "Tout va bien à la maison sauf que j'ai une alarme incendie qui hurle..."

edit : le fond de ton problème est le suivant => fonction preg_replace

Lorsque vous utilisez l'option obsolète e, cette fonction échappe quelques caractères (', ", \ et NULL) dans la chaîne qui remplace les références arrières. Ce comportement se justifie afin d'assurer qu'aucune erreur de syntaxe ne survient lors de l'utilisation des références arrières avec des guillemets simples et doubles (e.g. 'strlen(\'$1\')+strlen("$2")'). Assurez-vous d'être familier avec la syntaxe des chaînes afin de savoir exactement à quoi la chaîne interprétée doit ressembler.

[mecmec]

L'étape que tu ne connais pas apparemment, c'est "comment accéder au manuel PHP' ?

http://php.net/manual/fr/function.pr...e-callback.php

Je connais déjà cette page j'y suis déjà aller voir. En fait j'ai ce problème depuis 1 an environ et j'ai consulté ce manuel il y a quelques mois mais je suis dans l'incapacité de le convertir, du coup j'avais abandonné l'affaire et j'ai eu l'idée de supprimer ce fichier "error_log" une fois par mois (environ 2Go) mais il est temps que je me bouge pour venir poster ça sur un forum pour qu'un connaisseur me fasse ça en 5-10 minutes car moi j'ai bien perdu 2h en cumulé depuis ses 12 derniers mois, je n'ai vraiment plus du tout l'envie de perdre mon temps avec la fonction "preg_replace_callback" que je ne comprend pas.

[grunk]

venir poster ça sur un forum pour qu'un connaisseur me fasse ça en 5-10 minutes car moi j'ai bien perdu 2h en cumulé depuis ses 12 derniers mois, je n'ai vraiment plus du tout l'envie de perdre mon temps avec la fonction "preg_replace_callback" que je ne comprend pas

Bonjour,
Je pense que tu n'as pas bien compris le fonctionnement du forum.

Nous sommes là pour apporter une aide , pas pour faire le travail à ta place. Si tu veux rien faire , y'a plein de gens qui sont près à te faire le boulot moyennant de l'argent.

Maintenant si tu veux faire l'effort d'essayer quelque chose avec les pistes évoquées précédemment , tu trouvera surement de l'aide pour arriver à faire ce que tu veux , mais effectivement faudra fournir un petit effort.

Les étapes :
- Utiliser preg_replace_callback à la place de preg_replace
- Supprimer le modificateur "e"
- Placer le second argument de la fonction ('preg_quote("$1", "/")') dans un callback ou une fonction anonyme et du coup changer $1 par le paramètre passer à la fonction.

[Celira]

En gros, il faut que tu écrives une fonction qui prend le tableau matches en entrée et applique la fonction de ton pattern de remplacement avant de renvoyer le résultat.
Exemple : si ton pattern de remplacement était

Code :

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

"Youpi  : $1 -- $2"

, la fonction callback sera

Code :

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

1
2
3
function ($m) {
        return 'Youpi : '.$m[1].' -- '.$m[2] ;
    }

[Invité]

Bonjour,

tu peux lire cette réponse.

[mecmec]

Code php 7 :

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
<?php
 
function match_string($pattern, $str)
{
	$pattern = preg_replace_callback('/([^*])/', 'testPRC', $pattern);
	$pattern = str_replace('*', '.*', $pattern);
	return (bool) preg_match('/^' . $pattern . '$/i', $str);
}
 
function testPRC($m) {
	return preg_quote($m[1],"/");
}
 
 
echo match_string("*world*","hello world"); // returns true
echo match_string("world*","hello world"); // returns false
echo match_string("*world","hello world"); // returns true
echo match_string("world*","hello world"); // returns false
echo match_string("*ello*w*","hello world"); // returns true
echo match_string("*w*o*r*l*d*","hello world"); // returns true

PHP Tester online: http://sandbox.onlinephpfunctions.co...fcf626b9acd274

Ca à l'air de fonctionner. C'est top merci de votre aide c'est la première fois que j'essaye de résoudre quelque chose sur un forum sans qu'on me donne le code directement. Il faut juste avoir de la patience et de la motivation et un petit coup de pouce par un forum.
Savez-vous si ce code peut tenir longtemps dans les versions PHP 7 ?

Je vais l'essayer dans mon script original que j'ai depuis 1 an juste pour la journée pour voir s'il peut tenir le coup.

[grunk]

Savez-vous si ce code peut tenir longtemps dans les versions PHP 7 ?

Personne peux vraiment le dire. Pour l'instant il n'ya pas de dépréciation en vue pour ces fonctions , donc tu devrais être tranquille pour les quelques années à venir.

[mecmec]

Je l'ai posté ici aussi : https://stackoverflow.com/questions/...-with-asterisk (tout en bas)

Je met sur résolu.


Salutations.

[grunk]

Pour la peine je te donne le premier +1 de ta carrière sur stackoverflow
C'est important de partager des modifs sur un code qui t'a rendu service

[CosmoKnacki]

À noter que la réponse de @Czechnology sur ce post est loin d'être mauvaise.

[Celira]

À noter que la réponse de @Czechnology sur ce post est loin d'être mauvaise.

Ce code là ?

Code :

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

1
2
3
4
function match_string($patt, $haystack) {
  $regex = '|^'. str_replace('\*', '.*', preg_quote($patt)) .'$|is';
  return preg_match($regex, $haystack);
}

Si je comprends bien, au lieu d'appliquer preg_quote via une callback, il l'appelle directement ? ça va avoir un résultat équivalent ?

[CosmoKnacki]

Oui, preg_quote va échapper les caractères spéciaux y compris les astérisques, ensuite il remplace les astérisques échappés par .*.
Là où il a été malin, c'est que pour éviter de renseigner le deuxième paramètre de preg_quote et échapper un caractère de plus, il a choisi le caractère spécial | comme délimiteur pour sa pattern.

La seule différence c'est que sa fonction ne renverra pas de booléen (sauf en cas d'erreur) et qu'il a ajouté le modificateur s pour que le point matche aussi les nouvelles lignes.