check if another app is running then do task(request & fix code)

Question

I wrote this code:

Process Processs = Process.GetProcessesByName("ProcessName")[0];
        
do
{
    await Task.Delay(1500);
}
while (!(Processs.MainWindowHandle != IntPtr.Zero));

// stuff to do here
Processs = (Process)null;

But I get an error

index was outside the bounds of the array

Can anyone help me fix it? Or write new, better code?

it seems that `Process.GetProcessesByName("ProcessName")` is returning 0 processes, i.e. the process you are looking for is not running. By the way, avoid to name a local variable as a class: `Process process` or `Process proc` or something similar is going to be much more readable — Gian Paolo, May 25 '22 at 04:53
Also, no need for this `Processs = (Process)null;` (and no need for the cast as well, you can assign `null` without casting) — Gian Paolo, May 25 '22 at 04:55
Well, `Process.GetProcessesByName("ProcessName")` returns a `Process[]` array, so you could check if its length is > 0. — DiplomacyNotWar, May 25 '22 at 05:26
Does this answer your question? [What is an IndexOutOfRangeException / ArgumentOutOfRangeException and how do I fix it?](https://stackoverflow.com/questions/20940979/what-is-an-indexoutofrangeexception-argumentoutofrangeexception-and-how-do-i-f) — knittl, May 25 '22 at 05:42
I got 'Process' is a type, which is not valid in the given context — esus, May 25 '22 at 06:54

score 0 · Answer 1 · edited May 26 '22 at 14:54

0

Process.GetProcessesByName is returning array type object. So, we can check length of the process

Process[] processes = Process.GetProcessesByName("ProcessName");
    if(processes.length >0){
        // task is running 
    
    }
    else {
       // task is not running 
    
    }

edited May 26 '22 at 14:54

knittl

216,605
51
293
340

answered May 25 '22 at 05:39

Ajay Gupta

319
2
8

I need System.Diagnostics.Process[] not System.Diagnostics.Process – esus May 25 '22 at 05:56

check if another app is running then do task(request & fix code)

1 Answers1