I have a
search page with the following scenarios listed below. I was told there was a better way to do it, but not how, and that I am using too many if statements, and that it's prone to causing an error through url manipulation:
Search.cfm will processes a
search made from a
search bar present on all pages, with one
search input (titleName).
If
search.cfm is accessed manually (through URL not through using the simple
search bar on all pages) it displays an advanced
search form with three inputs (titleName, genreID, platformID) or it evaluates searchResponse variable and decides what to do.
If simple
search query is blank, has no results, or less than 3 characters it displays an error
If advanced
search query is blank, has no results, or less than 3 characters it displays an error
If any successful
search returns results, they come back normally.
The top-of-page logic is as follows:
<!---SET DEFAULT VARIABLE--->
<cfparam name="variables.searchResponse" default="">
<!---CHECK TO SEE IF SIMPLE
SEARCH A FORM WAS SUBMITTED AND EXECUTE
SEARCH IF IT WAS--->
<cfif IsDefined("Form.simpleSearch") AND Len(Trim(Form.titleName)) LTE 2>
<cfset variables.searchResponse = "invalidString">
<cfelseif IsDefined("Form.simpleSearch") AND Len(Trim(Form.titleName)) GTE 3>
<!---EXECUTE METHOD AND GET DATA--->
<cfinvoke component="myComponent" method="simpleSearch" searchString="#Form.titleName#" returnvariable="simpleSearchResult">
<cfset variables.searchResponse = "simpleSearchResult">
</cfif>
<!---CHECK IF ANY RECORDS WERE FOUND--->
<cfif IsDefined("variables.simpleSearchResult") AND simpleSearchResult.RecordCount IS 0>
<cfset variables.searchResponse = "noResult">
</cfif>
<!---CHECK IF ADVANCED
SEARCH FORM WAS SUBMITTED--->
<cfif IsDefined("Form.AdvancedSearch") AND Len(Trim(Form.titleName)) LTE 2>
<cfset variables.searchResponse = "invalidString">
<cfelseif IsDefined("Form.advancedSearch") AND Len(Trim(Form.titleName)) GTE 2>
<!---EXECUTE METHOD AND GET DATA--->
<cfinvoke component="myComponent" method="advancedSearch" returnvariable="advancedSearchResult" titleName="#Form.titleName#" genreID="#Form.genreID#" platformID="#Form.platformID#">
<cfset variables.searchResponse = "advancedSearchResult">
</cfif>
<!---CHECK IF ANY RECORDS WERE FOUND--->
<cfif IsDefined("variables.advancedSearchResult") AND advancedSearchResult.RecordCount IS 0>
<cfset variables.searchResponse = "noResult">
</cfif>
I'm using the searchResponse variable to decide what the the page displays, based on the following scenarios:
<!---ALWAYS DISPLAY SIMPLE
SEARCH BAR AS IT'S PART OF THE HEADER--->
<form name="simpleSearch" action="
search.cfm" method="post">
<input type="hidden" name="simpleSearch" />
<input type="text" name="titleName" />
<input type="button" value="
Search" onclick="form.submit()" />
</form>
<!---IF NO
SEARCH WAS SUBMITTED DISPLAY DEFAULT FORM--->
<cfif searchResponse IS "">
<h1>Advanced Search</h1>
<!---DISPLAY FORM--->
<form name="advancedSearch" action="
search.cfm" method="post">
<input type="hidden" name="advancedSearch" />
<input type="text" name="titleName" />
<input type="text" name="genreID" />
<input type="text" name="platformID" />
<input type="button" value="
Search" onclick="form.submit()" />
</form>
</cfif>
<!---IF
SEARCH IS BLANK OR LESS THAN 3 CHARACTERS DISPLAY ERROR MESSAGE--->
<cfif searchResponse IS "invalidString">
<cfoutput>
<h1>INVALID SEARCH</h1>
</cfoutput>
</cfif>
<!---IF
SEARCH WAS MADE BUT NO RESULTS WERE FOUND--->
<cfif searchResponse IS "noResult">
<cfoutput>
<h1>NO RESULT FOUND</h1>
</cfoutput>
</cfif>
<!---IF SIMPLE
SEARCH WAS MADE A RESULT WAS FOUND--->
<cfif searchResponse IS "simpleSearchResult">
<cfoutput>
<h1>Search Results</h1>
</cfoutput>
<cfoutput query="simpleSearchResult">
<!---DISPLAY QUERY DATA--->
</cfoutput>
</cfif>
<!---IF ADVANCED
SEARCH WAS MADE A RESULT WAS FOUND--->
<cfif searchResponse IS "advancedSearchResult">
<cfoutput>
<h1>Search Results</h1>
<p>Your
search for "#Form.titleName#" returned #advancedSearchResult.RecordCount# result(s).</p>
</cfoutput>
<cfoutput query="advancedSearchResult">
<!---DISPLAY QUERY DATA--->
</cfoutput>
</cfif>
Is my logic a) not efficient because my if statements/is there a better way to do this? And b) Can you see any scenarios where my code can break? I've tested it but I have not been able to find any issues with it. And I have no way of measuring performance. Any thoughts and ideas would be greatly appreciated.
Many thanks
