Discussion :

[lolodev]

bonsoir

voici ma méthode : (tiréé de la classe cart de webcart.com)

Code :

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

1
2
3
4
5
6
7
8
9
10
11
function add_item($itemid,$qty=1,$price = FALSE, $info = FALSE, $datej = FALSE)
	{ // adds an item to cart
 
			$this->items[]=$itemid;
			$this->itemqtys[$itemid] = $qty;
			$this->itemprices[$itemid] = $price;
			$this->iteminfo[$itemid] = $info;
			$this->itemdate[$itemid] = $datej;
			$this->itemcount++;
		//}
 }

j'appelle cette classe via:
${'ka'}->add_item($product['id'],1,$product['price'],$product['code'],$datelib);

"ka" étant une session de mon panier.

je ne comprends pas pourquoi lorsque j'ai deux produits avec le même ID, la méthode met à jour chaque item[id] avec les même valeurs INFO, DATEJ que celle le produit ajouté.

en fait,

Code :

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

$this->items[]=$itemid;

concatenne deux items identiques
et donc modifie la valeur du precedent item

que dois je faire ??
array_push ne change rien ..

merci

(j'espère être clair)

[Séb.]

en fait,

Code :

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

$this->items[]=$itemid;

concatenne deux items identiques
et donc modifie la valeur du precedent item

que dois je faire ??
array_push ne change rien ..

Comment ça "concatène" ? (on parle de concaténation pour des châines de caractères)
La commande $items[ ] = $itemid *empîle* $itemid sur le tableau $items.

je ne comprends pas pourquoi lorsque j'ai deux produits avec le même ID, la méthode met à jour chaque item[id] avec les même valeurs INFO, DATEJ que celle le produit ajouté.

Je ne vois pas de item[id] dans le snippet fourni.
Mais tu veux peut-être dire que qque chose comme :

Code :

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

1
2
$panier->add_item(123, 1) ;
$panier->add_item(123, 1) ;

Ne donne pas une quantité de l'article #123 à 2 mais à 1, est-ce cela ? Auquel cas c'est "normal", le script est fait comme cela :

Code :

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

$this->itemqtys[$itemid] = $qty;

PS :

"ka" étant une session de mon panier.

Y'a-t-il un intérêt à procéder de la sorte ?

[lolodev]

oui et même:

Code :

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

1
2
$panier->add_item(123, 1, p1) ;
$panier->add_item(123, 1,p2) ;

et mon pb : lorsque je lis $panier, le nombre d'element du panier est bien egale à 2, mais le code produit de l'element 1 à été modifie par p2.

pourquoi le code du premier element a t il ete modifié ?

(le fait de la quantité qui n'increment pas est normal, c'est un choix que j'ai fait)

[Séb.]

Code :

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

1
2
$panier->add_item(123, 1, p1) ;
$panier->add_item(123, 1,p2) ;

et mon pb : lorsque je lis $panier, le nombre d'element du panier est bien egale à 2, mais le code produit de l'element 1 à été modifie par p2.

Parceque itemcount a été incrémenté, mais en réalité il n'y en a qu'un :

Code :

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

1
2
$this->items[]=$itemid;
$this->itemqtys[$itemid] = $qty;

Après les deux add_item( ) $this->items vaut [ 123, 123 ], $this->itemqtys vaut [ 123 => 1 ].
=> C'est incohérent.
N'hésite pas à explorer ton instance avec print_r( ) / var_dump( ) afin de bien comprendre comment elle fonctionne.

pourquoi le code du premier element a t il ete modifié ?

Le principe est le même que présenté plus haut : tu identifies tes articles avec un ID, à chaque ajout l'éventuelle entrée précédente est écrasée par la nouvelle.
Que voudrais-tu avoir au final ?

Si je puis me permettre, à ta place je reverrai le fonctionnement du script qui ne m'a pas l'air très souple tel quel.

[lolodev]

bj

j'avoue avoir du mal à comprendre ce phénomène ...

tu identifies tes articles avec un ID, à chaque ajout l'éventuelle entrée précédente est écrasée par la nouvelle.
Que voudrais-tu avoir au final ?

mais celà n'est valable que si ID existe déjà dans la tableau

Que voudrais-tu avoir au final ?

et bien lorsque j'ajoute un ID qui existe déjà dans mon tableau item[], je veux qu'il ne modifie pas les autres ID et qu'il l'ajoute come un ID différent

merci

[Séb.]

celà n'est valable que si ID existe déjà dans la tableauet bien lorsque j'ajoute un ID qui existe déjà dans mon tableau item[]

C'est bien là le fond du pb. Mais comment se fait-il que tu veuilles avoir deux fois le même produit au sein du panier ? Ne serait-il pas plus pertinent d'incrémenter la quantité de l'existant ?

je veux qu'il ne modifie pas les autres ID et qu'il l'ajoute come un ID différent

C'est impossible avec la structure que tu as adopté puisque tu références les quantités et d'autres propriétés avec l'ID.

Pour éviter ce genre de conflit il faudrait TOUT regrouper au sein du même tableau :

Code :

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

1
2
3
4
function add_item(...)
{
    $this->items[ ] = array('id' => $id, 'qty' => ...) ;
}