[Tableaux] Supprimer 1 élément particulier d'1 tableau

**harlock59** · 16/05/2006, 11h51

bonjour, je voudrais supprimer un element particulier d'un tableau.
je suis allé dans la faq php, mais le "unset" n'enleve pas l'élément que je voudrais enlever. merci pour votre aide.
(voici la page de la faq que j'ai consulté: http://php.developpez.com/faq/?page=...b_supprelement)

voici mon code:

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
<?
session_start();
 
$str= $_SESSION["tpl_panier"];
$tab=explode(" ",$str);
 
$compteur=count($tab);
 
echo "<table>";
 
for ($i=2;$i<$compteur;$i++)
    {   mysql_connect("localhost","root","") or die (mysql_error());
        mysql_select_db("test_gesstock") or die (mysql_error());
        $req="select DESIGNATION from produits_fournit_bureau where id=$tab[$i]";
 
        $sql=mysql_query($req);
        $result=mysql_fetch_row($sql);
 
 
        $j=$i-1;
 
            echo "<br>article ".$j.": ".$tab[$i]." ".$result[0]." ";
            echo "<a href=?enlever_art=".$tab[$i]."> retirer cet article</a>";
 
 
    }
echo $_GET['enlever_art'];
 
for ($i=2;$i<$compteur;$i++)
echo $tab[$i];
    {
    if ($_GET['enlever_art']==$tab[$i])
        {
        unset($tab[$i]);
        }
}
 
?>

**Fladnag** · 16/05/2006, 12h24

for ($i=2;$i<$compteur;$i++)
echo $tab[$i];
{
if ($_GET['enlever_art']==$tab[$i])
{
unset($tab[$i]);
}
}

le unset fonctionne tres bien pour les elements d'un tableau... a condition evidemment de passer dans la boucle ;o)

Là ton for ne fait qu'un echo a chaque tour de boucle...

**Fladnag** · 16/05/2006, 12h25

petite parenthese :

c'est entre autre pour ca que je prefere la syntaxe :

for (...) {
...
}

if (...) {
...
}

ca evite ce genre de problemes ;o)

Invité · 16/05/2006, 12h26

perso je ne suis pas fan des codes raccourci , je prefere tjs mettre des accolade et edentté mon code pour plus de clarté.

Code :

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

1
2
3
4
5
6
7
8
9
 
for ($i=2;$i<$compteur;$i++)
{
       echo $tab[$i];
       if ($_GET['enlever_art']==$tab[$i])
      {
           unset($tab[$i]);
      }
 }

**Yogui** · 16/05/2006, 12h33

Salut

La vraie question est : pourquoi as-tu besoin d'unset() ?

**harlock59** · 16/05/2006, 13h32

parceque je fais un systeme de caddy (ou panier d'achat) et je voudrais pouvoir suprimer l'un des articles insérés.

**harlock59** · 16/05/2006, 13h36

j'ai modifié mon script en mettant l'accolade avant le echo, mais ca ne change pas le fait que mon unset ne marche pas. merci pour l'aide.

**Fladnag** · 16/05/2006, 13h41

mon unset ne marche pas

c'est pas tres precis...

avant le for, ecrit :

echo "<pre>"; var_dump($ton_tableau); echo "</pre><br>";

et ecrit la meme chose apres le for.

Si tu as exactement la meme chose, c'est qu'il ne passe jamais dans le if, donc ta condition est mauvaise. Sinon, il passe dans le if est fait le unset (et tu en as la preuve)

**Fladnag** · 16/05/2006, 13h42

Envoyé par harlock59

le panier n'est pas dans une variable locale, elle est dans une
variable de session $_SESSION["tpl_panier"].

ben faut faire du unset($_SESSION....) sur l'item qui va bien alors...

**harlock59** · 16/05/2006, 13h45

oui, mais j'ai ça:

Code :

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

1
2
$str= $_SESSION["tpl_panier"];
$tab=explode(" ",$str);

et ça:

Code :

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

1
2
3
4
5
6
7
8
for ($i=2;$i<$compteur;$i++)
{
    echo $tab[$i];
    if ($_GET['enlever_art']==$tab[$i])
        {
        unset($tab[$i]);
        }
}

**Fladnag** · 16/05/2006, 14h07

Envoyé par harlock59

ça affiche ça:
NULL

25NULL

rassure moi... t'as bien mis $tab et pas $ton_tableau dans le var_dump ???
a mon avis non...

bon...
...
...
apres ton for...

$tab=array_values($tab);

et ajoute les fonctions inverses de ce que tu as fait au debut...

**harlock59** · 16/05/2006, 14h25

ça donne:
array(4) {
[0]=>
string(0) ""
[1]=>
string(0) ""
[2]=>
string(1) "2"
[3]=>
string(1) "5"
}

25array(3) {
[0]=>
string(0) ""
[1]=>
string(0) ""
[2]=>
string(1) "2"
}

**harlock59** · 16/05/2006, 14h37

la fonction inverse, ce serait l'inverse du explode ?

**Fladnag** · 16/05/2006, 14h51

Comme tu le voit grace aux var_dump, une case a bien été supprimée de ton tableau.

Maintenant, oui, il te faut faire la fonction inverse du explode, puis ecraser l'ancienne valeur dans $_SESSION pour la remplacer par la nouvelle valeur.

**harlock59** · 16/05/2006, 14h53

ok, merci.

**harlock59** · 16/05/2006, 14h58

maintenant, je suis obligé de cliquer 2 fois sur "retirer cet article" pour qu'il soit retiré.
voici mon code:

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
 
session_start();
 
 
$str= $_SESSION["tpl_panier"];
 
$tab=explode(" ",$str);
 
 
$compteur=count($tab);
 
echo "<table>";
if ($_SESSION["tpl_panier"]!="")
{
for ($i=1;$i<$compteur;$i++)
    {  
        mysql_connect("localhost","root","") or die (mysql_error());
        mysql_select_db("test_gesstock") or die (mysql_error());
        $req = "select DESIGNATION from produits_fournit_bureau where id='$tab[$i]'";
        $sql = mysql_query($req) or die (mysql_error());
 
        $result = mysql_fetch_row($sql);
 
            $j=$i+1;
            echo "<br>article ".$i.": ".$tab[$i]." ".$result[0]." ";
            echo "<a href='?enlever_art=".$tab[$i]."'> retirer cet article</a>";
 
 
    }
}
else
 {echo "votre panier est vide";}
 
 for ($i=1;$i<$compteur;$i++)
{
 
    if ($_GET['enlever_art']==$tab[$i])
        {
        unset($tab[$i]);
        $_SESSION["tpl_panier"]=implode (" ",$tab);
        }
}
 
?>

quelqu'un saurait il comment faire pour ne pas avoir à cliquer 2 fois pour retirer un article du paniuer ? merci.

**Yogui** · 17/05/2006, 10h15

Salut

Plutôt que :

Code :

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

1
2
3
4
5
6
7
8
9
 for ($i=1;$i<$compteur;$i++)
{
 
    if ($_GET['enlever_art']==$tab[$i])
        {
        unset($tab[$i]);
        $_SESSION["tpl_panier"]=implode (" ",$tab);
        }
}

Pourquoi ne pas faire qqch comme :

Code :

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

1
2
3
4
if(!empty($_GET['enlever_art']) and (is_numeric($_GET['enlever_art']))){
   unset($tab[ intval($_GET['enlever_art']) ]);
   $_SESSION['tpl_panier'] = implode (' ', $tab);
}

Remarque générale : tu peux stocker des tableaux dans ta session, ça t'éviterait de faire des implode() et explode() à tout bout de champ. Tu pourrais aussi utiliser directement tes variables de session plutôt que de passer par un tableau temporaire.