Hi all. I'm building an app using model glue and reactor. I'm having an issue
with a function I wrote that returns my navigation to me, appropriately nested.
The function is a recursive one, and works as designed, no problem there. BUT,
it's painfully slow, even with debugging and reactor regeneration disabled. SO,
I'm wondering if any of you can look at how I've approached this and tell me if
anything jumps out at you as obviously inefficient. Thanks!
The specs:
End value returned:
<div id="menu0">Administration</div>
<div id="menu1">view reports</div>
<div id="menu1">search employees</div>
<div id="menu2">easy on the bottom line</div>
<div id="menu1">enrollment stats</div>
<div id="menu1">system reports</div>
<div id="menu1">Company profile</div>
<div id="menu1">manage system users</div>
<div id="menu1">message center</div>
<div id="menu0">Enrollment</div>
<div id="menu1">new hire enrollment</div>
<div id="menu1">life status event</div>
<div id="menu1">view current benefits</div>
<div id="menu0">About Us</div>
<div id="menu0">Cwes features</div>
<div id="menu1">streamlined open enrollment</div>
<div id="menu1">customizable reports</div>
<div id="menu1">ease of use</div>
<div id="menu1">one stop shop</div>
<div id="menu0">Contact Us</div>
My nav table structure:
NavigationID (primary key, auto incrementing integer);
name (varchar, name of my nav item);
output_text (varchar, text to use for the nav item);
link (varchar, link for the nav item);
parentid (integer, ID of the parent navigation item (= navigationID));
navorder (tinyint, sort order for navigation)
The CFC performing the nav building:
<cfcomponent displayname="Controller" output="false" hint="I am a sample
model-glue controller." extends="ModelGlue.Core.Controller">
<cfset variables.reactor = application.recordfactory />
<cfset variables.navcontainer = "" />
<cffunction name="Init" access="Public"
returnType="controller.navcontroller" output="false" hint="I build a new
SampleController">
<cfargument name="ModelGlue" required="true"
type="ModelGlue.ModelGlue" />
<cfargument name="InstanceName" required="true" type="string" />
<cfset super.Init(arguments.ModelGlue) />
<cfreturn this />
</cffunction>
<cffunction name="getNav" access="public" returnType="void"
output="false">
<cfargument name="event" type="ModelGlue.Core.Event"
required="false">
<cfargument name="parentID" type="numeric" required="yes"
default=0 />
<cfargument name="level" type="numeric" required="yes"
default=0 />
<cfset var objNav = "" />
<cfset var objItem = "" />
<!--- grab all the elements whose parent is
'arguments.parentID' --->
<cfset objNav =
variables.reactor.creategateway("navparent").getByFields(parentid=arguments.parentid)
/>
<!--- loop through query --->
<cfloop query="objNav">
<!--- add this item to global variable --->
<cfset variables.navcontainer = variables.navcontainer
& "<div id=""menu" & arguments.level & """>" & objNav.name & "</div>" & chr(10)
/>
<!--- check for children. If there are any, call this
function recursively --->
<cfif
variables.reactor.createrecord("navparent").load(navigationid=objNav.navigationid).getnavchilditerator().hasmore()>
<cfset getNav(parentID =
objNav.navigationID, level = arguments.level + 1) />
</cfif>
</cfloop>
<!--- loop finished --->
<!--- return final variable --->
<cfif arguments.level eq 0>
<cfset
arguments.event.setValue("navtext",variables.navcontainer) />
</cfif>
</cffunction>
</cfcomponent>
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~|
Message: http://www.houseoffusion.com/lists.cfm/link=i:4:237681
Archives: http://www.houseoffusion.com/cf_lists/threads.cfm/4
Subscription: http://www.houseoffusion.com/lists.cfm/link=s:4
Unsubscribe: http://www.houseoffusion.com/cf_lists/unsubscribe.cfm?user=89.70.4
Donations & Support: http://www.houseoffusion.com/tiny.cfm/54
