增加90000至的XElement的XDocument

Question

我有一个Dictionary<int, MyClass>

它包含100,000个项目

万件值填充，而90000为空。

我有这样的代码：

var nullitems = MyInfoCollection.Where(x => x.Value == null).ToList(); 
nullitems.ForEach(x => LogMissedSequenceError(x.Key + 1)); 

private void LogMissedSequenceError(long SequenceNumber) 
     { 
      DateTime recordTime = DateTime.Now; 

      var errors = MyXDocument.Descendants("ERRORS").FirstOrDefault(); 
      if (errors != null) 
      { 

       errors.Add(
        new XElement("ERROR", 
         new XElement("DATETIME", DateTime.Now.ToString("dd/MM/yyyy HH:mm:ss:fff")), 
         new XElement("DETAIL", "No information was read for expected sequence number " + SequenceNumber), 
         new XAttribute("TYPE", "MISSED"), 
         new XElement("PAGEID", SequenceNumber) 
         ) 
       ); 
      } 
     } 

这似乎需要约2分钟即可完成。我似乎无法找到瓶颈可能出现的位置，或者这种时机听起来正确吗？

任何人都可以看到为什么采取这么久？

Answer 1

如何关于用LINQ查询替换方法调用？

static void Main(string[] args) 
{ 

    var MyInfoCollection = (from key in Enumerable.Range(0, 100000) 
          let value = (MoreRandom() % 10 != 0) 
                ? (string)null 
                : "H" 
          select new { Value = value, Key = key } 
          ).ToDictionary(k => k.Key, v => v.Value); 

    var MyXDocument = new XElement("ROOT", 
            new XElement("ERRORS") 
           ); 
    var sw = Stopwatch.StartNew(); 
    //=== 
    var errorTime = DateTime.Now.ToString("dd/MM/yyyy HH:mm:ss:fff"); 
    var addedIndex = MyInfoCollection.Select((item, index) => 
                new 
                { 
                 Value = item.Value, 
                 Key = item.Key, 
                 Index = index 
                }); 
    var errorQuery = from item in addedIndex 
        where string.IsNullOrEmpty(item.Value) 
        let sequenceNumber = item.Key + 1 
        let detail = "No information was read for expected " + 
            "sequence number " + sequenceNumber 
        select new XElement("ERROR", 
         new XElement("DATETIME", errorTime), 
         new XElement("DETAIL", detail), 
         new XAttribute("TYPE", "MISSED"), 
         new XElement("PAGEID", sequenceNumber) 
         ); 

    var errors = MyXDocument.Descendants("ERRORS").FirstOrDefault(); 
    if (errors != null) 
     errors.Add(errorQuery); 
    //=== 
    sw.Stop(); 
    Console.WriteLine(sw.ElapsedMilliseconds); //623 
} 
static RandomNumberGenerator rand = RandomNumberGenerator.Create(); 
static int MoreRandom() 
{ 
    var buff = new byte[1]; 
    rand.GetBytes(buff); 
    return buff[0]; 
} 

Answer 2

这是我最有可能做的。

private void BuildErrorNodes() 
{ 
    const string nodeFormat = @"<ERROR TYPE=""MISSED""><DATETIME>{0}</DATETIME><DETAIL>No information was read for expected sequence number {1}</DETAIL><PAGEID>{1}</PAGEID></ERROR>"; 

    var sb = new StringBuilder("<ERRORS>"); 
    foreach (var item in MyInfoCollection) 
    { 
     if (item.Value == null) 
     { 
      sb.AppendFormat(
       nodeFormat, 
       DateTime.Now.ToString("dd/MM/yyyy HH:mm:ss:fff"), 
       item.Key + 1 
      ); 
     } 
    } 

    sb.Append("</ERRORS>"); 

    var errorsNode = MyXDocument.Descendants("ERRORS").FirstOrDefault(); 
    errorsNode.ReplaceWith(XElement.Parse(sb.ToString())); 
} 

Answer 3

如果您MyInfoCollection是巨大的，我不会把ToList()它只是让你可以使用ForEach扩展方法。调用ToList()将创建并填充一个巨大的列表。我会删除ToList()调用，并将.ForEach变成for each语句，或者写一个。 ForEach扩展方法为IEnumerable<T>。

然后对其进行配置并查看需要多长时间。还有一件事是删除ERRORS元素的find和null检查。如果它不在那里，那么请不要拨打上面的for each声明。这样你空一次检查一次而不是90,000次。

加上迈克尔葡萄汁指出的那样，我会定义一个字符串来保存的价值DateTime.Now.ToString("dd/MM/yyyy HH:mm:ss:fff")，然后引用它或者通过它在此外，你甚至不使用此电话：

DateTime recordTime = DateTime.Now;