[rille77]

Nån som kan se vad som är galet den här ASP koden.

Funktionen

Function IsAZ09(strCheck)
  With New RegExp
   .IgnoreCase = True
   .Pattern = "^[a-z][0-9][A-Z]$"
   IsAZ09 = .Test(strCheck)
  End With
End Function

Anropet

IF NOT IsAZ09(strUser) THEN 
 userFel = "2"
 bolFel = True
END IF

Tilldelningen av strUser

strUser = Replace(Trim(Request.Form("user")), "'", "''")

Tacksam för hjälp då jag har kört fast helt.

[Mikael Simonsson]

.Pattern = "^[a-z][0-9][A-Z]$"

Jag skulle gissa på att det är här felet ligger, tyvärr har jag inte möjlighet att testa koden. Som det är skrivet nu måste användarnamnet bestå av 3 tecken, där första måste vara en liten bokstav, andra en siffra osv.

Prova att ändra till:

.Pattern = "^[a-z0-9]+$"

Du behöver inte ha med A-Z när du använder:

.IgnoreCase = True

[Mikael Simonsson]

strUser = Replace(Trim(Request.Form("user")), "'", "''")

Kan tillägga att Replace funktionen här är helt onödig då du inte tillåter ' i IsAZ09() och för att undvika fel bör du testa mot SQL-injection när du bygger din SQL-fråga, helst inte tidigare.

[rille77]

QUOTE
CODE
strUser = Replace(Trim(Request.Form("user")), "'", "''")



Kan tillägga att Replace funktionen här är helt onödig då du inte tillåter ' i IsAZ09() och för att undvika fel bör du testa mot SQL-injection när du bygger din SQL-fråga, helst inte tidigare.

Stort tack för hjälpen Mikael, är inte så hemma på RegExp än.
Fråga bara, varför bär man inte testa för SQL injections före SQL frågan???

[Mikael Simonsson]

Originally posted by rille77@Oct 13 2004, 23:19
Fråga bara, varför bär man inte testa för SQL injections före SQL frågan???

(Ursäkta mitt lite sena svar.)

Jo, för att kunna hantera alla variabler under samma förutsättningar. Säg att du vill skriva ut det felaktiga användarnamnet eller fylla i ett formulär åt användaren, så att användaren kan rätta till sitt fel utan att behöva skriva om allt. Då måste du komma ihåg om du har bytt ut alla ' mot '' för att sen göra det motsatta när du ska skriva ut variablen.

Så en bra regel är att hantera alla variabler som "fientliga", oavsett om du hämtar data från en databas, cookies, Request objektet osv. När du bygger en SQL-fråga testar du mot SQL-injection och när du skriver ut en variabel använder du Server.HTMLEncode().

Exempel:

strUser = Trim(Request.Form("user"))
intUserId = Request.Form("userid")

' Bygga en SQL-fråga (sträng)
strSql = "SELECT * FROM users WHERE username = '" & Replace(strUser, "'", "''") & "'"

' Bygga en SQL-fråga (tal)
strSql = "SELECT * FROM users WHERE userid = " & Int(intUserId)

' Skriva ut en variabel
Response.Write(Server.HTMLEncode(strUser))
Response.Write(Server.HTMLEncode(intUserId))

[digiArt]

Din regexpsträng borde också kunna skrivas som

^\w$

[Mikael Simonsson]

Originally posted by digiArt@Oct 16 2004, 14:50
Din regexpsträng borde också kunna skrivas som

^\w$

Din kod fungerar bara med ett tecken och enligt dokumentationen är \w med underscore:

\w - Matches any word character including underscore. Equivalent to "[A-Za-z0-9_]".

Sen är nog "^[a-z0-9]+$" lättare att förstå om man inte jobbar med Reg.Exp ofta.