Tim,

I wouldn't call that bad practice, It should actually be called good
practice (at least in my book).  You are setting a variable and the ##
around the variable show that it is a variable, makes for easier reading.
Now, ## on the set side would be considered bad practice.

my .02

Paul Giesenhagen
QuillDesign
http://www.quilldesign.com
SiteDirector Commerce Builder


> Not sure if this is an error or just a bad practice, but you don't need
the
> # signs in your variable declaration (cfset)
>
> <CFSET var = ListGetAt(Form.FIELDNAMES, i)>
>
> works just fine.  Also you don't need to generate that var at all you can
> just do a list loop like this:
>
> <CFLOOP list="#form.fieldnames#" index="formField">
> #formField# = #evaluate(formField)#
> <CFLOOP>
>
> Tim Heald
> ACP/CCFD :)
> Application Development
> www.schoollink.net
>
> > -----Original Message-----
> > From: Chakka, Sudheer [mailto:[EMAIL PROTECTED]]
> > Sent: Thursday, May 09, 2002 4:16 PM
> > To: CF-Talk
> > Subject: Form.FIELDNAME - error!!!!
> >
> >
> > Hi,
> >
> >  Can any one tell me what is the mistake in
> >
> > <CFLOOP FROM="1" TO="#ListLen(FORM.FIELDNAMES)#" STEP="1" INDEX="i">
> > <CFSET var = #ListGetAt(Form.FIELDNAMES, i)#>
> > #ListGetAt(Form.FIELDNAMES, i)# = #Evaluate(var)#
> > <BR>
> > </CFLOOP>
> >
> >
> > I am getting the following error:
> >
> > An error occurred while evaluating the expression:
> > "#ListLen(FORM.FIELDNAMES)#"
> >
> >
> >  Any help on this is appreciated.
> >
> > thanks,
> > Sudheer Chakka.
> >
> >
> 
______________________________________________________________________
Your ad could be here. Monies from ads go to support these lists and provide more 
resources for the community. http://www.fusionauthority.com/ads.cfm
FAQ: http://www.thenetprofits.co.uk/coldfusion/faq
Archives: http://www.mail-archive.com/cf-talk@houseoffusion.com/
Unsubscribe: http://www.houseoffusion.com/index.cfm?sidebar=lists

Reply via email to