Want to win a PS4? Go Premium and enter to win our High-Tech Treats giveaway. Enter to Win

x
?
Solved

xml parsing, output to txt

Posted on 2009-04-01
19
Medium Priority
?
214 Views
Last Modified: 2013-12-24
I have the following code and am trying to output to a file which works but it only grabs the last record in the db and outputs to a txt.  It should loop through all records and output all records the the file.  also, how would I adjust it so that there is a line break after each record in the txt?
<cfif qOrdersToProcess.recordcount GT 0>
 
<cfloop query=qOrdersToProcess>
<cfset xmlString="#orderString#">
<cfif isXML(xmlString)>
<cfset parsed=XmlParse(xmlString)>
<cfset mystring="">
 
<cfloop from=1 to="#arrayLen(parsed.xmlRoot.Order.xmlChildren)#" step=1 index=i>
<cfset mystring=mystring & #evaluate("parsed.xmlRoot.Order.x#i#.xmlText")# & ','>
</cfloop>
 
<cfset myString = Left(myString, Len(myString)-1)>
<cffile action="write" file="#srcDir#CNV#counter##dateFormat(endDate,'mmdd')#"  output="#mystring#">
 
</cfif>
</cfloop>
 
</cfif>

Open in new window

0
Comment
Question by:COwebmaster
[X]
Welcome to Experts Exchange

Add your voice to the tech community where 5M+ people just like you are talking about what matters.

  • Help others & share knowledge
  • Earn cash & points
  • Learn & ask questions
  • 10
  • 9
19 Comments
 
LVL 52

Accepted Solution

by:
_agx_ earned 2000 total points
ID: 24041461
Using action="write" overwrites any existing file.  Since you are inside a loop, each iteration overwrites the previous line, leaving only the last line in the query.

Try using append instead:

<cffile action="append" addNewLine="yes"
file="#srcDir#CNV#counter##dateFormat(endDate,'mmdd')#"  
output="#mystring#">
0
 
LVL 52

Expert Comment

by:_agx_
ID: 24041480
Also, have you tried using array notation to eliminate the need for evaluate(..)?

<cfset mystring = listAppend(mystring, parsed.xmlRoot.Order["x"& i ].xmlText)>
0
 

Author Comment

by:COwebmaster
ID: 24041542
_agx_, is using an array notation better than what I'm doing?  Let me know if you see a better way.
0
Looking for a new Web Host?

Lunarpages' assortment of hosting products and solutions ensure a perfect fit for anyone looking to get their vision or products to market. Our award winning customer support and 30-day money back guarantee show the pride we take in being the industry's premier MSP.

 
LVL 52

Expert Comment

by:_agx_
ID: 24041588
Yes, it saves a little extra processing that you really do not need.  IMO it is also cleaner code.  

 (BTW, using listAppend will prevent trailing commas in your variable.)
0
 

Author Comment

by:COwebmaster
ID: 24042031
_agx_, I'm trying to pull back the ids and place them in a ValueList.  I've tried this but it pulls back all ids and repeats them for every iteration..

<cfif isXML(xmlString)>
<cfoutput>#ValueList(qOrdersToProcess.id)#</cfoutput>
<cfset updateids = ValueList(qOrdersToProcess.id)>
<cfoutput>#updateids#</cfoutput>
..

Any ideas?
0
 
LVL 52

Expert Comment

by:_agx_
ID: 24042193
Yes, you can't really do that. If you try and combine valueList (all values) with a loop through _individual_ records you are going to get duplicates.

How does this relate to the xml in the original post?  
0
 

Author Comment

by:COwebmaster
ID: 24042261
actually, I figured it out.
0
 

Author Comment

by:COwebmaster
ID: 24042274
I just placed an update statment under the cffile statement and that worked.
0
 
LVL 52

Expert Comment

by:_agx_
ID: 24042284
So what did you do differently?  Also, is the original question answered as well?
0
 
LVL 52

Expert Comment

by:_agx_
ID: 24042293
> I just placed an update statment under the cffile statement and that worked.

Outside of the loop, yes? Otherwise the update would run many, many times.
0
 

Author Comment

by:COwebmaster
ID: 24042305
Actually, there is one last piece I need to do.  I have a isXML statement in there to make that the record is correctly formatted.  However, I need to make sure the parent node is a certain value, say "Order".  So my xml statement would look like..

<?xml version="1.0" encoding="UTF-8"?><ns1:Order xmlns:ns1="http://www.url.com"><Order><x1>module6-1</x1><x2>module6-2</x2></Order></ns1:Order>

Is there a statement to check for that?                                                            
0
 

Author Comment

by:COwebmaster
ID: 24042311
Yes, I placed it outside the loop.
0
 
LVL 52

Expert Comment

by:_agx_
ID: 24042835
> I need to make sure the parent node is a certain value, say "Order"

You are already referencing the parent node here:

 .... <cfloop from=1 to="#arrayLen(parsed.xmlRoot.Order.xmlChildren)#"
0
 

Author Comment

by:COwebmaster
ID: 24043381
Ok, but how can I be sure that the parent node is correct?  In other words, a third party will be inserted those xml strings into the db so I want to make sure Order is correct and not Orders and so on.  Is there a way to check for that?
0
 
LVL 52

Expert Comment

by:_agx_
ID: 24043485
Yes, you can test for the existence of the key "Order". Then do something else if it was not found

<cfif structKeyExists(parsed.xmlRoot, "Order")>
      do your existing code here
<cfelse>
      "Order" key not found. do something else
</cfif>
0
 

Author Comment

by:COwebmaster
ID: 24043663
_agx_, would that go on the inside or outside of the cfloop?
0
 
LVL 52

Expert Comment

by:_agx_
ID: 24044030
It depends on what you want to do if the key does not exist. But probably Inside the loop, after the isXML check

<cfif isXML(xmlString)>
    <cfset parsed=XmlParse(xmlString)>
    <cfif structKeyExists(parsed.xmlRoot, "Order")>
    ... do the loop and file append here
   </cfif>
</cfif>

Open in new window

0
 

Author Comment

by:COwebmaster
ID: 24046131
Cool, thx _aqx_
0
 

Author Closing Comment

by:COwebmaster
ID: 31565421
Thx
0

Featured Post

Enroll in October's Free Course of the Month

Do you work with and analyze data? Enroll in October's Course of the Month for 7+ hours of SQL training, allowing you to quickly and efficiently store or retrieve data. It's free for Premium Members, Team Accounts, and Qualified Experts!

Question has a verified solution.

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

The Problem How to write an Xquery that works like a SQL outer join, providing placeholders for absent data on the outer side?  I give a bit more background at the end. The situation expressed as relational data Let’s work through this.  I’ve …
Lease-to-own eliminates the expenditure of hardware replacement and allows you to pay off the server over time. Usually, this is much cheaper than leasing servers. Think of lease-to-own as credit without interest.
In this video, Percona Solutions Engineer Barrett Chambers discusses some of the basic syntax differences between MySQL and MongoDB. To learn more check out our webinar on MongoDB administration for MySQL DBA: https://www.percona.com/resources/we…
Are you ready to place your question in front of subject-matter experts for more timely responses? With the release of Priority Question, Premium Members, Team Accounts and Qualified Experts can now identify the emergent level of their issue, signal…

609 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