Problem appending a new element to xml file...

[sherifffruitfly]

Hi all,

I'm trying to distill all of the info from google searches into what I
need, with partial success. In truth, the whole xmlNode, Document,
Element, etc group of classes & methods is going over my head - lol!

The structure of the xml file I'm trying to append to is as follows:

<?xml version="1.0" encoding="UTF-8"?>
<!-- stuff -->
<!-- more stuff -->
<Scenarios attr="0">
<Scenario num="0">
<TimeSeries name="cdj" >
<Observation date="01/01/2001" value="1000"/>

a whole bunch more observations

</TimeSeries>
</Scenario>
</Scenarios>

I'm trying to code up a routine that adds another Observation to the
list. Here's what I've got:

public void WriteHistory(Hi storyType histype, double val)
{
try
{
XmlDocument cdjDoc = new XmlDocument();
XmlNode parentNode;

cdjDoc.LoadXml( monthlyPath + filename);
parentNode = cdjDoc.Document Element;

XmlNode newElement = cdjDoc.CreateNo de(XmlNodeType. Element,
"Observatio n", null);
cdjDoc.AppendCh ild(newElement) ;

XmlAttribute date = cdjDoc.CreateAt tribute("date") ;
date.InnerText = DateTime.Now.To String("MM/dd/yyyy");
newElement.Attr ibutes.Append(d ate);

XmlAttribute rate = cdjDoc.CreateAt tribute("value" );
rate.InnerText = val.ToString();
newElement.Attr ibutes.Append(r ate);

XmlTextWriter writer = new XmlTextWriter(m onthlyPath +
filename,null);
cdjDoc.Save(wri ter);
}
catch(Exception ex)
{
MessageBox.Show (ex.Message,"Sc rewed!");
}

This doesn't work - throws an exception with message:

The data at the root level is invalid. Line 1, position 1.

I believe the problem is that I'm not "finding" the proper spot in the
xml structure to AppendChild my new element to. Can someone help me
with this? (Or any other problem, if I'm mistaken in my diagnosis?)

Thanks for any guidance,

cdj

[Marc Gravell]

You're making it very hard on yourself.

Try this (and note than most of the "code" is the XML block to make it
demonstrable); I have also used a more exotic query in SelectSingleNod e
than is probably necessary... depends if you need to select a
*specific* scenario/series; if only one you can drop all the
[@something='val ue'] stuff...

using System;
using System.Xml;
class Program {
static void Main()
{
const string xml = @"<?xml version=""1.0"" encoding=""UTF-8""?>

<Scenarios attr=""0"">
<Scenario num=""0"">
<TimeSeries name=""cdj"" >
<Observation date=""01/01/2001"" value=""1000""/>
<Observation date=""02/02/2001"" value=""2000""/>
<Observation date=""03/03/2001"" value=""3000""/>
</TimeSeries>
</Scenario>
</Scenarios>";
XmlDocument doc = new XmlDocument();
doc.LoadXml(xml ); //TODO: doc.Load(filena me);
XmlElement el = (XmlElement)
doc.SelectSingl eNode(@"Scenari os[@attr='0']/Scenario[@num='0']/TimeSeries[@name='cdj']");
XmlElement newObservation =
(XmlElement)el. AppendChild(doc .CreateElement( "Observation")) ;
newObservation. SetAttribute("d ate",
DateTime.Today. ToString("MM/dd/yyyy"));
newObservation. SetAttribute("v alue",
DateTime.Today. ToString("3000" ));

Console.WriteLi ne(doc.OuterXml ); //TODO: doc.Save(filena me);
}
}

[sherifffruitfly]

Marc Gravell wrote:

You're making it very hard on yourself.

Hey! I'm tryin here!

lolol! Just teasin - thanks - I'll give it a whirl!

cdj

[Marc Gravell]

Actually, I note your example hints at another level of xml nodes (not
shown)... in which case either include this in the XPath/XQuery, or
replace doc.SelectSingl eNode(...) with
doc.DocumentEle ment.SelectSing leNode(...), which will take you inside
the outer element.

Marc

[sherifffruitfly]

Marc Gravell wrote:

Actually, I note your example hints at another level of xml nodes (not
shown)... in which case either include this in the XPath/XQuery, or
replace doc.SelectSingl eNode(...) with
doc.DocumentEle ment.SelectSing leNode(...), which will take you inside
the outer element.

Marc

As the example structure I gave indicates, it is theoretically possible
for there to be multiple X's within a given Y for a variety of Y's.

In actual fact, the only case of interest to what I need is the
multiple Observations within a TimeSeries. Or to put it equivalently (I
hope): Observations are the only nodes with siblings.

Thanks again!

cdj

[Marc Gravell]

Glad to help - any problems, be sure to get back...

For completeness: your original code touched on XmlWriter; this is
typically used *instead of* the XmlDocument model in those scenarios
where XmlDocument is too heavy - just as very large xml fragments; with
XmlWriter you could write a 500Mb file from a database by writing one
row at a time directly to hte stream; with the XmlDocument you would
need to hold it all in memory first, probably taking about 4-times
that, so 2Gb (better hope for Win64 memory addressing!).
XmlDocument, however, uses the "all in memory" model to provide
services such as XPath/XQuery, and simple element manipulation. There
are other differences, but that is the general principle; for
small-to-mid-size xml, XmlDocument is a reasonable choice.

Marc

[sherifffruitfly]

Marc Gravell wrote:

Glad to help - any problems, be sure to get back...

For completeness: your original code touched on XmlWriter; this is
typically used *instead of* the XmlDocument model

The biggest problem I'm having now is preserving all of the pretty
tabs/formatting in the xml document. The PreserveWhiteSp ace property
does better than nothing, but falls substantially short of what might
be expected. For example, even with PreserveWhiteSp ace, a newline is
*not* inserted after the new Observation is added - this surprises me.

Is there a straightforward way to maintain the tab/newline structure?
(It's intuitively obvious what structure I'm talking about - tabs to
indicate hierarchy-level-descent, newlines for new elements.)

You brought up a new issue to me: the xmlDocument vs XmlWriter. The
code I've been working with, per your suggestion (I thought - lol) is:

XmlDocument cdjDoc = new XmlDocument();
cdjDoc.Preserve Whitespace = true;

XmlTextReader reader = new XmlTextReader(m onthlyPath + filename);
cdjDoc.Load(rea der);
reader.Close();

XmlElement el =
(XmlElement)cdj Doc.SelectSingl eNode(@"Scenari os[@version='1']/Scenario[@id='0']/TimeSeries[@name='cdj']");
XmlElement newObservation =
(XmlElement)el. AppendChild(cdj Doc.CreateEleme nt("Observation "));
newObservation. SetAttribute("d ate",
DateTime.Today. ToString("MM/dd/yyyy"));
newObservation. SetAttribute("v alue", val.ToString()) ;

XmlTextWriter writer = new XmlTextWriter(m onthlyPath +
filename,null);
cdjDoc.Save(wri ter);
writer.Close();

Is there a better way to load/save the xml file to/from disk besides
this? Do tell. lol

thanks again!

cdj

[sherifffruitfly]

sherifffruitfly wrote:

Marc Gravell wrote:

Glad to help - any problems, be sure to get back...

For completeness: your original code touched on XmlWriter; this is
typically used *instead of* the XmlDocument model

The biggest problem I'm having now is preserving all of the pretty
tabs/formatting in the xml document.

Nevermind on the prettyprinting. The following lines, in the right
place, completely suffice for me:

cdjDoc.CreateNo de(XmlNodeType. Text, "\n", null);

writer.Formatti ng = Formatting.Inde nted;
writer.Indentat ion = 4;

Thanks for your previous help tho!

cdj

[Marc Gravell]

I was going to suggest:

XmlWriterSettin gs settings = new XmlWriterSettin gs();
settings.Indent = true;
settings.NewLin eHandling = NewLineHandling .Entitize;
using (Stream fs = File.OpenWrite( @"c:\out.xml "))
using (XmlWriter writer = XmlWriter.Creat e(fs, settings))
{
doc.Save(writer );
writer.Close();
fs.Close();
}

I guess either route achieves the same...

Marc

[Marc Gravell]

Or (shorter)

XmlWriterSettin gs settings = new XmlWriterSettin gs();
settings.Indent = true;
settings.NewLin eHandling = NewLineHandling .Entitize;
using (XmlWriter writer = XmlWriter.Creat e(@"c:\out.xml" ,
settings))
{
doc.Save(writer );
writer.Close();
}