oops, forgot the loop in method 1:
action.cfm ==========
<cfparam name="form.myBox" default="">
<cfif listlen(form.myBox)>
<cfloop list="#form.myBox#" index="id">
<cfquery.....
Do insert.....
</cfloop>
</cfif>
=====================
still less code though.
> -----Original Message-----
> From: Rich Wild [mailto:[EMAIL PROTECTED]
> Sent: 06 June 2003 17:12
> To: '[EMAIL PROTECTED]'
> Subject: RE: [ cf-dev ] form.fieldnames
>
>
> > It does seem a little overcomplicated doing it this way
> > though. Surely it
> > would be easier to loop the names of the fields to set up
> form params
> > defaulted to 0 and then if a value of 1 is passed, insert
> > into the join
> > table?
>
> not really, as that way you need to find all your ids again (prob by a
> database query), then loop over these using <cfparam> to
> default them all,
> and then check whether each one is zero or not before inserting.
>
> If you do it the way Ian was doing it, then you only need one
> <cfparam> to
> ensure that there the form variable exists, even if its
> empty, and then you
> just need to loop over the that list.
>
> If its empty, no inserts happen, if its not, then those ids
> get entered into
> the database. You don't need to check if they're zero or
> something else.
>
> so compare this:
>
> form.cfm ===========
> <cfoutput query="myIds">
> <input type="checkbox" name="myBox" value="#id#"><br>
> </cfoutput>
> ====================
>
> action.cfm ==========
> <cfparam name="form.myBox" default="">
>
> <cfif listlen(form.myBox)>
> <cfquery.....
> Do insert.....
> </cfif>
> =====================
>
> to this:
>
> form.cfm ===========
> <cfoutput query="myIds">
> <input type="checkbox" name="myBox_#id#" value="1"><br>
> </cfoutput>
> =====================
>
> action.cfm ==========
> <cfquery name="myBoxes" datasource="#dsn#">
> SELECT id FROM myBoxes
> </cfquery>
>
> <cfloop query="myBoxes">
> <cfparam name="form.myBox_#id#" default="0">
> </cfloop>
>
> <cfloop query="myBoxes">
> <cfif form["myBox_" & id]>
> <cfquery...
> Do Insert.....
> </cfif>
> </cfloop>
> =====================
>
>
> Method 1 is faster <cf_emphasis>in this instance</cf_emphasis>.
>
> There are of course, other times when you method would be
> more correct,
> which I can't even begin to think of this close to the booze hour.
>
> Rich
>
>
> --
> ** Archive: http://www.mail-archive.com/dev%40lists.cfdeveloper.co.uk/
>
> To unsubscribe, e-mail: [EMAIL PROTECTED]
> For additional commands, e-mail: [EMAIL PROTECTED]
> For human help, e-mail: [EMAIL PROTECTED]
>
--
** Archive: http://www.mail-archive.com/dev%40lists.cfdeveloper.co.uk/
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
For human help, e-mail: [EMAIL PROTECTED]
