Solved

condition failing within cfif statement

Posted on 2011-09-26
8
476 Views
Last Modified: 2012-05-12
Hi,

I have a cfloop statement, within that there is an if statement, I am moving 2 files if statusflag is approved (1), to determine the type of file being moved, I am looking at a field to see if it holds a value.  The problem is the condition fails where I'm testing that field ListResult.NPSGroupNumber. I tried going about it a few different ways, switching order, creating another cfif, instead of cfelseif but no luck.  Any suggestions?

<CFLOOP from = "1" to = "#RecordCount#" index = "i">

   <cfif isdefined('FORM.GP#i#')>
   
   
         <cfset gpValue = FORM["gp"& i]>
         
                                 <CFQUERY DATASOURCE="Obi-default"  NAME="ListResult" >
                            select *
                            from NF_WMI
                            where TrackingNumber = #i#
                        </CFQUERY>
                       
            <cfif #val(gpValue)# eq 1 and len(ListResult.NPSGroupNumber) eq 0>  
           
           
            <cffile action="move"
                    source="#ListResult.BnftChart_Path#"
                    destination="\\pwn401av213\obi-dev\WMI\OBI\Approved\#ListResult.LOB#\#ListResult.GroupNo#bnftchart.obi">
                   
                   
                   <!--- <cffile action="move"
                    source="#ListResult.CertRider_Path#"
                    destination="\\pwn401av213\obi-dev\WMI\HTML\Approved\#ListResult.LOB#\#ListResult.GroupNo#bnftcert.html">
           
            </cfif>      
                        --->
                       
                       
            <cfelseif #val(gpValue)# eq 1 and isdefined(ListResult.NPSGroupNumber) >

                        <cfset FileWrite("\\pwn401av213\obi-dev\WMI\OBI\Approved\testing.txt\","Hello World")>
           
           
            <cfelseif #val(gpValue)# eq 0>
                   
                        <cfinclude template="rejected_email.cfm">
            </cfif>





            <CFQUERY name="apprvd" DATASOURCE="Obi-default" result="test">  
            UPDATE      NF_WMI  
            SET         StatusFlag = #val(gpValue)#,
            Approved_By = #submittedByQry.employeeNumber#,
            Approved_Date =  #Now()#
            WHERE TrackingNumber = #i#
       
                 </CFQUERY>  
     
     

   </cfif>
   
   
   
</CFLOOP>
0
Comment
Question by:armanbena
8 Comments
 
LVL 19

Expert Comment

by:Bhavesh Shah
ID: 36599211
have you check what gpvalue holding ?

I have doubt in this line

<cfset gpValue = FORM["gp"& i]>

try with

<cfset gpValue = evaluate("gp" & i)>

0
 
LVL 39

Accepted Solution

by:
gdemaria earned 500 total points
ID: 36599293

This is the correct format:   <cfset gpValue = FORM["gp"& i]>
anytime you feel like using evaluate() you should turn off your computer until the feeling goes away
:)


I suggest using this nested format with the <h1> flags I put in to see where the code is being entered, that will tell you which variables are not acting as you feel they should...

Note that since the isDefined() is referencing a query, it will always be true..  isdefined(ListResult.NPSGroupNumber)  will always be true
<cfif val(gpValue) eq 1>
               <h1>gpValue is 1</h1>
               <cfif len(ListResult.NPSGroupNumber) eq 0>  
                    <h1>NPSGroupNumber is empty</h1>
                    <cffile action="move"
                            source="#ListResult.BnftChart_Path#"
                            destination="\\pwn401av213\obi-dev\WMI\OBI\Approved\#ListResult.LOB#\#ListResult.GroupNo#bnftchart.obi">
               <cfelse>
                    <h1>NPSGroupNumber has a value</h1>
                    <cfset FileWrite("\\pwn401av213\obi-dev\WMI\OBI\Approved\testing.txt\","Hello World")> 
               </cfif>    
            
           <cfelse> 
               <h1>gpValue is Zero</h1>
               <cfinclude template="rejected_email.cfm">
           </cfif>

Open in new window

0
 
LVL 19

Expert Comment

by:Bhavesh Shah
ID: 36599327
Thank you Sir....
0
Master Your Team's Linux and Cloud Stack!

The average business loses $13.5M per year to ineffective training (per 1,000 employees). Keep ahead of the competition and combine in-person quality with online cost and flexibility by training with Linux Academy.

 
LVL 39

Expert Comment

by:gdemaria
ID: 36599400
wait, the isDefined() statement is not correct...

When testing a variable to see if it exists, you need to use quotes...

Typically like this....    isDefined("form.firstName")

without the quotes, you are testing the VALUE of the variable to see if THAT exists as a variable...

so, in this case...
  isdefined(ListResult.NPSGroupNumber)

Let's say ListResult.NPSGroupNumber is 123.   That means isDefined is testing to see if variables.123 exists as a variable... which I assume it does not.

You would want to use    isdefined("ListResult.NPSGroupNumber")

but as I said above, since you are testing a query, the value will always exist as long as that column exists in the cfquery

In short, remove the isDefined() from the code

0
 

Author Comment

by:armanbena
ID: 36710362
Okay, I removed the isDefined tag but it still fails.  Not sure what is wrong. This is what I have now:
<CFLOOP from = "1" to = "#RecordCount#" index = "i">


   <cfif isdefined('FORM.GP#i#')>
   
   
   
         <cfset gpValue = FORM["gp"& i]>
         
                                 <CFQUERY DATASOURCE="Obi-default"  NAME="ListResult" >
                            select *
                            from NF_WMI
                            where TrackingNumber = #i#
                        </CFQUERY>
                       
                       
                       
             <cfif #val(gpValue)# eq 0 > 
                   <cfinclude template="rejected_email.cfm">
                   
                 
                       
                       
            <cfelseif #val(gpValue)# eq 1 and len(ListResult.NPSGroupNumber) eq 0>  
           
           
           
            <cffile action="move"
                    source="#ListResult.BnftChart_Path#"
                    destination="\\pwn401av213\obi-dev\WMI\OBI\Approved\#ListResult.LOB#\#ListResult.GroupNo#bnftchart.obi">
                   
                   
                   <!--- <cffile action="move"
                    source="#ListResult.CertRider_Path#"
                    destination="\\pwn401av213\obi-dev\WMI\HTML\Approved\#ListResult.LOB#\#ListResult.GroupNo#bnftcert.html">
           
            </cfif>      
                        --->
                       
                       
            <cfelseif #val(gpValue)# eq 1>
                  
                        <cfset FileWrite("\\pwn401av213\obi-dev\WMI\OBI\Approved\testing.txt\","Hello World")>
           
           
            </cfif >





            <CFQUERY name="apprvd" DATASOURCE="Obi-default" result="test">  
            UPDATE      NF_WMI  
            SET         StatusFlag = #val(gpValue)#,
            Approved_By = #submittedByQry.employeeNumber#,
            Approved_Date =  #Now()#
            WHERE TrackingNumber = #i#
       
                 </CFQUERY>  
     
     

   </cfif>
   
   
   
</CFLOOP>
0
 
LVL 39

Expert Comment

by:gdemaria
ID: 36710417
> Okay, I removed the isDefined tag but it still fails.  Not sure what is wrong. This is what I have now:

what does "fails"  mean exactly?

What happened with the flags (the <h1> statements) ... which ones displayed and which ones didn't.   Which area did you expect to enter and did not, which flag is showing when you had that situation?
0
 

Expert Comment

by:kcjuliff
ID: 36710463
There is the strong possibility that the condition will never be met.

You have no final cfelse statement.

So if  #val(gpValue)# eq 1 and len(ListResult.NPSGroupNumber) eq 0 does not evaluate to true, then no path will be taken.

There are 4 possibilities  for the outer cfif

val(gpval) 1  len(ListResult.NPSGroupNumber) 1
val(gpval) 0  len(ListResult.NPSGroupNumber)1
val(gpval) 1  len(ListResult.NPSGroupNumber) 0
val(gpval)0  len(ListResult.NPSGroupNumber) 0


0
 
LVL 39

Expert Comment

by:gdemaria
ID: 36710568
> So if  #val(gpValue)# eq 1 and len(ListResult.NPSGroupNumber) eq 0 does not evaluate to true, then no path will be taken.

well, not quite, if gpvalue and 1, then it will go to the last CFELSEIF statement regardless of the value of len(NPSgroupNumber)

The cases are these...
val(gpval) 0  and   len(ListResult.NPSGroupNumber) is any value
val(gpval) 1  and   len(ListResult.NPSGroupNumber) is 0
val(gpval) 1  and   len(ListResult.NPSGroupNumber) is any value

what if gpval is not 0 or 1 ... then nothing happens...


So, the changes are your gpval is some number other than 1 or 0



That's why I proposed the CFIF structure above...  you don't have to re-test conditions and some condition is always met.  Add the <h1> tags to display and let you know where you are in the code...


0

Featured Post

Master Your Team's Linux and Cloud Stack!

The average business loses $13.5M per year to ineffective training (per 1,000 employees). Keep ahead of the competition and combine in-person quality with online cost and flexibility by training with Linux Academy.

Question has a verified solution.

If you are experiencing a similar issue, please ask a related question

PROBLEM:  How to open a cfwindow or run a function on double click of a cfgrid row. One of my clients wanted to be able to double click on a row item to get more detailed information about a transaction and to be able to modify the line items i…
Recently while working on a project I got a very annoying cfdocument has no body error message. I had never seen this error before. So I checked the code. The code was pretty simple; it was Just showing me the cfdocumnt tag and inside that tag a …
Microsoft Active Directory, the widely used IT infrastructure, is known for its high risk of credential theft. The best way to test your Active Directory’s vulnerabilities to pass-the-ticket, pass-the-hash, privilege escalation, and malware attacks …
I've attached the XLSM Excel spreadsheet I used in the video and also text files containing the macros used below. https://filedb.experts-exchange.com/incoming/2017/03_w12/1151775/Permutations.txt https://filedb.experts-exchange.com/incoming/201…

856 members asked questions and received personalized solutions in the past 7 days.

Join the community of 500,000 technology professionals and ask your questions.

Join & Ask a Question