Question

我的代码中有一个奇怪的错误，我无法理解为什么它的行为方式如此。我不是最有经验的代码战士，所以希望有更多经验的人可以理解为什么。

我将对象列表保存到XML文件中。我将对象列表传递给我的函数以进行保存，并且我已经有代码来检查对象是否已经存在，并且只有在不存在的情况下才将对象保存到XML文件中。如果我按一下“保存”按钮，它会将文件设置为正常，并按预期保存。如果我再次按“保存”，它将再次按预期工作，只保存新对象。

我遇到的错误是......

如果我按下保存按钮，比如说3次然后删除文件，当我按保存并重新创建文件时，列表会被保存3次。好像以前的列表仍然漂浮在一起并且只是相互叠加。

这是我保存的代码...如果有帮助，我的代码HasElement（）是XElement的扩展方法，只返回FirstOrDefault（）。保存仅在返回为null时发生。

  public void SaveDB(List<ContactList> cl)
         {
             if (cl != null)
             {
                 if (!File.Exists(DBPath))
                 {
                     XDocument doc = new XDocument(
                         new XDeclaration("1.0", "utf-8", "yes"),
                         new XElement("Contacts")
                         );
                     doc.Save(DBPath);
                     MessageBox.Show("File Created: " + DBPath);
                 }
                 MessageBox.Show(DBPath + " already exists!");
                 XDocument Doc = XDocument.Load(DBPath);
                 List<XElement> elmAdd = new List<XElement>();
                 XElement root = Doc.Element("Contacts");

                 foreach (ContactList CL in cl)
                 { 
                    if (root.HasElement(CL.Name) == null)
                    {
                     if (CL.Selected == true)
                         {
                             XElement eName = new XElement(CL.Name, "true");
                             elmAdd.Add(eName);
                         }
                         else if (CL.Selected == false)
                         {
                             XElement eName = new XElement(CL.Name, "false");
                             elmAdd.Add(eName);
                         }
                     }
                 }


                 MessageBox.Show("Lists saved");
                 Doc.Element("Contacts").Add(elmAdd);
                 Doc.Save(DBPath);
             } // End if null
             else
             {
                 MessageBox.Show("Debug: List is empty");
             }
         } // end method

Answer 1

很可能这个bug超出了这个函数的范围，并且已经传递了具有重复条目的List。您可以通过使用Distinct进行分组或更改您的代码来逐个附加子元素来处理您的功能。但更合适的解决方案是确定重复项如何添加到源列表中。

Answer 2

我检查了代码，在我看来，调用部分有问题。现在，当我点击保存，然后将contact，contact1，contact2修改为其他内容时，再次点击，旧的联系人值将与新的联系人一起保存。没关系。删除文件后，单击“保存”，只会写入新值。在记忆中不会记住接触中的'a'少。

private ContactList MyContact { get; set; } // MyContact is initialized with name contact

void Clickery(object o, RoutedEventArgs e)
{
    MyContact = new ContactList
    {
        Name = MyContact.Name + "a",
        Selected = false
    };
}

void Clicky(object o, RoutedEventArgs e)
{
    string DBPath = "somefile.txt";
    List<ContactList> cl = new List<ContactList>() { 
        MyContact,
        new ContactList { Name = "contact1", Selected = false },
        new ContactList { Name = "contact2", Selected = true } };
    if (cl != null)
    {
        if (!File.Exists(DBPath))
        {
            XDocument doc = new XDocument(
                new XDeclaration("1.0", "utf-8", "yes"),
                new XElement("Contacts")
                );
            doc.Save(DBPath);
            MessageBox.Show("File Created: " + DBPath);
        } else {
            MessageBox.Show(DBPath + " already exists!");
        }
        XDocument Doc = XDocument.Load(DBPath);
        List<XElement> elmAdd = new List<XElement>();
        XElement root = Doc.Element("Contacts");

        foreach (ContactList CL in cl)
        {
            if (root.Element(CL.Name) == null)
            {
                if (CL.Selected == true) {
                    XElement eName = new XElement(CL.Name, "true");
                    elmAdd.Add(eName);
                } else {
                    XElement eName = new XElement(CL.Name, "false");
                    elmAdd.Add(eName);
                }
            }
        }

        MessageBox.Show("Lists saved");
        Doc.Element("Contacts").Add(elmAdd);
        Doc.Save(DBPath);
    } else { // End if null
        MessageBox.Show("Debug: List is empty");
    }
} // end method

class ContactList
{
    public string Name { get; set; }
    public bool Selected { get; set; }
}

错误：如果文件被删除，列表将保留在内存中，如果删除文件，则列出数据两次

2 个答案: