Beste Allemaal,

Ik ben bezig met het maken van een eigen webshop voor school. Ik loop echter op dit moment tegen twee problemen aan:

Probleem 1:

Na een hoop gepiel is het me gelukt om het aantal van een product in de cart aan te passen en dit aantal door te voeren in de cart_array session. Maar er treedt een probleem op als ik de aantallen van verschillende producten tegelijk wil aanpassen, want dan pakt hij het nieuwe ingevulde aantal en voert die door op alle producten. Dus als ik bijv. product 1 aantal aanpas naar 5, past hij het aantal in de session van product 2,3,4 etc. ook aan naar 5. Ik weet dat de fout in het pakken van het nieuwe ingevulde aantal zit, want hij pakt dus maar één aantal en voert die door voor alle producten, maar ik weet niet hoe ik het kan oplossen.

Probleem 2:
Mijn tweede probleem komt bij het verwijderen van de verschillende producten uit de winkelwagen. Het verwijderen gaat goed, het lukt om er één te verwijderen, maar ook meerdere tegelijk. Echter als je bijv. het eerste product verwijdert uit de $_SESSION['cart_array'] en je hebt bijv. nog 3 andere producten erin staan, worden de indexen: 1-2-3, terwijl het eerst 0-1-2-3 was. En dan kloppen mijn checkboxen niet meer, want als je dan weer het eerste product wil verwijderen, wil hij de 0-index pakken, maar die is er dus niet meer. Ik heb de regel sort($_SESSION['cart_array']; onder de regel unset($_SESSION['cart_array'][''.$remove_id.'']); proberen te plakken, maar dan verwijdert hij gewoon alle indexen. Hoe kan dit?

Dit is de code:

<?php
/////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////
//       Section 1 (if user attempts to add something to the cart from the product page)
/////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////
if(isset($_POST['pid']))
{
    $pid = $_POST['pid'];
    $wasFound = false;
    $i = 0;
    // If the cart session variable is not set or cart array is empty
   	if (!isset($_SESSION['cart_array']) || count($_SESSION['cart_array']) < 1) { 
    // RUN IF THE CART IS EMPTY OR NOT SET
    $_SESSION['cart_array'] = array(0 => array('product_id' => $pid, 'qty' => 1));
	} else {
		// RUN IF THE CART HAS AT LEAST ONE ITEM IN IT
		foreach ($_SESSION['cart_array'] as $each_item) { 
            $i++;
		      while (list($key, $value) = each($each_item)) {
				  if ($key == "product_id" && $value == $pid) {
					  // That item is in cart already so let's adjust its quantity using array_splice()
					  array_splice($_SESSION["cart_array"], $i-1, 1, array(array("product_id" => $pid, "qty" => $each_item['qty'] + 1)));
					  $wasFound = true;
				  } // close if condition
		      } // close while loop
	       } // close foreach loop
   		       if ($wasFound == false) {
			   array_push($_SESSION["cart_array"], array("product_id" => $pid, "qty" => 1));
                }
        
    }
   	header("location: cart.php"); 
    exit();
}
?>

<form action="" method="post" enctype="multipart/formdata">
                        
<table align="center" width="700" bgcolor="skyblue">
                                
<tr align="center">
    <th>Remove</th> 
    <th>Product(s)</th> 
    <th>Quantity</th> 
    <th>Total Price</th>   
</tr>
<?php

/////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////
//       Section 4 (render the cart for the user to view)
/////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////
$cartOutput = '';
$cartTotal = '';
if (!isset($_SESSION['cart_array']) || count($_SESSION['cart_array']) < 1) { 
    echo $cartOutput = "<h2 align='center'>Your shopping cart is empty</h2>";
    } else {
        $i = 0;
        foreach ($_SESSION['cart_array'] as $each_item) { 
                                                
        $product_id = $each_item['product_id'];
                                                
        $sql = $db->prepare("SELECT * FROM products WHERE product_id = '$product_id' LIMIT 1");
        $sql->execute();
                                                
        while ($row = $sql->fetch()) {
        $product_price = $row['product_price'];
        $product_title = $row['product_title'];
        $product_image = $row['product_image'];

        }
                                                
        $price_total = $product_price * $each_item['qty'];
        $cartTotal = $price_total + $cartTotal;
        $i++;
        
// Items verwijderen
if(isset($_POST["remove_cart"]))
    {    
                                    
    // Access the array and run code to remove that array index
    foreach($_POST['remove_item'] as $remove_id)
        {
            unset($_SESSION['cart_array'][''.$remove_id.'']);
        } 
            header('location: cart.php');
    }
                                    
// Itemaantallen aanpassen                           
if (isset($_POST['item_to_adjust']) && $_POST['item_to_adjust'] != "") {
$item_to_adjust = $_POST['item_to_adjust'];
$quantity = $_POST['qty'];
$i = 0;
foreach($_POST['item_to_adjust'] as $adjust_id)
    {
        $i++;
        array_splice($_SESSION["cart_array"], $i-1, 1, array(array("product_id" => $adjust_id, "qty" => $quantity)));                         
    } 
 }                                   
?>
<tr align="center">
    <td>
    <td><input type="checkbox" name="remove_item[]" value="<?php echo $i - 1;?>" /></td>
    <td><?php echo $product_title; ?><br /> 
    <img src="admin_area/product_images/<?php echo $product_image;?>" width="60" height="60" /> </td>
    <input name="item_to_adjust[]" type="hidden" value="<?php echo $each_item['product_id'];?>" />
    <td><input type="text" size="4" name="qty" value="<?php echo $each_item['qty'];?>" maxlength="1" /></td>
    <td><?php echo '&euro;'.$price_total; ?></td>                               
</tr>

<?php       
                            
            }
                } 
                                    
?>

<tr align="right">
<td colspan="4"><b>Sub Total:</b></td>
<td colspan="4"><?php echo '&euro;'.$cartTotal; ?></td>
</tr>
                            
<tr align="center">
    <td colspan="2"><input type="submit" name="remove_cart" value="Remove Item(s)" /></td>
    <td></td><input name="adjustBtn" type="submit" value="Update" /></td>
    <td><input type="submit" name="continue" value="Continue shopping" /></td>
    <td><a href="checkout.php"><button>Checkout</button></a></td>
    <td><a href="cart.php?cmd=emptycart">Empty Cart!</a></td>
                            
</tr>
                            
</table>
                        
</form>


Ik kom er maar niet uit!

Alvast bedankt, Mike
Waarom een loze dimensie toevoegen met nietszeggende indexen?

Voeg direct onder $_SESSION['cart_array'] key-value paren toe met als key het product id, en als value de hoeveelheid. Dat is alle informatie die je nodig hebt.

Dus 5 producten van product met id 3 zitten als volgt in de sessie:
(edit: andersom dus :s)
$_SESSION['cart_array'][3] = 5;

In plaats van
$_SESSION['cart_array'][??? onbekende index ???][3] = 5;

De problemen ontstaan ook doordat je [??? onbekende index ???] verwart met het product id volgens mij.
Nu je het zegt is dat inderdaad overbodig. De enige informatie die ik nodig heb is het product_id en het aantal ervan. Ik heb het nu zo gedaan:
<?php if(isset($_POST['pid']))
{
    $pid = $_POST['pid'];
    $qty = 1;
    $wasFound = false;
    // If the cart session variable is not set or cart array is empty
   	if (!isset($_SESSION['cart_array']) || count($_SESSION['cart_array']) < 1) { 
    // RUN IF THE CART IS EMPTY OR NOT SET
    $_SESSION['cart_array'] = array($pid => $qty);
    $wasFound = true;
	} 
    
    // als cart niet leeg is, push item in de sessie                           
    if ($wasFound == false) 
    {
        foreach($_SESSION['cart_array'][$pid] as $checkItem)
        {
            if ($checkItem == $pid)
            {
                $_SESSION['cart_array'][$pid][$qty]++;
            }
            echo 'lol';
        }
        
        $_SESSION['cart_array'][$pid] = $qty;
    } 
}
?>


Nu zet hij alle producten mooi onder elkaar met de aantallen als values. Maar nu kom ik er even niet uit, hoe ik ervoor kan zorgen dat als iemand hetzelfde product nog een keer toevoegt, dat de quantity dan met 1 opgehoogd wordt.
Hij voert de foreach loop ook niet uit, want hij echo't de lol ook niet. Hoe kan dat?

Alvast bedankt, Mike
Bestudeer de constructie van de foreach() nog eens goed, ik denk dat hier de verwarring zit.

Er zijn in principe twee vormen:

<?php
foreach ($array as $key => $value) {
    // ...
}
?>

Dus met twee "argumenten": as A => B, waarbij A de huidige key is van $array, en B de value, en
<?php
foreach ($array as $value) {
    // ...
}
?>

Dus met één "argument": as B, waarbij B de huidige value is, de keys worden in deze variant verder buiten beschouwing gelaten.

Kijk nu nog eens naar je code. $checkItem representeert op dit moment in jouw code een value (een hoeveelheid), en geen key (geen product id).
Wat dacht je van

$product_id = 1;
$amount = 3;

if(array_key_exists($product_id, $_SESSION['cart_array']))
  $_SESSION['cart_array'][$product_id] = $_SESSION['cart_array'][$product_id] + $amount;
else
  $_SESSION['cart_array'][$product_id] = $amount;


Weet je trouwens zeker dat je een winkelwagen als sessie wil opslaan?
De gebruiker verliest de producten zodra deze de browser afsluit.


---

regel 105 mis je trouwens je [] voor name="qty" => name="qty[]"
@Dennis, na invoering van jouw stukje werkt hij eindelijk!
Maar wat was dan precies het verschil tussen deze twee stukjes:
<?php             if (isset($_SESSION['cart_array'][$pid]))
                {
                    echo 'Hij bestaat';
                    $_SESSION['cart_array'][$pid] = $qty++;
                }
         
        $_SESSION['cart_array'][$pid] = $qty; ?> 

Dit had ik namelijk voordat ik jou stukje invoerde en hij deed het wel, maar hij telde het aantal maar tot maximaal 2 bij. Ik weet ook waarom, want telkens als $_POST['pid'] wordt gezet wordt $qty weer op 1 gezet, dus kan hij maximaal tot 2 ophogen. Maar ik krijg het niet helemaal helder in mijn hoofd wat er anders is aan jouw stukje. Ik denk dat het nu wel werkt door de [$pid] + $qty i.p.v. [$pid] = $qty++;, is dat correct?

<?php         if(array_key_exists($pid, $_SESSION['cart_array'])) {
        $_SESSION['cart_array'][$pid] = $_SESSION['cart_array'][$pid] + $qty;
        } else { 
        $_SESSION['cart_array'][$pid] = $qty; ?> 


Over of ik de winkelwagen in een sessie wil opslaan, ik dacht van wel, want ik wil sowieso niet de producten meteen in de database opslaan. Want als een klant uiteindelijk niet besteld, staan er overbodige producten in de database en daarom heb ik ervoor gekozen om het op te slaan in iets tijdelijks. Of is er een betere mogelijkheid?

In ieder geval erg bedankt voor de reacties, want het werkt nu!

Mvg, Mike
Hi Mike,

ik heb niet je hele code bekeken, maar een stukje van mijn cart class gegeven.

Zoals ik ook al aangaf, vergeet je de [] bij de qty in je formulier.
Hierdoor heb je uiteindelijk maar 1 $_POST['qty']
Voor producten gebruik je wel []

Je hebt een array: $_POST['item_to_adjust'] maar geen array $_POST['qty'] door de bovengenoemde reden.
Daarom veranderen zowiezo alle producten naar die ene zelfde $_POST['qty'], met de waarde wat bij je laatste product staat


met [$pid] = $qty++; zal nooit werken. Ik denk dat je $qty++; alleen apart kan gebruiken

$a = 10;
$b = 0;

$b = $a++;
echo $b;

Dit werkt ook niet :) Dan is $b gewoon 10.
Maar doe ik nou

$a = 10;
$b = 0;

$a++; // <--

$b = $a; // <--
echo $b;

Dan is $b wel 11

Ik weet niet hoe ik dit goed moet uitleggen, wellicht dat een van de 'main' leden hier op het forum dit beter kan uitleggen.

Ikzelf heb gekozen om een cart in een cookie op te slaan, dit is volledig aan de gebruiker in zijn browser.
Uiteraard wil je geen carts van gebruikers bijhouden in je database, maar ik denk dat potentiele klanten het fijn vinden als hun winkelwagen nog bestaat wanneer ze weer eens terug komen.
Zo maak ik ook een 'wishlist' aan.

Dit doe ik met $_COOKIE en die laat ik voor max. 30 dagen bewaren.
Dit werkt bijna hetzelfde als $_SESSION, maar ik krijg geen array klakkeloos in een COOKIE, daarvoor gebruik ik dan json_encode($myCartArray); om op te slaan
@Dennis
Allereerst, ik denk niet dat het vaak zal voorkomen dat je je browser "per ongeluk" afsluit tijdens het shoppen. Daarbij hangt het van meerdere instellingen af of "de gebruiker (direct) de producten verliest zodra deze de browser afsluit.", zowel aan de serverside als de clientside. Mijn browser herstelt bijvoorbeeld alle tabs bij het opnieuw opstarten (en de cookies met sessie id's die op zouden moeten houden met bestaan indien de browser afsluit -lifetime 0- zijn er ook nog steeds, ik weet nog steeds niet of je dit als een browserbug zou kunnen beschouwen). Dan is er ook nog andere functionaliteit om producten die je wilt hebben, te onthouden. Denk bijvoorbeeld aan een wishlist ofzo. Niet alles hoeft in je cart te zitten.

Ik weet niet of je voor de opslag van je cart cookies zou moeten gebruiken. Ik zou dit niet doen. Je ontsluit hiermee namelijk een stukje "interne administratie". Dit hoort simpelweg niet aan de gebruikerskant thuis als je het mij vraagt.

Dan snap ik het advies niet helemaal: predik je nu voor of tegen het gebruik van databases voor de opslag van carts? Er zijn trouwens ook sessiemanagementsystemen die via de database opereren, in plaats van de normale -file-based- sessies :).

On a sidenote:
$a++ wil zeggen "post increment", je hoogt hierbij de waarde van $a op nadat je deze hebt uitlezen / toegekend. Hierbij heeft de variabele aan welke je de waarde toekent dus de oude waarde van voor het ophogen.
++$a wil zeggen "pre increment", je hoogt hierbij de waarde van $a (direct) op voordat je deze waarde mogelijk toekent aan een andere variabele. In dat geval heeft de variabele aan welke je deze waarde toekent direct de nieuwe waarde van $a.
Als je dit soort verwarring wilt voorkomen, gebruik dan gewoon $a + 1 in plaats van fancy shorthands waarvan je de werking niet helemaal duidelijk is.
Dennis WhoCares op 18/03/2016 07:04:18

Ikzelf heb gekozen om een cart in een cookie op te slaan, dit is volledig aan de gebruiker in zijn browser.
Uiteraard wil je geen carts van gebruikers bijhouden in je database, maar ik denk dat potentiele klanten het fijn vinden als hun winkelwagen nog bestaat wanneer ze weer eens terug komen.
Zo maak ik ook een 'wishlist' aan.

Dit doe ik met $_COOKIE en die laat ik voor max. 30 dagen bewaren.
Dit werkt bijna hetzelfde als $_SESSION, maar ik krijg geen array klakkeloos in een COOKIE, daarvoor gebruik ik dan json_encode($myCartArray); om op te slaan


Wat als ik cookies uitschakel, of ze opruim bij het sluiten van de browser?
En waarom zou je de cart niet in de database willen opslaan?
In m'n privacy en cookie statement benadruk ik dat mijn site cookies nodig heeft voor bepaalde functies van m'n site.

Omdat ik daarvoor gekozen heb.
Ik ga geen carts en wishlists bewaren in m'n database op basis van ip adressen en/of browser fingerprints

(aangezien niet alle klanten een account nemen/willen)
Overigens krijg je een popup als je cookies niet aan hebt staan :)
Omdat ik daarvoor gekozen heb.

Dit is geen onderbouwing.

(aangezien niet alle klanten een account nemen/willen)

Ik weet niet helemaal zeker of die mensen het dan ook kunnen waarderen indien jij dan allerlei informatie op hun PC dumpt, waar ze een volgend bezoek mee geconfronteerd worden... Zal hun gemoedsrust ook niet helpen als je het mij vraagt.

Overigens krijg je een popup als je cookies niet aan hebt staan :)

Behalve als JavaScript (ook) uit staat?

Reageren