Hjælp til PHP Session

Tror jeg ville ændre lidt på logikken, så din kode bliver noget i stil med:

<?php

session_start();
$cart = $_SESSION['cart'];
$basket = $_GET['basket'];
$amount = (int)$_GET['antal'];
$id = (int)$_GET['id'];

switch ($basket) {
case 'add':
    if (!$cart) {
        $cart = '';
    }
   
    for($i = 1; $i <= $amount; $i++) {
        $cart .= ','.$id;
    }
   
    $cart = trim($cart, ",");

    break;
}

$_SESSION['cart'] = $cart;
echo $cart;

?>

Jeg har enormt mange negative kommentarer til koden, men jeg vil lade mit spørgsmål tale for sig selv: hvordan vil du fjerne noget fra indkøbskurven, hvis du senere i forløbet ændrer mening?

Normalt vil man bruge et array til at administrere en indkøbskurv. Noget ala:


Det giver dig et array med varens id samt hvor mange af af netop den vare du vil have.

Det er mere fleksibel og endda pænere kode.

Det virker jo perfekt nu.
Kan du ikke lige forklare hvad du har gjort.

Hvem af os mener du?

alphabits... Skrev min besked før din så så ikke lige din før jeg havde skrevet..

Du skrev din besked før min? Hvordan kan min besked være et kvarter ældre end din, så? Ak, ja - livets små mysterier...

Problemet i din oprindelige kode var at du skrev


i den løkker der blev kørt når $_SESSION['cart'] ikke fandtes i forvejen. Dvs. at løkken køre x antal gange, men i hvert gennemløb sætter den $cart til $_GET['id'], hvilket i dit system svarer til at lægge 1 produkt i kurven. Jeg gætter på at du havde lidt problemer med at når $cart var en tom streng kunne du ikke bare skrive


i løkken, fordi du så ville få et komma i starten af $cart. Min løsning er sådan set bare at være ligeglad, i første omgang, med at der kommer et komma i starten og først når løkken er kørt fjerner jeg kommaet med


Ikke super elegant, men det virker :)

Når det så er sagt, må jeg give repox ret i at der findes bedre måder at gemme en indkøbskurv i ens session. Så hvis du er frisk på en udfordring, kan du se på koden i hans svar, og se om det giver mening.

Smid venligst et svar.

Svar :)

Host.. Svar :)