Discussion :

[Neckara]

Bonjour,

Est-ce que quelqu'un pourrait m'expliquer comment un simple parcours de tableau par référence, sans rien faire, peut modifier le tableau ?
Je suis très perplexe, et ne comprends pas tout à fait ce qui se passe.

Quelle serait la bonne méthode pour le parcourir via référence ?

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
$test = [
        '0' => ['id'=>0],
        '1' => ['id'=>1]
    ];
 
    foreach($test as $id => &$res) {}
 
   var_dump($test);
 
    foreach($test as $key => $res) {
            //echo $key . ' : ' . $res['id'] . "\n";
    }
 
    var_dump($test);

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
array(2) {
  [0]=>
  array(1) {
    ["id"]=>
    int(0)
  }
  [1]=>
  &array(1) {
    ["id"]=>
    int(1)
  }
}
array(2) {
  [0]=>
  array(1) {
    ["id"]=>
    int(0)
  }
  [1]=>
  &array(1) {
    ["id"]=>
    int(0)
  }
}

[mathieu]

je pense qu'à la fin du 1er "foreach", $res contient un pointeur vers $test["1"].
et donc au 1er tour de la 2e boucle "foreach", $test["1"] est écrasé avec le contenu de $test["0"].

[Neckara]

je pense qu'à la fin du 1er "foreach", $res contient un pointeur vers $test["1"].

Ce qui est marrant, c'est qu'à la fin du 1er tour, le dernier élément semble avoir été remplacé par une référence.

Je pense avoir résolu le problème en ajoutant unset($res) à la fin de la première boucle.

[Celira]

Mathieu me parait avoir une bonne piste. Un test rapide avec un tableau avec un peu plus de données montre que visiblement c'est la dernière ligne qui est écrasée par l'avant-dernière

array (size=5)
  0 => 
    array (size=2)
      'id' => int 0
      'nom' => string 'A' (length=1)
  1 => 
    array (size=2)
      'id' => int 1
      'nom' => string 'B' (length=1)
  2 => 
    array (size=2)
      'id' => int 2
      'nom' => string 'C' (length=1)
  3 => 
    array (size=2)
      'id' => int 3
      'nom' => string 'D' (length=1)
  4 => 
    array (size=2)
      'id' => int 4
      'nom' => string 'E' (length=1)
// ----
array (size=5)
  0 => 
    array (size=2)
      'id' => int 0
      'nom' => string 'A' (length=1)
  1 => 
    array (size=2)
      'id' => int 1
      'nom' => string 'B' (length=1)
  2 => 
    array (size=2)
      'id' => int 2
      'nom' => string 'C' (length=1)
  3 => 
    array (size=2)
      'id' => int 3
      'nom' => string 'D' (length=1)
  4 => 
    array (size=2)
      'id' => int 3
      'nom' => string 'D' (length=1)

Sinon, une solution simple est d'utiliser des noms de variables différents

[mathieu]

Sinon, une solution simple est d'utiliser des noms de variables différents

ou alors ne plus utiliser les références

[grunk]

Mathieu à la bonne réponse ,
à la fin de ta première boucle $res est toujours une référence vers le dernier élément.La valeur est donc modifié par la première itération du deuxième foreach.

Il faut donc faire un unset($res) entre tes 2 boucles.

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
$test = [
    '0' => ['id'=>0],
    '1' => ['id'=>1]
];
 
foreach($test as $id => &$res) {}
 
var_dump($test);
 
unset($res);
 
foreach($test as $key => $res) {
        //echo $key . ' : ' . $res['id'] . "\n";
}
 
var_dump($test);

La question est de savoir si tu as vraiment besoin de faire une itération par référence. En 15 ans je crois que ca ne m'est jamais arrivé

[Neckara]

Merci pour vos réponses.

La question est de savoir si tu as vraiment besoin de faire une itération par référence. En 15 ans je crois que ca ne m'est jamais arrivé

Le but est de modifier les éléments du tableau lors de son parcours, donc par référence, c'est tout aussi bien.

[grunk]

Rien ne t'empèche de modifier sans passer par référence.
C'est un peu plus lourd à écrire mais ça évite ce genre de "bug" souvent difficile à identifier.

Et accessoirement passer par valeur est plus rapide que par référence (sauf dans le cas d'une écriture ou c'est kif kif)

[Neckara]

C'est un peu plus lourd à écrire mais ça évite ce genre de "bug" souvent difficile à identifier.

Justement, je suis très fainéant.

Et accessoirement passer par valeur est plus rapide que par référence (sauf dans le cas d'une écriture ou c'est kif kif)

La rapidité d'exécution du PHP n'est pas vraiment une contrainte pour moi.

J'utilise les références, juste pour les parcours, ou quand je veux qu'un élément "pointe" sur un autre, e.g. maStruct["c"] = &maStruct["a"]["c"].

[mathieu]

si on reprend l'exemple du 1er message et que vous vouliez absolument réutiliser le même nom de variable, vous pouvez mettre "unset($res);" juste après la 1re boucle "foreach".