looping over form fields and then adding in DB

loop and check the existence of the input element when form is submited..

StructKeyExists(form,p#i#) -- Do insert...

upon form submission

<cfloop from="1" to="5" index="i">
   <cfif structKeyExists(FORM,'p#i#') >
          -- DO YOUR INSER --
    </cfif>
</cfloop>

the way you specified i know about it, but my question is will it insert the 1 and 4th row id specified

or there is some way using currentRow like stuff not sure abt that

Yu are only considering field (p) while leaving all other fields
why can i know

or what is yuor point of view

You can just check for the existence of value.. this will ONLY insert the values what user have entered

<cfloop from="1" to="5" index="i">
    <cfif FORM['p#i#'] NEQ ''>
    	--- DO YOUR QUERY HERE ---    
    </cfif>
</cfloop>

Select allOpen in new window

<cfloop from="1" to="5" index="i">
    <cfif FORM['p#i#'] NEQ ''>
    	--- DO YOUR QUERY HERE ---    
    </cfif>
    <cfif FORM['r#i#'] NEQ ''>
    	--- DO YOUR QUERY HERE ---    
    </cfif>
    <cfif FORM['s#i#'] NEQ ''>
    	--- DO YOUR QUERY HERE ---    
    </cfif>
    <cfif FORM['t#i#'] NEQ ''>
    	--- DO YOUR QUERY HERE ---    
    </cfif>
</cfloop> 

Select allOpen in new window

so ur point is this

<cfif StructKeyExists(FORM,'produc_name#i#') AND FORM['produc_name#i#'] NEQ ''>

in this case they need to add the produc_name for this to work

so you are saying is for singlefield, i need to rewite the query again and again

is that so

Just checking  <cfif FORM['p#i#'] NEQ ''> would work ...

I usually create a function if it is the same query ..

<cffunction name="doInsert" access="public" output="no">
	<cfargument name="yorArgument" type="string" required="yes">
		--- YOUR QUERY HERE ---
</cffunction>

<cfloop from="1" to="5" index="i">
    <cfif FORM['p#i#'] NEQ ''>
    	<cfset doInsert(#FORM['p#i#']#)  /> 
    </cfif>
    <cfif FORM['r#i#'] NEQ ''>
    	<cfset doInsert(#FORM['r#i#']#)  />   
    </cfif>
    <cfif FORM['s#i#'] NEQ ''>
    	<cfset doInsert(#FORM['s#i#']#)  />    
    </cfif>
    <cfif FORM['t#i#'] NEQ ''>
    	<cfset doInsert(#FORM['t#i#']#)  />     
    </cfif>
</cfloop> 

Select allOpen in new window

the thing u are doing is something new to me as you are saying that you will create a function, can you explain a bit or write the queery complete , well my case is bit different as i am using this inside a function and i have no issues if i can use the same inside another function

<cffunction name="doInsert" access="public" output="no" returntype="void">
      <cfargument name="name" type="string" required="yes">
      <cfquery name="insertdata" datasource="#dsn#">
                   insert into yourtable() values()
                    where name = '#arguments.name#'
        </cfquery>
</cffunction>

<cfloop from="1" to="5" index="i">
    <cfif FORM['p#i#'] NEQ ''>
          <cfset doInsert(#FORM['p#i#']#)  />
    </cfif>
    <cfif FORM['r#i#'] NEQ ''>
          <cfset doInsert(#FORM['r#i#']#)  />  
    </cfif>
    <cfif FORM['s#i#'] NEQ ''>
          <cfset doInsert(#FORM['s#i#']#)  />    
    </cfif>
    <cfif FORM['t#i#'] NEQ ''>
          <cfset doInsert(#FORM['t#i#']#)  />    
    </cfif>
</cfloop>

Dude, I still did not understood your work folio, u are passing a null value right in the insert statement, and suppose i move tis function move out and wwrite somewhere eelse, will this scenario work, if yes, how , can you please explain in detail

that will be nice of u

You can move the function anywhere you want to .. say in a cfc called test.cfc

Test.cfc

<cffunction name="doInsert" access="public" output="no" returntype="void">
      <cfargument name="name" type="string" required="yes">
      <cfquery name="insertdata" datasource="#dsn#">
                   insert into yourtable(a) values('#arguments.name#')
        </cfquery>
</cffunction>

Select allOpen in new window

Your main code

<cfset queryObj = createObject("component","test") />
<cfloop from="1" to="5" index="i">
    <cfif FORM['p#i#'] NEQ ''>
          <cfset queryObj.doInsert(#FORM['p#i#']#)  />
    </cfif>
    <cfif FORM['r#i#'] NEQ ''>
          <cfset queryObj.doInsert(#FORM['r#i#']#)  />  
    </cfif>
    <cfif FORM['s#i#'] NEQ ''>
          <cfset queryObj.doInsert(#FORM['s#i#']#)  />    
    </cfif>
    <cfif FORM['t#i#'] NEQ ''>
          <cfset queryObj.doInsert(#FORM['t#i#']#)  />    
    </cfif>
</cfloop>

>> Dude, I still did not understood your work folio, u are passing a null value right in the insert statement <<

Didn't get you on this,

1. Check if the FORM submit value is NOT NULL

<cfif FORM['p#i#'] NEQ ''>

2. IF it is NOT null then call the function which has the query to do the insert

<cfquery name="insertdata" datasource="#dsn#">
                   insert into yourtable(a) values('#arguments.name#')
</cfquery>

Does that make sense ?

ok what it is doing, if any value is defined, it is going to insert right!

ok suppose i fill 1st field of row 1 and 3rd field of row 4, then it will gonna create 4 records or will do in the same one ewcord, the 1st and the 4th one

ok what it is doing, if any value is defined, it is going to insert right! -- YES, if the user inputs the value, it is going to call the insert function, otherwise it will NOT...

ok suppose i fill 1st field of row 1 and 3rd field of row 4, then it will gonna create 4 records or will do in the same one ewcord, the 1st and the 4th one   -- the rows that you filled are the ones which will be inserted.. as the ones which are not filled would be NULL..

myselfrandhawa,

 I assume p,t,r and S are COLUMNs in the row, is that right?  

 Question - do you need all 4 columns (p,t,r and S)  to be entered to create the record or will only one column be enough to create the record?  

 The follow example shows that IF ANY ONE field of p,t,r or s have a value, the record will be created.   If they are all empty, the record will not be inserted.

 You can enter a value in Row 1 column P and a record in Row 3 column S and the result will be two records created; the first with just a value in P and the second with a value in S.

 Is that what you want?

 If you want to UPDATE and DELETE as well, let me know, there are simple changes to make to allow for insert, update and delete.

-- note that you do not need to test if the field exists (unless one of the fields is a checkbox)

<cfloop from="1" to="5" index="i">
   <cfset inputExists = false>
   <cfif len(FORM["p#i#"]) + FORM["rp#i#"] + FORM["s#i#"] + FORM["t#i#"]>
      <cfset inputExists = true>
   </cfif>
   <cfif inputExists>
     <cfquery name="doInsert">
       insert into my table (
         p
        ,r
        ,s
        ,t
       ) values (
          '#form["p" & i]#'
         ,'#form["r" & i]#'
         ,'#form["s" & i]#'
         ,'#form["t" & i]#'
       )     
     </cfquery>
   </cfif>
</cfloop>

Select allOpen in new window

sorry, I forgot to use the len() on all form field in the CFIF.

here it is corrected...

<cfloop from="1" to="5" index="i">
   <cfset inputExists = false>
   <cfif len(FORM["p#i#"]) + len(FORM["rp#i#"]) + len(FORM["s#i#"]) + len(FORM["t#i#"])>
      <cfset inputExists = true>
   </cfif>
   <cfif inputExists>
     <cfquery name="doInsert">
       insert into my table (
         p
        ,r
        ,s
        ,t
       ) values (
          '#form["p" & i]#'
         ,'#form["r" & i]#'
         ,'#form["s" & i]#'
         ,'#form["t" & i]#'
       )     
     </cfquery>
   </cfif>
</cfloop>

Select allOpen in new window

so which is ur correct verison last one or the above used one

the last one, where it says "here it is corrected...."

Great, i will implemented it and get back to u asap

That's great well what if the last comment of yours can be made as a function and then used, how that can be done, same as above or some changes neded

You can wrap the function tag around the code, no problem.

But why?   If you are already using all CFCs and functions, then you can make it a function.   But if you're not already usiing them, then why would you want to create one for this?

The reason you may want to make it a function  is that you will call the function from many different places in your code.  This seems like something you would only use in one place.  So no need to add the extra code..

no i would like this to created as function just it will be portable if used somewhere else

ok, just wrap the function tag around it and pass in the form fields as arguments

Ok will update 2marrow on this

i will write the function & show here if something is wrong or right

in a hurry if u can wrap the functionality as custom function that will be nice, as i am leaving for the day or two might not be able to work on this later