Correct use of Lambda query

Question

Consider the following code:

       private static void WriteProcesses(StreamWriter sw, DateTime d) {
            sw.WriteLine("List of processes @ " + d.ToString());
            Process[] localAll = Process.GetProcesses().Where(o => o.ProcessName.ToLower() != "svchost");            
            if(localAll.Length > 0) {
                for(int i = 0; i < localAll.Length; i++) {                    
                    sw.WriteLine("      " + localAll[i].ProcessName);
                }
            }
        }

But i get a red squiggly line saying:

Cannot implicitly convert type System.Collections.Generic.IEnumerable' to 'System.Diagnostics.Process[]'. An explicit conversion exists (are you missing a cast?)

I tried changing the array to a List but didnt work.

FYI, the generally accepted spelling is "lambda" ("lamda" is a little closer to the original greek but not the accepted English spelling). — Ian Varley
– Ian Varley, Commented Jan 11, 2009 at 1:14

jason · Accepted Answer · 2009-01-11 01:22:10Z

5

Change

Process[] localAll = Process.GetProcesses().Where(o => o.ProcessName.ToLower() != "svchost");

to

Process[] localAll = Process.GetProcesses().Where(o => o.ProcessName.ToLower() != "svchost").ToArray();

Where is returning an IEnumerable<Process> on which you can call ToArray to convert to an array of type Process.

Alternatively, you can enumerate through the IEnumerable<Process> that Where returns.

var processes = Process.GetProcesses().Where(o => o.ProcessName.ToLower() != "svchost");
foreach (Process process in processes) {
    sw.WriteLine("      " + process.ProcessName);
}

edited Jan 11, 2009 at 1:22

answered Jan 11, 2009 at 1:17

jason

243k36 gold badges437 silver badges532 bronze badges

Sign up to request clarification or add additional context in comments.

Comments

tvanfosson · Accepted Answer · 2009-01-11 01:21:00Z

2

I think you'd be better off to just deal with it as an IEnumerable

 private static void WriteProcesses(StreamWriter sw, DateTime d) {
     sw.WriteLine("List of processes @ " + d.ToString());
     var localAll = Process.GetProcesses()
                           .Where(o => o.ProcessName.ToLower() != "svchost");            
     foreach(Process process in localAll) {                    
         sw.WriteLine("      " + process.ProcessName);
     }
 }

answered Jan 11, 2009 at 1:21

tvanfosson

534k102 gold badges703 silver badges801 bronze badges

Comments

Garry Shutler · Accepted Answer · 2009-01-11 01:25:36Z

I would rewrite your code like this:

       private static void WriteProcesses(StreamWriter sw, DateTime d) {
            sw.WriteLine("List of processes @ " + d.ToString());
            var localAll = Process.GetProcesses().Where(o => o.ProcessName.ToLower() != "svchost");            
            foreach(var local in localAll) {                    
                    sw.WriteLine("      " + local.ProcessName);
            }
        }

Your problem is coming from the fact that Where returns an IEnumerable which cannot be mapped to an array. But there is no need for you to use an array so I've taken the use of it out. The length check is just making the code less clear for me so I changed to a foreach as well.

John Sheehan · Accepted Answer · 2009-01-11 01:16:55Z

0

If you don't want to use ToArray(), change Process[] to var then hover over it to see what type is being returned from the Where statement. (Or just hover over the Where() method, which will tell you the same thing)

answered Jan 11, 2009 at 1:16

John Sheehan

78.3k30 gold badges162 silver badges194 bronze badges

Comments

Matt Fenelon · Accepted Answer · 2009-01-11 01:18:37Z

0

The .Where method returns an IEnumerable, where T is the type being filtered. For this example you would need to call .ToArray( ) after the Where to convert the collection to an array.

answered Jan 11, 2009 at 1:18

Matt Fenelon

1461 silver badge7 bronze badges

Collectives™ on Stack Overflow

Correct use of Lambda query

5 Answers 5

Comments

Comments

Comments

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

5 Answers 5

Comments

Comments

Comments

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related