获得满足条件的集合项索引的哪种方式更好?

时间:2012-12-25 06:01:47

标签: c# linq collections

假设我有一组字符串。

List<string> lines = new List<string>(File.ReadAllLines("Test.txt"));

一个正则表达式,用于搜索该集合中的匹配项:

Regex r = new Regex(@"some regular expression");

如何获得与正则表达式匹配的元素的权限?

我有三个想法。


第一

var indeces1 = lines.Where(l => r.IsMatch(l))
                    .Select(l => lines.IndexOf(l));
foreach (int i in indeces1)
{
    Console.WriteLine(i);//Do the index-based task instead...
}

我不喜欢它,在原始集合上使用IndexOf。我错了,没关系?


var indeces2 = lines.Select((l, i) => new { Line = l, Index = i })
                    .Where(o => r.IsMatch(o.Line));

foreach (var o in indeces2)
{
    Console.WriteLine(o.Index);//Do the index-based task instead...
}

它似乎比第一个更好,但有没有办法在不创建匿名对象的情况下做同样的事情?


最后一个:

for (int i = 0; i < lines.Count; i++)
{
    if (r.IsMatch(lines[i]))
    {
        Console.WriteLine(i);//Do the index-based task instead...
    }
}

实际上我现在正在使用这个。但是我喜欢LINQ,我希望有一种LINQ方式来做同样的事情。

那么有更好的LINQ方式来完成这个简单的任务吗?

3 个答案:

答案 0 :(得分:2)

如果您喜欢LINQ,可以使用Enumerable.Range更简单:

var indexes = Enumerable.Range(0, lines.Count)
                        .Where(i => r.IsMatch(lines[i]));

修改

而不是使用File.ReadAllLines将所有行都存入内存:

List<string> lines = new List<string>(File.ReadAllLines("Test.txt"));

如果您的文件很大,您应该考虑使用延迟执行的ReadLines来提高效率:

var lines = File.ReadLines("C:\\Test.txt"));

答案 1 :(得分:2)

如果您只需要索引,那么为什么不尝试。选择(t =&gt; t.Index);在你的选项2. (在结尾)只获得IEnumerable索引。你将摆脱Anonymous对象。

所以你的查询是:

var indeces2 = lines.Select((l, i) => new { Line = l, Index = i })
        .Where(o => r.IsMatch(o.Line))
        .Select(t => t.Index);

答案 2 :(得分:0)

在这种情况下,我会使用你简陋的迭代器版本:

for (int i = 0; i < lines.Count; i++)
{
    if (r.IsMatch(lines[i]))
    {
        Console.WriteLine(i);//Do the index-based task instead...
    }
}

对于这种情况,LINQ并没有真正减少行数并增加可读性,与选项3相比。所以我会选择最简单的版本。