Discussion :

[RowanMayfair]

Bonjour,

Je sèche, sans comprendre ce qui cloche.

J'ai 2 arrays :

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
46
$foo =
array (size=436)
  0 => 
    array (size=3)
      'id' => string '2459' (length=4)
      'prenom' => string 'bird' (length=4)
      'nom' => string 'Jean' (length=4)
 
  1 => 
    array (size=3)
      'id' => string '2444' (length=4)
      'prenom' => string 'Luc' (length=3)
      'nom' => string 'CestPasMoi' (length=10)
 
  2 => 
    array (size=3)
      'id' => string '2474' (length=4)
      'prenom' => string 'Steph' (length=5)
      'nom' => string 'CestMoi' (length=7)
  3 => 
    array (size=3)
      'id' => string '2446' (length=4)
      'prenom' => string 'Rasta' (length=5)
      'nom' => string 'CestLui' (length=6)
 
  4 => 
    array (size=3)
      'id' => string '2537' (length=4)
      'prenom' => string 'Thia' (length=4)
      'nom' => string 'CestElle' (length=8)
 
 
$bar =
array (size=8)
  0 => 
    array (size=2)
      'idCherche' => string '8' (length=1)
      'idUneAutre' => string '2459' (length=4)
  1 => 
    array (size=2)
      'idCherche' => string '8' (length=1)
      'idUneAutre' => string '2474' (length=4)
  2 => 
    array (size=2)
      'idCherche' => string '8' (length=1)
      'idUneAutre' => string '2537' (length=4)

Dans $foo['id'], je voudrai supprimer les entrées qui correspondent à celles qui sont dans $bar['idUneAutre'].

J'ai donc fait un truc comme ça :

Code :

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

1
2
3
4
5
6
7
8
9
foreach($foo as $unFoo) {
        foreach($bar as $leBar) {
            if ($bar['idUneAutre'] == $foo['id']) {
                unset($unFoo, $foo);
            }
        }        
    }
 
    return $foo;

Mais ça ne fonctionne pas Wamp me répond

Undefined variable: unFoo in c:\blablabla on line 145

ça ne fonctionne pas comme ça unset ?
où c'est autre chose ?

D'avance merci pour vos éclairages

[laurentSc]

Bonjour,
il me semble que unset attend un seul paramètre, donc essaye unset($foo['id']);

[RowanMayfair]

Non ça ne fonctionnait pas non plus.

Mais je viens de trouver une solution

Code :

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

1
2
3
4
5
6
7
8
 
for($i = 0; $i < count($foo); $i++) {
        for ($j = 0; $j < count($bar); $j++) {
            if($foo[$i]['id'] == $bar[$j]['idUneAutre']) {
                array_splice($foo, $i, 1);
            }
        }
    }

Déjà une boucle foreach n'était pas forcément adaptée.
Et ensuite, array_splice c'est top en fait, ça te fait le petit découpage qui va bien.

Merci

[laurentSc]

Ca s'invente pas ; comment as-tu trouvé ?

[RowanMayfair]

Ca s'invente pas ; comment as-tu trouvé ?

Euh ben si ça s'invente drôle de remarque.
J'ai passé 1 heure à tripoter mon bout de code dans tous les sens, avec des var_dump avant et après.

En fait en parcourant avec une boucle for déjà ça m'a paru + clair (pour moi en tout cas), et sinon avec une recherche google j'avais lu qu'il y avait 2 solutions pour supprimer des éléments dans un array : unset et array_splice

Donc ben voilà ....

[CosmoKnacki]

Parcourir un tableau avec des boucles for, c'est un peu beaucoup la loose depuis l'apparition de foreach en php4, et même si ça fonctionne avec array_splice cette fonction est plutôt destinée à enlever un pan contiguë d'items d'un tableau que des items un par un.

Il faut bien comprendre que par défaut une boucle foreach travaille sur des copies des items du tableau par sur les items eux-mêmes, c'est pourquoi unset($unFoo); n'effacera que la copie de l'item courant dans la boucle, mais que le tableau $foo restera inchangé.

Il est possible de demander dans une boucle foreach de ne pas renvoyer une copie de l'item, mais bien une référence vers l'item du tableau en plaçant & devant le nom de la variable:

Code :

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

1
2
3
4
5
6
7
8
foreach($foo as &$unFoo) {
    foreach($bar as $leBar) {
        if ( $leBar['idUneAutre'] == $unFoo['id'] ) {
            unset($unFoo);
            break; // l'item a été enlevé, on interrompt la boucle sur $bar
        }
    }        
}

Néanmoins, pas mal de développeurs n'aiment pas utiliser les références ou s'en méfient et préfèrent utiliser la clef de l'item que tu peux obtenir via foreach:

Code :

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

1
2
3
4
5
6
7
8
foreach($foo as $k => $unFoo) {
    foreach($bar as $leBar) {
        if ( $leBar['idUneAutre'] == $unFoo['id'] ) {
            unset($foo[$k]);
            break; 
        }
    }        
}

---

Les boucles imbriquées c'est pas la panacée, car avec elles on augmente très vite les temps de traitements ( m items de tableau1 X n items du tableau2 ).
Ce que tu peux faire à la place c'est extraire une bonne fois pour toutes les ids à enlever de $bar grâce à array_column puis de retourner le tableau obtenu avec array_flip. De cette manière, pour contrôler qu'une id est présente dans le tableau obtenu, il suffit de regarder si la clef (donc l'id) existe avec isset:

Code :

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

1
2
$ids = array_flip(array_column($bar, 'idUneAutre'));
$result = array_filter($foo, function ($item) use ($ids) { return !isset($ids[$item['id']]); });

Et à partir de PHP 7.4, tu peux même écrire cette deuxième (et dernière) ligne comme ça:

Code :

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

$result = array_filter($foo, fn($item) => !isset($ids[$item['id']]));

C'est pas beau la vie?