syntax errpr near parenthesis

[syedshaffee]

this is my code

dim i
dim checkboxValue
dim TextBoxValue
 
dim Count
dim Count1
dim postion
dim postion1
position=0
postion1=0
i=0
Checkbox1= Request.Form("same")
strdate=Request.Form("Date")
a=Split(strdate ,",")
b=Split(Checkbox1,",") 
if strdate<>"" and Checkbox1<>"" then
TextBoxValue=UBound(a)
checkboxValue=UBound(b)
'Response.Write(TextBoxValue)
if TextBoxValue  > checkboxValue then
for i=0 to TextBoxValue
Set objRIMSConn1 = Server.CreateObject("ADODB.Connection")
    objRIMSConn1.Open RIMS_CONNECT
    objRIMSConn1.Execute "Insert into Eot_returned(Asset_id,Date) values("+b(i)+",'"+a(i)+"');"
next
objRIMSConn1.Close
    Set objRIMSConn1 = nothin
    else
    for i=0 to checkboxValue
Set objRIMSConn1 = Server.CreateObject("ADODB.Connection")
    objRIMSConn1.Open RIMS_CONNECT
    objRIMSConn1.Execute "Insert into Eot_returned(Asset_id,Date) values(" +b(i)+ "," +a(i)+ ")"
next
end if
 
    Response.Write("<script language='javascript' type='text/javascript'> alert('The value hass been Inserted'); </script>")
     else
    Response.Write("<script language='javascript' type='text/javascript'> alert('Do you want to return any asset'); </script>")
end if

Erro:Near ")" please an u people help me???????????

[Rabbit]

You've only defined one field to insert into but you include two values.

[syedshaffee]

yes i can but the problem is in two its giving an error

[Rabbit]

I have no idea what you're saying. I'm saying that you can't specify two values when you only specify one column.

[multinett]

you have no space between ")" and "values"
if you want any help, it's better to post the full error message.
you can get the error message with
response.write err.description

[syedshaffee]

thnaks u guys and mlutinett i will post the full error message

[jhardman]

Rabbit is definitely right: the insert statement has the wrong number of fields.
If you were inserting into just one field the command would look like this:

insert into eot_returned (asset_id) values (4)

but if you were inserting into two different fields, the command would look like this:

insert into eot_returned (asset_id, asset_name) values (4, 'John')

If you were trying to insert into one field but two different records, you would need two different insert statements

insert into eot_returned (asset_id) values (4);
insert into eot_returned (asset_id) values (5)

the way you have it is definitely wrong, but I can't tell which you are trying to do.

Jared

[syedshaffee]

jhardman i have changed the code and still it is giving me an error the following is the error:

Error Type:
Microsoft OLE DB Provider for ODBC Drivers (0x80040E14)
[Microsoft][ODBC SQL Server Driver][SQL Server]Incorrect syntax near ')'.
, line 876

[syedshaffee]

OKAY now the values are going to the database but its still displaying the same page

[Rabbit]

What do you mean it displays the same page?

[syedshaffee]

the error page in asp
syedshaffee
re: syntax errpr near parenthesis
jhardman i have changed the code and still it is giving me an error the following is the error:

Error Type:
Microsoft OLE DB Provider for ODBC Drivers (0x80040E14)
[Microsoft][ODBC SQL Server Driver][SQL Server]Incorrect syntax near ')'.
but the date time column in sql is like 1/01/1900 for each column

[Rabbit]

You said the data was going into the database, it shouldn't be giving you an error message if it was. Are you sure the data is getting there?

[syedshaffee]

yes i checked it even i'm wondering how???? is this possible

[Rabbit]

It would help to see the updated code.

[syedshaffee]

here is the code that is edited

dim i
dim checkboxValue
dim TextBoxValue
 
dim Count
dim Count1
dim postion
dim postion1
position=0
postion1=0
i=0
Checkbox1= Request.Form("same")
strdate=Request.Form("Date")
a=Split(strdate ,",")
b=Split(Checkbox1,",")
 
if strdate<>"" and Checkbox1<>"" then
TextBoxValue=UBound(a)
checkboxValue=UBound(b)
'Response.Write(TextBoxValue)
while not rstAssetdesc.EOF
Des = rstAssetdesc("Asset_type_description")
if TextBoxValue  > checkboxValue then
for i=0 to  checkboxValue
 
objRIMSConn.Execute "Insert into Eot_returned (Contract_no,Eot_action,Asset_id,Asset_description,Date) values ('" + Session("Contract_no") + "','" + Session("EotAction") + "'," + b(i) + ",'" + Desc + "', '" + a(i)+ "');"
next
 
    else
    for i=0 to TextBoxValue
    objRIMSConn.Execute "Insert into Eot_returned (Contract_no,Eot_action,Asset_id,Asset_description,Date) values ('" + Session("Contract_no") + "','" + Session("EotAction") + "'," + b(i) + ",'" + Desc + "', '" + a(i)+ "');"
next
end if
Response.Write("<script language='javascript' type='text/javascript'> alert('The value hass been Inserted'); </script>")
     else
    Response.Write("<script language='javascript' type='text/javascript'> alert('Do you want to return any asset'); </script>")
end if 
 

[Rabbit]

date is a reserved keyword. If you're going to use it as a field name, you need to demarcate it as such. Different DBMS will have different methods of demarcation. So you'll need to look up the method for your particular DBMS.

[syedshaffee]

ok i checked that and i changed the name of the field but now can u help me with the loop b'coz the values are doubled in the table

[Rabbit]

I have no idea what values are being passed and what your intentions are. I can't help without knowing those.

[jhardman]

OK, this is a really useful technique I have been doing for over 10 years whenever I don't get the response from the db that I expect:

print out (output to the screen) the SQL statement instead of sending it to the dbms. That way you can look at it and see where it is going wrong. 90% of the time as soon as you look at it you can tell why you are getting a funny response like getting two records added instead of one.

Jared