Insert into fejl

connection.execute("INSERT INTO ordre (kunde,dato) VALUES ("+ id + ", " + toSqlDate(new Date()) +")");
Er det ikke [ ] der driller?

connection.execute("INSERT INTO ordre ([kunde,dato]) VALUES ("& id & ", #" & toSqlDate(new Date()) &"#)");

Så kommer denne Either BOF or EOF is True, or the current record has been deleted. Requested operation requires a current record.

connection.execute("INSERT INTO ordre ([kunde,dato]) VALUES ("& id & ", #" & toSqlDate(dato()) &"#)");
  res = connection.execute("SELECT id FROM ordre WHERE kunde='& id &' ORDER BY id DESC");

Hvorfor ikke bare:

connection.execute("INSERT INTO ordre (kunde,dato) VALUES ("+ id + ", #" + toSqlDate(new Date()) +"#)");

under antagelse af at din toSqlDate-funktion leverer en lovlig dato ifht. databasen.

Det er her datoen kommer fra:
function toSqlDate(dato) {
  return "'" + date.getDate() + "-" + (date.getMonth() + 1) + "-" + date.getYear() + "'";
}

softspot hvis jeg bruger den du viser får jeg; Expected ';'

efter rettelse får jeg af vide at datoen er undefine altså new date

Det er det der med dato der ikke virker som det skal!

Har ændret lidt i det så linj:en er flg:
connection.execute("INSERT INTO ordre (kunde,dato) VALUES ("+ id + ", #" + showDate(new Date()) +"#)");

og har included dato.js som ser således ud:

<%
function showDate(){
    dt = new Date();    //gets today's date right now (to the millisecond)
    month = dt.getMonth()+1;
    day = dt.getDate();
    year = dt.getFullYear();
   
}

%>

Fejlen lyder så flg:
Microsoft OLE DB Provider for ODBC Drivers error '80040e14'

[Microsoft][ODBC Microsoft Access Driver] Syntax error in INSERT INTO statement.

Du mangler vel at returnere den dato du får stykket sammen i showDate...?

Dvs. return skal sættes ind så det bliver:
<%
function showDate(){
    dt = new Date();    //gets today's date right now (to the millisecond)
    month = dt.getMonth()+1;
    day = dt.getDate();
    year = dt.getFullYear();
return dt;
   
}

%>

Ja, det vil i det mindste returnere den dato du får skabt med new Date(), men de andre ting du gør har ikke nogen effekt, med mindre du har nogle globale variable som hedder, month, day og year...

Prøv evt. dette i stedet:

<%
function showDate(){
  dt = new Date();    //gets today's date right now (to the millisecond)
  month = dt.getMonth()+1;
  day = dt.getDate();
  year = dt.getFullYear();
  return year + '-' + month + '-' + day;
}

%>

Var det her bedre?:

<%
function showDate( ){
    dt = new Date();    //gets today's date right now (to the millisecond)
    month = dt.getMonth()+1;
    day = dt.getDate();
    year = dt.getFullYear();
    return "'" + day() + "-" + (month() + 1) + "-" + year() + "'";
}

%>

...og så kan du jo forkorte din SQL-sætning lidt:

connection.execute("INSERT INTO ordre (kunde,dato) VALUES ("+ id + ", #" + showDate() + "#)");

Ja, hvis din egen giver lidt problemer (pga. formatet kan tolkes forskelligt alt efter hvilket land du er i og hvilken landekode din side er sat til at benytte), så kan du jo prøve den jeg foreslog i post 16/01-2006 13:38:11...

Får denne fejl ved det:
Microsoft OLE DB Provider for ODBC Drivers error '80040e14'

[Microsoft][ODBC Microsoft Access Driver] Syntax error in INSERT INTO statement.

Prøv lige at ændre denne linie:

connection.execute("INSERT INTO ordre (kunde,dato) VALUES ("+ id + ", #" + showDate() + "#)");

til flg. linier:

var sql = "INSERT INTO ordre (kunde,dato) VALUES ("+ id + ", #" + showDate() + "#)";
Response.Write(sql);
Response.End();
connection.execute(sql);


og fortæl/vis så hvad der skrives i din browser når du udfører koden...

Hvis du bruger din egen showDate fra 16/01-2006 13:38:17, så skal du lige undlade at lægge 1 til month 2 gange... det kan resultere i en ugyldig dato (da måned kan blive 13)...

Faktisk så vil jeg anbefale dig at bruge min version af showDate, da dn egen har flere fejl (med mindre du har funktioner som hedder day, month og year).

var sql = "INSERT INTO ordre (kunde,dato) VALUES ("+ id + ", #" + showDate() + "#)";
Response.Write(sql);
Response.End();
connection.execute(sql);

  res = connection.execute("SELECT id FROM ordre WHERE kunde="+ id +" ORDER BY id DESC");
  ordrenummer = res.Fields("id");

Skal det se således ud så?

Ja, indtil videre er det de første 3 linier der er interessante (da siden stopper efter Response.End(). Det er bare for at se hvilken SQL der kommer ud af det...

Når jeg gør som du har skrevet får jeg flg resultat:
INSERT INTO ordre (kunde,dato) VALUES (, #day() + "-" + month() + "-" + year() +#)

Det er hvis du bruger din egen version af showDate. Jeg er ret overbevist om at hvis du bruger den version jeg har flikket sammen, så kommer det ikke til at se sådan ud...

Resulatet af din:
showdate og det du skrev bliver :
INSERT INTO ordre (kunde,dato) VALUES (, #2006-1-16#)

Jeps, så begynder det at ligne noget. Det der mangler er en værdi i den variabel der hedder id. Som du kan se er den første værdi tom i VALUES-listen og det må den ikke være. Derfor skal du sikre dig at den variabel der hedder id har en gyldig værdi (dvs. en der stemmer ovenens med den type som feltet kunde har i databasen...

// registrer ordren
  Application.Lock();
connection.execute = "INSERT INTO ordre (kunde,dato) VALUES ("+ id + ", #" + showDate() + "#)";
  res = connection.execute("SELECT id FROM ordre WHERE kunde='+ id +' ORDER BY id DESC");
  ordrenummer = res.Fields("id");
  Application.Unlock();

Sådan her, ser det ud samlet.

Altså hvis man holder sig til det der var før det,da det så ikke hænger sammen med resten hvis jeg bruger response osv...

Ja, men du skal lige have sikret dig at id indeholder en gyldig værdi... hvor kommer den værdi fra?

fra en access database

Ok, hvordan kommer den over i variablen, for der er tydeligvis noget galt med det udtræk (måske antager du der returneres en række hvor der ikke gør eller sådan noget). Prøv at vise den kode hvor denne værdi tildeles...

Her er hele sidens kode:
<%

  OK = 0;
  FORKERT_KODEORD = 1;
  KUNDE_EKSISTERER = 2;

  con = createDatabaseConnection();

  // kontroller først, om kunden er logget ind
  id = -1;
  if (Request.Cookies("kundeid") != "") {
      id = opdaterKunde(Request.Cookies("kundeid"));
  } else {
      id = tilfojKunde();
  }

  // registrer ordren
  Application.Lock();
connection.execute = "INSERT INTO ordre (kunde,dato) VALUES ("+ id + ", #" + showDate() + "#)";
  res = connection.execute("SELECT id FROM Ordre WHERE kunde='+ id +' ORDER BY id DESC");
  ordrenummer = res.Fields("id");
  Application.Unlock();

  albumantal = Session("albumantal");
  for (i = 0; i < albumantal; i++) {
    sql = "INSERT INTO ordrelinie (album, antal, ordre) VALUES (";
    sql += Session("album"+i);
    sql += ", ";
    sql += Session("antal"+i);
    sql += ", ";
    sql += ordrenummer;
    sql += ")";
      connection.execute(sql);
  }
  printResult(OK);

  function opdaterKunde(id) {
    sql = "UPDATE kunde SET ";
    sql += "Navn='" + Request.Form("Navn") + "', "
    sql += "Adresse='" + Request.Form("Adresse") + "', ";
    sql += "Postnummer=" + Request.Form("Postnummer") + " , ";
    sql += "Postdistrikt='" + Request.Form("By") + "', ";
    sql += "Email='" + Request.Form("Email") + "'";
    sql += " WHERE id=" + id;
    connection.execute(sql);
      return id;
  }

  function tilfojKunde(id) {
    // kontroller, at de to kodeord er ens
    if (!strcmp(Request.Form("kodeord1"),Request.Form("kodeord2"))) {
        printResult(FORKERT_KODEORD);
    }

    // kontroller, at der ikke allerede eksisterer en kunde med den email-adresse
    res = connection.execute("SELECT id FROM kunde WHERE email='" + Request.Form("Email") + "'");
    if (!res.EOF) { // kunden eksisterer
        printResult(KUNDE_EKSISTERER);
    }

      sql = "INSERT INTO kunde (Navn, Adresse, Postnummer, By, Email, Kodeord) VALUES (";
      sql += "'" + Request.Form("Navn") + "', ";
      sql += "'" + Request.Form("Adresse") + ", ";
      sql += Request.Form("Postnummer") + ", ";
      sql += "'" + Request.Form("By") + "', ";
      sql += "'" + Request.Form("Email") + "', ";
      sql += "'" + Request.Form("Kodeord") + "')";

 
      res = connection.execute("SELECT id FROM kunde WHERE Email='" + Request.Form("Email") + "'");
      return res.Fields("id");
  }

  function printResult(status) {
%>


i toppen er database, dato inkluderet...

Der er et par fejl i rutinen tilfojKunde. Dels bliver den INSERT du har lavet ikke udført, dels mangler der en apostrof efter adressefeltet. Prøv evt. denne version i stedet:

  function tilfojKunde(id) {
    // kontroller, at de to kodeord er ens
    if (!strcmp(Request.Form("kodeord1"),Request.Form("kodeord2"))) {
        printResult(FORKERT_KODEORD);
    }

    // kontroller, at der ikke allerede eksisterer en kunde med den email-adresse
    res = connection.execute("SELECT id FROM kunde WHERE email='" + Request.Form("Email") + "'");
    if (!res.EOF) { // kunden eksisterer
        printResult(KUNDE_EKSISTERER);
    }

      sql = "INSERT INTO kunde (Navn, Adresse, Postnummer, By, Email, Kodeord) VALUES (";
      sql += "'" + Request.Form("Navn") + "', ";
      sql += "'" + Request.Form("Adresse") + "', ";
      sql += Request.Form("Postnummer") + ", ";
      sql += "'" + Request.Form("By") + "', ";
      sql += "'" + Request.Form("Email") + "', ";
      sql += "'" + Request.Form("Kodeord") + "')";

      connection.execute(sql);

      res = connection.execute("SELECT @@IDENTITY AS id");
      return res.Fields("id");
  }

Desuden vil jeg til hver en tid anbefale at du benytter SELECT @@IDENTITY til at finde den række du lige har oprettet, da den er mere sikker (og generisk) end alle mulige andre metoder der selecter på de data man lige har indsat.

...og man skal naturligvis sørge for at lukke sine recordset når man er færdig med at bruge dem, så databasen kan få frigjort sine resurser igen. :)

Microsoft OLE DB Provider for ODBC Drivers error '80040e14'

[Microsoft][ODBC Microsoft Access Driver] Syntax error in INSERT INTO statement.

og linie 74 = connection.execute(sql);

Flere spørgsmål:

1. Hvilken af de mindst 2 linier med connection.execute(sql) er der tale om?

2. Hvordan ser SQL-sætningen ud (benyt tricket med Response.Write + Response.End)?

3. Hvad ligger der i den cookie du bruger til at afgøre om der skal opdateres eller oprettes en kunde?


Jeg kan stadig finde flere syntaktiske fejl i koden, så det er interessant at vide lidt mere hvilken vej igennem koden du kører i øjeblikket. Hvis du er mere specifik med hvor i koden fejlen opstår, kan jeg hurtigere hjælpe dig med at rette fejlen... :)

Jeg lukker softspot smider du ikke et svar?

Jo, det kan jeg da godt, men fik vi løst problemet, eller stiller jeg bare formange træls spørgsmål...? :)

Altså det hele endte meget uoverskueligt og det vil kræve jeg laver nogen større rettelser, så derfor lukker jeg :-). . . .

Ellers mange tak for de uddybende svar! :)

Fint nok, men vi kunne (og kan stadig) godt have fundet en løsning på din udfordring, der er bare lige nogle steder i din kode som skal ændres lidt for at det kommer til at spille... så umulig er koden jo heller ikke ;-)