如何使用线程完成,然后重用线程?

时间:2013-05-21 00:03:28

标签: c# .net multithreading winforms wmi

我正在构建一个表单,我正在尝试使用线程,以便从WMI查询中获取一些结果,以便在文本框中显示,而不会让表单冻结在用户身上。但是,当我使用下面的代码并在调试时使用Break-All时,代码就位于getPrinterThread.Join()。我知道我一定错过了什么。

我的目标是获取一个线程来运行ObtainPrinterPort方法来完成,然后获得一个线程来运行InstallPrinterPort方法来完成。我将下面的代码作为另一种方法中的内联代码。代码不是在一个单独的类或任何东西,我没有后台工作者,因为我见过的所有例子,直到现在,只是让我感到困惑。

这是我公认的糟糕线程尝试:

Thread printThread = new Thread(ObtainPrinterPort);
printThread.Start();
while (!printThread.IsAlive) ;
Thread.Sleep(1);
printThread.Join();         // Form sits and does nothing; Break-all reveals this line as statement being executed.

Thread installThread = new Thread(InstallPrinterPort);
installThread.Start();
while (!installThread.IsAlive);
Thread.Sleep(1);
installThread.Join();

是否有一种简单的方法可以使某些工作变得安全并允许我在文本框中向用户显示方法中显示的结果?希望有一种方法可以让我继续使用我在表单类中编写的实例变量/方法/代码...否则,如果我'我将不得不重写大量代码我将实现一个"DoWork"-type example(从DoWork方法/构造函数或Worker类调用我的方法)。

请记住,我的方法需要将文本从线程返回到文本框以向用户显示结果。我有代码,我假设将允许我返回文本线程如果有效,但我只是想确保任何建议/帮助记住这一点。我正在使用的代码如下:

public void AppendTextBox(string value)
{
    if (InvokeRequired)
    {
        this.Invoke(new Action<string>(AppendTextBox), new object[] { value });
        return;
    }

    txtResults.Text += value;
}

对于它的价值,这是我的ObtainPrinterPort方法和随之而来的CreateNewConnection方法...... InstallPrinterPort方法非常相似,所以发布它不会真正揭示​​多少:

private ManagementScope CreateNewConnection(string server, string userID, string password)
{
    string serverString = @"\\" + server + @"\root\cimv2";
    ManagementScope scope = new ManagementScope(serverString);

    try
    {
        ConnectionOptions options = new ConnectionOptions
        {
            Username = userID,
            Password = password,
            Impersonation = ImpersonationLevel.Impersonate,
            EnablePrivileges = true

        };
        scope.Options = options;
        scope.Connect();
    }
    catch (ManagementException err)
    {
        MessageBox.Show("An error occurred while querying for WMI data: " +
                        err.Message);
    }
    catch (System.UnauthorizedAccessException unauthorizedErr)
    {
        MessageBox.Show("Connection error (user name or password might be incorrect): " +                              unauthorizedErr.Message);
    }

    return scope;
}

private void ObtainPrinterPort()
{
    string computerName = "";
    string userID = "";
    string password = "";
    string printerQuery = "SELECT * FROM Win32_Printer WHERE Name = ";
    string portQuery = "SELECT * FROM Win32_TCPIPPrinterPort WHERE Name = ";
    string search = "";
    SelectQuery query;

    foreach (var s in lstServer)
    {
        computerName = s.ServerName;
        userID = s.UserID;
        password = s.Password;
    }

    ManagementScope scope = CreateNewConnection(computerName, userID, password);

    foreach (Printers p in lstPrinters)
    {
        AppendTextBox("Obtaining printer/port info for " + p.PrinterName + "\r\n");

        search = printerQuery + "'" + p.PrinterName + "'";

        query = new SelectQuery(search);

        try
        {
            using (var searcher = new ManagementObjectSearcher(scope, query))
            {
                ManagementObjectCollection printers = searcher.Get();

                if (printers.Count > 0)
                {
                    AppendTextBox("\tStoring printer properties for " + p.PrinterName + "\r\n");

                    foreach (ManagementObject mo in printers)
                    {
                        StorePrinterProperties(p, mo);
                    }
                }
                else
                {
                    lstPrinterExceptions.Add("Printer: " + p.PrinterName);

                    AppendTextBox("\t**Printer " + p.PrinterName + " not found**\r\n");
                }
            }
        }
        catch (Exception exception)
        {
            MessageBox.Show("Error: " + exception.Message, "Error",
                            MessageBoxButtons.OK, MessageBoxIcon.Exclamation);
        }

        if (!lstPrinterExceptions.Contains("Printer: " + p.PrinterName)
           && !lstPrinterExceptions.Contains("Port: " + p.PortName))
        {
            search = portQuery + "'" + p.PortName + "'";

            query = new SelectQuery(search);

            try
            {
                using (var searcher = new ManagementObjectSearcher(scope, query))
                {
                    ManagementObjectCollection ports = searcher.Get();

                    if (ports.Count > 0)
                    {
                        AppendTextBox("\tStoring port properties for " + p.PortName + " (" + p.PrinterName + ")\r\n");

                        foreach (ManagementObject mo in ports)
                        {
                            StorePortProperties(p, mo);
                        }
                    }
                    else
                    {
                        lstPrinterExceptions.Add("Port: " + p.PortName);

                        AppendTextBox("\t**Port " + p.PortName + " for " + p.PrinterName + " not found**\r\n");
                    }
                }
            }
            catch (Exception exception)
            {
                MessageBox.Show("Error: " + exception.Message, "Error",
                                MessageBoxButtons.OK, MessageBoxIcon.Exclamation);
            }

            AppendTextBox("\tSuccessfully obtained printer/port info for " + p.PrinterName + "\r\n");
        }
    }
}

感谢。

1 个答案:

答案 0 :(得分:0)

您的问题是Invoke调用阻塞等待主线程DoEvents,因此可以处理它们,但主线程在Thread.Join中阻塞。你有一个僵局。

这是在不阻塞UI线程的情况下运行线程的方法。 Thread.Join阻塞,直到另一个线程完成,所以在这里我只阻塞最多100ms,然后调用DoEvents以便表单可以响应消息(包括处理来自其他线程的Invoke调用),然后循环直到后台线程完成。

Thread printThread = new Thread(ObtainPrinterPort);
printThread.Start();
while (printThread.IsAlive) {
    Application.DoEvents();        
    printThread.Join(100);        
}

在这样的循环中调用DoEvents有点hacky,但它会起作用。

您还可以查看BackgroundWorker,这会让整件事变得更加安全和轻松。

一个简单的例子:

var bw = new BackgroundWorker();            
bw.DoWork += (worker, args) => {
    ObtainPrinterPort();
};
bw.RunWorkerAsync();