probleme de boucle (foreach)

**papouuu86** · 20/10/2009, 14h51

Bonjour alors voila, j'éspere que mon post est adapté a cette section, a vrai dire c'est ici que j'ai pensé après avoir parcouru les autres..

Mon soucis est que j'ai un panier, tout fonctionne très bien si ce n'est que je n'ai pas ajouté de fonction qui incrémente la quantité d'un produit deja existant dans le panier ainsi le produit sera affiché 2 fois dans le panier jusque la aucun soucis mais lors de l'enregistrement de la commande dans ma BDD, j'ai une table "contient" qui récupère l'ID du produit contenu dans chaque commande, l'id de cette derniere et la quantité associée a chaque produit. Cependant le produit présent 2 fois dans le panier ne sera ajouté qu'une fois dans la BDD.

j'ai donc créé cette boucle de vérification qui fonctionne lorsque le dernier article ajouté correspond au nouveau mais si j'intercale un autre article entre alors le produit reviendra.

Exemple: si j'ajoute un clavier il est bien présent dans le catalogue,
si je réessaie d'intégrer le meme clavier l'ajout n'est pas effectué

Par contre si j'ajoute un clavier et ensuite une souris et que je
rajoute encore le meme clavier, il est bien intégré dans le panier...
c'est ce que je ne veux pas..

voici ma boucle:

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
$ref=$_POST['reference'];
 
	$cadd=$monpanier->getBasket();
	foreach ($cadd as $art)
	{ $ouch=$art;}
 
		$position=array_search($ref, $ouch);
 
		if ($position !== false)
			{					
				echo'votre article existe deja, veuillez utiliser les bouttons + et - pour ajuster la quantité désirée';				
			}
			else
			{
                                 echo'article ajouté';
$monpanier->addArticle(array('code'=>$_POST['reference'],'nom'=>$_POST['nom'],'prix'=>$_POST['prix'],'quantite'=>$_POST['quantite']));// on insere les valeurs dans le tableau "monpanier"
	}

**openaccess** · 20/10/2009, 15h13

utilise in_array() plutôt que array_search()
ou alors dans ta condition met

Code :

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

if ($position != false)

!= au lieu de !== car array_search peut te sortir 0, false ou "" en fonction des situations...

**papouuu86** · 20/10/2009, 15h54

merci pour ta réponse, j'utilise in_array() du coup

mais la vérification ne se fait toujours que sur le dernier article ajouté au panier et non sur le panier en entier...
ca fait 2 jours que je suis la dessus a faire des bidouilles par-ci par-là pour toujours le même résultat

si quelqu'un à une idée de mon erreur ou une piste de résolution vous êtes les bienvenus!!

**openaccess** · 20/10/2009, 16h24

Tu peux nous montrer le contenue de $monpanier->getBasket() avec ceci ?

Code :

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

1
2
 
print_r ($monpanier->getBasket())

**papouuu86** · 20/10/2009, 16h31

Bien sur voici le résultat avec 1 article dans le panier:

Array ( [0] => Array ( [code] => 88-RRF-43 [nom] => suite office [prix] => 500,00 [quantite] => 1 ) )

**openaccess** · 20/10/2009, 16h43

Si tu as seulement le dernier article, c'est à cause de la boucle

Code :

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

1
2
3
4
5
 
foreach ($cadd as $art)
{ 
	$ouch=$art;
}

la variable $ouch est écrasé à chaque itération de la boucle... Et donc ne conserve que la dernière valeur

Essaie plutot ceci

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
 
$ref = $_POST['reference'];
 
$cadd=$monpanier->getBasket();
foreach ($cadd as $art)
{ 
	if (in_array($ref, $art))
        {					
	      echo'votre article existe deja, veuillez utiliser les bouttons + et - pour ajuster la quantité désirée';				
        }
        else
        {
	      echo'article ajouté';
              $monpanier->addArticle(array('code'=>$_POST['reference'],'nom'=>$_POST['nom'],'prix'=>$_POST['prix'],'quantite'=>$_POST['quantite']));// on insere les valeurs dans le tableau "monpanier"
        }
}

**papouuu86** · 20/10/2009, 17h47

ah non j'ai deja essayé ca en fait.
ca doit pouvoir marcher mais avec un count sur le nombre d'enregistrements par ce que la il n'accepte même pas 1 article dans le panier puisque la vérification est effectuée pour chaque enregistrement du coup dès qu'on ajoute un article il va vérifier et interdire (= afficher "votre article existe deja, veuillez utiliser les boutons + et - pour ajuster la quantité désirée")

je vais essayer avec une boucle for avec $i<nombre d'articles

qui vérifiera sur $cadd[$i]

**papouuu86** · 21/10/2009, 10h16

Bon j'ai éssayé ca ce matin:

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
if(isset($_POST['reference'],$_POST['nom'],$_POST['prix'],$_POST['quantite']))
{	
	$ref=$_POST['reference'];
 
	$cadd=$monpanier->getBasket();
	$nb_art = $monpanier->getNbrArticle();
	$yo=$nb_art+1;
	for($i=0; $i<$nb_art; $i++)
	{
		$tabs=$cadd[$i];
	}
	$x=0;
	while($x<$yo)
	{
	$position=array_search($ref, $tabs);
	$x++;
	}
		if ($position !== false)
        {					
	      echo'votre article existe deja, veuillez utiliser les bouttons + et - pour ajuster la quantité désirée';				
        }
        else
        {
	      echo'article ajouté';
             $monpanier->addArticle(array('code'=>$_POST['reference'],'nom'=>$_POST['nom'],'prix'=>$_POST['prix'],'quantite'=>$_POST['quantite']));// on insere les valeurs dans le tableau "monpanier"
        } 
}

mais toujours pas de résultat, il me prend toujours la dernière valeur ajoutée
j'ai fais ca a la base pour pouvoir mettre

Code :

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

$position=array_search($ref, $tabs[$x])

mais il n'aime pas le [x]....
je pensais que ca allait me parcourir chaque liste de valeurs de $tabs mais non..
comment faire pour la vérification sur toutes les valeurs du tableau?

**openaccess** · 21/10/2009, 13h52

C'est toujours le même problème...
Ta boucle for écrase à chaque itération ta variable $tabs... Du coup elle n'a qu'un élément: le dernier de $cadd

**papouuu86** · 21/10/2009, 14h05

oui je sais ca m'énerveeeeeeeeeeeeeeeeee car si je mets tout dans ma boucle il n'accepte pas 1 seul enregistrement :'( je déséspere

**openaccess** · 21/10/2009, 14h35

dans ta boucle au lie de mettre

Code :

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

$tabs=$cadd[$i];

met

Code :

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

$tabs[]=$cadd[$i];

**papouuu86** · 21/10/2009, 14h56

j'ai aussi éssayé... sans résultat.... ou alors la vérification ne fonctionne même plus pour le dernier article... je pète un câble la...

sinon est-ce qu'il y a un équivalent de array_search mais pour les variables?
par ce que je pourrais éssayer:

foreach($tabs as $articles)

et faire la verif la dessus c'est peut-etre moins compliqué de le faire sur autre chose qu'un tableau...
j'aurais bientôt tout essayé
je vous dis pas quand j'aurais trouvé l'explosion de joie que je vais connaître!!

**micetf** · 21/10/2009, 15h22

Et quelque chose comme ça,
ça ne ferait pas l'affaire ?

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
$trouve=false;
foreach ($cadd as $art) {
	if (in_array($ref,$art)) {
		$trouve=true;
		break;
	}
}
if ($trouve) 
	echo'votre article existe deja, veuillez utiliser les bouttons + et - pour ajuster la quantité désirée';				
else {
	echo'article ajouté';
	$monpanier->addArticle(array('code'=>$_POST['reference'],'nom'=>$_POST['nom'],'prix'=>$_POST['prix'],'quantite'=>$_POST['quantite']));// on insere les valeurs dans le tableau "monpanier"
}

Fred