List<T>.ForEach 带索引

Question

我正在尝试找到以下代码的 LINQ 等效项：

NameValueCollection nvc = new NameValueCollection();

List<BusinessLogic.Donation> donations = new List<BusinessLogic.Donation>();
donations.Add(new BusinessLogic.Donation(0, "", "", "");
donations.Add(new BusinessLogic.Donation(0, "", "", "");
donations.Add(new BusinessLogic.Donation(0, "", "", "");

for(var i = 0; i < donations.Count(); i++)
{
    // NOTE: item_number_ + i - I need to be able to do this
    nvc.Add("item_number_" + i, donations[i].AccountName);
}

我希望我可以使用类似的东西：

NameValueCollection nvc = new NameValueCollection();

List<BusinessLogic.Donation> donations = new List<BusinessLogic.Donation>();
donations.Add(new BusinessLogic.Donation(0, "", "", "");
donations.Add(new BusinessLogic.Donation(0, "", "", "");
donations.Add(new BusinessLogic.Donation(0, "", "", "");

donations.ForEach(x => nvc.Add("item_name_" + ??, x.AccountName);

但我还没有找到一种方法来确定循环在哪个迭代上。任何帮助将不胜感激！

Answer 1

LINQ 没有ForEach 方法，这是有充分理由的。 LINQ 用于执行查询。它旨在从某些数据源获取信息。它不是旨在改变数据源。 LINQ 查询不应引起副作用，这正是您在这里所做的。

List 类确实有一个 ForEach 方法，这就是您正在使用的方法。因为它实际上不在 System.Linq 命名空间中，所以从技术上讲它不是 LINQ 的一部分。

您的问题中的 for 循环没有任何问题。尝试以您尝试的方式更改它是错误的（从良好实践的角度来看）。

Here 是一个更详细地讨论此事的链接。

现在，如果您想忽略该建议并使用ForEach 方法，编写一个提供操作索引的方法并不难：

public static void ForEach<T>(this IEnumerable<T> sequence, Action<int, T> action)
{
    // argument null checking omitted
    int i = 0;
    foreach (T item in sequence)
    {
        action(i, item);
        i++;
    }
}

Answer 2

如果你真的想使用 List.ForEach，很简单：

//[...]
int i=0;
donations.ForEach(x => nvc.Add("item_name_" + i++, x.AccountName);

Answer 3

这有点令人费解并创建了一个中间集合，但是怎么样：

donations.Select((x, i) => new {Name = "item_name_" + i, x.AccountName})
    .ToList()
    .ForEach(x=> nvc.Add(x.Name, x.AccountName));

这使用the overload of Enumerable.Select which incorporates the index。

我不得不说，这样做并没有什么真正的好处。您使用中间集合创建了更多开销，恕我直言，您失去了原始 for 循环的可读性。

如果您愿意使用foreach 循环而不是List.ForEach，也可以跳过中间集合。请参阅@wageoghe's answer（再次强烈推荐）。

Answer 4

这是一篇旧帖子，但在 Google 中排名很高，所以我认为更通用的方法会更合适。 （另外，我往往会忘记这是如何完成的，这意味着我每次都必须谷歌......）

假设一个整数列表：

var values = new List<int>() { 2, 3, 4, 0x100, 74, 0xFFFF, 0x0F0F };

要迭代列表并创建索引，请执行以下操作：

values.Select((x, i) => new
{
    item = x,
    index = i
})
.ToList()
.ForEach(obj =>
{
    int value = obj.item;
    int valueIndex = obj.index;
});

Answer 5

您是否有任何理由不使用Dictionary<string, string>，因为您的姓名/键似乎是独一无二的？这会更快，您可以使用ToDictionary 标准查询运算符。

此外，如果您确实希望使用扩展方法（尽管 Servy 说这里的 for 循环是正确的解决方案），那么您可以编写自己的方法 - 请参阅 here。

Answer 6

捎带@lc 的答案。

foreach (var x in donations.Select((d, i) => new {ItemName = "item_name_" + i, AccountName = d.AccountName}))
{
  nvc.Add(x.ItemName, x.AccountName);
}

Answer 7

这也可以使用聚合来完成。检查以下示例：

var data = new[]
{
    "A",
    "B",
    "C",
    "D"
};

var count = data.Aggregate(0, (index, item) =>
{
    Console.WriteLine("[{0}] => '{1}'", index, item);

    return index + 1;
});

Console.WriteLine("Total items: {0}", count);

Aggregate 在这里充当for 语句。唯一的缺点是每次迭代都需要将索引的值加一并返回。

Answer 8

我喜欢这样做：

NameValueCollection nvc = new NameValueCollection();

List<BusinessLogic.Donation> donations = new List<BusinessLogic.Donation>();
donations.Add(new BusinessLogic.Donation(0, "", "", ""));
donations.Add(new BusinessLogic.Donation(0, "", "", ""));
donations.Add(new BusinessLogic.Donation(0, "", "", ""));

Enumerable
    .Range(0, donations.Count())
    .ToList()
    .ForEach(i => nvc.Add("item_number_" + i, donations[i].AccountName));

Answer 9

一个糟糕的解决方案是简单地使用 select 并返回一个无用的值。

items.toList().Select((el, index) =>
{
    el.doStuff();
    return 0;
});

Answer 10

C# 7（大约 2017 年）添加了更简洁的元组语法，可以与 Select 的索引形式一起使用（无需手动计算索引），以提供简洁明了的语法。例如：

foreach ((var item, int n) in TheItems.Select((i,n)=>(i,n))) {
    // item is the current item, n is its index
}

或者，如果您愿意：

foreach (var element in TheItems.Select((item,n)=>(item,n))) {
    // element.item is the current item, element.n is its index
}

所以在 OP 的情况下，例如：

foreach ((var item, int n) in donations.Select((i,n)=>(i,n)))
    nvc.Add($"item_number{n}", item.AccountName);

或者，如果您愿意：

foreach ((var k, var v) in donations.Select((i,n)=>($"item_number_{n}",i.AccountName)))
    nvc.Add(k, v);

或者，类似于this answer，但略显冗长，但请注意，这将通过调用ToList() 杀死惰性求值，这会在调用ForEach() 之前强制枚举整个源：

TheItems.Select((item,n)=>(item,n)).ToList().ForEach(element => { 
    // element.item is the current item, element.n is its index
});

Answer 11

试试这个 -

donations.ForEach(x =>
         {
             int index = donations.IndexOf(x);
             nvc.Add("item_name_" + index, x.AccountName);
         });