5

我有这个基本代码,它将检查每分钟运行的记事本。

namespace Watcher
{
    class Program
    {
        static void Main(string[] args)
        {
            for (int i = 0; ; i--)
            {
                foreach (Process clsProcess in Process.GetProcesses())
                {
                    if (clsProcess.ProcessName.Contains("notepad"))
                    {
                        Console.WriteLine("True");
                    }
                    Console.WriteLine("NFalse");
                }
                Thread.Sleep(10000);
            }
        }
    }
}

问题是它为每个正在运行的进程返回“NFalse”(例如,它将打印 100 个)。我怎样才能只打印一次以表明进程没有运行?

4

4 回答 4

7

重构你的代码。

你在一种方法上做得太多了。将检查记事本是否正在运行的代码放入单独的方法中:

static bool CheckIfProcessIsRunning(string nameSubstring)
{
    foreach (Process clsProcess in Process.GetProcesses())
    {
        if (clsProcess.ProcessName.Contains(nameSubstring))
        {
            return true;
        }
    }
    return false;
}

您可以使用 LINQ 进一步简化此操作:

static bool CheckIfProcessIsRunning(string nameSubstring)
{
    return Process.GetProcesses().Any(p => p.ProcessName.Contains(nameSubstring));
}

一旦你编写了这个方法,剩下的就是调用它并根据它返回的是 true 还是 false 来打印正确的消息。

while (true)
{
    string message = CheckIfProcessIsRunning("notepad") ? "True" : "NFalse";
    Console.WriteLine(message);
    Thread.Sleep(10000);
}

现在,您有两个非常简单的方法,而不是一个具有复杂逻辑的长方法。

于 2012-08-23T21:24:12.837 回答
2

您只需要检查您感兴趣的进程。不要费心循环所有正在运行的进程。使用Process.GetProcessByName()

for (int i = 0; ; i--)
{
     Process[] processes = Process.GetProcessByName("notepad++"); // Without extension
     if(processes.Length > 0){
          Console.WriteLine("True");
     }
     else{
          Console.WriteLine("False");
     }

     Thread.Sleep(10000);
}
于 2012-08-23T21:24:47.223 回答
0

好吧,这似乎运作良好。

Dim x = Process.GetProcesses().ToList().FirstOrDefault(Function(p) p.ProcessName.Contains("Notepad"))
if x Is Nothing then
    Console.WriteLine("false")
end if
于 2012-08-23T21:33:29.380 回答
0

只需更改为这个,所以你只打印一次。

var b = false;
foreach (Process clsProcess in Process.GetProcesses())
            {
                if (clsProcess.ProcessName.Contains("notepad"))
                {
                    if (!b) b = true;
                }
            }
                Console.WriteLine(b);
于 2012-08-23T21:24:50.013 回答