It does not sit well with me to make the one and only sequence operator that is only useful for its side effects.

class Cargo
{
    public int UniqueId { get; set; }
    
    public int Weight { get; set; }
    
    public bool WeightLimited { get; set; }
}

const int LIMIT = 50;

...

var cargo = new[] { new Cargo { UniqueId = 1, Weight = 150, WeightLimited = true},
    new Cargo { UniqueId = 2, Weight = 100, WeightLimited = false } };
    
cargo.ForEach(c => c.WeightLimited && c.Weight > LIMIT, c => ThrowException(c.UniqueId, c.Weight));

It does not sit well with me to make the one and only sequence operator that is only useful for its side effects.

Не стоит делать extension method для IEnumerable, единственным смыслом которого будет достижение побочных эффектов

Методы, обрабатывающие последовательности IEnumerable со спецэффектами не должны быть методами-расширениями для IEnumerable

ВВ>cargo.ForEach(c => c.WeightLimited && c.Weight > LIMIT, c => ThrowException(c.UniqueId, c.Weight));
ВВ>

            var wrongCargo = cargo.FirstOrDefault(c => c.WeightLimited && c.Weight > LIMIT);

            if (wrongCargo != null)
                ThrowException(wrongCargo.UniqueId, wrongCargo.Weight));

foreach(var c in cargo.Where(c => c.WeightLimited && c.Weight > LIMIT))
  ThrowException(c.UniqueId, c.Weight));

int result;
result = seq.NullCount();
result = seq.NullCount(); // usually good

result = seq.ForEach(action);
result = seq.ForEach(action); // usually bad

ВВ>var arr = new int[]{ 1, 1, 1, 2, 2, 2, 3, 3, 3};            
ВВ>var it = arr.ForEach(i => i < 2, Console.WriteLine).ForEach(i => i < 3, Console.WriteLine);
ВВ>

ВВ>А куда стандартная делась?

ВВ>Да, пусть вероятность наличия побочных эффектов в данном действии 99 к 1 — но мы же не вероятность обсуждаем, а дизайн?

if (!sequence.All(condition))
  throw new InvalidOperationException();

    int[] a = new[] {1, 2, 4, 5, 7, 8, 9};
    int t = 0;
    var w = 6;
    var rucksack = a.Where(x => (t += x) <= w);

public void ForEach<T>(this ICollection<T> source, Action<T> action) {
  foreach(T item in source) action(item);
}

ВВ>>var arr = new int[]{ 1, 1, 1, 2, 2, 2, 3, 3, 3};            
ВВ>>var it = arr.ForEach(i => i < 2, Console.WriteLine).ForEach(i => i < 3, Console.WriteLine);
ВВ>>

using System;
using System.Linq;
using System.Collections.Generic;

class Program
{
    static void Main()
    {
        var arr = new int[]{ 1, 1, 1, 2, 2, 2, 3, 3, 3};            
        var it = arr.ForEach(i => i < 2, Console.WriteLine).ForEach(i => i < 3, Console.WriteLine);
        
        foreach (var v in it)
        {}
    }
}

static class Iterator
{
    public static IEnumerable<T> ForEach<T>(this IEnumerable<T> self, Func<T,Boolean> condition, Action<T> action)
    {
        foreach (var v in self)
            if (condition(v))
            {
                action(v);
                yield return v;
            }
    }
}

ВВ>>А куда стандартная делась?

ВВ>>Да, пусть вероятность наличия побочных эффектов в данном действии 99 к 1 — но мы же не вероятность обсуждаем, а дизайн?