Insert from a field

[parsec]

I have this code

Code: Select all

on MouseUp
   global gConID
put field "FName" into tFName 
     put "INSERT INTO apps (appName) "  into tSQL
   put "VALUES (" & tFName & ");" after tSQL
   revExecuteSQL gConID, tSQL
   if the result is not 1 then
      answer warning the result
      exit  to top
   end if
   end mouseUp

that suppose to get the values from a FName
however I get this error "Microsoft ODBC SQL SERVER driver invalid column name" and what ever is the input from the field
here are my table and the table names
TABLE "apps" (
"appID" INT NOT NULL,
"appName" VARCHAR(50) NOT NULL,
"appRegCode" VARCHAR(50) NULL DEFAULT NULL,
"appUserID" VARCHAR(50) NULL DEFAULT NULL,
PRIMARY KEY ("appID")
)
I try to understand what I am doing wrong
Thank you
J

[AxWald]

Hi,

try:
put "VALUES ('" & tFName & "');" after tSQL

Since "appName" is a string you need some quotes around. And since you're constructing the string with concatenation (that already uses quotes) the use of " ' " (single quote) is the way to go!

Have fun!

[parsec]

thank you for the help
May I ask you if that is the proper and most efficient way to do this, is any other way as well
Thank you again

[AxWald]

Hi,

May I ask you if that is the proper and most efficient way to do this, is any other way as well

Well, I'm used to construct strange SQL statements "manually", since I'm doing this in MS Access VBA since ages - after a while you get quite accustomed to work with multitudes of:

Code: Select all

( ) , ' " & ...

;-)

If you had a function:

Code: Select all

function swote Txt    -- usefull to get those nasty 'singlequotes'
   return "'" & txt & "'"
end swote

you could write:

Code: Select all

put "VALUES (" & swote(tFName) & ");" after tSQL

as well.

I'm sure there are other ways to make it easier though. You just have to make sure to have the data formatted in a way that the target SQL engine understands ...

Have fun!