God validering

1:

http://activedeveloper.dk/download/default.asp?mode=showdownload&id=140

2 + 3.

<input type="text" name="test">

<%
if isnumeric(request.form("test")) then
  if len(request.form("test")) > 4 then
      'Fejl
  end if
end if
%>

/Rune

rune > kigger lige lidt på det....

rune > Jeg har flg kode :

****************************************
if Request.ServerVariables("REQUEST_METHOD") = "POST" then


Navn = Request.form("Navn")
Brugernavn = Request.form("Brugernavn")
Password = Request.form("Password")
Adresse = Request.form("Adresse")
Postnummer = Request.form("Postnummer")
Bynavn = Request.form("Bynavn")
Telefonnummer = Request.form("Telefonnummer")
Email = Request.form("Email")
Timenow = Now()
Godkendt = False

SQL = "INSERT INTO tbl_users (Brugernavn, Password, Email, Navn, Adresse, Postnummer, Bynavn, Telefonnummer, PID, Timenow, Godkendt) VALUES ('" & Brugernavn & "', '" & Password & "', '" & Email & "', '" & Navn &"', '" & Adresse &"', " & Postnummer &", '" & Bynavn &"', " & Telefonnummer &", '" & IDpas &"', '" & Timenow & "', " & IIf(Godkendt, "True", "False") &  ")"

Conn.execute(SQL)
****************************************
Hvor skal lortet sættes ind? Jeg forestiller mig at det skal ind umiddelbart efter jeg har deklareret mine vars???

Jeg går ud fra at det er feltet postnummer du vil tjekke og er 4 tal

<%
if Request.ServerVariables("REQUEST_METHOD") = "POST" then

Navn = Request.form("Navn")
Brugernavn = Request.form("Brugernavn")
Password = Request.form("Password")
Adresse = Request.form("Adresse")
Postnummer = Request.form("Postnummer")
Bynavn = Request.form("Bynavn")
Telefonnummer = Request.form("Telefonnummer")
Email = Request.form("Email")
Timenow = Now()
Godkendt = False


if isnumeric(Postnummer) then
  if not len(Postnummer) > 4 then
      SQL = "INSERT INTO tbl_users (Brugernavn, Password, Email, Navn, Adresse, Postnummer, Bynavn, Telefonnummer, PID, Timenow, Godkendt) VALUES ('" & Brugernavn & "', '" & Password & "', '" & Email & "', '" & Navn &"', '" & Adresse &"', " & Postnummer &", '" & Bynavn &"', " & Telefonnummer &", '" & IDpas &"', '" & Timenow & "', " & IIf(Godkendt, "True", "False") &  ")"

    Conn.execute(SQL)
  end if
end if

end if
%>

Hvis jeg indsætter
if isnumeric(Postnummer) then
  if not len(Postnummer) > 4 then
    Response.write "<p>FEJL</p>"
  else
SQL....

Sker der ikke en skid...

Altså den kommer hverken m fejl eller smækker indhold i db...

Det må da være smartere at lave

if isnumeric(Postnummer) then
  if len(Postnummer) > 4 then
    Response.write "<p>FEJL</p>"
  else

eller hvad?

den sidste er til at foretrække...

du kan også vælge at sætte en maxlength på din input:

<input type="text" name="Postnummer" maxlength="4">

Du skal vel også gøre dette ved telefonnumret:

<input type="text" name="Telefonnummer" maxlength="8">

rune > Det sidstnævnte er meget smart, men det giver jo absolut ingen garanti for at det kun bliver numeriske værdier ):

nej men så behøver du ikke at teste for det i dit script! Det er bare en smagssag

Kommandoen isNumeric dur ikke umiddelbart til at kontrollere med.
isNumeric returnere true hvis Request.QueryString("postnr") er blank. ! (og dermed bliver Postnummer blank.)
Den skal se således ud:

if isnumeric(CStr(Postnummer)) then
  if CInt(Postnummer) < 1000 or CInt(Postnummer) > 9999 then
    Response.write "<p>FEJL</p>"
  else

Med mindre du vil acceptere udenlandske postnumre som eks: 100 = Thorshavn

En kontrol af e-mail skal mindst kontrollere for følgende:
1. Kun et @ og kun et .
2. Noget før @ og noget efter .
3. og noget i mellem @ og .

kunne også lave et klient script der tjekkede om den var tom, før man overhovedet begyndte at sende oplysningerne.