所以我有这个代码，它基本上读取一个XML文件并做一些if来检查一些事情，然后将适当的控件放在画布（面板）上。
但它很长。或者，比我希望的要长。这段代码就像...现在2或3岁。
我想做的是使用 Linq to XML。
从 XML 文件读取：
OpenFileDialog o = new OpenFileDialog();
o.Filter =
"wordreplaced Multimedia Format (*.mf)|*.mf|" +
"Word Document (*.docx)|*.docx|" +
"PDF Document (*.pdf)|*.pdf|" +
"Text FIle (*.txt)|*.txt";
o.Title = "wordreplaced 11 - Open Document";
using (o)
{
if (o.ShowDialog() == DialogResult.OK)
{
foreach (var controlTag in XDocument.Load(o.FileName).Root.Elements())
{
var controlType = Type.GetType(
string.Format(
"System.Windows.Forms.{0}, System.Windows.Forms, Version=2.0.0.0, Culture=neutral, PublicKeyToken=b77a5c561934e089",
controlTag.Name.LocalName), false);
if (controlType == null || !typeof(Control).IsAssignableFrom(controlType))
{
continue;
}
var control = (Control)Activator.CreateInstance(controlType);
control.Text = controlTag.Attributes("Content").First().Value;
try
{
control.ForeColor = Color.FromArgb(
int.Parse(controlTag.Attributes("A").First().Value),
int.Parse(controlTag.Attributes("R").First().Value),
int.Parse(controlTag.Attributes("G").First().Value),
int.Parse(controlTag.Attributes("B").First().Value));
Font font = FromString(controlTag.Attributes("Font").First().Value);
control.Font = font;
}
catch { continue; }
control.BackColor = Color.Transparent;
control.MouseDown += new MouseEventHandler(control_MouseDown);
control.MouseMove += new MouseEventHandler(control_MouseMove);
control.MouseUp += new MouseEventHandler(control_MouseUp);
control.MouseClick += new MouseEventHandler(control_MouseClick);
control.MouseDoubleClick += new MouseEventHandler(control_MouseDoubleClick);
string boldness = Convert.ToString(controlTag.Attributes("Bold"));
if (boldness == "yeah")
control.Font = new Font(control.Font.Name, control.Font.Size, FontStyle.Bold);
Type t = control.GetType();
if (t.Name == "Label")
{
Label label = (Label)control;
label.AutoSize = true;
label.Location = new Point(
Convert.ToInt32(controlTag.Attributes("LocationX").First().Value),
Convert.ToInt32(controlTag.Attributes("LocationY").First().Value));
Canvas.Controls.Add(label);
// handlers.
label.MouseDown += new MouseEventHandler(label_MouseDown);
label.MouseMove += new MouseEventHandler(label_MouseMove);
label.MouseUp += new MouseEventHandler(label_MouseUp);
label.MouseClick += new MouseEventHandler(label_MouseClick);
label.MouseDoubleClick += new MouseEventHandler(label_MouseDoubleClick);
}
else if (t.Name == "LinkLabel")
{
control.Tag = controlTag.Attributes("Address").First().Value;
LinkLabel link = (LinkLabel)control;
link.AutoSize = true;
link.Location = new Point(
Convert.ToInt32(controlTag.Attributes("LocationX").First().Value),
Convert.ToInt32(controlTag.Attributes("LocationY").First().Value));
if (boldness == "yeah")
control.Font = new Font(control.Font.Name, control.Font.Size, FontStyle.Bold);
link.LinkColor = Color.White;
Canvas.Controls.Add(link);
// Add handlers.
link.MouseDown += new MouseEventHandler(link_MouseDown);
link.MouseMove += new MouseEventHandler(link_MouseMove);
link.MouseUp += new MouseEventHandler(link_MouseUp);
link.MouseClick += new MouseEventHandler(link_MouseClick);
link.MouseDoubleClick += new MouseEventHandler(link_MouseDoubleClick);
}
else if (t.Name == "PictureBox")
{
PictureBox p = (PictureBox)control;
p.Image = Base64ToImage(controlTag.Attributes("Content").First().Value);
p.AutoSize = true;
p.Location = new Point(
Convert.ToInt32(controlTag.Attributes("LocationX").First().Value),
Convert.ToInt32(controlTag.Attributes("LocationY").First().Value));
Canvas.Controls.Add(p);
// Add handlers.
p.MouseDown += new MouseEventHandler(p_MouseDown);
p.MouseMove += new MouseEventHandler(p_MouseMove);
p.MouseUp += new MouseEventHandler(p_MouseUp);
p.MouseClick += new MouseEventHandler(p_MouseClick);
p.MouseDoubleClick += new MouseEventHandler(p_MouseDoubleClick);
}
}
this.Text = "wordreplaced 11 - " + o.FileName;
}
}
写入 XML 文件：
SaveFileDialog s = new SaveFileDialog();
s.Filter =
"wordReplaced Multimedia Format (*.mf)|*.mf|" +
"Word Document (*.docx)|*.docx|" +
"PDF Document (*.pdf)|*.pdf|" +
"Text FIle (*.txt)|*.txt";
s.Title = "wordReplaced 11 - Save Document";
s.CheckFileExists =
false;
XElement d;
using (s)
{
if (s.ShowDialog() == DialogResult.OK)
{
if (File.Exists(s.FileName))
d = XElement.Load(s.FileName);
else
d = new XElement("cs");
foreach (Control control in Canvas.Controls)
{
Type t = control.GetType();
switch (t.Name)
{
case "JTS.TextBox":
XElement xe0 = new XElement("JTS.TextBox",
new XAttribute("Content", control.Text),
new XAttribute("LocationX", control.Location.X),
new XAttribute("LocationY", control.Location.Y),
new XAttribute("A", control.ForeColor.A),
new XAttribute("R", control.ForeColor.R),
new XAttribute("G", control.ForeColor.G),
new XAttribute("B", control.ForeColor.B),
new XAttribute("Font", control.Font),
new XAttribute("Bold", "yeah"));
d.Add(xe0);
break;
case "LinkLabel":
LinkLabel ll = (LinkLabel)control;
try
{
XElement xe1 = new XElement("LinkLabel",
new XAttribute("Content", control.Text),
new XAttribute("LocationX", control.Location.X),
new XAttribute("LocationY", control.Location.Y),
new XAttribute("A", ll.LinkColor.A),
new XAttribute("R", ll.LinkColor.R),
new XAttribute("G", ll.LinkColor.G),
new XAttribute("B", ll.LinkColor.B),
new XAttribute("Font", ll.Font),
new XAttribute("Address", control.Tag),
new XAttribute("Bold", "yeah"));
d.Add(xe1);
}
catch
{
XElement xe1 = new XElement("LinkLabel",
new XAttribute("Content", control.Text),
new XAttribute("LocationX", control.Location.X),
new XAttribute("LocationY", control.Location.Y),
new XAttribute("A", ll.LinkColor.A),
new XAttribute("R", ll.LinkColor.R),
new XAttribute("G", ll.LinkColor.G),
new XAttribute("B", ll.LinkColor.B),
new XAttribute("Font", ll.Font),
new XAttribute("Bold", "yeah"));
d.Add(xe1);
}
break;
case "PictureBox":
PictureBox px = (PictureBox)control;
string ie = ImageToBase64(px.InitialImage, System.Drawing.Imaging.ImageFormat.Bmp);
XElement xe2 = new XElement("PictureBox",
new XAttribute("Content", ie),
new XAttribute("LocationX", px.Location.X),
new XAttribute("LocationY", px.Location.Y));
d.Add(xe2);
break;
default:
break;
}
d.Save(s.FileName);
FilePath = s.FileName;
Text = s.FileName;
ds = true;
}
}
}
哪个工作正常。没有一个问题。但是尝试扩展/使用等就是这么一团糟。我需要一种更高效/更干净/优雅的方式来写入和读取 XML 文件，并对它们执行 linq 查询。
我读过一些教程和一篇文章，但我不太明白。一个网站提到了 StreamWriter，这没有任何意义，而且似乎有许多不同的方法来使用 Linq 写入/读取 XML 文件，这对我来说非常令人困惑。
任何帮助都非常感谢，
谢谢
我认为你的问题不是使用错误的类或工具，而是你需要学习如何提取方法并停止以这种方式重复代码。
理想情况下，第一个代码应如下所示：
// In whatever method loads the file
string fileName = GetFileNameFromUser();
if (fileName != null)
{
var rootElement = GetRootElementOfXmlFile(fileName);
foreach (var controlTag in rootElement)
{
ProcessControlTag(controlTag);
}
}
private static void ProcessControlTag(XElement controlTag)
{
var type = GetControlType(controlTag);
if (type == null)
{
return;
}
var control = CreateControl(controlType, controlTag);
Canvas.Controls.Add(control);
}
private static void CreateControl(Type controlType, XElement controlTag)
{
var control = (Control)Activator.CreateInstance(controlType);
AddCommonControlModifications(control, controlTag);
if (controlType.Name == "Label")
{
AddLabelModifications(control, controlTag);
}
else if (controlType.Name == "LinkLabel")
{
AddLinkLabelModifications(control, controlTag);
}
else if (controlType.Name == "PictureBox")
{
AddPictureBoxModifications(control, controlTag);
}
}
这只是为了让你开始;我没有实现其中的几种方法。任何时候您有重复的代码（例如位置设置代码，或任何使您在 try/catch 中重复代码两次的事情），您都可以将其提取到方法中。
方法一般应为3--5行长，而不是几十行。将事物分解为小的、可重用的、最重要的是可读的块，用明确的名称声明它们的意图，这样你的代码读起来更像你的思维过程，而不是一长串神奇的咒语，以某种方式让计算机做你想做的事情。
代码有效吗？您有什么新要求要实施吗？你没有更重要的事情要做吗？
没坏就别修。你只会引入错误。