执行在多个线程上访问全局数组的函数

时间:2013-05-14 08:13:26

标签: c# multithreading .net-4.0

我有一个功能' graph1'调用另一个函数,如下所示。当我打电话

graph1threader(copy, date);

我得到了我想要的答案,但这需要30秒,所以我尝试使用多线程。但是当我使用

thread[copy] = new Thread(() => graph1threader(copy, date));//pass date,copy
thread[copy].Start();

我没有得到结果,即全局变量只保持0.为什么会这样?我怎么能纠正这个?

void graph1()
    {

        chart1.ChartAreas[0].AxisX.MajorGrid.Enabled = false;
        chart1.ChartAreas[0].AxisY.MajorGrid.Enabled = false;

        for (int k = 0; k < 5; k++)
        {                
            int copy = k;
            DateTime date = G.AddDays(copy);
            refValues[copy] = 0;
            threshValues[copy] = 0;
            y_Values[copy] = 0;
            y__Values[copy] = 0;
            yValues[copy] = 0;

            //task[copy] = new Task(() => graph1threader(copy, date));//pass date,copy
            //ask[copy].Start();
            graph1threader(copy, date);

        }



            for (int j = 0; j < 5; j++)
            {
                DateTime temp = G.AddDays(j);
                string temper = temp.ToShortDateString();
                y__Values[j] = y__Values[j] - y_Values[j];
                xNames[j] = temper;
            }

        chart1.Series[1].Points.DataBindXY(xNames, y_Values);

        chart1.Series[2].Points.DataBindXY(xNames, y__Values);

        chart1.Series[3].Points.DataBindXY(xNames, refValues);

        chart1.Series[4].Points.DataBindXY(xNames, threshValues);

        chart1.Series[5].Points.DataBindXY(xNames, yValues);

    }

void graph1threader(int x, DateTime date)
    {
        DBConnect A = new DBConnect();
        List<string>[] list = new List<string>[4];
        list = A.mandetselect();
        int numberofmen = A.Countmandet();
        string[] man_name = list[0].ToArray();
        string[] trade = list[1].ToArray();
        string[] license = list[2].ToArray();
        string[] training = list[3].ToArray();
        string[] display_status = list[4].ToArray();
        //string abc;
        List<string>[] lista = new List<string>[5];
        List<string>[] listc = new List<string>[14];


        for (int j = 0; j < numberofmen; j++)
        {
            int flag = 0;
            if (!display_status[j].Equals("NO") && (selection == "ALL" || (selection == "LAE" && license[j] != "") || (selection == "NON LAE" && license[j] == "") || (selection == "ALL AVIONICS" && trade[j] == "Avionics") || (selection == "NON LAE AVIONICS" && trade[j] == "Avionics" && license[j] == "") || (selection == "LAE AVIONICS" && trade[j] == "Avionics" && license[j] != "") || (selection == "ALL AIRFRAMES" && trade[j] == "Airframes") || (selection == "NON LAE AIRFRAMES" && trade[j] == "Airframes" && license[j] == "") || (selection == "LAE AIRFRAMES" && trade[j] == "Airframes" && license[j] != "")))
            {
                refValues[x]++;
                threshValues[x] = 0.8 * refValues[x];
                string abc = man_name[j].Replace(" ", "_");
                int no_of_proj = A.Countproj(abc);//required
                lista = A.manprojselect(abc);//required
                string[] projname = lista[0].ToArray();
                string[] country = lista[2].ToArray();
                string[] startofproj = lista[3].ToArray();
                string[] endofproj = lista[4].ToArray();
                string Status = "";
                listc = A.Select();
                int numberc = A.Count();//number of projects, not required
                string[] nameofproj = listc[0].ToArray();
                string[] status = listc[13].ToArray();

                for (int l = 0; l < A.Countproj(abc); l++)
                {

                    for (int m = 0; m < numberc; m++)
                    {
                        if (nameofproj[m] == projname[l])
                        {
                            Status = status[m];
                        }
                    }


                    DateTime shuru = DateTime.ParseExact(startofproj[l],
                                   "dd-MM-yyyy hh:mm:ss",
                                   CultureInfo.InvariantCulture);
                    DateTime anth = DateTime.ParseExact(endofproj[l],
                                   "dd-MM-yyyy hh:mm:ss",
                                   CultureInfo.InvariantCulture);
                    if (date >= shuru && date <= anth)
                    {


                        if (Status != "PLANNED" && Status != "LO" && flag == 0)
                        {
                            y_Values[x]++;//BASIC UTILISATION
                            flag = 1;
                        }
                        if (Status == "IP" || Status == "OTD")
                            y__Values[x]++;//EXCESS
                        if (Status == "PLANNED")
                        {
                            yValues[x]++;//UNUTILISED

                        }

                    }

                }
            }
        }

    }

我最近才遇到多线程。如果代码看起来不是很好,那么请原谅。 threshValue[]refValues[]y_Values[]y__Values[]yValues[]都是全局变量

2 个答案:

答案 0 :(得分:0)

多线程不会自动使程序运行得更快,Thread.Join只是等待线程完成。

基本上如果主线程在线程完成之前无法继续,则不应使用多线程。

多线程适合的示例:

  • 当尝试从多个位置下载信息时,您可以让多个线程等待来自每个位置的数据
  • 当您想要在保持用户界面响应的同时执行耗时的任务时

第二个例子可能是这里的情况,但是Thread.Join是一个阻塞操作:它仍然在线程工作时停止更新用户界面。在这种情况下,您必须让线程通知主线程完成它。

我不知道你在这个平台上工作,但是在Windows窗体上,Form类有一个Invoke方法,可以让你在该窗体的线程上调用一个方法:

class TheForm
{
    // the method running in a separate thread
    void myThread()
    {
        // do the time consuming work
        byte[] myData = ProcessData();

        // send data to the main thread 
        Invoke(new Action<byte[]>(ThreadCompleted), myData);
    }

    // will be executed in the main thread
    void ThreadCompleted(byte[] data)
    {
        // process the data
    }
}

关于全局变量和多线程:任何线程都可以访问它们,但是当时,哪个线程无法访问它们。如果可以,您应该避免让多个线程访问它们,或者使用lock机制来保护它们。

答案 1 :(得分:0)

经过大量的研究和实验。我知道问题不是由于缺少多线程,而是因为经常连接到数据库(在我的循环中)。

我的解决方案是在进入我的循环之前从数据库中批量获取所有数据。这节省了大量宝贵的时间。