Bug:如果文件被删除,列表会保留在内存中,如果文件被删除,列表会保存两次数据

Bug: List remains in memory if file deleted, and saves data twice if file is deleted

我的代码中有一个奇怪的错误,我不明白为什么它会这样。我不是最有经验的代码战士,所以希望有更多经验的人能明白为什么。

我正在将对象列表保存到 XML 文件中。我将对象列表传递给我的函数进行保存,并且我有代码来检查对象是否已经存在,如果对象不存在则只将对象保存到 XML 文件。如果我按一次“保存”按钮,它会很好地创建文件,并按预期保存。如果我再次按保存,它会再次按预期工作,并且只会保存新对象。

我遇到的错误是...

如果我按保存按钮,比如 3 次,然后删除文件,当我按保存并重新创建文件时,列表会保存 3 次。就好像之前的列表还在四处浮动,只是被叠加在一起。

这是我的保存代码...如果有帮助,我的代码 HasElement() 是 XElement 的扩展方法,并且只是 returns 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

很可能这个错误超出了这个函数的范围,并且以某种方式传递了具有重复条目的列表。您可以通过使用 Distinct 进行分组或更改您的代码以一个接一个地附加子元素来在您的函数中处理这种情况。但更合适的解决方案是确定如何将重复项添加到源列表。

我检查了代码,在我看来,调用部分有问题。现在,当我点击保存,并将 contactcontact1contact2 修改为其他内容,然后再次点击,旧的联系人值将与新的联系人值一起保存。没关系。当我删除文件并单击保存时,只会写入新值。联系方式中少'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; }
}