嵌套foreach代码重构：能否用单foreach、Linq或更优雅方式实现？

你的代码本质是遍历customers和notifiers的所有组合，对每一组调用notifyCustomer(notifier, customer)。以下是几种更简洁优雅的实现方式：

1. 用SelectMany生成扁平集合+单个foreach #

通过LINQ的SelectMany将嵌套的集合展开为所有(notifier, customer)对的扁平集合，再用单个foreach遍历执行操作：

foreach (var (notifier, customer) in customers.SelectMany(c => notifiers.Select(n => (n, c))))
{
    notifyCustomer(notifier, customer);
}

2. LINQ ForEach链式调用 #

如果不需要中断循环或处理复杂逻辑，可以用LINQ的ForEach链式调用（注意需先转为List，因为原生IEnumerable没有ForEach方法）：

customers.SelectMany(c => notifiers.Select(n => (n, c)))
         .ToList()
         .ForEach(pair => notifyCustomer(pair.n, pair.c));

3. 自定义扩展方法（复用性更强） #

如果这类"遍历两个集合所有组合执行操作"的场景多次出现，可以封装一个扩展方法，让代码语义更清晰：

public static class EnumerableExtensions
{
    public static void ForEachCrossPair<TLeft, TRight>(this IEnumerable<TLeft> leftItems, IEnumerable<TRight> rightItems, Action<TRight, TLeft> action)
    {
        foreach (var left in leftItems)
        {
            foreach (var right in rightItems)
            {
                action(right, left);
            }
        }
    }
}

// 调用时只需一行代码
customers.ForEachCrossPair(notifiers, notifyCustomer);

注意事项 #

• 如果notifyCustomer执行过程中需要中断循环、捕获异常或处理复杂分支，原生的嵌套foreach其实是最直观易读的选择；
• LINQ方式更适合逻辑简单、追求代码简洁的场景，扩展方法则胜在语义清晰和复用性。

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