Question

我在程序中想要完成的是知道某些进程是否正在运行（我需要了解所有正在运行的实例）。我想把它们放在一个组合框中，作为一个对象存储，所以我可以稍后再把它们抛回去。我觉得这很容易但事实证明，这让我有些头痛：P 我不确定这是怎么做的，但是它有效。但是，我对这个代码解决方案感到很难过。我不知道有什么好的编程模式，这就是为什么我问你的编码员，帮助我。

我想到的第一件事是使用计时器来频繁检查进程并添加它们，并使用Exited事件将它们从我的组合框中删除。所以这是我在计时器的Tick事件上的代码：

    private void timer_ProcessCheck_Tick(object sender, EventArgs e)
    {
        Process[] tmpArray = Wow_getCurrentlyRunning(); // this returns Process[]
        if (comboBox_processes.Items.Count == 0)
        {
            if (tmpArray.Count() > 0) 
                for (int Index = 0; Index < tmpArray.Count(); Index++)
                    Add(tmpArray[Index]); // adding to combobox
        }
        else
        { 
            if (tmpArray.Count() > comboBox_processes.Items.Count)
            {
                List<Process> result;
        /*Diff compares the two array, and returns to result variable.*/
                if (Diff(tmpArray, comboBox_processes, out result))                 
                    foreach(Process proc in result)
                        Add(proc); // adding to combobox
            }
        }
    }

我的Diff方法看起来像这样，这将差异变为diff变量。

    public bool Wow_differsFrom(Process[] current, ComboBox local, out List<Process> diff)
    {
        List<int> diffIndex = new List<int>();

        foreach (Process proc in current)
            diffIndex.Add(proc.Id);

        for (byte Índex = 0; Índex < current.Count(); Índex++)
        {
            for (byte Index = 0; Index < local.Items.Count; Index++)
            {
                if (current[Índex].Id == (local.Items[Index] as Process).Id)
                {
                    diffIndex.Remove(current[Índex].Id);
                    break;
                }
            }
        }

        diff = new List<Process>();

        for (int x = 0; x < current.Count(); x++)
            for (int i = 0; i < diffIndex.Count; i++)
                if (current[x].Id == diffIndex[i])
                    diff.Add(current[x]);

        if (diff.Count == 0)
            return false;
        return true;
    }

这是在进程退出时调用的Exited事件处理程序

    private void Wow_exitedEvent(object o, EventArgs e)
    {
        RemoveCBItem(comboBox_processes, (o as Process).Id); // this will remove the process from combobox, also threadsafe.
    }

我的问题：

你会怎么做？我接近这个吗？我有一种感觉，我不喜欢。
申请开始有什么事吗？就像有一个出口。也许在Win32 API中很深？

Answer 1

总的来说，我认为这个想法是正确的 - 如果你需要每次刷新活动进程列表。所以使用计时器更新列表是可以的。我不太了解Win32 API，但我认为如果任何人都可以订阅process_run和process_retminate wineows事件，这将是一个安全问题，因此不太可能。

但你真的需要一直更新吗？也许只有当组合框扩展时才能阅读进程列表？当用户下次将其展开时，您将再次重新启动项目。我认为这种方法会减少问题。

作为您的实施，我认为它不是最有效和优雅的：

将整个Process对象存储在组合框项目中对我来说并不好。更好地创建您的类，它将只存储您需要的那些属性（进程ID，进程名称）
在循环中使用current.Count（）是极其低效的 - 它是一种在调用时始终迭代IEnumerable的扩展方法。所以你的

for（byteÍndex= 0;Índex＆lt; current.Count（）;Índex++）

导致O（N * N）复杂性。幸运的是，进程数量不会太大而不会影响您的应用程序，但您应该知道这一事实，并且不习惯在循环中使用此方法。使用current.Length，因为它是一个数组。
您的收藏同步过于复杂和奇怪。为什么不使用添加 - 删除操作来创建一个接收集合以更改和集合到init的方法并使第一个集合等于第二个集合？你们将按照某些属性（例如进程名称）对这两个集合进行排序，这可以非常简单有效地完成 - 使用binary search。在WPF中，您可以使用ObservableCollection作为dataSource来最有效地使用此方法。在WinForms中，您可能也可以使用带有更改通知的集合，但我还没有使用它们。

你可以做得更简单：

//Somewhere in Form_Load
combobox.DisplayMember = "Name";//name of the property in your MyProcessInfo class
combobox.ValueMember = "Id";//name of the property in your MyProcessInfo class

//In your timer.Tick handler
combobox.DataSource = Wow_getCurrentlyRunning().Select(p=>new MyProcessInfo(p.Id, p.Name)).ToList();

但如果可以使用组合框和一些闪烁，这种方法将始终重新启动所有项目。

流程操作（监控）

1 个答案: