C # Обновление TextBox из нескольких потоков - PullRequest
3 голосов
/ 29 марта 2012

Я собирал простую WinForm, которая порождала бы количество потоков для цикла от 0 до 10000 - цель этого - чтобы замедлить Windows, чтобы некоторые другие программы работали медленно.

В основном формаимеет текстовое поле, в которое я хочу записать индекс цикла из каждого потока.Все было хорошо для одного потока, но так как я ввел больше потоков, я, кажется, зависал, когда нажимал на кнопку «Стоп» - я не очень уверен, куда идти дальше.

Мой образецВероятно, не написано хорошо.Я хочу лучше понять многопоточность, взаимоблокировки и т. Д. В прошлом я немного баловался с BackgroundWorker, но большую часть последних 2+ лет занимался Java.

Form1.cs

public delegate void SetTextDelegate(string text);

public partial class Form1 : Form
{
    private Thread[] _slow;
    private object lockTextBox = new object();

    public Form1()
    {
        InitializeComponent();
    }

    #region Event Handlers

    private void ui_btnClose_Click(object sender, EventArgs e)
    {
        this.Close();
    }

    private void Form1_Load(object sender, EventArgs e)
    {

    }

    private void ui_btnStart_Click(object sender, EventArgs e)
    {
        if (_slow != null)
        {
            StopAllThreads();
        }

        _slow = new Thread[ (int) numNoOfTheads.Value ];
        for( int i = 0; i < numNoOfTheads.Value; i++)
        {
            _slow[i] = new Thread(ThreadRunLoop);
            _slow[i].Start();
        }
    }

    private void Form1_FormClosing(object sender, FormClosingEventArgs e)
    {
        if (_slow != null)
        {
            StopAllThreads();
        }
    }

    private void ui_btnStop_Click(object sender, EventArgs e)
    {
        if (_slow != null)
        {
            StopAllThreads();
        }
    }

    private void ui_btnClear_Click(object sender, EventArgs e)
    {
        this.textBox1.Clear();
    }

    #endregion

    protected void ThreadRunLoop()
    {
        try
        {
            for (int i = 0; i < 10000; i++)
            {
                UpdateText("Loop " + i + " for " + Thread.CurrentThread.ManagedThreadId);
            }
        }
        catch (ThreadInterruptedException ex)
        {
            Console.WriteLine("Thread has been interrupted.");
        }
    }

    private void UpdateText(string text)
    {
        //lock (lockTextBox)
        //{
            if (textBox1.InvokeRequired)
            {
                textBox1.Invoke(new SetTextDelegate(UpdateText), text);
            }
            else
            {
                textBox1.SuspendLayout();
                textBox1.Text = textBox1.Text + text + System.Environment.NewLine;
                textBox1.SelectionStart = textBox1.Text.Length;
                textBox1.ScrollToCaret();
                textBox1.ResumeLayout();

            }
        //}
    }

    private void StopAllThreads()
    {
        for (int i = 0; i < _slow.Length; i++)
        {
            if (_slow[i] != null)
            {
                _slow[i].Interrupt();
                _slow[i] = null;
            }
        }
        _slow = null;
    }
}

Form1.Designer.cs

partial class Form1
{
    /// <summary>
    /// Required designer variable.
    /// </summary>
    private System.ComponentModel.IContainer components = null;

    /// <summary>
    /// Clean up any resources being used.
    /// </summary>
    /// <param name="disposing">true if managed resources should be disposed; otherwise, false.</param>
    protected override void Dispose(bool disposing)
    {
        if (disposing && (components != null))
        {
            components.Dispose();
        }
        base.Dispose(disposing);
    }

    #region Windows Form Designer generated code

    /// <summary>
    /// Required method for Designer support - do not modify
    /// the contents of this method with the code editor.
    /// </summary>
    private void InitializeComponent()
    {
        this.ui_btnClose = new System.Windows.Forms.Button();
        this.ui_btnStart = new System.Windows.Forms.Button();
        this.ui_btnStop = new System.Windows.Forms.Button();
        this.textBox1 = new System.Windows.Forms.TextBox();
        this.ui_btnClear = new System.Windows.Forms.Button();
        this.numNoOfTheads = new System.Windows.Forms.NumericUpDown();
        this.label1 = new System.Windows.Forms.Label();
        ((System.ComponentModel.ISupportInitialize)(this.numNoOfTheads)).BeginInit();
        this.SuspendLayout();
        // 
        // ui_btnClose
        // 
        this.ui_btnClose.Location = new System.Drawing.Point(433, 268);
        this.ui_btnClose.Name = "ui_btnClose";
        this.ui_btnClose.Size = new System.Drawing.Size(75, 23);
        this.ui_btnClose.TabIndex = 0;
        this.ui_btnClose.Text = "Close";
        this.ui_btnClose.UseVisualStyleBackColor = true;
        this.ui_btnClose.Click += new System.EventHandler(this.ui_btnClose_Click);
        // 
        // ui_btnStart
        // 
        this.ui_btnStart.Location = new System.Drawing.Point(12, 12);
        this.ui_btnStart.Name = "ui_btnStart";
        this.ui_btnStart.Size = new System.Drawing.Size(75, 23);
        this.ui_btnStart.TabIndex = 1;
        this.ui_btnStart.Text = "Start";
        this.ui_btnStart.UseVisualStyleBackColor = true;
        this.ui_btnStart.Click += new System.EventHandler(this.ui_btnStart_Click);
        // 
        // ui_btnStop
        // 
        this.ui_btnStop.Location = new System.Drawing.Point(12, 41);
        this.ui_btnStop.Name = "ui_btnStop";
        this.ui_btnStop.Size = new System.Drawing.Size(75, 23);
        this.ui_btnStop.TabIndex = 2;
        this.ui_btnStop.Text = "Stop";
        this.ui_btnStop.UseVisualStyleBackColor = true;
        this.ui_btnStop.Click += new System.EventHandler(this.ui_btnStop_Click);
        // 
        // textBox1
        // 
        this.textBox1.Location = new System.Drawing.Point(93, 12);
        this.textBox1.Multiline = true;
        this.textBox1.Name = "textBox1";
        this.textBox1.ScrollBars = System.Windows.Forms.ScrollBars.Both;
        this.textBox1.Size = new System.Drawing.Size(415, 241);
        this.textBox1.TabIndex = 3;
        // 
        // ui_btnClear
        // 
        this.ui_btnClear.Location = new System.Drawing.Point(352, 268);
        this.ui_btnClear.Name = "ui_btnClear";
        this.ui_btnClear.Size = new System.Drawing.Size(75, 23);
        this.ui_btnClear.TabIndex = 4;
        this.ui_btnClear.Text = "Clear";
        this.ui_btnClear.UseVisualStyleBackColor = true;
        this.ui_btnClear.Click += new System.EventHandler(this.ui_btnClear_Click);
        // 
        // numNoOfTheads
        // 
        this.numNoOfTheads.Location = new System.Drawing.Point(12, 98);
        this.numNoOfTheads.Name = "numNoOfTheads";
        this.numNoOfTheads.Size = new System.Drawing.Size(74, 20);
        this.numNoOfTheads.TabIndex = 5;
        this.numNoOfTheads.Value = new decimal(new int[] {
        1,
        0,
        0,
        0});
        // 
        // label1
        // 
        this.label1.AutoSize = true;
        this.label1.Location = new System.Drawing.Point(9, 82);
        this.label1.Name = "label1";
        this.label1.Size = new System.Drawing.Size(83, 13);
        this.label1.TabIndex = 6;
        this.label1.Text = "No. Of Threads:";
        // 
        // Form1
        // 
        this.AutoScaleDimensions = new System.Drawing.SizeF(6F, 13F);
        this.AutoScaleMode = System.Windows.Forms.AutoScaleMode.Font;
        this.ClientSize = new System.Drawing.Size(520, 303);
        this.Controls.Add(this.label1);
        this.Controls.Add(this.numNoOfTheads);
        this.Controls.Add(this.ui_btnClear);
        this.Controls.Add(this.textBox1);
        this.Controls.Add(this.ui_btnStop);
        this.Controls.Add(this.ui_btnStart);
        this.Controls.Add(this.ui_btnClose);
        this.Name = "Form1";
        this.Text = "Slow My Machine";
        this.Load += new System.EventHandler(this.Form1_Load);
        this.FormClosing += new System.Windows.Forms.FormClosingEventHandler(this.Form1_FormClosing);
        ((System.ComponentModel.ISupportInitialize)(this.numNoOfTheads)).EndInit();
        this.ResumeLayout(false);
        this.PerformLayout();

    }

    #endregion

    private System.Windows.Forms.Button ui_btnClose;
    private System.Windows.Forms.Button ui_btnStart;
    private System.Windows.Forms.Button ui_btnStop;
    private System.Windows.Forms.TextBox textBox1;
    private System.Windows.Forms.Button ui_btnClear;
    private System.Windows.Forms.NumericUpDown numNoOfTheads;
    private System.Windows.Forms.Label label1;
}

Обновление Если переместить замок в другое вметод UpdateText и добавьте Thread.Sleep (20);в цикле мой графический интерфейс более отзывчив, и я могу нажать на кнопку «Стоп» и переместить форму.

Любые отзывы, исправления будут оценены.

Ответы [ 4 ]

3 голосов
/ 29 марта 2012

lock внутри UpdateText вызовет тупик. Рабочий поток получает блокировку и затем вызывает Invoke. Затем поток пользовательского интерфейса вызывает попытку получить такую ​​же блокировку , но должен ждать, пока она не будет снята. Дело в том, что блокировка никогда не будет снята, потому что Invoke блокируется, пока поток пользовательского интерфейса не завершит выполнение делегата. Этого никогда не происходит, потому что поток пользовательского интерфейса все еще ожидает получения блокировки. Тупик!

1 голос
/ 29 марта 2012

Измените цикл for на

   for (int i = 0; i < 10000; i++)
    {
       var text = "Loop " + i + " for " + Thread.CurrentThread.ManagedThreadId;
       if (textBox1.InvokeRequired)
          textBox1.Invoke(new SetTextDelegate(UpdateText), text);
       else
          UpdateText(text);
    }

И измените текст обновления на

    private void UpdateText(string text)
    {
       textBox1.SuspendLayout();
       textBox1.Text = textBox1.Text + text + System.Environment.NewLine;
       textBox1.SelectionStart = textBox1.Text.Length;
       textBox1.ScrollToCaret();
       textBox1.ResumeLayout();
     }

РЕДАКТИРОВАТЬ : Моя ошибка.Это только улучшит организацию, а не в каком-либо аспекте.Если вы хотите обновлять пользовательский интерфейс так часто, вы должны использовать BackgroundWorker , что сказал rdkleine.

0 голосов
/ 29 марта 2012

Используйте BackgroundWorker, который обновляет поток пользовательского интерфейса.

Вот хороший пример использования BackgroundWorker:

http://msdn.microsoft.com/en-us/library/cc221403(v=vs.95).aspx

Это не объясняет, как извлечь данные (ваше значение типа int) и поместить их в текстовое поле, но это хорошее начало.

0 голосов
/ 29 марта 2012

Попробуйте переместить блокировку в UpdateText внутрь else.

...