[thelist] Effcient ASP Script?

David at softv.net David at softv.net
Mon Nov 25 14:55:01 CST 2002


Why are you are storing your connection string in a session object? This
creates unnecessary overhead. Unless you are using multiple connection
strings it would be better to store it in an Application variable.

You should also explicity name the fields you are selecting rather then
"SELECT * ".

Also, look into using disconnected recordsets.

If you are using SQL Server you can run a SQL profiler to see what other
activity is happeing with the DB server at the same time which may be
hurting performance, and you can also run performance monitors on IIS to see
if there are any bottlenecks.

Dave

<%
David Mitchell
Application Developer
david at sofTV.net
sofTV.net
www.sofTV.net
613.729.6400 x268
%>



-----Original Message-----
From: Adam Slesinger [mailto:badlanners at hotmail.com]
Sent: Monday, November 25, 2002 3:45 PM
To: thelist at lists.evolt.org
Subject: [thelist] Effcient ASP Script?


Hi folks!

I am writing a page that selects all records with company names and then
updates the type of those records if they are the first 2000 records.  The
database has a total of 13,000 records.  When I run this on my server, it
takes a long time, and then times out.

My 1st questions is, how can I keep any script in general from timing out?

My 2nd question is - is my code efficient?  I wouldn't think that it would
take so long for this process to run, but it only gets through about 1400
records before the timeout happens.

Any help is appreciated!

Thanks,
adam

set conn = server.createObject("ADODB.Connection")
conn.open Session("connection")
set rstemp = server.createobject("ADODB.Recordset")

SQL = "SELECT * FROM registrants WHERE company <> ''"
rstemp.open SQL, conn

do while not rstemp.eof
    strCount= strCount+ 1

    if strCount <= 2000 and strCount > 0 then
        'Update Type
        set typeUpdate= conn.execute("UPDATE registrants SET type= 1 WHERE
company = '" & rstemp("company") & "'")
    end if

rstemp.moveNext
loop

conn.close
set conn = nothing
--
* * Please support the community that supports you.  * *
http://evolt.org/help_support_evolt/

For unsubscribe and other options, including the Tip Harvester
and archives of thelist go to: http://lists.evolt.org
Workers of the Web, evolt !



More information about the thelist mailing list