I've got an array in inc_header that looks like this: intAllowedCategoryArray = Array(2,3,5,10,11,12,13)
I want to show a new post button on forum.asp and topic.asp beside the 'New Topic' buttons.
I've got a code check that works on post.asp:
For x = 0 to UBound(intAllowedCategoryArray)
If strRqForumID = intAllowedCategoryArray(x) Then
blnForumCheck = 1
x = UBound(intAllowedCategoryArray)
End If
Next
If blnForumCheck = 1 Then
I need this modified to work on forum.asp and topic.asp where the strRqForumID variable is different .. or not called that at all. Anyone got any ideas?
Do I need to declare something related to the Forum_ID at the top of each page?
if (Request("FORUM_ID") = "" or IsNumeric(Request("FORUM_ID")) = False) and (Request.Form("Method_Type") <> "login") and (Request.Form("Method_Type") <> "logout") then
Response.Redirect "default.asp"
else
Forum_ID = cLng(Request("FORUM_ID"))
end if
On another note, rather than looping through an array until you find a match, make inAllowedCategoryArray a string and us an if statement to see if Forum_ID is in that string as follows:
if instr(","&intAllowedCategoryArray&",",","&Forum_ID&",")>0 then ...
Search is your friend “I was having a mildly paranoid day, mostly due to the fact that the mad priest lady from over the river had taken to nailing weasels to my front door again.”
It works for Forum.asp but I get a type mismatch error in topic.asp
For x = 0 to UBound(intAllowedCategoryArray)
If rsTopic("FORUM_ID") = intAllowedCategoryArray(x) Then
blnForumCheck = 1
x = UBound(intAllowedCategoryArray)
End If
Next
If blnForumCheck = 1 then Response.Write ...etc..
I tried replacing that block of code with
if instr(","&intAllowedCategoryArray&",",","&Forum_ID&",")>0 then
For x = 0 to UBound(intAllowedCategoryArray)
If rsTopic("FORUM_ID") = intAllowedCategoryArray(x) Then
blnForumCheck = 1
x = UBound(intAllowedCategoryArray)
End If
Next
up before
rsTopic.close
set rsTopic = nothing
line 169 Shaggy, if you get a chance I'd like to explore using your suggestion of a string instead of the way I'm doing it if it is faster and uses less resources. Thanks for all your help (again!).
If you needed to, you could place it back where it was and use Forum_ID instead of rsTopic("FORUM_ID").
Unless you have a huge array of forums you're checking for, you're not going to notice any significant improvement in execution time or resource usage using instr. Still, no harm in keeping your options open. The reason you got a type mismatch error when you tried using it was because intAllowedCategoryArray is an array - to use instr, you'll have to make it a string instead.
Search is your friend “I was having a mildly paranoid day, mostly due to the fact that the mad priest lady from over the river had taken to nailing weasels to my front door again.”
Topic Locked
Posted - 12 May 2005 : 21:46:44
