操作现有的 XDocument（失败）

Question

我从 Silverlight 应用程序中获得了以下代码 sn-p：

var messages = from message in XcurrentMsg.Descendants("message")
                               where    DateTime.Parse(message.Attribute("timestamp").Value).CompareTo(DateTime.Parse(MessageCache.Last_Cached())) > 0
                               select new
                               {
                                   ip = message.Attribute("ip").Value,
                                   timestamp = message.Attribute("timestamp").Value,
                                   text = message.Value,
                               };
                if (messages == null)
                    throw new SystemException("No new messages recorded. Application tried to access non existing resources!");

            foreach (var message in messages)
            {
                XElement temporaryElement = new XElement("message", message.text.ToString(), new XAttribute("ip", message.ip.ToString()), new XAttribute("timestamp", message.timestamp.ToString()));
                XcurrentMsg.Element("root").Element("messages").Add(temporaryElement);

                AddMessage(BuildMessage(message.ip, message.timestamp, message.text));
                msgCount++;
            }

            MessageCache.CacheXML(XcurrentMsg);
            MessageCache.Refresh();

XcurrentMsg 是从我的服务器获取的包含消息的 XDocument： 结构

<root>
    <messages>
          <message ip="" timestamp=""> Text </message>
    </messages>
</root>

我想获得比我上次缓存 XcurrentMsg 时更新的所有“消息”。 只要我剪掉“XElementtemporaryElement”和“XcurrentMsg.Element ....”，这就可以正常工作 并简单地使用 currentMsg 字符串作为输出。 但我希望将“新消息”保存在我的 XcurrentMsg / Cache 中。 现在，如果我不删除这部分，我的应用程序会变得非常疯狂。 我认为它会不停地向 XcurrentMsg 写入无限元素。

我不知道是什么问题。

问候，

Answer 1

这是一个经典的 LINQ 陷阱。

变量messages 持有对IEnumerable<someAnonymousType> 的引用。您犯的错误是假设在分配给messages 之后，所有Descendents 都已被枚举并且someAnonymousType 的集合已被构建。

实际上，此时没有发生任何事情。只有当您开始使用 foreach 枚举集合时，才会枚举 Descendents 并创建匿名类型。即使在这一点上整套Descendents 还没有被枚举，实际上直到第一个满足where 子句条件的项目，然后一次只创建一个投影（选择的结果）为foreach 循环通过。

因此，您添加的消息也包含在枚举中，并且因为这会引发其他消息，您会以无限循环结束，或者至少是抱怨您尝试修改正在枚举的集合的错误。

如果您想确保要枚举的项目列表在支持循环之前已修复，请将ToList() 添加到查询中，这将创建一个List<T>。

var messages = (from message in XcurrentMsg.Descendants("message")
                           where    DateTime.Parse(message.Attribute("timestamp").Value).CompareTo(DateTime.Parse(MessageCache.Last_Cached())) > 0
                           select new
                           {
                               ip = message.Attribute("ip").Value,
                               timestamp = message.Attribute("timestamp").Value,
                               text = message.Value,
                           }).ToList();

注意，无论哪种情况都不需要测试messages是否为null，它不会为null，它可能是一个空的枚举或列表。

Answer 2

不确定 AddMessage 做了什么，但您应该推迟到 XcurrentMsg 的所有添加，直到 foreach 完成。所以拿这个：

foreach (var message in messages)
{
    XElement temporaryElement = new XElement("message", 
        message.text.ToString(), new XAttribute("ip", message.ip.ToString()), 
        new XAttribute("timestamp", message.timestamp.ToString()));
    XcurrentMsg.Element("root").Element("messages").Add(temporaryElement);

    AddMessage(BuildMessage(message.ip, message.timestamp, message.text));
    msgCount++;
}

把它变成这样：

List<XElement> elementsToAdd = new List<XElement>();
foreach (var message in messages)
{
    XElement temporaryElement = new XElement("message", 
        message.text.ToString(), new XAttribute("ip", message.ip.ToString()), 
        new XAttribute("timestamp", message.timestamp.ToString()));
    elementsToAdd.Add(temporaryElement);

    AddMessage(BuildMessage(message.ip, message.timestamp, message.text));
    msgCount++;
}

XcurrentMsg.Element("root").Element("messages").Add(elementsToAdd.ToArray());

希望这会有所帮助！