Discussion :

[DeveloppeurDimanche]

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 :

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

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

est remplacé par cela :

Code :

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

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

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

-

Mais j'ai un doute avec ceci :

Code :

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

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

Est-ce que ce remplacement va fonctionner ?

Code :

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

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 :

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

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 :

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

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.

[cavo789]

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.

[DeveloppeurDimanche]

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.

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

[cavo789]

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.

[Séb.]

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

Code :

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

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 :

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

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 :

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

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 :

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

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

[DeveloppeurDimanche]

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.