DEV Community

loading...

C# lambda vs foreach readability.

Ulisses Cavalcante
I'm dotnet dev trying to improve angular skills
・1 min read

Hey guys, recently a receive an feedback that i need to change my code, turn an lambda expression into a foreach loop because its more adherent to clean code, i create this example to explain.
Overall code example

The both versions give me same results, but in my opinion this lambda expression is easy to understand, and easy to write and maintain, and i like to use that.

lambdaExample

i receive an recommendation to change that to a for-each loop, like this, to be more clean code adherent.

foreach

I really want see other people opinions and arguments about this, and i like lambda expression for me they are very powerful and easy to use.

Discussion (5)

Collapse
klawingco profile image
Marc Lawingco

I don't understand,
LINQ is much more readable than for each in anyday.
While foreach is faster than LINQ, its not that noticeable enough.

So the idea is
You want readability over performance = LINQ
You want a slightly faster performance over readiblity = For Each.

Collapse
ucavalcante profile image
Ulisses Cavalcante Author

Totally agree, ty.

Collapse
gruenwaldt profile image
Michael Grünwaldt

Hello,
since you asked for opinions.

For me the linq statement Is much easier to understand. Clear structure and short.

When one is used to linq i don‘t know how to write it clearer.
Even when not used to linq it is almost a sentence.

For me, in this case definately pro linq!
Regards
Michael

Collapse
ucavalcante profile image
Ulisses Cavalcante Author

Ty for your opinion Michael.

Collapse
ucavalcante profile image
Ulisses Cavalcante Author

Here an great article about.

link