Discussion :

[Yohgars]

Bonjour,

Je développe un petit webservice de type REST où les options produit sont stockées sous la forme : 10-251,11-197,18-249

Pour mettre à jour ce produit, je dois remplacer par exemple la sous-chaine 11-197 par 20-202.

La difficulté vient du fait que cette sous-chaine peut être au début, au milieu ou à la fin de la chaine principale.

Ma première idée a été de décomposer en plusieurs regex pour chaque cas de figure et cela fonctionne.

Cependant je suis un peu rouillé dans ce domaine, et ce n'est sans doute pas optimisé. Comment feriez-vous de votre coté ?

Edit : j'ai pu simplifier déjà un peu les patterns

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
16
17
18
19
20
21
22
23
24
25
26
27
28
29
 
// batterie de test
$stringa = '11-1978';
$stringb = '211-197';
$string1 = '11-197';
$string2 = '11-197,18-249';
$string3 = '10-251,11-197,11-1977';
$string4 = '411-197,18-249,11-197';
$string5 = '10-251,11-1972,11-197,811-197';
 
$patterns = array();
$patterns[] = '/^11-197$/';
$patterns[] = '/^11-197,/';
$patterns[] = '/,11-197,/';
$patterns[] = '/,11-197$/';
 
$replacements = array();
$replacements[] = '20-202';
$replacements[] = '20-202,';
$replacements[] = ',20-202,';
$replacements[] = ',20-202';
 
echo preg_replace($patterns, $replacements, $stringa); echo '<br/>';
echo preg_replace($patterns, $replacements, $stringb); echo '<br/>';
echo preg_replace($patterns, $replacements, $string1); echo '<br/>';
echo preg_replace($patterns, $replacements, $string2); echo '<br/>';
echo preg_replace($patterns, $replacements, $string3); echo '<br/>';
echo preg_replace($patterns, $replacements, $string4); echo '<br/>';
echo preg_replace($patterns, $replacements, $string5);

[rawsrc]

Bonsoir,

c'est clair y'a plus simple :

Code :

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

1
2
3
4
5
6
7
8
<?php
 
$haystack = '10-251,111-1977,11-197,18-249';
$needle   = '11-197';
$replaced = '20-202';
echo str_replace(array("$needle,", ",$needle,", ",$needle"), array("$replaced,", ",$replaced,", ",$replaced"), $haystack);
 
?>

La seule condition c'est de bien avoir des valeurs sans aucun espace sinon il fadrait les retirer avant le traitement.

[Yohgars]

Merci de cette réponse, mais ce code ne fonctionne pas dans plusieurs des cas testés par le bout de code :

- 1 seule option ($string1)
- 1 seule option : pas de remplacement si chiffre par devant ($stringa)
- 1 seule option : pas de remplacement si chiffre derrière ($stringb)
- plusieurs options : pas de remplacement si chiffre par devant ($string4)
- plusieurs options : pas de remplacement si chiffre par derrière ($string3)

C'est important puisqu'il s'agit de couples "option_id - value_id"

Voici une sortie OK :

Code :

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

1
2
3
4
5
6
7
11-1978
211-197
20-202
20-202,18-249
10-251,20-202,11-1977
411-197,18-249,20-202
10-251,11-1972,20-202,811-197

Il me semble que le preg_replace est nécessaire pour repérer le début et la fin de chaine. Je pensais simplement que l'on pouvait peut-être tout regrouper dans 1 seul pattern.

[rawsrc]

Salut,

Ne me remercie pas quand je me suis sèchement lourdé

Bon ce coup-ci ça devrait gazer sans regex :

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
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
<?php
 
$data = array(
   '11-1978',
   '211-197',
   '11-197',
   '11-197,18-249',
   '10-251,11-197,11-1977',
   '411-197,18-249,11-197',
   '10-251,11-1972,11-197,811-197'
);
 
 
$needle   = '11-197';
$replaced = '20-202';
$clean    = array();
 
foreach($data as $haystack) {
   $nums = explode(',', $haystack);
   $temp = array();
   array_map(
      function($p) use ($needle, $replaced, &$temp) {
         $temp[] = ($p === $needle) ? $replaced : $p;
      }, $nums
   );
   $clean[] = implode(',', $temp);
}
 
$out = array(
   '11-1978',
   '211-197',
   '20-202',
   '20-202,18-249',
   '10-251,20-202,11-1977',
   '411-197,18-249,20-202',
   '10-251,11-1972,20-202,811-197'
);
 
$nb = count($data);
echo '<pre>';
for($i = 0; $i < $nb; ++$i) {
   echo 'in    : ', $data[$i], '<br />', 'out   : ', $out[$i], '<br />', 'clean : ', $clean[$i], '<br />';
}
 
?>

Et je t'ai mis le code de test qui va avec.

[Yohgars]

Ok, donc si j'ai bien compris, pas de regex si on peut faire sans.

J'en vois beaucoup dans le code source sur lequel je travaille et j'ai fini par penser que c'était mieux.

Edit : Je viens de tester sur un produit réel et le temps d'exécution est divisé par plus de 2

Edit 2 : Ouille le serveur de production est en php 5.1.6 donc pas de use.

[rawsrc]

Edit : Je viens de tester sur un produit réel et le temps d'exécution est divisé par plus de 2

Bah c'est plutôt normal, les regex gèrent la complexité syntaxique très synthétiquement mais cela a un certain coût en terme de performance.

Edit 2 : Ouille le serveur de production est en php 5.1.6 donc pas de use.

Pas de problème, tu peux aussi faire comme ça :

Code :

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

1
2
3
4
5
6
7
8
foreach($data as $haystack) {
   $nums = explode(',', $haystack);
   $temp = array();
   foreach($nums as $num) {
      $temp[] = ($num === $needle) ? $replaced : $num;
   }
   $clean[] = implode(',', $temp);
}

[Yohgars]

J'avais trouvé comment faire pour php5.1.6.

Cette fois ci je peux dire merci, je crois et même résolu.