What changes i need to apply in the following code?

Question

0.00/5 (No votes)

See more:

Hi,
I was just trying to check a particular program is currently running or not using the following code.But it always showing "Not running".What is the mistake ihave done?Kindly help me to rectify it.

C#

static void Main(string[] args)
{
    Process[] myProcesss;
    myProcesss = Process.GetProcesses();
    foreach (Process myProcess in myProcesss)
    {
        if (myProcess.ProcessName.ToUpper() == "C:\\MyWebServerroot\\ConsoleApplication7\\ConsoleApplication7\\bin\\Debug\\ConsoleApplication7.exe")
        {
            Console.WriteLine("Running");
        }
        else
        {
            Console.WriteLine("Not Running");
        }
    }
    Console.ReadLine();
}

Posted 18-Oct-11 0:54am

sreenathpktr

Add a Solution

4 solutions

Add a Solution

Add your solution here

Treat my content as plain text, not as HTML

Preview 0

…

Existing Members

Sign in to your account

...or Join us

Download, Vote, Comment, Publish.

Your Email
Password
Forgot your password?

Your Email
This email is in use. Do you need your password?
Optional Password

I have read and agree to the Terms of Service and Privacy Policy
Please subscribe me to the CodeProject newsletters

When answering a question please:

Read the question carefully.
Understand that English isn't everyone's first language so be lenient of bad spelling and grammar.
If a question is poorly phrased then either ask for clarification, ignore it, or edit the question and fix the problem. Insults are not welcome.
Don't tell someone to read the manual. Chances are they have and don't get it. Provide an answer or move on to the next question.

Let's work to help developers, not make them feel stupid.

This content, along with any associated source code and files, is licensed under The Code Project Open License (CPOL)

Nicholas Butler · Answer 1 · 2011-10-18T00:59:00

Solution 1

For a non-case-sensitive string comparison, use:

String.Equals( myProcess.ProcessName, "...", StringComparison.OrdinalIgnoreCase );

Posted 18-Oct-11 0:59am

Nicholas Butler

sabbi26 · Answer 2 · 2011-10-18T01:05:00

Solution 2

Hi,

Use This

C#

foreach (Process myPro in myProcesss)
    {
        if (myPro.ProcessName.Equals("ConsoleApplication7.exe")
        {
            Console.WriteLine("Running");
        }
        else
        {
            Console.WriteLine("Not Running");
        }
    }

Thanks...

Posted 18-Oct-11 1:05am

sabbi26

Wayne Gaylard · Answer 3 · 2011-10-18T01:10:00

Solution 3

First of all, you parse each processname to upper case and then compare it to a normal case string. That won't ever be true. Second ProcessName returns the name of the process only, without the extension, and without the directory info. Thus a programme that is called myProgramme.exe will return myProgram without the extension. Try something liek this:-

C#

Process[] myProcesss;
           myProcesss = Process.GetProcesses();
           bool isRunning = false;
           foreach (Process myProcess in myProcesss)
           {
               if (myProcess.ProcessName.Equals("myProgramme",StringComparison.InvariantCultureIgnoreCase ))
               {
                   isRunning = true;
               }
           }
           string message = isRunning ? "Running" : "Not Running";
           MessageBox.Show(message);

obviously substitute MessageBox for Console.WriteLine for a console app and myProgramme for the name of your programme.

Hope this helps

Posted 18-Oct-11 1:10am

Wayne Gaylard

Updated 18-Oct-11 1:41am

v3

Comments

sreenathpktr 18-Oct-11 7:22am

Thanks for the reply.I applied your code.But it shoes the "Running" only..

Wayne Gaylard 18-Oct-11 7:40am

I made an error in the if shorthand ( ? : ). I have updated my answer including a way to ensure there are no case errors.

sreenathpktr 18-Oct-11 7:51am

Thanks for the updation...But now also it always shows "Not Runnng"..

Wayne Gaylard 18-Oct-11 8:38am

Have you checked in task manager that the process is running ? and how it is spelt ? Did you substitute myProgramme for your process name ? I have checked the code I posted and it works properly, so I think the issue is with one of those I mentioned.

sreenathpktr 18-Oct-11 9:04am

Yes..i checked everything...this is the code i have applied.And still it's showing "Not Running"..
static void Main(string[] args)
{
Process[] myProcesss;
myProcesss = Process.GetProcesses();
bool isRunning = false;
foreach (Process myProcess in myProcesss)
{
if (myProcess.ProcessName.Equals("ConsoleApplication7", StringComparison.InvariantCultureIgnoreCase))
{
isRunning = true;
}
}
string message = isRunning ? "Running" : "Not Running";
Console.WriteLine(message);
Console.ReadLine();
}
Could you tell me what is the error?

Wayne Gaylard 18-Oct-11 9:19am

I think your application is not running. Try running this with explorer as the application(.Equals("explorer", StringComparison..etc) and see what you get.

sreenathpktr 18-Oct-11 9:25am

Thank you sir..i got it...i added "ConsoleApplication7.vshost"

Mohammad A Rahman · Answer 4 · 2011-10-19T19:46:00

It might help you,

C#

class Program
    {
        static void Main(string[] args)
        {
            string processToCheck = "???????.exe";
            if (ProcessCheck(processToCheck))
            {
                Console.WriteLine("Running");
            }
            else
            {
                Console.WriteLine("Not Running");
            }
        }

        private static bool ProcessCheck(string processToCheck)
        {
            return Process.GetProcesses().Where(process => process.ProcessName.Equals(processToCheck.Remove(processToCheck.IndexOf('.')), StringComparison.OrdinalIgnoreCase)).Count() > 0;
        }
    }

:)