Click here to Skip to main content
15,885,668 members
Please Sign up or sign in to vote.
0.00/5 (No votes)
See more:
Hi i am first creating a xml document, and adding date to it, and then i am checking if the document exists then add value to the xml document
i want to generate like this in my xml document
HTML
<book>
  <firstname>Balle</firstname>
  <lastname>Largen</lastname>
</book>
<book>
  <firstname>Kalle</firstname>
  <lastname>Svensson</lastname>
</book>


But my code right now generates like this, when adding data another time

XML
<?xml version="1.0" encoding="utf-8"?>
<book>
  <firstname>Balle</firstname>
  <lastname>Largen</lastname>firstnameKallelastnameSvensson</book>


here is my function
C#
private void CreateXmlDoc()
       {

           var firstname = TextBoxFirstName.Text;
           var lastname =   TextBoxLastName.Text;

           var settings = new XmlWriterSettings();
           settings.Indent = true;
           settings.IndentChars = ("    ");
           settings.CloseOutput = true;
           settings.OmitXmlDeclaration = true;
           DateTime now = DateTime.Now;
           var fileName = Server.MapPath("~") + "ansokningar"+ now.Year+"-"+now.Month+"-"+now.Day+".xml";

           if (!File.Exists(fileName))
           {
               var writer = XmlWriter.Create(fileName, settings);

               writer.WriteStartElement("book");
               writer.WriteElementString("firstname", firstname);
               writer.WriteElementString("lastname", lastname);
               writer.WriteEndElement();
               writer.Flush();
               writer.Close();

           }
           else
           {

               var xDocument = XDocument.Load(fileName);
               var xDoc = new XmlDocument();

               xDoc.CreateElement("book");
               XElement root = xDocument.Element("book");
               if (root != null)
               {

                   root.Add("firstname",firstname);
                   root.Add("lastname", lastname);
               }
               xDocument.Save(fileName);

           }




       }
Posted
Comments
Zoltán Zörgő 26-Dec-14 3:57am    
Any progress? If you have found my answer of use, please feel free to accept it formally.

Take a look here: http://msdn.microsoft.com/en-us/library/bb387089.aspx[^]
In your case:
C#
XElement books =
    new XElement("books",
        new XElement("book",
            new XElement("firstname", "Balle"),                                                 
            new XElement("lastname", "Largen")
        ),
        new XElement("book",
            new XElement("firstname", "Kalle"),                                                 
            new XElement("lastname", "Svensson")
        )
    );
 
Share this answer
 
v2
0) You can have only one root: you need an outer element like books for example.
1) XmlDocument.DocumentElement[^] will get the root.
2) Why mixing XDocument and XmlDocument like this?

I would use something like this (LinqPad demo):
XDocument addBook(string filename, string firstname, string lastname)
{
	XDocument xml;
	XElement books;
	if (File.Exists(filename))
   	{
		xml = XDocument.Load(filename);
		books = xml.Root;
	}
	else
	{
		xml = new XDocument();
		books = new XElement("books");
		xml.Add(books);
	}
	
	books.Add(
		new XElement("book",
			new XElement("firstname", firstname),
			new XElement("lastname", lastname))
	);
	
	xml.Save(filename);
	return xml;
}

void Main()
{
	Console.WriteLine(addBook(@"d:\temp\t.xml","John","Smith"));
}
 
Share this answer
 

This content, along with any associated source code and files, is licensed under The Code Project Open License (CPOL)



CodeProject, 20 Bay Street, 11th Floor Toronto, Ontario, Canada M5J 2N8 +1 (416) 849-8900