Question

遇到问题，希望有人可以帮助我。

我尝试在循环中启动4个任务但是我得到了一个ArgumentOutOfRangeException：

 private byte[] GetData(int id, PLC plc)
    {
        switch (id)
        {
            case 0:
                return plc.ReadBytes(DataType.DataBlock, 50, 0, 200);
            case 1:
                return plc.ReadBytes(DataType.DataBlock, 50, 200, 200);
            case 2:
                return plc.ReadBytes(DataType.DataBlock, 50, 500, 200);
            case 3:
                return plc.ReadBytes(DataType.DataBlock, 50, 700, 200);
            case 4:
                return plc.ReadBytes(DataType.DataBlock, 50, 900, 117);
            default:
                return null;
        }
    }

循环获得溢出，因为i = 4

如果我启动没有循环的任务，它们运行没有任何问题：

{{1}}

不知道为什么？来自西门子PLC的任务GetData通过套接字连接。 PLC最多支持32个连接。我每个连接收到200字节。

{{1}}

任何想法？

问候山姆

Answer 1

这可能是由closure problem引起的。

试试这个：

 for (int i = 0; i < 4; i++)
 {
      //start task with current connection
      int index = i;
      tasks[index] = Task<byte[]>.Run(() => GetData(index, plcPool[index]));
 }

可能发生的事情是，当最后一个线程开始运行时，循环已经将i递增到4，这就是传递给GetData()的值。将i的值捕获到单独的变量index中并使用它来代替该问题。

例如，如果您尝试此代码：

public static void Main()
{
    Console.WriteLine("Starting.");

    for (int i = 0; i < 4; ++i)
        Task.Run(() => Console.WriteLine(i));

    Console.WriteLine("Finished. Press <ENTER> to exit.");
    Console.ReadLine();
}

它通常会给你这种输出：

Starting.
Finished. Press <ENTER> to exit.
4
4
4
4

将该代码更改为：

public static void Main()
{
    Console.WriteLine("Starting.");

    for (int i = 0; i < 4; ++i)
    {
        int j = i;
        Task.Run(() => Console.WriteLine(j));
    }

    Console.WriteLine("Finished. Press <ENTER> to exit.");
    Console.ReadLine();
}

你会得到像

这样的东西

Starting.
Finished. Press <ENTER> to exit.
0
1
3
2

注意它是如何仍然不是必须的！您将看到打印出的所有正确值，但顺序不确定。多线程很棘手！

Answer 2

在 2021 年，您真的应该为此使用内置的 Parallel.For。达到你想要的效果非常简单：

ConcurrentBag<byte[]> results = new ConcurrentBag<byte[]>();

ParallelLoopResult result = Parallel.For(0, 4, (i, state) => {
   results.Add(GetData(i, plcPool[i]));
});

Answer 3

在C＃5中，foreach的循环变量将在逻辑上位于循环内部，因此闭包每次都会关闭变量的新副本。 “for”循环不会更改。

另一种方式是：

创建任务
运行任务。
等待所有任务完成。

演示代码如下：

 internal class Program
{
    private static void Main(string[] args)
    {
        Task[] tasks = new Task[4];

        for (int i = 0; i < 4; i++)
        {
            //start task with current connection
            tasks[i] = new Task<byte[]>(GetData,i);
        }

        foreach (var task in tasks)
        {
            task.Start();
        }

        Task.WaitAll(tasks);

        Console.Read();
    }

    private static byte[] GetData(object index)
    {
        var i = (int) index;
        switch (i)
        {
            case 0:

                //return plc.ReadBytes(DataType.DataBlock, 50, 0, 200);
                Console.WriteLine(i);
                return new byte[] { };
            case 1:
            //return plc.ReadBytes(DataType.DataBlock, 50, 200, 200);
            case 2:
                Console.WriteLine(i);
                return new byte[] { };
            //return plc.ReadBytes(DataType.DataBlock, 50, 500, 200);
            case 3:
                Console.WriteLine(i);
                return new byte[] { };
            //return plc.ReadBytes(DataType.DataBlock, 50, 700, 200);
            case 4:
                //return plc.ReadBytes(DataType.DataBlock, 50, 900, 117);
                Console.WriteLine(i);
                return new byte[] { };

            default:
                return null;
        }
    }
}

输出：

注意：它是new Task<byte[]>(GetData,i);而不是new Task<byte[]>(()=>GetData(i));

（）=＆gt; v表示“返回变量v的当前值”，而不是“在创建委托时返回值v。” 闭包关闭变量，而不是超过值。

因此new Task<byte[]>(GetData,i);没有“关闭问题”

For循环结果使用Task.Run或Task.Start溢出

3 个答案: