Transformer des list = each en foreach pour PHP 8

Bonjour,

Je suis un développeur PHP du dimanche (comprendre, je fais juste des bidouilles mineures) et j'essaie de convertir un vieux logiciel PHP5 pour PHP8.

Notamment il faut remplacer toutes les instructions each.

La majorité du temps, j'utilise ce modèle :

Code:

---

while (list($key, $value) = @each($_POST['form']))

---

est remplacé par cela :

Code:

---

foreach ($_POST['form'] as $key => $value)

---

J'en ai corrigé des dizaines dans le style.

-

Mais j'ai un doute avec ceci :

Code:

---

1 2 3

while (list($key) = each($str)) { $str[$key] = $this->xss_clean($str[$key]); }

---

Est-ce que ce remplacement va fonctionner ?

Code:

---

1 2 3

foreach ($str as $key => $value) { $str[$key] = $this->xss_clean($str[$key]); }

---

Il y a le même type de code plus loin :

Code:

---

1 2 3

while (list($key) = each($data)) { if ( is_array($data[$key])) continue; if ($data[$key] != 'TRUE' && $data[$key] != 'FALSE' && ($key != $data[$key] OR $embedded_tags !== TRUE))

---

Que je remplacerais par :

Code:

---

1 2 3

foreach ($data as $key => $value) { if ( is_array($data[$key])) continue; if ($data[$key] != 'TRUE' && $data[$key] != 'FALSE' && ($key != $data[$key] OR $embedded_tags !== TRUE))

---

Vous me direz, il suffit de tester, mais ces bouts de code se trouvent dans des méandres de librairies, je ne sais pas à quel moment ils sont appelés...

Merci d'avance à celui ou ceux qui me confirmeront ou non ces changements, sans doute triviaux pour un vrai développeur PHP.

Bonjour

Comme tu es étudiant alors je me permets de te répondre avec quelques outils qui demandent un peu de temps à prendre en main mais, misère de misère, comme c'est GRANDIOSE de travailler de la sorte.

Regarde p.ex. cette petite vidéo : https://spatie.be/videos/front-line-...8-using-rector

Cela fait exactement ce que tu demandes; convertir un code legacy et le porter sur PHP 8.

L'exemple montre un seul fichier mais que ce soit un ou des dizaines de milliers, cela fonctionnera. Tout en automatique et sans aucune erreur possible !

La magie se nomme rector.

Dans l'exemple, il est fait mention de Docker; autre outil absolument fabuleux qui permet, simplifions, d'installer un environnement de travail sur ta machine; cela sous la forme d'un unique fichier nommé "container". Tu peux donc installer Rector sans te prendre la tête d'installer Rector ;), juste de l'utiliser au travers d'un container ayant été créé et configuré par l'équipe derrière l'outil.

Simple, rapide et méchamment efficace. Reste qu'il faut investir du temps à apprendre Docker et rector mais, si tu es étudiant en informatique, c'est VRAIMENT ce qu'il faut faire. Ces outils te seront fortement utiles dans le futur.

Bonne journée.

Merci cavo789 d'avoir pris le temps de répondre.

J'ai mis étudiant à la va vite, mes études d'informatiques datent d'il y a 25 ans et ça fait longtemps que je ne suis plus professionnellement dans le milieu. :D

Je vais regarder l'outil que tu suggères, je ne connaissais pas et je me complique la vie pour rien !

Citation:

---

Envoyé par DeveloppeurDimanche

J'ai mis étudiant à la va vite...

---

Dans notre milieu, même si on n'est plus actif à 100% dans la branche, on est toujours étudiant ;) C'est le propre de l'informaticien; sans cesse renouveler et acquérir de nouvelles compétences.

Bon dev et remise à l'étrier.

while/list/each c'est plutôt du PHP3, un des grands apport de PHP4 ayant été foreach ;)

Code:

---

1 2 3

while (list($key) = each($str)) { $str[$key] = $this->xss_clean($str[$key]); }

---

Le but est de parcourir une chaîne et d'appliquer xss_clean() sur chaque caractère ? Si oui ça devait déjà être bugué sur les chaînes aux caractères multi-octets comme UTF-8.

Tu peux essayer :

Code:

---

1 2 3 4 5 6

$new_str = ''; $n = mb_strlen($str); for ($i = 0; $i < $n; $i++) { $new_str .= $this->xss_clean(mb_substr($str, $i, 1)); // Reste à voir si xss_clean() prend en charge UTF-8 } $str = $new_str;

---

Pour celui-ci :

Code:

---

1 2 3

while (list($key) = each($data)) { if ( is_array($data[$key])) continue; if ($data[$key] != 'TRUE' && $data[$key] != 'FALSE' && ($key != $data[$key] OR $embedded_tags !== TRUE))

---

Essaie ceci :

Code:

---

1 2 3 4 5 6 7 8

foreach ($data as $key => $value) { if (is_array($value)) { continue; } if ($value != 'TRUE' && $value != 'FALSE' && ($key != $value || $embedded_tags !== true)) { ... } }

---

Bon courage

Citation:

---

Envoyé par Séb.

Le but est de parcourir une chaîne et d'appliquer xss_clean() sur chaque caractère ? Si oui ça devait déjà être bugué sur les chaînes aux caractères multi-octets comme UTF-8.

---

Merci Seb !

C'est effectivement du code qui date de 2005, et la BDD n'est pas en UTF-8.

C'est tout à fait anachronique, mais ça fonctionne bien, sauf quand il y a des migrations de PHP à gérer !

Bonne journée à toi.