valider form nummer

If len(formfelt) <> 8 And isNumeric(formfelt) = FALSE Then
  'fejl
End If

Det er mærkeligt, det er som om den springer valideringen over??

Her er min kode

<%
If Request.Form("Action") = "Send" Then
%>

<%
If len(Request.Form("sms")) <> 8 And isNumeric(Request.Form("sms")) = FALSE Then
%>
      <script language="javascript">
      <!--
      window.alert ("Nmmeret kan kun indeholde tal");
      window.history.go(-1);
      //-->
      </script>
<%
End If

Dim rs1, rs2
If Request.Form("RADIO") = "false" Then
        rs1 = "DELETE * FROM sms WHERE sms=" & Request.Form("sms") & ""
        objConn.Execute(rs1)
        Session("strDone") = "<strong><font color=""#990000"">("& Request.Form("sms") &" er slettet)</font></strong>"
        else
        rs2 = "INSERT INTO sms (sms) VALUES ('"& Request.Form("sms") &"')"
        objConn.Execute(rs2)
        On Error GoTo 0
        Session("strDone") = "<strong><font color=""#990000"">("& Request.Form("sms") &" er tilføjet)</font></strong>"
    End If

Hvis jeg skriver 1234abcd, kommer der bare fejl i SQl sætningen:
Datatyperne stemmer ikke overens i kriterieudtrykket.

Jeg tror lige at du skal vise dín form-kode.

Hvis der står:

<form action="Send" method="post">

- så er der i hvert fald noget galt allerede der.

Min form starter med:
<form method="POST" action="addSMS.asp">
og slutter med
<input type="submit" name="Action" value="Send">

Formen ligger også på addSMS.asp

Det er en rigtig dårlig ide at bruge navnet action for sin submit-knap. Det har jo allerede en betydning i <form>-taggen.

Kald din submit-knap ved et andet navn (og helst ikke for "submit" i stedet).

Aarrggghh, det er mig der fucker rundt i det...
keysersoze dit svar virker...

Tak for hjælpen

skulle lige til at skrive en masse - men den korte version nu hvor du har fået det til at virke; hvis du vil alerte med js så tjeck også med js og ikke med asp - dette skal bare køres på onsubmit. Validering med ASP kan ikke altid undgås - det er for at "fange" folk med javascript slået fra - og her skal du som udgangspunkt så holde dig fra at blande js ind i det men i stedet bare skrive en almindelig besked til skærmen.

Man bør *både* tjekke på client- og serverside.

På clientside gør man det for st hjælpe sine brugere.

PÅ serverside gør man det for at sikker sig imod ukorrekt input.

Folk kan have slået JavaScript fra, eller onde hacker-typer kan have lavet deres egen forms med det specifikke formål at angribe din side.

Tak for jeres input