C# do while循环开始停止

Question

我试图为此循环制作一个停止按钮，但它无限期运行，当我单击按钮 2 时没有任何反应，

bool dowhile = false;
private void button1_Click(object sender, EventArgs e)
{
    do
    {
        for (int i = listbox1.Items.Count - 1; i >= 0; i--)
        {
            string textstring = listbox1.Items[i].ToString();
            richTextBox1.AppendText("" + textstring + ": Done\n");
            Thread.Sleep(1000);
        }
    } while (!dowhile);
}
private void button2_Click(object sender, EventArgs e)
{
    this.dowhile = true;
}

我哪里出错了？

抱歉，“lvlchanger”拼写错误，代码现在没问题，没有遗漏

我也在寻找一个不太长的修复程序:))

Answer 1

在您的 button1_Click 完成之前，系统无法处理消息队列中的任何内容（即按钮点击、重绘等） - 但它永远不会。这正是导致所有“{blah} 未响应”消息的原因 - 代码未及时响应消息队列。

基本上，不要这样做。一个hacky修复方法是一些DoEvents()，但是不行！正确地做；基本上，从 button2_Click 处理事件。也许您应该通过计时器滴答运行刷新？

Answer 2

Thread.Sleep 几乎总是错误的方法；当您发现自己想要在代码中使用类似的东西或 Application.DoEvents 时，是时候退一步思考一下您真正在做什么以及为什么它不起作用。您的未来应该有一个新的设计。

这里真正的问题是您正在 UI 线程上工作，这会阻止用户与您的应用程序交互。在循环完成处理之后，您对 Button2 的点击将被识别并处理，此时它绝对不会执行任何操作。

这是因为线程一次只能完成一项任务。事实上，Thread.Sleep(100) 只会让情况变得更糟，因为它强制线程旋转并且在 100 毫秒内不执行任何操作。完成循环还需要 100 毫秒，而且绝对没有任何增益。

此（常见）问题的正确解决方案是将循环旋转到单独的线程上，使用类似 BackgroundWorker 类。 MSDN 条目有一个很好的使用示例，包括您的具体用例：允许用户取消长时间运行的后台任务。

Answer 3

private void button1_Click(object sender, EventArgs e)
{
    bool dowhile = false;private void button1_Click(object sender, EventArgs e){
    do 
    {
        for (int i = listbox1.Items.Count - 1; i >= 0; i--)
        {
            string textstring =listbox1.Items[i].ToString();
            richTextBox1.AppendText("" + textstring + ":` `Done\n");
            Thread.Sleep(1000);
        }
    } while (!lvlchanger && dowhile == false);
}

private void button2_Click(object sender, EventArgs e)
{
    this.dowhile = true;
}

Answer 4

将 Application.DoEvents() 添加到循环中以允许应用程序处理来自其他源的事件。

/编辑
这应该可行，但是......

“这几乎从来都不是正确的答案”——科迪·格雷