序列不包含元素

时间:2015-06-18 21:57:04

标签: c# linq random

有人可以解释为什么以下LINQ查询会抛出InvalidOperationException吗?    (不要说列表没有元素,我正在寻找的值总是存在于集合中)

class Program
{
     static int lastNumber;
     static void Main()
     {
          int loopCount = 100; int minValue = 1; int maxValue = 10000;
          var numbers = Enumerable.Range(minValue, maxValue).ToList();//or ToArray();
          Random random = new Random();

          for (int i = 0; i < loopCount; i++)
          {
              //.First() throws the exception but it is obvious that the value exists in the list
              int x = numbers.Where(v => v == NewMethod(minValue, maxValue, random)).First();
          }
          Console.WriteLine("Finished");
          Console.ReadLine();

     }

     private static int NewMethod(int minValue, int maxValue, Random random)
     {
         var a1 = random.Next(minValue + 1, maxValue - 1);
         lastNumber = a1;
         return a1;
     }
}

只有当我在lambda exssion中调用NewMethod时才会出现问题 如果这样做就可以了

int temp=NewMethod(minValue, maxValue, random);
int x = numbers.Where(v => v == temp).First();

我添加了lastNumber字段以帮助调试代码,您可以看到该值在崩溃时存在于该集合中

PS
问题不是随机变量,我删除了参数并在方法中创建了一个新的局部随机但问题仍然存在

更新

事实证明,您不需要循环使其崩溃。 如果多次运行程序,则会再次出现错误

using System;
using System.Collections.Generic;
using System.Diagnostics;
using System.Linq;
class Program
{
    static int lastNumber;
    static void Main()
    {
        int minValue = 1, maxValue = 100000;
        var numbers = Enumerable.Range(minValue, maxValue).ToArray();
        //Crashes sometimes
        int x = numbers.Where(v => v == NewMethod(minValue, maxValue)).First();
        Console.WriteLine("Finished");
        Console.ReadLine();
     }

     private static int NewMethod(int minValue, int maxValue)
     {
          Random random = new Random();
          var a1 = random.Next(minValue + 1, maxValue - 1);
          lastNumber = a1;
          return a1;
      }
 }

2 个答案:

答案 0 :(得分:5)

@Oleg是对的,但

Where扫描列表,查找符合给定条件的元素。在这种情况下,每个元素的标准都会发生变化。如果您将问题缩小,请说出包含5个元素的数组:

List<Int32> (5 items)
1 
2 
3 
4 
5 

然后通过寻找与某个随机数匹配的值(x为Item[i]r为随机数)循环播放:

Item 1: x = 1, r = 2  // fail
Item 2: x = 2, r = 3  // fail
Item 3: x = 3, r = 2  // fail
Item 4: x = 4, r = 3  // fail
Item 5: x = 5, r = 2  // fail

请注意,没有项目与该特定随机数匹配,因此没有项目符合条件且First会引发异常!

正如您所发现的,修复是在枚举之前生成随机数

int temp=NewMethod(minValue, maxValue, random);  // say 2

Item 1: x = 1, temp = 2  // fail
Item 2: x = 2, temp = 2  // success!

旁注:

在这里使用maxValue有点误导:

 Enumerable.Range(minValue, maxValue)

由于Enumerable.Range的第二个参数是结果集合的长度,而不是最大值。在这种情况下,它起作用的原因是你从1开始,但如果使用了99和100,那么结果会出乎意料 - 你得到99到198的范围。

答案 1 :(得分:4)

这是因为 NewMethod会在每次迭代时调用,并且每次都会生成新的随机数。

但是在这段代码中,首先生成数字,然后与数字集合的每个元素进行比较。

int temp=NewMethod(minValue, maxValue, random);
int x = numbers.Where(v => v == temp).First();