Where子句仅返回匹配索引数组的部分产品：原因排查与修复

首先咱们拆解下你的问题：用Where结合索引筛选产品时，只拿到了前两个匹配项，索引5、7、10对应的产品完全没命中，大概率是以下几个原因之一：

最可能的原因：目标索引超出orderedProducts的实际范围 #

C#里集合的索引是从0开始的——比如集合有3个元素，最大可用索引是2（对应第三个元素）。如果你的orderedProducts.Count小于11，那索引5、7、10对应的元素根本不存在，自然不会被筛选出来。

你可以先加一行代码确认集合长度：

Console.WriteLine($"orderedProducts的元素总数：{orderedProducts.Count}");

如果总数小于11，那问题就出在这——你要取的索引超出了集合的实际范围。

另一个可能：混淆了「1-based序号」和「0-based索引」 #

很多人会下意识把第一个元素当成序号1，但C#里集合的索引是从0开始的。如果你的indexes列表里的数值是元素的序号（比如你想拿第1、2、5、7、10个元素），那实际对应的索引应该是0、1、4、6、9，这时候需要把索引列表转成0-based：

var zeroBasedIndexes = indexes.Select(x => x - 1).ToList();

隐藏坑：orderedProducts是延迟执行的Linq查询 #

如果orderedProducts不是直接的List<T>或数组，而是之前通过Where、Skip、Take等Linq操作生成的延迟查询，那Where((pr, i) => ...)里的i是当前查询的枚举索引，不是原始集合的索引。这种情况下，需要先把查询固化成内存集合：

var fixedProducts = orderedProducts.ToList(); // 先把查询转换成List，固定索引

修复后的优化代码 #

另外，你原来用indexes.Any(x => x == i)的效率偏低（每次查找都是O(n)），建议用HashSet<int>优化查找速度（O(1)），最终代码可以根据情况选择：

情况1：indexes是0-based索引，且orderedProducts长度足够 #

var indexes = new List<int> { 1, 2, 5, 7, 10 };
var indexSet = new HashSet<int>(indexes); // 转成HashSet优化查找
var myProducts = orderedProducts
    .Where((pr, i) => indexSet.Contains(i))
    .ToList();

情况2：indexes是1-based序号 #

var indexes = new List<int> { 1, 2, 5, 7, 10 };
var zeroBasedIndexSet = new HashSet<int>(indexes.Select(x => x - 1));
var myProducts = orderedProducts
    .Where((pr, i) => zeroBasedIndexSet.Contains(i))
    .ToList();

情况3：orderedProducts是延迟查询 #

var indexes = new List<int> { 1, 2, 5, 7, 10 };
var indexSet = new HashSet<int>(indexes);
var fixedProducts = orderedProducts.ToList(); // 先固化集合
var myProducts = fixedProducts
    .Where((pr, i) => indexSet.Contains(i))
    .ToList();

内容的提问来源于stack exchange，提问作者toshiomagic