> <cfset TotalOptPrice = 0>
> <!--- Find how many options are added to this
> product, if any
> <cfloop index="pricenum" from="1"
> to="#ListLen(Options, "^")#">
> <!--- Set variable to option price --->
> <cfset CurrentOptionPrice = #ListGetAt(OptionPrice,
> PriceNum,
> "^")#>
> <cfset TotalOptPrice = (#TotalOptPrice# +
> #CurrentOptionPrice#)>
> </cfloop>
I'd start by using a list loop instead of an iterative loop...
<cfloop index="thisprice" list="#Options#" delimiters="^">
<cfset TotalOptPrice = TotalOptPrice + thisPrice>
</cfloop>
Or better yet, eliminate the loop altogether
<cfset TotalOptPrice = ArraySum(ListToArray(Options,"^"))>
This is probably more efficient. It's _definitely_ less code.
Isaac
Certified Advanced ColdFusion 5 Developer
www.turnkey.to
954-776-0046
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~|
Archives: http://www.houseoffusion.com/cf_lists/index.cfm?forumid=4
Subscription: http://www.houseoffusion.com/index.cfm?sidebar=lists&body=lists/cf_talk
FAQ: http://www.thenetprofits.co.uk/coldfusion/faq
Get the mailserver that powers this list at http://www.coolfusion.com
