Une regex de plus

Version imprimable

09/01/2009, 16h33
Thorvaldsson

Une regex de plus
Bonjour,

Voilà, j'ai pas l'habitude de poster, mais là ça fait vraiment longtemps que cherche sur le web, le/les forums, etc, sans résultat... ^^
J'en viens donc à solliciter votre aide. =)

Bon !
Mon problème : Je souhaite faire un preg_replace sur une chaîne qui nous vient directement d'un code source. De ce type :

Code:

<td id="2-5" style="padding:0px;width:30px;height:30px;background:url(maps/2_petit_couloir/2-5.jpg);" onClick="propriete_case='';type='';test(this.id)"></td>

Je travaille donc sur UNE ligne venant du fichier source et je souhaite modifier une partie de ce code par une expression regulière en php, de cette façon :
Code:

1 2 $a_editer='/onClick=".*">/i'; //la regex qui va trouver ce que je veux changer $editer='onClick="test()">'; //ce que je veux y mettre
Je souhaite donc pouvoir changer tout ce qui peut se trouver dans l'attribut onClick...

Dans mon code, j'en viens à faire ceci, donc :

Code:

preg_replace($a_editer,$editer,$line);

Où $line est la ligne source à editer du type donné au-dessus.

Donc, ce devrait être tout bête, mais ma regex marche pas !
en faisant simplement

Code:

$a_editer='/onClick.*/i';

ça marche, mais ça va trop loin, évidement, cela va jusqu'à la fin de la ligne ; et puisque j'aimerais que ça s'arrête à la première balise de fermeture qu'elle rencontre, je devrais lui dire de s'arrêter à ceci : > ; mais ça ne veut pas ! lol
Est-ce dû aux caractères contenu dans le code source, au fait de lui dire de tout prendre (.*) jusque (>)... Ce me semblait être une regex toute bête, mais j'avoue que là je comprend pas ! ^^
J'ai fait des tonnes de tests, mais il semble que dès que je lui demande de chercher des symboles du genre >,<,ou ", ça ne veut plus marcher... même avec les \ devant ! ou entre crochets aussi ... je bloc !

Si vous avez une idée, une solution ? Je vous en remercierais d'avance !
09/01/2009, 16h47
s.n.a.f.u

C'est sûrement dû au caractère greedy (gourmand) de l'expression .*

Tu peux utiliser ceci qui est plus sélectif :

Code:

$pattern = '/onClick="[^"]*"/i';
09/01/2009, 16h52
Thorvaldsson

Il ne veut rien savoir non plus.
Cela réagit comme s'il ne trouvait tout simplement pas l'expression...
Il n'effectue donc aucun changement sur la chaîne... =(

Je crois que je vais vous présenter mon code (qui n'en est qu'au début) au complet pour la mise en situation :
Voici :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
 
if (isset ($_GET['map'])){
	include ($_GET['map']);
 
	$nb_ligne=0;
 
	$id='2-5';//à recevoir depuis un autre script. (Celui-ci s'utilisera d'ailleurs en fonction...).
 
	$id_a_trouver='/'.$id.'/';
 
	$a_editer='/onClick="[^"]*"/i';//Ceci aussi variera selon les demandes... pré-établies, évidement, avec des if...
	$editer='onClick="test()">';
 
	$fmap=fopen($_GET['map'],"r+");
 
	while($line=fgets($fmap)){
		$nb_ligne++;
		if (preg_match($id_a_trouver,$line)){
			echo 'Le code source de la case <strong>'.$id.'</strong> se trouve à la ligne <strong>'.$nb_ligne.'</strong>.<br/>';
			echo 'Voici le code de la ligne :<br/>';
			echo htmlspecialchars($line);
			echo '<br/><br/>';
 
			echo preg_replace($a_editer,$editer,htmlspecialchars($line));
 
			break;
		}
	}
	fclose($fmap);
}

Je sais pas si ça peut aider...

09/01/2009, 17h03
Thorvaldsson

Ainsi, sans voir à travailler sur le code en lui-même, mon problème est vraiment la regex !
Puisque le but final de ce code est :

- On inclus le fichier source dans la page (affichage d'origine);
- On voit à modifier les attributs onClick des cases (car c'est un tableau);
- On inscrit les changements effectués dans le fichier (pour sauvegarde);

Et cette regex est donc la clef puisqu'elle va me permettre de changer, voir à plusieurs reprises, tout ce qui pourrait être contenu dans l'attribut onClick ...

Voili ! ...
Je pensais vraiment pas avoir autant de mal à réaliser cela ! (la regex ! lol )
09/01/2009, 17h43
goodpz
La chaîne de remplacement ($editer) va ajouter un ">" qui n'est pas souhaité à mon avis:
Code:

1 2 $a_editer='/onClick="[^"]*"/i'; $editer='onClick="test()">';
Code:

preg_replace($a_editer,$editer,htmlspecialchars($line));

Tu devrais plutôt appliquer htmlspecialchars au retour de preg_replace plutôt qu'à l'entrée.

A part ça, le pattern de jml94 me paraît correct

CA Y'EST !!!

Ca marche ! lol

Merci beaucoup pour vos lumières !

C'était bien cela goodpz : il fallait appliquer le htmlspecialchars au retour par variable (chose importante) de preg_replace plutôt que directement à l'entrée.

Merci encore !

Voici donc ce que j'ai fait (pour ceux qui voudraient voir, qui en auraient besoin) :
Code:

1 2 3 4 5 6 7 $a_editer='/onClick="[^"]*"/i'; //La regex de recherche $editer='onClick="test()"'; //Le changement souhaité //Puis dans la boucle, j'en viens à appliquer tout ceci : $test=preg_replace($a_editer,$editer,$line); //Dans la variable $test, le changement appliqué sur la chaîne stockée dans $line echo htmlspecialchars($test); //Puis on fait ce que l'on veut du résultat ^^
Mon problème était donc le htmlspecialchars

Bonne continuation à vous !