0

我有以下代码正在运行，但它很脏。实际上代码很好，除了我添加的部分：暂停和停止按钮。我是 c# 的新手，所以任何帮助都会受到赞赏。

    private void pause_button_Click(object sender, EventArgs e)
    {
        start = false; pause = true; stop = false;
        guiUpdate();
        PauseEvent.Reset();
    }

    private void stop_button_Click(object sender, EventArgs e)
    {
        if (pause == true)
        {
            PauseEvent.Set();
            pause = false;
            this.start_button.Click -= new System.EventHandler(this.resume_button_Click);
        }
        start = false; stop = true;
    }

    private int activeThreads = 0;
    private Thread thread;
    private void DoWork(object sender)
    {
        string line = null;
        ereader = new StreamReader(MY_LIST);
        do
        {
            lock (ereader)
            {
                PauseEvent.WaitOne();
                line = ereader.ReadLine();
            }

            //
            //other commands for processing & building the argument
            //

            lock (signal)
            {
                ++activeThreads;
            }

            thread = new Thread(new ParameterizedThreadStart(
                o =>
                {
                    processit((object)o);
                    lock (signal)
                    {
                        --activeThreads;
                        Monitor.Pulse(signal);
                    }
                }));
            thread.Start(argument);

            lock (signal)
            {
                while (activeThreads > maxthreads)
                    Monitor.Wait(signal);
            }

            lock (signal)
            {
                if (!start)
                {
                    showwaiting(true);//shows an animated gif with a "please wait" msg
                    while (activeThreads > 0)
                        Monitor.Wait(signal);
                    showwaiting(false);
                    if (stop == true)
                    {
                        this._BackgroundWorker.CancelAsync();
                        break;
                    }
                }
            }
        }
        while (ereader.Peek() != -1);
        showwaiting(true);
        lock (signal)
        {
            while (activeThreads > 0)
                Monitor.Wait(signal);
        }
        showwaiting(false);
    }


    private void _BackgroundWorker_RunWorkerCompleted(object sender, RunWorkerCompletedEventArgs e)
    {
        start = false; stop = true;
        guiUpdate();
    }

我究竟能做些什么来避免循环内外的重复命令？

4

1 回答 1

1

问题评论中提到的是有关代码实践更改的建议。

  • 在条件中检查布尔变量时不要使用== trueor 。== false（杰夫）
  • 使用枚举来表示状态而不是多个布尔变量。

我还要补充：

  • 使用正确的 C# 命名约定。对于您的方法，如果有多个单词，请将所有单词大写。（例如，而不是showwaiting使用ShowWaiting）。

对于您的小循环，您实际上并不需要优化它们，尽管不仅仅是一个简单的循环。因此，您可以将它们与所有相同的代码一起重构为一个单独的方法：

private void ShowAndWait()
{
    showwaiting(true);
    lock (signal)
    {
        while (activeThreads > 0)
            Monitor.Wait(signal);
    }
    showwaiting(false);
}

请注意，同一线程可以锁定它之前获得的监视器。这是一个快速操作，因为线程已经拥有锁。然后，您可以从这两个地方调用此方法。

于 2012-06-09T00:53:29.980 回答