More fun with LINQ

[andrewducker]

Having finally gotten back to the stuff I was looking at a couple of weeks ago, I've now discovered that I can replace:

doggies.ForEach(
    dog => dog.Paws.ForEach(
        paw => paw.Claws.ForEach(
            claw => claw.Clip())));

with

var myClaws = from dog in doggies from paw in dog.Paws from claw in paw.Claws select claw;
foreach (Claw claw in myClaws)
{
    claw.Clip();
}

Sadly, LINQ doesn't seem to have any way of executing queries rather than returning result sets. But then it is Language INtegrate Query. I think that's about as simple as you can get. Although I could replace the foreach loop with

myClaws.ForEach(claw => claw.Clip());

if I really wanted to...

Comments

[figg.livejournal.com]

Can you do

(from dog in doggies from paw in dog.Paws from claw in paw.Claws select claw).ForEach(claw => claw.Clip());

?

[andrewducker]

Oooh. Yes. Should have thought of that last night :->

[figg.livejournal.com]

p.s. maybe you should stick a clipAllClaws() method in Dog such that it contains the inner for each loops. hurr hurr

[andrewducker]

Well, yes. In real life I'd do something more like that :->

[surliminal.livejournal.com]

you'd have loved the doggies at the pub today:)

[darkoshi]

It sounded like your original intent was to make the code more easily understandable, but you appear to be going the other way now, from my perspective.

[figg.livejournal.com]

His original intent was to replace an older iteration idiom (for(i=o,i++;i<lim){...}) with a newer one (foreach (i in collection) {...}). (Because he liked the newer style, and disliked the older one) Since the original was just as readable and understandable before :D

[andrewducker]

Oh no it's not!

It's possibly simpler when it's A[i].B[j].C[k], but not when it's

A[i].B.C.D.E[j].F.G.H.I[k]
especially when there are multiple lines referencing each level.

[holyoutlaw.livejournal.com]

Dog dog doggity dog paw paw clippity clip clip clip Next!

[robhu.livejournal.com]

Come on, I thought the standard Hello World! for LINQ was the one line / one query ray tracer.

[andrewducker]

Which, you'll notice, purely calculates in the LINQ - the pixels are _set_ by a foreach following it...

[robhu.livejournal.com]

Well, now it doesn't seem very impressive.

[andrewducker]

It's still a neat bit of code - to calculate all of the pixel values in 60 lines of select statement.

But yes - more of a "look what I can do!" than anything you'd want in real life.