error checking doesn't appear to be working

S

Steven Scaife

Below is my ASP page, I have changed the update to read DRIAL which doesn't
exist, shouldnt this throw an error, or if the connection cannot be made
shouldnt it throw an error as well

thanks in advance

<%@language="VBScript"%>
<% option explicit %>
<% response.buffer = True %>

<%
dim telNo, comp, adbTable
telNo = request.form("telNo")

response.write telno & "<BR>"
comp = inStr(1, telNo, " ", 1)

if comp <> 0 then
response.redirect "phone.asp?QS=yes&num=" & telno
end if
%>

<%
Dim conn, RS, strSQL, strConnImpreza, strConnDom_Con,
strConnImprezaTouchStar

'connection details
<% server.execute ("connections.inc") %>
'error checking starts here, if an error is encountered display a msg to the
user and carry on
on error resume next

conn.open strConnImpreza

'this sql statement can be run on both impreza and dom_con tables Amcat'
strSQL = "INSERT INTO MasterDoNotCall(phonenum) VALUES ('" & telNo & "');"
conn.execute strSQL 'run the sql statement

disperror("imprezanocall") 'call the error function

response.write strSQL & "<BR>"
conn.close 'close the connection'


conn.open strConnDom_Con 'open the connection to dom_con'

strSQL = "INSERT INTO MasterDoNotCall(phonenum) VALUES ('" & telNo & "');"
conn.execute strSQL

disperror("DomConNoCall")

response.write strSQL & "<BR>"
conn.close


'for this table we need to run two sql statements, first open the
connection'
conn.open strConnImprezaTouchStar
disperror("connTouchstar")

strSQL = "INSERT INTO Nevercall (PhoneNum, EffectiveDate) " & _
"VALUES ('" & telNo & "', Getdate())"
conn.execute strQSL

disperror("imprezaTouchstarNC")

response.write strSQL & "<BR>"

strSQL = "UPDATE DIAL SET CRC = 'DNC' WHERE PhoneNum = '" & telNo & "';"
conn.execute strSQL

disperror("imprezaTouchstar")

response.write strSQL & "<BR>"


conn.close
set conn = nothing


%>
<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.0 Transitional//EN">

<html>
<head>
<title>Updated Databases</title>
<style>
<!--
body { font-family: arial, helvetica, sans-serif; font-size: 10px; }
//-->
</style>
</head>

<body>
Detail have been inserted into the corresponding databases


</body>
</html>
<%
sub dispErr(adbTable)
' if an error is raised then trap it here and end processing
IF err.number <> 0 then
%>

<p>Record already exists on the server and table.</p>
<%
SELECT CASE adbTable
case "imprezanocall" %>
<p>Server: Impreza, Database: AMcat, Table: MasterDonotcall; Was not
updated</p>
<% case "DomConNoCall" %>
<p>Server: Dom_Call, Database: AMcat, Table: MasterDonotcall; Was not
updated</p>
<% case "imprezaTouchstar" %>
<p>Server: Impreza, Database: Touchstar, Table: Dial; Was not
updated</p>
<% case "imprezaTouchstarNC" %>
<p>Server: Impreza, Database: Touchstar, Table: Nevercall; Was not
updated</p>
<% case "connTouchstar" %>
<p>Connection to Touchstar failed</p>
<% END SELECT
end if
Err.Clear

end sub
%>
 
S

Steven Scaife

Tarwn said:
Your first error is in the last line I pasted above. Since you are already
in an ASP script block you cannot (and do not need to) enter another script
block in order to execute the Server.Execute call. If you weren't receiving
an error on this line (at te very least) then you mayhave deaper problems, as
this line is a syntax error that occurs before your On Error Resume Next

-T
Click to expand...

In my page that line isn't there, the strings of my connection are there, i
just put that there to show that some form of connection method is there,
didn't wanna show the different server connection strings, paranoia on my
part
 
T

Tarwn

That explains a lot, I was trying to figure out how you would not be getting
an error in that situation :)

I've looked through the rest of your code and if it is a logic error, I am
not seeing it. My assumption is that the line you rendered inoperable was
the last SQL entry.

An error should be thrown at that point, something to the effect of table
not found. What type of database are you operating on (MS Access, SQL Server,
etc)? Not sure it will matter but it may.

One thing I would try is to cut it down to the minimal amount of code to try
and duplicate the lack of error. Comment out everything except the connection
string, creation of the connection object, the sql string, execution of the
sql string, and a one line "If err.number <> 0 Then Response.Write
err.description"

If you get the error in that case then uncomment the
dispError("imprezaTouchstar") line. Etc. Work your way from the most simple
to the most broad.

-T
 
S

Steven Scaife

Just noticed why it wouldn't work, i am calling disperror instead of
disperr. Doh!

thanks for the help
 

Ask a Question

Want to reply to this thread or ask your own question?

You'll need to choose a username for the site, which only take a couple of moments. After that, you can post your question and our members will help you out.

Ask a Question

Similar Threads

PHP/MySQL UPDATE not working for second table 1
Can't execute php to delete multiple rows in database 3
Registration Form 7
How can i fix Mysql images not found error? 0
Survey details won't go through using php, ajax, Mysql 0
Cannot Find Error on this ASP 4
Item cannot be found in the collection corresponding to the requested name or ordinal 2
Using Arrays to write data to SQL Server 2

Members online

Total: 58 (members: 3, guests: 55)
Robots: 391

Forum statistics

Threads
473,968
Messages
2,570,153
Members
46,699
Latest member
AnneRosen

Latest Threads

Top