if sætning driller

denne linie
if(!$_SESSION[tjek] == "") {
skal for det første være:
if($_SESSION[tjek] != "") {

Nej det virkede før har bare tilføjet antal funktionen i det og så gik det lidt galt

før havde jeg bare
hvis $SESSION[tjek] sat ind læg vare i db

hvis ikke $_SESSION[tjek] tom kør ikke logget_ind function

det her var det orginale script jeg havde fra starten
if(isset($_POST[kob_x])) {
    if(!$_SESSION[tjek] == "") {
        mysql_query("INSERT INTO kurv (vare_id, unikt_id, antal) VALUES ('$_POST[varenummer]','$_SESSION[unikt_id]','$_POST[antal]')") or die(mysql_error());
    } else {
        ikke_logget_ind();
    }
}

Grunden til fejlen er at du har to else. Du kan ikke have en if-sætning og så have to else.

og det virkede !!

det der virkede var altså det jeg skrev i indlæg:
Kommentar: krydset
10/12-2003 15:52:20
det gamle script
'

Hvordan ville du sætte det op ??

og hvis ikke  'tjek' og 'kob_x' og 'vare_id' og 'unikt_id' og 'antal' er defineret som konstanter skal der anførselstegn om

til gengæld er der ingen grund til at sætte anførselsteng omkring "$_POST[varenummer]" i linien:
                if( $row[vare_id] == $_POST[varenummer] ) {

fire linier under den linie du peger på
        /*linie 395*/ } else {
er der en anden linie med
            } else {
som ikke hører til nogen if-sætning. det vil gå galt.

mvh JakobA

Svært lige at se hvad du vil, men ser ud til at det er sådan du vi:

$antal = mysql_query("SELECT * FROM kurv"); //henter ud fra kurv
if(isset($_POST['kob_x']))
{
    //hvis køb knappen aktiveres tjek om sessione er sat
    if(!$_SESSION['tjek'] == "")
    {
        //hvis sessionen er sat. Tjek så om varen findes
        while($row = mysql_fetch_array($antal))
        {
            if($row['vare_id'] == $_POST['varenummer'])
            {
                echo "OPDATER ANTAL";
                //hvis varen findes skal antal i db opdateres
                /*linie 395*/
            }
            else
            {
                //hvis ikke varen findes i kurv skal den lægge varen i kurv
                mysql_query("INSERT INTO kurv (vare_id, unikt_id, antal) VALUES ('$_POST[varenummer]','$_SESSION[unikt_id]','$_POST[antal]')") or die(mysql_error());
            }
        }
    }
    else
    {
        //hvis sessionen $_SESSION[tjek] er tom kør funtionen ikke_logget_ind
        ikke_logget_ind();
    }
}

det ser meget rigtigere ud (og endda med ordentlige indrykninger:-))
kommer der stadig fejl?

jeg holder også på at det burde være korrekt nu... det var også den eneste fejl jeg kunne se...

Morten

tester lige nu er lidt noob endnu :-( så hav lidt tålmodighed ik?

Det er ikke helt korrekt.

Hvis man ikke er logget ind så kan man ikke købe det virker!
men hvis man logger ind så kan man godt købe men ikke putte noget i kurven

Jeg har det her nu


$antal = mysql_query("SELECT * FROM kurv"); //henter ud fra kurv
if(isset($_POST['kob_x']))
{
    //hvis køb knappen aktiveres tjek om sessione er sat
    if(!$_SESSION['tjek'] == "")
    {
        //hvis sessionen er sat. Tjek så om varen findes
        while($row = mysql_fetch_array($antal))
     
        {
            echo "TJEK VAREN";
            if($row['vare_id'] == $_POST['varenummer'])
            {
                echo "OPDATER ANTAL";
                mysql_query("UPDATE kurv antal = '$_POST[antal]' WHERE vare_id = '$_POST[varenummer]' AND unikt_id = '$_SESSION[unikt_id]'") or die(mysql_error());
                //hvis varen findes skal antal i db opdateres
            }
            else
            {
                //hvis ikke varen findes i kurv skal den lægge varen i kurv
                mysql_query("INSERT INTO kurv (vare_id, unikt_id, antal) VALUES ('$_POST[varenummer]','$_SESSION[unikt_id]','$_POST[antal]')") or die(mysql_error());
                echo "indsæt i db";
            }
        }
    }
    else
    {
        //hvis sessionen $_SESSION[tjek] er tom kør funtionen ikke_logget_ind
        ikke_logget_ind();
    }
}

den udskriver ikke nogen af de echo jeg har sat ind

der ligger en fejl idat at else-betingelsen ligger INDE i der hvor du looper kurven igennem - dette medføre 2 ting

1) du kan kun tilføje ting til kurven hvis der allerede ligger noget !
2) der bliver oprettet en ordre HVER gang du har en anden ting i kurven (da den jo stadig looper gennem kurven)

derfor skal du lige omstrukturere dit script lidt

Mortne

oki prøver mig lidt frem skriver efter mere hjælp hvis det kokser helt :-O

altid velkommen ;) (du er ikke den værste at hjælpe ;))

Nu har dea_pg allerede skrevet det, her er et forslag til jvordan koden kunne være:

du har et problem i den der while sætning:

        //hvis sessionen er sat. Tjek så om varen findes
        while($row = mysql_fetch_array($antal))
        {
            if($row['vare_id'] == $_POST['varenummer'])
            {
                echo "OPDATER ANTAL";
                //hvis varen findes skal antal i db opdateres
                /*linie 395*/
            }
            else
            {
                //hvis ikke varen findes i kurv skal den lægge varen i kurv
                mysql_query("INSERT INTO kurv (vare_id, unikt_id, antal) VALUES ('$_POST[varenummer]','$_SESSION[unikt_id]','$_POST[antal]')") or die(mysql_error());
            }
        }

while løkken løber igennem alle de varer der er i indkøbskurven og for hver af varerne udføres if-sætningen.
Så hvis det er tredie vare i indkøbskurven vil den prøve at lægge varen i to gange før den kommer til at forsøge at opdatere kurven.

Så jeg ville flytte indsætning af ny vare udenfor while løkken så det først sker nå alle varer er blevet tested igennem og vi er sikre på at varen ikke er der:

        //hvis sessionen er sat. Tjek så om varen findes
        while($row = mysql_fetch_array($antal))
        {
            if($row['vare_id'] == $_POST['varenummer'])
            {
                echo "OPDATER ANTAL";
                //hvis varen findes skal antal i db opdateres
                /*linie 395*/
                break;              // springer ud af while løkken
            }
        }
        if ( !$row )  // hvis while-løkken fik lov at køre færdig blev varen ikke fundet
        {
            //hvis ikke varen findes i kurv skal den lægge varen i kurv
            mysql_query("INSERT INTO kurv (vare_id, unikt_id, antal) VALUES ('$_POST[varenummer]','$_SESSION[unikt_id]','$_POST[antal]')") or die(mysql_error());
        }

tror det virker efter det jacoba skal lige rette lidt små fejl vender tilbage senere

Der er et lille problem med det.

Hvis man starter med at putte en vae i kurven der ikke eksistere virker det fint, men hvis man derimod putter en vare i der eksistere opdatere den osse antallet, MEN skriver udskriver osse functionen ikke_logget_ind();, hvorfor gør den det??

og hvis man prøver at putte en vare i kurven kører den IKKE "ikke_logget_ind functionen. Men jeg kan ikk eirgtig få det på plads:-O

fordi du nok har mistet det else der skal stå foran  ikke_logget_ind(...).

dt jeg skrev var kun løllem og en if i samme niveau som løkken. alt det udenom (med mindre indrykning) skal ikke ændres.

Ups.    løllem  ->  løkken

Jeg har det her

$antal = mysql_query("SELECT * FROM kurv"); //henter ud fra kurv
if(isset($_POST['kob_x']))
{
    //hvis køb knappen aktiveres tjek om sessione er sat
    if(!$_SESSION['tjek'] == "")
    {
  //hvis sessionen er sat. Tjek så om varen findes
        while($row = mysql_fetch_array($antal))
        {
            if($row['vare_id'] == $_POST['varenummer'])
            {
                  mysql_query("UPDATE kurv SET antal = '$_POST[antal]' WHERE vare_id = '$_POST[varenummer]' AND unikt_id = '$_SESSION[unikt_id]'") or die(mysql_error());

                echo "OPDATER ANTAL";
                //hvis varen findes skal antal i db opdateres
                /*linie 395*/
                break;              // springer ud af while løkken
            }
        }
        if ( !$row )  // hvis while-løkken fik lov at køre færdig blev varen ikke fundet
        {
            //hvis ikke varen findes i kurv skal den lægge varen i kurv
            mysql_query("INSERT INTO kurv (vare_id, unikt_id, antal) VALUES ('$_POST[varenummer]','$_SESSION[unikt_id]','$_POST[antal]')") or die(mysql_error());
        }
   
    else
    {
        //hvis sessionen $_SESSION[tjek] er tom kør funtionen ikke_logget_ind
        ikke_logget_ind();
    }
}
}

Mangler der en else her nogen steder her

der mangler en } foran else ikke_logget_ind()

tror jeg - har kin lige skimmet den - sidder og lavet matematik aflevering...

Morten

så var den der det endelig eresultat blev det her



$antal = mysql_query("SELECT * FROM kurv"); //henter ud fra kurv
if(isset($_POST['kob_x']))
{
    //hvis køb knappen aktiveres tjek om sessione er sat
    if(!$_SESSION['tjek'] == "")
    {
  //hvis sessionen er sat. Tjek så om varen findes
        while($row = mysql_fetch_array($antal))
        {
            if($row['vare_id'] == $_POST['varenummer'])
            {
                  mysql_query("UPDATE kurv SET antal = '$_POST[antal]' WHERE vare_id = '$_POST[varenummer]' AND unikt_id = '$_SESSION[unikt_id]'") or die(mysql_error());

                echo "OPDATER ANTAL";
                //hvis varen findes skal antal i db opdateres
                /*linie 395*/
                break;              // springer ud af while løkken
            }
        }
        if ( !$row )  // hvis while-løkken fik lov at køre færdig blev varen ikke fundet
        {
            //hvis ikke varen findes i kurv skal den lægge varen i kurv
            mysql_query("INSERT INTO kurv (vare_id, unikt_id, antal) VALUES ('$_POST[varenummer]','$_SESSION[unikt_id]','$_POST[antal]')") or die(mysql_error());
        }
   
    }else
    {
        //hvis sessionen $_SESSION[tjek] er tom kør funtionen ikke_logget_ind
        ikke_logget_ind();
    }
}


gider i alle 3 lægge et svar så i kan få lidt for hjælpen?

svaa'r

Morten

takker

Svar :-)

Takker for hjælpem :-D