Why? Maybe for refactoring? Maybe you are repeating your code, and realize, "hey, I can make a function that does this and save like 100000 lines of code!"? Maybe someone trying to grok the differences between JavaScript and Coldfusion functions and whatnot?

I don't know and it doesn't matter. Read the damn blog before you post a damn question. GAAAH!

:-P

Just ribbin ya, Mr. ecommerce pro. Cuz uh felt like it. Nothing really wrong with questioning and all. :-)

***

Nice little post Barney, some CF noob will no doubt find it useful. I liked the short exp. of the diff. between javascript funcs and cf funcs myself. :-)

I'd still recommend a CFLOOP LIST="" loop if you're using tags, but if you're using script (where you don't have a native list loop), converting to an array is almost always a good idea.

Basically, as you said, performing a ListToArray() first and looping over the Array is MUCH quicker than my original suggestion.

My only excuse is that I extrapolated quote from the performance doc to also mean "for" loops (for some reason???).

In any case, I created a test that created a list with 10,000 items with randomly generated data that varied in length between 5 and 15 characters.

I kept the test very simple and merely looped over the list (or array) and set a variable to the item value.

The results are quite staggering.

With a list of 10,000 items these are the results running on CFMX7 on a P4 2Ghz with 1GB Ram:

List Time: ~32,500 ms
Array Time: ~0 ms (max 15 ms)

Perhaps an additional line needs to be added to that coding standards docs about lists in CFSCRIPT!! ;)

I'm posting this code on our blog.

Thanks for the challenge! ;)

Regards

Niklas

It would be interesting to compare the performance of a for loop with listToArray() and a for loop with listGetAt() for 100, 1000 items.

Maybe I'll throw something together if I have some time.

Regards

Niklas

Yeah, for what I was doing, a CFLOOP LIST="" would be perfect, but as the title of the post indicates, I was looking for CFSCRIPT-based version of that loop. As the MM coding guidelines indicate, listGetAt is inefficient, and so I avoided it by using an array. Using a CFLOOP tag was impossible, because it can't be run in CFSCRIPT.

"Don't slavishly convert lists to arrays

Even though manipulating an array is generally faster than manipulating a list in CFMX, if you simply need to iterate over a list of items and process each one in turn the faster construct is <cfloop list="#itemList#" index="x"> … </cfloop>. Don't convert itemList to an array and then loop over that – it's not worth it because it probably won't be faster."

I understand that looping over an array is quicker than looping through a list.

However, in simple loops (as above), it was my understanding that if you were to convert the list to an array and then loop over the array you would not be increasing performance as some work needs to happen to convert that list to array in the listToArray() function anyway. So the performance saving is negligable.

This understanding was also confirmed by the ColdFusion MX Coding Guidelines and specifically this bit:

"Don't slavishly convert lists to arrays

Even though manipulating an array is generally faster than manipulating a list in CFMX, if you simply need to iterate over a list of items and process each one in turn the faster construct is … . Don't convert itemList to an array and then loop over that – it's not worth it because it probably won't be faster."

http://livedocs.macromedia.com/wtg/public/coding_standards/performance.html

Towards the end of the article.

And:

"Use arrays instead of lists – in general

In CFMX, lists suffer from the generally slow string processing in Java which means that list manipulation can be slower than in CF5. In general, it is better to work with arrays of items instead of lists of items: listGetAt() is not an efficient way to work with individual items in a data set! However, see the list vs array caveat in the Don't section below."

Regards

Niklas

Niklas: Dereferecing an array is faster than dereferecing a list, particularly when the list is long. Each list operation has to tokenize the list, while an array is already broken down. So by doing the tokenization onces (the listToArray call), I save a bunch of time. I did some quick test, and with just a couple elements, the list operations were quicker. By the time you had 10 elements, the array method took about 50% of the time, and when you got to 20 elements, the array method took just under 10% of the time. So for lists that have more than a few elements, converting to an array first will be quicker.