Looping through an array using ColdFusion

Question

I have a shopping cart array, which has a variable to tell me if the product is an accessory or not, this will be either yes or no. I need to loop through the cart and find out the following:

If the cart contains accessories only; do whatever.
If the cart is products only; do whatever.
If the cart has products and accessories; do whatever.

I have been trying this:

<cfloop index="i" from="1" to="#arrayLen(session.mycart)#">
  <cfif session.mycart[i].accs EQ "yes">
    <cfset accPresent = "yes">
  </cfif>
  <cfif session.mycart[i].accs EQ "no">
    <cfset prodpresent = "yes">
  </cfif>
</cfloop>

<cfif accPresent EQ "yes" and prodPresent EQ "no">
  <cfset  bothPresent EQ "yes">
</cfif>

This falls down as accPresent is not found, this i think is due to the fact the loop goes through one at a time and the accs is not equal to yes once it find a non accessory product. What's the best way to achieve what I'm trying to do?

Dale Fraser · Accepted Answer · 2012-02-12 21:28:55Z

7

Do this

<cfset accPresent = "no" />
<cfset prodPresent = "no" />
<cfloop index="i" from="1" to="#arrayLen(session.mycart)#">
    <cfif session.mycart[i].accs EQ "yes">
        <cfset accPresent = "yes">
    </cfif>
    <cfif session.mycart[i].accs EQ "no">
        <cfset prodpresent = "yes">
    </cfif>
</cfloop>

<cfif accPresent EQ "yes" and prodPresent EQ "no">
    <cfset  bothPresent EQ "yes">
</cfif>

edited Feb 12, 2012 at 21:28

answered Feb 12, 2012 at 21:23

Dale Fraser

4,7687 gold badges43 silver badges76 bronze badges

Sign up to request clarification or add additional context in comments.

2 Comments

Dale Fraser Over a year ago

Also shouldn't <cfif accPresent EQ "yes" and prodPresent EQ "no"> be <cfif accPresent EQ "yes" and prodPresent EQ "yes">

Dale Fraser Over a year ago

Also the second <cfif could just be an else assuming .accs is only ever yes or no

Mark A Kruger · Accepted Answer · 2012-02-12 21:27:00Z

3

Jason,

Your 3rd statement assumes that AccPresent and ProdPresent will both exist. Did you create them first and give them default values? Try this:

<cfparam name="accPresent" default="no"/>
<cfparam name="prodPresent" default="no"/>
<cfloop index="i" from="1" to="#arrayLen(session.mycart)#">
<cfif session.mycart[i].accs EQ "yes">
<cfset accPresent = "yes">
</cfif>
<cfif session.mycart[i].accs EQ "no">
<cfset prodpresent = "yes">
</cfif>
</cfloop>    
<cfif accPresent EQ "yes" and prodPresent EQ "no">
<cfset  bothPresent EQ "yes">
</cfif>

This assumes of course that each of these should be set to "no" by default.

answered Feb 12, 2012 at 21:27

Mark A Kruger

7,19322 silver badges21 bronze badges

4 Comments

Mark A Kruger Over a year ago

(leigh suggested a CFBREAK as well. That's a good thought - but I suspect that a shopping cart would have a pretty small array, so you would not getting any performance kick out of that. Still, it would actually be the most efficient way to go with 2 loops. As soon as you have a "yes" you break out of the loop and move on to the next block.

Leigh Over a year ago

Actually I misread the question ;) I thought they were only concerned with two conditions (hasAccessories=true/false) so cfbreak made more sense there.

Leigh Over a year ago

Though if it is a small array, they could just count the number of each. Then use the counts to determine if both are present ie <cfif accessoryCount and productCount>has both </cfif>

Jason Congerton Over a year ago

Thanks for your answers, both answers are correct!! I have selected the first as correct and voted up the second answer! Completely forgot to declare the variables! Tired eyes!!! Generally this cart will only have a max of two products, so performance should be good!!

James A Mohler · Accepted Answer · 2014-05-07 20:54:21Z

2

In versions of ColdFusion 8 and higher, a <cfloop> can use an array directly

<cfloop index="i" array="#session.mycart#">
  <cfif i.accs EQ "yes">
    <cfset accPresent = "yes">
  </cfif>
  <cfif i.accs EQ "no">
    <cfset prodpresent = "yes">
  </cfif>
</cfloop>

<cfif accPresent EQ "yes" and prodPresent EQ "no">
  <cfset  bothPresent EQ "yes">
</cfif>

Note: That i refers to the struct containing the data, not position of the data

http://livedocs.adobe.com/coldfusion/8/htmldocs/help.html?content=Tags_j-l_15.html

answered May 7, 2014 at 20:54

James A Mohler

11.1k15 gold badges51 silver badges76 bronze badges

Comments

Russ · Accepted Answer · 2012-07-06 04:28:30Z

1

In CF 10 (or Railo 4), this can be done more elegantly using cfscript and the Underscore.cfc library:

_ = new Underscore();

myCart = duplicate(session.myCart);

accPresent = _.any(myCart, function(val) {
    return val.accs;
});

prodPresent = _.any(myCart, function(val) {
    return !val.accs;
});

bothPresent = accPresent && prodPresent;

The great thing about _.find() is that it stops as soon as the iterator function returns true, so you don't have to iterate over every single element in the array.

Note: Using duplcate() is recommended when accessing shared-scope variables in order to prevent deadlocks.

(Disclaimer: I wrote Underscore.cfc)

answered Jul 6, 2012 at 4:28

Russ

1,9511 gold badge15 silver badges16 bronze badges

Collectives™ on Stack Overflow

Looping through an array using ColdFusion

4 Answers 4

2 Comments

4 Comments

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

4 Answers 4

2 Comments

4 Comments

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related