correct way to free memory

hello,

can someone please tell me if the above example is totally corerct to free OLEDB connection and memory of asp.net applications?

VB Code:

'connection
Dim StrConnect as String, StrOpen as string
Dim DBconn As OleDbConnection, DBcomm as OleDbCommand, DBread as OleDbDataReader
StrConnect = "Provider=Microsoft.Jet.OLEDB.4.0; Data Source=mydb.mdb"
 
DBconn=New OleDbConnection(StrConnect)  'connection to DB
DBconn.Open()
'connect
Stropen="Select * " & _
   "FROM structure " & _
   "WHERE " & _
   "(pid = " & pid & ")"
DBcomm=New OleDbCommand(Stropen,DBconn)
DBread=DBcomm.ExecuteReader()
 
if DBread.read() then
   Title.text = DBread("name")
end if
 
DBread.close()
DBconn.close()
 
'free memory
DBcomm.dispose()
DBconn.dispose()
DBread = nothing

thank you very much,

wc.

I believe that you can dispose of DBread as well. Not sure because I havn't used this kind of data reader before. I use the SQL type of data read.

For this line.

VB Code:

Dim StrConnect as String, StrOpen as string

The more efficient way to do this would be

VB Code:

Dim StrConnect, StrOpen as string

Besides that, it looks good to me. With .Net, the garbage collector will clean up the memory for you. However, it is still good practice to dispose of the variables like you are doing.

:thumb:

thank you indydavid32,

even though .net is a complete new programming environment, i'm pretty sure as well that manually settings objects to null is the best way to do so. i'm an old vb6 programmer :)

but why

VB Code:

Dim StrConnect, StrOpen as string

would be a more efficient way?

in old (again) vb6, this would mean that the first variable would be declared as variant. is this different in .net?

finally, i'm pretty sure you cannot dispose of a OleDbDataReader, this is why i'm setting it to nothing instead.

cheers,

wc.

In .Net, dim string1, string2, string3, string4 as String will set all of them to string.

ok, thank you!

Your welcome. Come to think of it, you can't dispose of the sql data reader either. I had a brain fart there. lol