Why RegExp Problem ???

[Guest_imported]

Hi,

I have been having problems finding out why this regular expression is not working. The error points to the first line of the below chunk of code.

--------------------------------
StringToSearch = File.FileName
Set RegularExpressionObject = New RegExp

RegularExpressionObject.Pattern = ".exe"
RegularExpressionObject.IgnoreCase = True
RegularExpressionObject.Global = True

Set expressionmatch = RegularExpressionObject.Execute(StringToSearch)

If expressionmatch Then
Response.Redirect "failed.html"
end if

 

[lobstah]

what is "File.FileName"?

 

[Guest_imported]

File.FileName gets a File's name and converts it into a like string

 

[lobstah]

is that the line that the error points to? if so, perhaps it's not your regex that's bad, but something with getting that filename?

 

[Guest_imported]

theres nothing wrong with the File.FileName coz if I do a response.write on it. It'll show

 

[lobstah]

oh, i thought you said that was the line the error was pointing to.

so which line is the error pointing to, and what is the error?

 

[Guest_imported]

The error for some reason points to that line and if I was to change File.FileName to something like "fhdsjkhfjkhf.exe" an error will show about the RegularExpressionObject is undefined ???

 

[vasah20]

you're looking for everything that ends in .exe?

you'll need to change the regexp pattern to look like
re.pattern = "\.exe"

the . is a wildcard char to mean anything, so .exe will always give you more than just the executables.

a better way to do it would be to limit the search to the end of the string. ie: re.pattern = "(\.exe)$"

so only files ending in .exe would get found by that pattern. you might want to check that, though, since I've been using perl the last couple months, and my syntax might be a bit off

hth leo

------------
Leo Mendoza
lmendoza@students.depaul.edu

 

[lobstah]

what browser and version are you using?

 

[tsuji]

Hello, md10943.

expressionmatch is a collection and will not stand ready for If Then test criteria. Hence, the revision would be:

If expressionmatch.count Then
or
If expressionmatch.count<>0 Then

regards - tsuji