Comments on Dale Fraser's Blog: ColdFusion Convert A Query Row to a Struct in 1 li...

Hmmm, you don't need those parentheses, and si...

2010-07-23T20:56:20.906+10:00

Hmmm, you don't need those parentheses, and since a column name must be at least one character, you should use + instead of *, so the expression can be: '\b[^,]+\b'
With the replace being: '"\0" : "##q.\0[1]##"'

Actually, no need for the word boundaries either, it can be '[^,]+' and '##q['\0'][1]##'

And case-sensitivity doesn't come into it, so overall it could be changed to:
<cfset qRow = deserializeJson("{#evaluate(DE("#REReplace(q.ColumnList, '[^,]+', '"\0" : "##q["\0"][1]##"', 'ALL')#"))#}") />

However, I can't think of any benefit to that over doing a simple loop:

<cfset qRow = StructNew() />
<cfloop index="CurCol" list="#q.ColumnList#">
<cfset qRow[CurCol] = q[CurCol][1] />
</cfloop>

If you want that to be one line, make it a function and do:
<cfset qRow = QueryRowToStruct(q,1) />

Oh and well done Steve, I didn't see that you ...

2010-07-23T12:28:42.404+10:00

Oh and well done Steve, I didn't see that you had written it :)

@Steve Onnis Yeah I was wondering the same thing. ...

2010-07-23T12:22:48.324+10:00

@Steve Onnis
Yeah I was wondering the same thing. I generally don't use Evaluate() if there is another efficient way to get around it (there often is).
Performance from what I hear has improved greatly and as for security in this case, the inputs look pretty controlled to me. Looks secure. A speed comparison to a looped solution might be cool to see.

Nice post Dale, bookmarked :)

So its isn't a performence issue then, a secur...

2010-07-22T22:19:14.891+10:00

So its isn't a performence issue then, a security issue? If you are in control over what is being evaluated then the argument is void as as far as i am concerned. Like anything..it's bad if you use it poorly

@Steve Onnis: It has everything to do with the pri...

2010-07-22T21:46:48.286+10:00

@Steve Onnis: It has everything to do with the principle behind the function: allowing unknown code to be executed on the server. It's just a potential security risk and should therefore be avoided at all times.

At least, that's what I think. :)

And why exactly is Evaluate() bad? Its performence...

2010-07-22T21:34:45.327+10:00

And why exactly is Evaluate() bad? Its performence has improved heaps since CF6 and i honestly don't see why it is so bad...unless someone can prove its evilness?

I agree, its very cool, and shows the possibilitie...

2010-07-22T20:38:44.698+10:00

I agree, its very cool, and shows the possibilities but I do use a small function and loop over in real use.

I would call this a nice proof-of-concept, but wou...

2010-07-22T20:32:22.022+10:00

I would call this a nice proof-of-concept, but would never use it in any serious production environment. I avoid usage of the Evaluate() function at all times because it just means trouble.

I would go with a UDF. Shouldn't be that hard.