Min varekurv++ tæller dobbelt

Du har et ; for meget i linje 120.
Fjern det og det virker igen. :o)

????

Hvilken linie ??

<ole>

Præcis ... "hvilken linje?"  ;o)

Hvordan skal man kunne gennemskue noget i en kode, man ikke kan se? Hvis vi skal gætte på, hvad der kan være galt, er vi nødt til at se koden - ellers kan det blive en _meget_ lang quiz  =)

/mvh
</bole>

Som Ole siger, det var ment som en joke, da vi intet kan se af de linjer du har vist herover.
Hvis koden som bruges når der laves et link ala:

buy.php?product=ID
Og så noget mere af koden fra indkøbskurven.

Måske et link til din side ?

okay. Det hele er bygget op i klasser og funktioner.

min kurv funktion ser sådan ud.

    function kurv()
    {
        $product_id = $_GET[id]; //the product id from the URL
        $action = $_GET[action]; //the action from the URL

    if($product_id ){
    switch($action) {     

                case "add":
                    $_SESSION['cart'][$product_id]++ -1;
                    header("location: index.php?side=varekurv");
                    break;

                case "empty":
                    unset($_SESSION['cart']);
                    header("location: index.php?side=varekurv");
                    break;

                case "remove":
                    $_SESSION['cart'][$product_id]--;
                    if ($_SESSION['cart'][$product_id] == 0)
                        unset($_SESSION['cart'][$product_id]);
                    header("location: index.php?side=varekurv");
                    break;


            }
        }
    }

Linket som jeg som jeg bruger til at ligge ting ind i kurven.

<a href="index.php?side=kurv&action=add&id=' . $row[vareid] . '">

selve funktionen smider jeg ud sådan her.
function gen_contents()
    {
        $id = $_GET["side"];
        switch($id)
        {
    case kurv :
            include_once ("include/kurv.php");
            $kurv = new kurv();
            $text .= $kurv->kurv();
            return $text;
            break;
}

Jeg har fundet ud af at det må være i den sidste switch case den er galt, for hvis jeg kalder funktionen direkte med $product_id kommer fejlen ikke.

Hvad betyder denne linje - og hvorfor ser den sådan ud?
    $_SESSION['cart'][$product_id]++ -1;

Derudover ser det ikke umiddelbart ud, somom der er fejl, der kan være skyld i dobbelt optælling i den kode, du viser her  =)

Efter som du kalder med add, så bør der jo kun blive tilføjet en.
Prøv at lav noget debug...
Inden du går ind i din funktion, print så:

print_r($_SESSION['cart']);
og efter kaldet af add osv...

olebole se bort fra -1 ville lige teste noget, nåede ikke at slette det.

jeg har nu prøvet med print_r($_SESSION['cart']); og det viser at det er så snart at der trykkes på linket det bliver tilføjet 2 stk vare
Array ( [29] => 1 ) Array ( [29] => 2 )

Det er i øvrigt eksemplet herfra jeg har brugt http://whoyouknow.co.uk/php/shop/ prøver jeg det af uden at smide det i klasser og funktioner funker det.

har også prøvet at sniffe lidt på hvad der bliver sendt frem og tilbage og der er hellere ikke noget mistænktsomt.

Så min konklusion er lidt at det er den her sidste case switch som gør et eller andet.

Men din sidste case, tilføjer jo ikke en vare.

Jo, den tilføjer faktisk en vare, hvis $_GET["id"] er sat og $_GET["action"] indeholder 'add' og $_GET["side"] indeholder 'kurv' i det øjeblik, gen_contents bliver kaldt. Men det er ikke til at sige, når man ikke kan se, hvad der sker, hvornår og hvorfor  =)

Ved godt at jeg er lidt sparsom med kode. men har fundet ud af lidt mærkeligt.

hvis jeg skriver min switch case sådan her.

    if($product_id && $action){
    switch($action) {    //decide what to do   

                case "add":
                    $_SESSION['cart'][$product_id]++;
                    header("location: index.php?side=varekurv");                   
                    break;

Hvor jeg sender brugeren videre til varekurven med det samme virker tingene, men hvis jeg fjerner den den tæller den dobbelt.

Er du _helt_ sikker på, den tæller dobbelt, hvis du skriver:

    if($product_id && $action){
    switch($action) {    //decide what to do   

                case "add":
                    // $_SESSION['cart'][$product_id]++;
                    header("location: index.php?side=varekurv");                   
                    break;

Ahhh, nu forstår jeg! Jamen, har du overhovedet læst, hvad jeg skriver i (11/10-2008 01:06:13)?

"Jo, den tilføjer faktisk en vare, hvis $_GET["id"] er sat og $_GET["action"] indeholder 'add' og $_GET["side"] indeholder 'kurv' i det øjeblik, gen_contents bliver kaldt."

- og så er der jo ikke noget at sige til, at det ikke sker, når du ingenting foretager dig - men blot sender brugeren videre.

Da du kun kan overtales til at levere drypvise informationer, hopper jeg ud her. Det fører vist ingen vegne

Fik det aldrig løst - Startede forfra, og fik det til at fungere.

svar