ASP - Calling a result set and populate as selection from DB

[yimma216]

Hi, I am trying to call a recordset from sql server and populate them into a html drop sown selection box.

 Set conn = CreateObject("ADODB.Connection") 
conn.Open "Driver={SQL Server};Server=cslproxy;Database=gra;Trusted_Connection=yes;"
sql = "select * from rerturn_reasons"
rs2.open sql, conn
If Not rs2.eof Then
    response.write vbcrlf & "<tr><td>Reason for Return:</td><td><select name=selectreason>"
 
Do While Not rs2.eof
    response.write vbcrlf & "<option"
If rs2(0) = rs2("option_text") Then
    response.write vbcrlf & "selected=selected"
    End If
response.write ">" & rs2("option_text") & "</option>"
    rs2.movenext
Loop
response.write vbcrlf & "</select></td></tr>"
End if
rs2.close
conn.close
 

but i got error on it. Could any body check my code?

[idsanjeev]

can you provide more detials about error

[DrBunchman]

As idsanjeev says we'll need the error to be sure of what the problem is. Glancing at your code though you don't seem to create your recordset object anywhere e.g.

Set rs2 = CreateObject("ADODB.RecordSet")

Dr B

[jeffstl]

Without knowing which line you are getting the error, I am seeing 2 problems that would cause errors for certain.

One you need to specifiy the server object to create a connection. Like this

Set conn = Server.CreateObject("ADODB.Connection") 
 

Second thing is in ASP code, Do While loops are ended with the syntax wend (short for While Loop End I would imagine) So you need this

Do While
 
Wend
 

[jhardman]

... in ASP code, Do While loops are ended with the syntax wend (short for While Loop End I would imagine) So you need this

Do While
 
Wend
 

no, "do while ... loop" is correct. There is an alternate syntax "while...wend" which does not use the "do". these two structures are completely redundant, both work perfectly.

Jared

[jeffstl]

no, "do while ... loop" is correct. There is an alternate syntax "while...wend" which does not use the "do". these two structures are completely redundant, both work perfectly.

Jared

woops

Scattered brain, correct...sorry

[yimma216]

Thanks every one, I got it fixed.