Writting a xml file

[Alberto]

I have this code to write a xml file:

public void Grabar()
{
SaveFileDialog sfd = new SaveFileDialog();
sfd.DefaultExt = "xml";

sfd.InitialDirectory = Config.Directory;
if (sfd.ShowDialog() == DialogResult.OK)
{
FileStream fs = new FileStream(sfd.FileName,
FileMode.Create);

XmlTextWriter tw = new XmlTextWriter(sfd.FileName, null);
<=====

tw.Formatting = Formatting.Indented;
tw.WriteStartDocument();
....

When I execute the code, it breaks in the selected line. It says that the
file it's been used by another process but the file it's new. I type a new
name in the filedialog form.
Thank you for your help.

 

[Göran Andersson]

I have this code to write a xml file:

public void Grabar()
{
SaveFileDialog sfd = new SaveFileDialog();
sfd.DefaultExt = "xml";

sfd.InitialDirectory = Config.Directory;
if (sfd.ShowDialog() == DialogResult.OK)
{
FileStream fs = new FileStream(sfd.FileName,
FileMode.Create);

XmlTextWriter tw = new XmlTextWriter(sfd.FileName, null);
<=====

tw.Formatting = Formatting.Indented;
tw.WriteStartDocument();
....

When I execute the code, it breaks in the selected line. It says that
the file it's been used by another process but the file it's new. I type
a new name in the filedialog form.
Thank you for your help.

You are creating the file in the previous line, but you are not closing
it. Therefore it's your own code that is using the file.

Don't use a FileStream to create the file, let the XmlTextWriter create
it for you instead.

 

[Martin Honnen]

You are creating the file in the previous line, but you are not closing
it. Therefore it's your own code that is using the file.

Don't use a FileStream to create the file, let the XmlTextWriter create
it for you instead.

Also note that since .NET 2.0 you should use XmlWriter.Create instead of
new XmlTextWriter e.g.
XmlWriterSettings xws = new XmlWriterSettings();
xws.Indent = true;
using (XmlWriter xw = XmlWriter.Create(sfd.FileName, xws))
{
// write stuff here
}