无法修改单独线程

时间:2015-06-20 08:26:03

标签: c# multithreading winforms text file-io

所以我制作了一个C#应用程序,它必须不断读取和显示文本文件的内容,同时允许用户在文本框中输入内容并将其附加到该文件的末尾。

我通过在单独的线程上运行read方法来实现这一点,但是更改存储显示文本文件内容的变量是导致问题的原因。最初我尝试了一种方法来做到这一点,但是它没有工作,并给出了一个跨线程操作 - 无效的'错误。然后我尝试应用我在MSDN上找到的一些代码,但现在在线程结束后更新变量之后!

请帮忙。

partial class MainForm
{
    delegate void SetTextCallback(string text);
    public static string msg;
    public static string name;
    public void InitClient()
    {
        name = "public.txt";
        Console.WriteLine(name);
        if(!File.Exists(name))
        {
            File.Create(name);
            File.AppendAllText(name, "Welcome to " + name);
        }
        Thread Read = new Thread(new ThreadStart(this.Client));
        Read.Start();
        while(!Read.IsAlive);
    }
    public void WriteText()
    {
        File.AppendAllText(name, this.InputBox.Text);
        this.InputBox.Clear();
    }
    private void SetText(string text)
    {
        if (this.OutPut.InvokeRequired)
        {    
            SetTextCallback d = new SetTextCallback(SetText);
            this.Invoke(d, new object[] { text });
        }
        else
        {
            this.OutPut.Text = text;
        }
    }
    public void Client()
    {
        msg = File.ReadAllText(name);
        Console.WriteLine(msg);
        Thread.Sleep(300);
        this.SetText(msg);
    }
}

为什么线程表现得像这样。如何修改我的代码,以便输出框的内容始终等于文本文件的内容。

欢迎任何建议。

2 个答案:

答案 0 :(得分:1)

你在这里遇到了多个问题,

  • 使用File可能不是线程安全的。
  • 您的方法不会重复
  • 你的Sleep()正在线程

你可以通过抛弃线程并使用一个简单的Timer来解决所有这些问题。

答案 1 :(得分:1)

尝试使用后台工作程序而不是创建新线程。后台工作程序将在单独的线程中运行其内容,并允许您在其工作时报告“进度”。此进度报告将始终在UI线程(或启动后台工作程序的线程)上运行。

它还有一个事件,在后台工作程序完成时调用。这也在UI线程上运行。

这个例子可以帮助你入门。

更新:根据建议添加了一些非常基本的错误处理

这个想法是使用ReportProgress的UserData(第二个参数)在需要时在UI线程上进行更新。在这种情况下,它是一个字符串,但这可以是任何对象。

此外,您可以使用DoWorkEventArgs的结果从后台工作中生成最终结果。在这种情况下,我返回抛出的任何异常,否则返回null,但您也可以返回任何您想要的内容。

正如Henk在评论中提到的那样,处理DoWork回调中发生的错误非常重要,因为这里发生的异常等将被吞下,工作人员将完成,好像没有发生任何不良事件。

private BackgroundWorker _backgroundWorker;

public Form1()
{
    InitializeComponent();

    _backgroundWorker = new BackgroundWorker();
    _backgroundWorker.WorkerReportsProgress = true;
    _backgroundWorker.WorkerSupportsCancellation = true;
    // This is the background thread
    _backgroundWorker.DoWork += BackgroundWorkerOnDoWork;

    // Called when you report progress
    _backgroundWorker.ProgressChanged += BackgroundWorkerOnProgressChanged;

    // Called when the worker is done
    _backgroundWorker.RunWorkerCompleted += BackgroundWorkerOnRunWorkerCompleted;
}

private void BackgroundWorkerOnRunWorkerCompleted(object sender, RunWorkerCompletedEventArgs runWorkerCompletedEventArgs)
{
    if (runWorkerCompletedEventArgs.Result != null)
    {
        // Handle error or throw it
        throw runWorkerCompletedEventArgs.Result as Exception;
    }

    textBox1.Text = "Worker completed";
}

private void BackgroundWorkerOnProgressChanged(object sender, ProgressChangedEventArgs progressChangedEventArgs)
{
    textBox1.Text = progressChangedEventArgs.UserState as string;
}

private void BackgroundWorkerOnDoWork(object sender, DoWorkEventArgs doWorkEventArgs)
{
    try
    {
        for (int i = 0; i < 100 && !_backgroundWorker.CancellationPending; i++)
        {
            _backgroundWorker.ReportProgress(0, i + " cycles");
            Thread.Sleep(100);
        }
    }
    catch (Exception ex)
    {
        doWorkEventArgs.Result = ex;
    }

}

private void startButton_Click(object sender, EventArgs e)
{
    if (!_backgroundWorker.IsBusy)
        _backgroundWorker.RunWorkerAsync();
}

private void cancelButton_Click(object sender, EventArgs e)
{
    if(_backgroundWorker.IsBusy)
        _backgroundWorker.CancelAsync();
}