Global.asa good practice?

[Andyza]

In my Global.asa file I have the following 2 subs for opening and
closing my db connection:

Sub OpenConn()
Dim conn
Set conn = Server.CreateObject("ADODB.Connection")
conn.Open Application("ConnString")
End Sub

Sub CloseConn()
conn.Close
Set conn = nothing
End Sub

Then in my asp pages I simply call the subs when I need to open and
close the db connection.

Eg.:

Dim Test
Test = "Select XXX from SomeTable"
OpenConn()
Set rs = conn.Execute(Test)
Do While Not rs.eof
Response.Write(rs("XXX"))
rs.MoveNext()
Loop
CloseConn()

Is this good practice to have the 2 db subs in the Global.asa?

 

[Andyza]

Ok, bad idea! I wasn't thinking! It won't work - the subs will be out
of scope in the Global.asa...

 

[Bob Barrows [MVP]]

Ok, bad idea! I wasn't thinking! It won't work - the subs will be out
of scope in the Global.asa...

Not only that, but read this:
http://www.aspfaq.com/show.asp?id=2053

Bob Barrows

 

[Aidy]

As you have found out, that doesn't work, but you can put those pages in a
separate file and INCLUDE them in each ASP page you need them.