1

我需要同时计算文本框中元素的总和和元素的数量。所以我决定创建两个线程——一个用于数字的长度，一个用于元素的总和。但是当我只启动一个线程时 - 它工作正常。但是当我启动第二个线程时 - 表单开始工作缓慢或根本停止工作。我创建了两个线程

thrd = new Thread(GetLength);
thrd.Start();
thrd1 = new Thread(SetSum);
thrd1.Start();

这些是线程的函数，用于计算文本框中数字的长度和计算其元素的总和。

private void SetSum()
{
    while (true)
    {
       if (this.label3.InvokeRequired)
           this.Invoke(new Action(() => label3.Text = this.GetSum().ToString()));
    }
}

private int GetSum()
{
    string n = textBox1.Text;
    int sum = 0;
    for (int i = 0; i < n.Length; i++)
     {
            try
            {
                sum += int.Parse(n[i].ToString());
            }
            catch (FormatException) { };
        }
        return sum;
     }

private void GetLength()
{
    while (true)
     {
            if (this.label2.InvokeRequired)
                this.Invoke(new Action(() => label2.Text = " |  Length = " + textBox1.Text.Length.ToString()));
     }
}

哪里有问题？同步？

我找到了一个解决方案 - 我Thread.Sleep(1)在 GetLength 方法中添加了 while 循环

4

3 回答 3

2

这里有几个问题。

  1. 手头的任务对于（完整的）线程来说太小了。创建线程的成本很高。
  2. 通过调用主操作，所有工作都在主线程上完成。你的解决方案毕竟不是多线程的。
  3. 计数很容易作为 Summing 的副产品完成（反之亦然），因此 2 个线程/任务是多余的。
  4. 循环会拖累你的while(true) ...进程，消耗过多的 CPU 时间

这里的简单答案是不使用任何线程，只需在textBox1.TextChanged.

于 2012-09-17T18:10:58.947 回答
1

是的，问题实际上是同步：太多了。

您正在生成仅执行Invokes 的线程，这意味着 UI 线程正在执行所有工作。

于 2012-09-17T18:10:51.957 回答
1

这部分代码是一个无限循环，没有任何 Thread.Sleep 或任何其他等待。这将使 CPU 达到 100%。您应该将其与某些事件或任何其他将触发 GetLength 的活动联系起来

private void GetLength() 
{ 
    while (true) 
     { 
            if (this.label2.InvokeRequired) 
                this.Invoke(new Action(() => label2.Text = " |  Length = " + textBox1.Text.Length.ToString())); 
     } 
}
于 2012-09-17T18:12:13.093 回答