Question

我从Silverlight应用程序中获得了以下代码片段：

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时更新“所有”消息“。只要我删除“XElement temporaryElement”和“XcurrentMsg.Element ......”，这样就可以正常工作。并简单地使用currentMsg字符串作为输出。但我希望在我的XcurrentMsg / Cache中保存“新消息”。现在，如果我不删除这部分，我的应用程序变得非常疯狂。我认为它会在不停止的情况下将无限元素写入XcurrentMsg。

我无法弄清楚问题是什么。

的问候，

Answer 1

这是一个经典的LINQ问题。

变量messages包含对IEnumerable<someAnonymousType>的引用。您所犯的错误是假设在分配到messages后，所有Descendents都已枚举且someAnonymousType已设置完毕。

实际上在这一点上没有发生任何事情。只有当您开始使用foreach枚举集合时，才会枚举Descendents并创建匿名类型。即使在整个Descendents集合尚未枚举的情况下，实际上只有在第一个符合where子句条件的项目之前，才会创建一个投影（select的结果）在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是做什么的，但是在foreach完成之前，你应该推迟对XcurrentMsg的所有添加。所以拿这个：

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());

希望这有帮助！

操纵现有的XDocument（失败）

2 个答案: