ForEach, a simple but very useful extension method

discount furniture » ForEach, a simple but very useful extension method

discount furniture » ForEach, a simple but very useful extension method — Tue, 19 Aug 2008 10:51:38 GMT

Krzysztof Koźmic — Tue, 19 Aug 2008 11:22:46 GMT

You're not checking for nulls :)

Other than that - similar method has been on my Utils.dll library for a long time now. It should have been in the framework in the first place.

Miguel Madero — Tue, 19 Aug 2008 15:20:11 GMT

It's there for List<T> since 2.0

some of those methods should've been refactored down as IEnumerable<T> extensions

Mike Brown — Tue, 19 Aug 2008 16:27:14 GMT

Was too lazy to do it myself so I just cheated and used Let ;)

Paul — Wed, 20 Aug 2008 20:25:13 GMT

I'm missing the point I guess - but why is this usefull over just using the foreach() w/o the extension?

JamesCurran — Thu, 21 Aug 2008 20:16:33 GMT

@Paul

Given:

List<string> titles = ....

You could:

foreach(string title in titles)

{

title = title.ToUpperCase();

}

Or you could:

titles.ForEach(t => t=t.ToUpperCase());

JamesCurran — Thu, 21 Aug 2008 20:17:25 GMT

I'd suggest making it:

public static IEnumerable<T> ForEach<T>(this IEnumerable<T> collection, Action<T> action)

{...

return collection;

}

so you could chain them.

Techie.notepad — Thu, 02 Oct 2008 02:51:43 GMT

Playing with Extension Methods - also explaining why they are dangerous!

Kirill Osenkov — Sun, 01 Feb 2009 10:41:38 GMT

In my recent post about coding styles one particular thing provoked the majority of feedback and discussions:

Magnus — Fri, 04 Sep 2009 13:11:11 GMT

foreach(var t in titles) t = t.ToUpperCase();

titles.ForEach(t => t = t.ToUpperCase());

Not much difference...