Uppdatera PM-fält

[Herr Bill]

Varför fungerar inte nedanstående kod om Presentation är ett PM-fält? Vad skall jag göra för att det skall fungera? Högst poäng till den som hjälper mig!

 

 

Set rsc = Server.CreateObject ("ADODB.Recordset")

Villkor = "SELECT Presentation From Foretagsregister2 WHERE ID=" & Request.Form("ID")

rsc.Open Villkor,Connect,adOpenDynamic,adLockOptimistic

 

rsc("Presentation") = request("Presentation")

 

rsc.Update

Connect.Close

Response.Redirect "main.asp"

[Sasja]

När jag kollar på det lite snabbt ser jag att du inte har avslutat en rad, eller?

 

Jag tänker på den andra raden där du borde avsluta med att lägga till &" "

 

D.v.s. det blir så här:

Set rsc = Server.CreateObject ("ADODB.Recordset")
Villkor = "SELECT Presentation From Foretagsregister2 WHERE ID=" & Request.Form("ID") &" "
rsc.Open Villkor,Connect,adOpenDynamic,adLockOptimistic

rsc("Presentation") = request("Presentation")

rsc.Update
Connect.Close
Response.Redirect "main.asp"

 

Om det ändå inte löser sig. Vad är det för felmeddelande du får?

[inlägget ändrat 2002-11-08 23:32:12 av Sasja]

[inlägget ändrat 2002-11-08 23:34:14 av Sasja]

[Anders N]

Jag brukar aldrig göra sådär, utan använder "update" istället.. men.. hmm.. vad är det hämtar presentationen från?

 

Borde det inte vara request.form("presentation") eller request.querystring("presentation")

 

Ciao,

Anders

 

[Anders N]

avsluta med att lägga till &" "

 

Va?

 

Den strängen är ju redan "avslutad", man behöver inte lägga på det där.

 

Jag ser MÅNGA som brukar lägga på & "" i slutet... varför vill man göra så? Jag tycker inte alls det känns logiskt.

 

Ciao,

Anders

 

[Sasja]

Fungarar lika bra med att skriva request som request.form eller request.querystring. Även om det inte är helt rätt att skriva endast request rent säkerthetsmässigt...

 

Tror det ligger till så iallafall.

 

[Sasja]

Själv tycker jag det är hur logiskt som helst. Har du skrivit ETT citationstecken måste du ju avsluta denna med ett TILL.

 

Men det kanske även fungerar i att inte göra så.

 

Har du provat det jag sa?

Annars... vad får du för felmeddelande?

 

[inlägget ändrat 2002-11-08 23:40:40 av Sasja]

[Anders N]

Själv tycker jag det är hur logiskt som helst. Har du skrivit ETT citationstecken måste du ju avsluta denna med ett TILL.

 

Ja...

 

Villkor = "SELECT Presentation From Foretagsregister2 WHERE ID="

 

Där har han skrivit ett citationstecken, och avslutat med ett till...

 

Sedan så vill du lägga till mer till strängen, och skriver &"", skriver ett citationstecken, och avslutar med ett... vilket lägger till ingenting.. (i ditt fall ville du iofs lägga till ett mellanslag

 

Ciao,

Anders

 

[Sasja]

Jo, jag förstår hur du tänker och jag respekterar det. =)

 

Däremot vill jag att du respekterar min metod, det är kvällens mål för mig. =)

Villkor = "SELECT Presentation From Foretagsregister2 WHERE ID="& Request.Form("ID") &" "

Förstår du nu hur jag tänker?

 

Jo... den sista är skitonödig nu när jag tänker efter. =)

 

Men fan... det känns som bättre att SQL-raden är ihopsatt typ... det är en barnsjukdom jag har i min kodning kanske =)

 

Tack för tipset annars !=)

[inlägget ändrat 2002-11-08 23:50:26 av Sasja]

[Anders N]

Mmm... kanske.. LITE logik.. någonstans i det.. visst respekterar jag det

 

Ciao,

Anders

"That you may retain your self-respect, it is better to displease the people by doing what you know is right, than to temporarily please them by doing what you know is wrong."

 

 

[Herr Bill]

Felmeddelandet jag får lyder:

 

ADODB.Recordset.1 error '80004005'

 

SQLState: 34000

Native Error Code: 0

[iNTERSOLV][ODBC SequeLink driver]Statement with cursor named in this statement's CURRENT-OF-CURSOR clause is not a SELECT FOR UPDATE.

 

 

/ekonomikonsulter2/andra_foretag_process.asp, line 66

 

(line 66 = rsc.Update)

 

 

Om jag ändrar Presentation till ett vanligt textfält i Access så fungerar det, men inte om jag låter fältet vara ett PM-fält.

 

Eftersom texten är längre än 255 tecken räcker ett vanligt textfält inte till, så jag måste verkligen ha ett PM-fält!

 

Hjälp mig!

 

 

[Magnus Gladh]

Däremot vill jag att du respekterar min metod, det är kvällens mål för mig. =)

 

Vet inte om jag vill respekterar det, det finns inget positivt i det du gör endast negativt.

 

1. Extra kommando för processorn, alltså tar det kraft (om än väldigt lite)

2. Ett extra moment alltså en möjlig felkälla.

 

Så visst du får gärna göra så om du vill, men jag vet inte om jag kommer att respekterar det

 

- Magnus

-----------------------------------------------------

ju mer jag lär mig ju mer inser jag så lite jag kan

 

[Magnus Gladh]

Enligt felmeddelandet så påstår IIS att du har öppnat ditt recordset fel, låter dock konstigt att det fungerar med en vanlig textrad.

 

Had jag varit som du hade jag slängt ut allt vad som heter recordset när du updaterar och lägger till data i databasen, och använder vanliga INSERT och UPDATE sqlsatser som ger bättre prestande och mindre felkällor.

 

- Magnus

-----------------------------------------------------

ju mer jag lär mig ju mer inser jag så lite jag kan

 

[Herr Bill]

Tack för ditt svar.

 

Hur ser en sql-sats ut??

 

[Sasja]

T.ex. så här:

SELECT * FROM anvandardata WHERE id = 3

 

Gå igenom WebStudios SQL-skola vettja! Adressen dit: http://www.idg.se/webstudio/pub/object.asp?id=14

 

[Herr Bill]

Det är ju det jag har skrivit?

 

Jag har försökt läsa WebStudios SQL-skola, men jag förstår inte ett sk*t av det..

 

Om du skulle ändra följande kod så att den blir rätt, skulle jag vara dig evigt tacksam..

 

Set rsc = Server.CreateObject ("ADODB.Recordset")

Villkor = "SELECT Presentation From Foretagsregister2 WHERE ID=" & Request.Form("ID")

rsc.Open Villkor,Connect,adOpenDynamic,adLockOptimistic

 

rsc("Presentation") = request("Presentation")

 

rsc.Update

Connect.Close

Response.Redirect "main.asp"

 

 

Tack på förhand!

 

[Sasja]

Här kommer det! Jag tror och hoppas det ska fungera. Givetvis måste du ha databaskopplingen ovanför detta.

 

SQL = "UPDATE Foretagsregister2 SET Presentation = '"& request("Presentation") &"' WHERE ID="& request("id") &""

Connect.Execute(SQL)

Connect.Close

 

[inlägget ändrat 2002-11-10 18:21:04 av Sasja]

[Herr Bill]

Sådär, nu erhöll du din första poäng!

 

Koden du skrev fungerar utmärkt. Men hur fasen gör jag om jag vill ändra fler fält? Att skriva något i stilen med...

 

SQL = "UPDATE Foretagsregister2 SET Presentation = '"& request("Presentation") &"' and SET Organisationsnummer = '"& request("Organisationsnummer ") &"'

 

...fungerar tydligen inte.

 

 

 

[Sasja]

TACK för poängen! =)

 

Ne, det är enklare än du tror.

 

SQL:en blir så här:

SQL = "UPDATE Foretagsregister2 SET Presentation = '"& request("Presentation") &"', Organisationsnummer = '"& request("Organisationsnummer") &"' WHERE ID="& request("id") &""

 

[inlägget ändrat 2002-11-10 19:48:12 av Sasja]

[Anders N]

Fast, det där är ju osäker kod, så...

 

För varje sträng du läser in vill du använda replace(strängen, "'",""), och för talen kan du använda CInt(talet).

 

SQL = "UPDATE Foretagsregister2 SET Presentation = '"& [b]replace([/b]request("Presentation")[b],"'","")[/b] &"', Organisationsnummer = '"& [b]replace([/b]request("Organisationsnummer")[b],"'","")[/b] &"' WHERE ID="& [b]CInt([/b]request("id")[b])[/b]

 

Ciao,

Anders

[inlägget ändrat 2002-11-10 19:58:26 av Anders N]

[Sasja]

Räcker det verkligen att bara ta bort de halva citationstecknena?

 

Ska man inte även ta bort hela citationstecken?

 

[Anders N]

Det räcker att ta bort apostrof (') eftersom det är den som används för strängarna i SQL-frågan.

 

Om resultatet ska visas på en hemsida vill man dock förmodligen filtrera bort "<" och ">" också, för att undvika "dumma" HTML-kommandon.

 

Ciao,

Anders