Discussion :

[jubourbon]

Bonjour,

Je débute en php, et j'ai suivi des tutos pour apprendre, notamment un tuto sur les paniers sur ce site.

Mais voila j'ai des probleme avec certaines fonctions qui me renvoi des erreurs. Apres avoir verifié sur la doicumentation php, je ne vois vraiment pas le probleme.

Voici mon code:

Code :

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

ajouterArticle("Test1",2,12,8);

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
 
function ajouterArticle($libelleProduit,$qteProduit,$prixProduit){
 
   //Si le panier existe
   if (creationPanier() && !isVerrouille())
   {
      //Si le produit existe déjà on ajoute seulement la quantité
      $positionProduit = array_search($libelleProduit,  $_SESSION['panier']['libelleProduit']);
 
      if ($positionProduit !== false)
      {
         $_SESSION['panier']['qteProduit'][$positionProduit] += $qteProduit ;
      }
      else
      {
         //Sinon on ajoute le produit
         array_push( $_SESSION['panier']['libelleProduit'],$libelleProduit);
         array_push( $_SESSION['panier']['qteProduit'],$qteProduit);
         array_push( $_SESSION['panier']['prixProduit'],$prixProduit);
      }
   }
   else
   echo "Un problème est survenu veuillez contacter l'administrateur du site.";
}

Creation panier et isVerouille permettent de creer et verouiller le panier.

L'excution de ce code me renvoi les erreurs suivantes:
Warning: array_search() [function.array-search]: Wrong datatype for second argument in /Users/.../fonctions/PanierFunctions.php on line 29

Warning: array_push() [function.array-push]: First argument should be an array in /Users/.../fonctions/PanierFunctions.php on line 38

Mes arguments sont pourtant de bon type.

Merci par avance pour votre aide.

[sabotage]

utilise var_dump() pour controler ta variable.

[jubourbon]

J'ai utilisé var_dump, ma variable existe bien, mais il ne veut pas que je la remplisse

[ThomasR]

Le problème c'est juste que $_SESSION['panier']['libelleProduit'] n'est pas un tableau je pense, et array_search cherche dans... des tableaux.

Que renvoie ?

Code :

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

1
2
 
var_dump(is_array($_SESSION['panier']['libelleProduit']));

Tu as mal compris l'utilisation des tableaux :

Code :

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

1
2
3
4
5
6
 
// on ajoute un produit au panier
$_SESSION['panier'][] = array(
    'libelleProduit' => 'Cacahuètes',
    'prix' => 5
);

[jubourbon]

Salut,

Merci pour ta réponse. Je suis au boulot, je fais le test en rentrant ce soir et publie le résultat.

Mais pour info, j'ai bien suivi ce tuto pour faire mon panier et il me semble que mon panier est bien un array.

[ThomasR]

Je trouve que dans ce tuto la manière dont sont stockées les produits dans le panier n'est pas logique, même si cela fonctionne évidemment.

Aussi, $_SESSION['panier'] est surement un array, mais dans ton code $_SESSION['panier']['libelleProduit'] ne l'est clairement pas.

[RunCodePhp]

ThomasR +1

La construction de ton tableau de session de produit ne t'aide pas à la gestion de tout ça.

Code :

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

1
2
3
4
5
6
 
// on ajoute un produit au panier
$_SESSION['panier'][] = array(
    'libelleProduit' => 'Cacahuètes',
    'prix' => 5
);

Donc ici, il se créé un indice automatiquement pour chaque produit.
Le principe est donc déjà bien mieux, car il permet de mieux représenter la réalité, comme tous les produit auront tous 1 libellé, un prix, une quantité etc ...

Cependant, pour encore facilité l'accès à ce tableau, au lieu de faire accorder un indice automatique, l'idéal serait de mettre l'ID du produit.
Exemple lors de la création d'un produit :

Code :

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

1
2
3
4
5
6
7
8
 
// on ajoute un produit au panier
$produit_id = (int)$_POST['produit_id'];
 
$_SESSION['panier'][$produit_id] = array(
    'libelleProduit' => 'Cacahuètes',
    'prix' => 5
);

Ce qui fait que (par exemple) lors de la récupération d'un produit, en supposant que tu prévois de toujours t'appuyer sur l'ID du produit :

Code :

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

1
2
3
4
5
6
 
$produit_id = (int)$_POST['produit_id'];
 
$info_produit = $_SESSION['panier'][$produit_id];
echo 'Le prix : '.$info_produit['prix'];
echo 'Libellé : '.$info_produit['libelleProduit'];

Ca devient très simple, et dans tous les cas : Ajout, suppression, mise à jours, affichage.

Donc on a pas à ce prendre le choux à savoir la position du produit, l'ID du produit DOIT être un élément connu et indispensable à la gestion d'un panier.
Bref ... le principe fourni par le tuto me semble pas aussi une bonne solution, c'est fort dommage vu le travail accompli.

[jubourbon]

J'avais bien pigé ce tuto. Certes je trouve ca bourrin, mais comme je suis pas un pro du php, je l'ai pris tel quel.

Merci pour vos conseils, j'essaye de mettre ca en place ce soir.

[sabotage]

il me semble que

La réponse était dans le var_dump() justement.