Question

我有以下代码正在运行，但它非常脏。实际上代码很好，除了我添加的部分：暂停和停止按钮。我是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();
    }

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

Answer 1

在问题中提到评论是关于代码实践变更的建议。

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

我还想补充一下：

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

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

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

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

暂停和停止线程

1 个答案: