Utilisation de str_replace sur des tableaux

Version imprimable

Bonjour à tous,

Mon but est de modifier tous les liens de champs text issus de ma BDD.
Après mon tri je me retrouve avec 3 tableaux: les anciens liens ($links), les nouveaux liens ($new_links) et le texte ($chaine). Les clefs sont les ID des textes.
Dans chaque texte, il peut y avoir plusieurs liens à changer :

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
31
32
33
34
35
36
37
38
39
40
41
42
43
 
Array
(
    [3018] => Array
        (
            [0] => https://www.toto.com/401-osmoseur-rgnrateur-aqua-avanti-em.html
            [1] => https://www.toto.com/401-osmoseur-rgnrateur-aqua-avanti-em.html
            [2] => https://www.toto.com/401-osmoseur-rgnrateur-aqua-avanti-em.html
            [3] => https://www.toto.com/401-osmoseur-rgnrateur-aqua-avanti-em.html
            [4] => https://www.toto.com/401-osmoseur-rgnrateur-aqua-avanti-em.html
        )
 
    [3024] => Array
        (
            [0] => https://www.toto.com/1619-la-mdecine-nouvelle-germanique-livre-du-dr-ryke-geerd-hamer-1619.html
            [1] => https://www.toto.com/1521-la-cure-de-tous-les-herpes-hulda-clark-9780984112937.html
        )
 
)
 
Array
(
    [3018] => Array
        (
            [0] => https://www.toto.com/401-osmoseur-rgnrateur-aqua-avanti-em.html?s=263322901
            [1] => https://www.toto.com/401-osmoseur-rgnrateur-aqua-avanti-em.html?s=263322901
            [2] => https://www.toto.com/401-osmoseur-rgnrateur-aqua-avanti-em.html?s=263322901
            [3] => https://www.toto.com/401-osmoseur-rgnrateur-aqua-avanti-em.html?s=263322901
            [4] => https://www.toto.com/401-osmoseur-rgnrateur-aqua-avanti-em.html?s=263322901
        )
 
    [3024] => Array
        (
            [0] => https://www.toto.com/1619-la-mdecine-nouvelle-germanique-livre-du-dr-ryke-geerd-hamer-1619.html?s=263322901
            [1] => https://www.toto.com/1521-la-cure-de-tous-les-herpes-hulda-clark-9780984112937.html?s=263322901
        )
)
 
Array
(
[3018] => Le texte 3018 qui contient les liens à remplacer
[3024] => Le texte 3024 qui contient les liens à remplacer
)

Quand j'ai mes 3 tableaux, je fais un bête

Code:

1
2
3
4
5
 
 
foreach($chaines as $id=>$content){
	$new_content[$id] = str_replace($links[$id], $new_links[$id], $content);
}

Mais le print_r du nouveau tableau sort n'importe quoi genre :

Code:

https://www.toto.com/401-osmoseur-rgnrateur-aqua-avanti-em.html?s=263322901?s=263322901?s=263322901?s=263322901?s=263322901

Pouvez-vous m'aider ?
D'avance merci.

Bonjour,

La boucle que tu essayes de passer ne permet pas de déterminer l'index des liens à remplacer.
Voici ma correction :

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
31
32
33
34
35
36
37
38
39
$links = [
    3018 => [
        'https://www.toto.com/401-osmoseur-rgnrateur-aqua-avanti-em.html',
        'https://www.toto.com/401-osmoseur-rgnrateur-aqua-avanti-em.html',
        'https://www.toto.com/401-osmoseur-rgnrateur-aqua-avanti-em.html',
        'https://www.toto.com/401-osmoseur-rgnrateur-aqua-avanti-em.html',
        'https://www.toto.com/401-osmoseur-rgnrateur-aqua-avanti-em.html'
    ],
    3024 => [
        'https://www.toto.com/1619-la-mdecine-nouvelle-germanique-livre-du-dr-ryke-geerd-hamer-1619.html',
        'https://www.toto.com/1521-la-cure-de-tous-les-herpes-hulda-clark-9780984112937.html'
    ]
];
 
$new_links = [
    3018 => [
        'https://www.toto.com/401-osmoseur-rgnrateur-aqua-avanti-em.html?s=263322901',
        'https://www.toto.com/401-osmoseur-rgnrateur-aqua-avanti-em.html?s=263322901',
        'https://www.toto.com/401-osmoseur-rgnrateur-aqua-avanti-em.html?s=263322901',
        'https://www.toto.com/401-osmoseur-rgnrateur-aqua-avanti-em.html?s=263322901',
        'https://www.toto.com/401-osmoseur-rgnrateur-aqua-avanti-em.html?s=263322901'
    ],
    3024 => [
        'https://www.toto.com/1619-la-mdecine-nouvelle-germanique-livre-du-dr-ryke-geerd-hamer-1619.html?s=263322901',
        'https://www.toto.com/1521-la-cure-de-tous-les-herpes-hulda-clark-9780984112937.html?s=263322901'
    ]
];
 
$chaines = [
    3018 => 'Le texte https://www.toto.com/401-osmoseur-rgnrateur-aqua-avanti-em.html, https://www.toto.com/401-osmoseur-rgnrateur-aqua-avanti-em.html, https://www.toto.com/401-osmoseur-rgnrateur-aqua-avanti-em.html, 
                https://www.toto.com/401-osmoseur-rgnrateur-aqua-avanti-em.html, https://www.toto.com/401-osmoseur-rgnrateur-aqua-avanti-em.html qui contient les liens à remplacer',
    3024 => 'Le texte https://www.toto.com/1619-la-mdecine-nouvelle-germanique-livre-du-dr-ryke-geerd-hamer-1619.html, https://www.toto.com/1521-la-cure-de-tous-les-herpes-hulda-clark-9780984112937.html qui contient les liens à remplacer'
];
 
foreach ($links as $id => $linksById) {
    foreach ($linksById as $index => $link) {
        $chaines[$id] = str_replace($link, $new_links[$id][$index], $chaines[$id]);
    }
}

30/03/2019, 11h32
Dendrite
Et pourquoi tu ne passerais pas par des requêtes SQL pour modifier des tables SQL avec une formule dans ce goût
Code:

1 2 3 update table liens l inner join table nouveaux_liens nl on l.id=nl.id set l.lien=nl.lien
Au préalable bien sûr, tu sauvegardes toute ta base ! Très important !
Et pour qu'on t'aide il faudrait que tu nous donnes la structure de tes tables SQL et les explications et un petit jeu de données.

Bonjour,

En fait ces liens sont disséminés dans un champ wp_content d'une table wp_post (un blog Wordpress). Avec des centaines d'autres liens. Donc si leur structure matche selon certains critères, je les remplace par les mêmes liens auxquels j'ajoute un ID d'affiliation du type ?s=xxxxxxxxx

Voici comment je récupère les tableaux:
1- Je récupère les contents
Code:

1 2 $resultats = $wpdb->get_results("SELECT ID, post_content FROM wp_posts WHERE post_content != '' AND post_content LIKE '%www.toto.com%' AND post_content LIKE '%.html%'") ;
2- Le traitement
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 foreach ($resultats as $row) { $chaines[$row->ID] = $row->post_content; $motif='#<a href="(.*?)"(.*?)>#is'; preg_match_all($motif,$chaine,$out,PREG_PATTERN_ORDER); foreach ($out[1] as $link) { if(strpos($link,'www.toto.com') && strpos($link, '.html')){ $links[$row->ID][] = $link; $new_links[$row->ID][] = $link.'?s=263322901'; } } }
3- Constitution du tableau final (tel que j'ai présenté dans mon 1er post) et affichage
Code:

1 2 3 4 5 foreach($chaines as $id=>$content){ $new_content[$id] = str_replace($links[$id], $new_links[$id], $content); } print_r($new_content);
Mais peut-être n'est ce pas la bonne méthode ? J'ignorais qu'on pouvait faire ce traitement directement dans MySQL...

31/03/2019, 10h32
renaud26

Bonjour Sam__ et merci de ta réponse,

En fait il y a des indices dans mes tableaux car pour chaque chaine il peut y avoir plusieurs liens à traiter.
Je n'avais pas donné assez de précisions, j'ai donc publié un autre post dans ce sujet.
01/04/2019, 16h01
Sam__

Oui je me suis bien rendu compte qu'il y avait plusieurs liens à traiter pour chaque id.
Tu trouvera dans ma réponse précédente la boucle corrigée te permettant de réaliser les remplacements.
Il faut boucler sur les liens et non les chaînes car sinon impossible de déterminer le nombre de liens à remplacer.

Enfin tu peux améliorer cette boucle avec une instruction SQL Update ou Replace pour enregistrer en BDD les modifications directement.
02/04/2019, 19h28
renaud26

Merci Sam__, je vais tester et je reviens.
Bonne soirée.

Edit:

C'est carrément top ! Je te remercie beaucoup. Et oui je vais faire ma requête update dans ma boucle.
02/04/2019, 20h12
renaud26
Damned ! J'ai écrit trop vite !
Si je fais un print_r($chaines) après les foreach de traitement, j'ai encore des liens du style
Code:

1 2 https://www.totocom/401-osmoseur-rgnrateur-aqua-avanti-em.html?s=263322901?s=263322901?s=263322901?s=263322901?s=263322901
Bien sûr dans les chaines où il y a plusieurs liens identiques à remplacer...

Si plusieurs liens identiques, alors pourquoi plusieurs index identiques aussi ?

Car dans ce cas, il suffit de :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
        $links = [
            3018 => [
                'https://www.toto.com/401-osmoseur-rgnrateur-aqua-avanti-em.html'
            ],
            3024 => [
                'https://www.toto.com/1619-la-mdecine-nouvelle-germanique-livre-du-dr-ryke-geerd-hamer-1619.html',
                'https://www.toto.com/1521-la-cure-de-tous-les-herpes-hulda-clark-9780984112937.html'
            ]
        ];
 
        $new_links = [
            3018 => [
                'https://www.toto.com/401-osmoseur-rgnrateur-aqua-avanti-em.html?s=263322901'
            ],
            3024 => [
                'https://www.toto.com/1619-la-mdecine-nouvelle-germanique-livre-du-dr-ryke-geerd-hamer-1619.html?s=263322901',
                'https://www.toto.com/1521-la-cure-de-tous-les-herpes-hulda-clark-9780984112937.html?s=263322901'
            ]
        ];

Parce que voici comment se construisent les tableaux $links et $new_links

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
 
foreach ($resultats as $row) {
 
	$chaines[$row->ID] = $row->post_content;
 
	$motif='#<a href="(.*?)"(.*?)>#is';
 
	preg_match_all($motif,$chaine,$out,PREG_PATTERN_ORDER);
 
	foreach ($out[1] as $link) {
		if(strpos($link,'www.toto.com') && strpos($link, '.html')){
			$links[$row->ID][] = $link;
			$new_links[$row->ID][] = $link.'?s=263322901';
		}
 
	}	
 
}

Et je ne vois pas le problème avec les index puisque j'obtiens ça:

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
31
32
33
34
35
36
37
 
Array
(
    [3018] => Array
        (
            [0] => https://www.toto.com/401-osmoseur-rgnrateur-aqua-avanti-em.html
            [1] => https://www.toto.com/401-osmoseur-rgnrateur-aqua-avanti-em.html
            [2] => https://www.toto.com/401-osmoseur-rgnrateur-aqua-avanti-em.html
            [3] => https://www.toto.com/401-osmoseur-rgnrateur-aqua-avanti-em.html
            [4] => https://www.toto.com/401-osmoseur-rgnrateur-aqua-avanti-em.html
        )
 
    [3024] => Array
        (
            [0] => https://www.toto.com/1619-la-mdecine-nouvelle-germanique-livre-du-dr-ryke-geerd-hamer-1619.html
            [1] => https://www.toto.com/1521-la-cure-de-tous-les-herpes-hulda-clark-9780984112937.html
        )
 
)
 
Array
(
    [3018] => Array
        (
            [0] => https://www.toto.com/401-osmoseur-rgnrateur-aqua-avanti-em.html?s=263322901
            [1] => https://www.toto.com/401-osmoseur-rgnrateur-aqua-avanti-em.html?s=263322901
            [2] => https://www.toto.com/401-osmoseur-rgnrateur-aqua-avanti-em.html?s=263322901
            [3] => https://www.toto.com/401-osmoseur-rgnrateur-aqua-avanti-em.html?s=263322901
            [4] => https://www.toto.com/401-osmoseur-rgnrateur-aqua-avanti-em.html?s=263322901
        )
 
    [3024] => Array
        (
            [0] => https://www.toto.com/1619-la-mdecine-nouvelle-germanique-livre-du-dr-ryke-geerd-hamer-1619.html?s=263322901
            [1] => https://www.toto.com/1521-la-cure-de-tous-les-herpes-hulda-clark-9780984112937.html?s=263322901
        )
)

On voit bien que pour le content 3018 chaque lien même identique a un index différent et pareil pour le tableau $new_links de remplacement...

Citation:

Parce que voici comment se construisent les tableaux $links et $new_links

Tu peux éventuellement mettre à jour tes entrées directement dans la boucle.

Citation:

On voit bien que pour le content 3018 chaque lien même identique a un index différent et pareil pour le tableau $new_links de remplacement...

Cela pose problème avec str_replace, car la chaîne trouvera toujours le lien, d'où ton résultat inattendu. Et il est inutile de conserver plusieurs liens identiques.
Tu peux optimiser ta boucle avec un in_array().
Code:

1 2 3 4 5 6 7 8 9 10 foreach ($out[1] as $link) { if (strpos($link, 'www.toto.com') && strpos($link, '.html')) { if (!in_array($link, $links[$row->ID])) { $links[$row->ID][] = $link; $new_links[$row->ID][] = "$link?s=263322901"; } } }

Merci, effectivement le in_array() simplifie les tableaux et il n'y a qu'une seule entrée "link" et "new_link" si le lien est répété plusieurs fois dans content.
Malgré cela, effectivement le str_replace plante dès qu'il n'y a plus d'un lien.
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 $out = ''; $content = array(); foreach ($links as $id => $linksById) { foreach ($linksById as $index => $link) { $out .= "Je remplace le lien ". $link ." par " .$new_links[$id][$index] . " dans la chaine ID ".$id." \n"; $content[$id] = str_replace($link, $new_links[$id][$index], $chaines[$id]); } $up = "UPDATE wp_post SET post_content = '".$content[$id]."' WHERE ID = '".$id."'<br>"; echo $up; } echo $out; print_r($content);
Dans ce test, le $out est parfait. Mais le print_r($content) est faux dès que $links et $new_links > 1 lien.
Et même le $up est faux
:arf:

Citation:

Dans ce test, le $out est parfait. Mais le print_r($content) est faux dès que $links et $new_links > 1 lien.

Oui c'est normal car si > 1 lien, tu écrases à chaque passage ton $content[$id], d'où la modification directe de $chaines, comme dans le code d'une de mes précédentes réponses.
Code:

1 2 3 4 5 6 7 8 9 foreach ($links as $id => $linksById) { foreach ($linksById as $index => $link) { $chaines[$id] = str_replace($link, $new_links[$id][$index], $chaines[$id]); } } $up = "UPDATE wp_post SET post_content = '{$content[$id]}' WHERE ID = '$id"; echo "$up<br>";
Penses également à utiliser une transaction avant ta première requête Update, et à valider une fois la dernière passée ;-)

Merci !!! C'était une erreur d'implémenter un nouveau tableau $content dans les boucles.
En utilisant ta méthode :
Code:

1 2 3 4 5 6 7 8 9 10 11 foreach ($links as $id => $linksById) { foreach ($linksById as $index => $link) { $chaines[$id] = str_replace($link, $new_links[$id][$index], $chaines[$id]); } $up = "UPDATE wp_post SET post_content = '{$chaines[$id]}' WHERE ID = '$id"; echo $up."<br>"; }
Nickel !

Qu'entends-tu par utiliser une transaction dans la requête UPDATE ?
Je pensais faire un truc du genre :
Code:

1 2 3 4 5 6 $wpdb->update( $wpdb->prefix.'ma_table', array('post_content' => {$chaines[$id]}), array('ID' =>$id) );
Je te remercie beaucoup pour ton aide précieuse, Sam. Super sympa.

Je ne vois pas l'intérêt d'utiliser des tableaux et encore moins de faire une recherche avec preg_match_all pour dans un deuxième temps effectuer un remplacement (toutes les fonctions de remplacement font déjà une recherche). La chaîne n'a besoin d'être parcourue qu'une seule fois pour appliquer le remplacement:

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
31
32
33
34
35
36
37
$pattern = '~ <a \s+ (?> [^>\s]+ \s+ )*? href=[\'"]? \K [^>"\'\s]+ ~ix';
 
$options = [ 'host' => 'www.toto.com',
             'extension' => 'html',
             'query' => 's=263322901' ]; // on regroupe les paramètres.
 
$replacement = function ($m) use (&$options) {
 
    $url = $m[0];
    $parts = parse_url( (stripos('http', $url) === 0 ? '' : 'http://') . $url ); // on parse l'url
 
    if ( !isset($parts['path']) )
        return $url;
 
    $tmp = explode('.', $parts['path']);
    $extension = array_pop($tmp);
 
    if ( $parts['host'] === $options['host'] && $extension === $options['extension'] ) {
        // on reconstruit l'url
        $url = $parts['scheme'] . '://' . $parts['host'] . '/' . $parts['path'] . '?';
 
        if ( isset($parts['query']) )
            $url .= $parse['query'] . '&';
 
        $url .= $options['query'];
 
        if ( isset($parts['fragment']) )
            $url .= '#' . $parts['fragment'];
    }
 
    return $url;
};
 
foreach ($resultats as $row) {
    $new_content = preg_replace_callback($pattern, $replacement, $row->post_content);
    echo $new_content;
}

C'est un code qui affiche le résultat (pour la démo). Pour ton usage, il suffit de préparer une requête SQL d'update avant la boucle puis d'y binder les valeurs et de l'exécuter dans la boucle.

Citation:

Envoyé par CosmoKnacki

Je ne vois pas l'intérêt d'utiliser des tableaux et encore moins de faire une recherche avec preg_match_all pour dans un deuxième temps effectuer un remplacement (toutes les fonctions de remplacement font une recherche). La chaîne n'a besoin d'être parcouru qu'une seule fois pour appliquer le remplacement:

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
31
32
33
34
35
36
37
$pattern = '~ <a \s+ (?> [^>\s]+ \s+ )*? href=[\'"]? \K [^>"\'\s]+ ~ix';
 
$options = [ 'host' => 'www.toto.com',
             'extension' => 'html',
             'query' => 's=263322901' ]; // on regroupe les paramètres.
 
$replacement = function ($m) use (&$options) {
 
    $url = $m[0];
    $parts = parse_url( (stripos('http', $url) === 0 ? '' : 'http://') . $url ); // on parse l'url
 
    if ( !isset($parts['path']) )
        return $url;
 
    $tmp = explode('.', $parts['path']);
    $extension = array_pop($tmp);
 
    if ( $parts['host'] === $options['host'] && $extension === $options['extension'] ) {
        // on reconstruit l'url
        $url = $parts['scheme'] . '://' . $parts['host'] . '/' . $parts['path'] . '?';
 
        if ( isset($parts['query']) )
            $url .= $parse['query'] . '&';
 
        $url .= $options['query'];
 
        if ( isset($parts['fragment']) )
            $url .= '#' . $parts['fragment'];
    }
 
    return $url;
};
 
foreach ($resultats as $row) {
    $new_content = preg_replace_callback($pattern, $replacement, $row->post_content);
    echo $new_content;
}

C'est un code qui affiche le résultat (pour la démo). Pour ton usage, il suffit de préparer une requête SQL d'update avant la boucle puis d'y binder les valeurs et de l'exécuter dans la boucle.

Woooooooooaaaawwwww !
Là je suis carrément sur le cul !
Un grand merci à toi aussi, ce bout de code va sacrément me servir !

04/04/2019, 08h40
Sam__
Citation:

Envoyé par renaud26

Qu'entends-tu par utiliser une transaction dans la requête UPDATE ?

Les transactions te permettent de t'assurer que toutes les lignes ont été modifiées sans erreur, sinon toute modification est annulée.

https://www.supinfo.com/articles/sin...solation-mysql

En l'occurence, sous WordPress :
Code:

1 2 3 4 5 6 7 8 $wpdb->query('START TRANSACTION'); $result = $wpdb->update(...); if ($result) { $wpdb->query('COMMIT'); } else { $wpdb->query('ROLLBACK'); }
05/04/2019, 14h37
renaud26

Merci pour ton aide !