System.ArgumentOutOfRangeException LINQ C＃

Question

我必须编写一个程序，以每列的域。

我的代码是这样，但是当我想播放时抛出异常（System.ArgumentOutOfRangeException）。

string[] all_lines =  {  "1, 2, 3" , "4, 5, 6", "4, 2, 3", "4, 2, 6", "9, 8, 7" };


string[] separator = new string[] { ", " };
int nAtt = all_lines[0].Split(separator, StringSplitOptions.RemoveEmptyEntries).Count() ;
List<IEnumerable> dom = new List<IEnumerable>();

for (int i = 0; i < nAtt; i++)
{
    var ele = (from lines in all_lines
                let data = lines.Split(separator, StringSplitOptions.RemoveEmptyEntries)
                select data.ElementAt(i)).Distinct();
    dom.Add(ele);
}

foreach (var row in dom)
{
    Console.Write("( ");
    foreach (var ele in row)
        Console.Write("{0} ", ele);
    Console.WriteLine(")");
}

此代码的输出应为：

（1 4 9）
（2 5 8）
（3 6 7）

有没有解决方案或替代方案？

Answer 1

有关闭问题。 在这里看到ElementAt(i) 。

for (int i = 0; i < nAtt; i++)
{
    var ele = (from lines in all_lines
                let data = lines.Split(separator, StringSplitOptions.RemoveEmptyEntries)
                select data.ElementAt(i)).Distinct();
    dom.Add(ele);
}

这个查询将使用变量i它的自我。 它不是副本。 因此，在循环完成后， i变成nAtt并且在尝试使用它时会遇到异常。 因此您必须先存储变量，然后再使用它。

var i1 = i;
var ele = (from lines in all_lines
        let data = lines.Split(separator, StringSplitOptions.RemoveEmptyEntries)
        select data.ElementAt(i1)).Distinct();
dom.Add(ele);

Answer 2

你的技术是因为非常低效的，通过使用.Count()和.ElementAt()多次，你是不必要重申你可枚举，许多倍。

一种更有效，更优雅，更不易出错的方法可能是：

var dom = all_lines
    .SelectMany(line => line
                         .Split(new[]{", "}, StringSplitOptions.RemoveEmptyEntries)
                         .Select((x, i) => new{x, i}))
    .GroupBy(x => x.i)
    .Select(g => g.Select(gi => gi.x).Distinct())
    .ToList();