Question

我正在将ForEach替换为Parallel.ForEach。 ForEach工作正常，并按预期给出结果，其中Parallel.ForEach给出了意想不到的结果。

            string[] ids= { };
            string input="AA;AA;111;T#BB;BB;222;T#CC;CC;333;F";
            string typeId, type, value;
            typeId= type=value=string.Empty;
            bool isValid = false;
            if (input.Contains("#"))
            { ids = input.Split('#'); }
            else
            { ids = new string[] { input };}

//using ForEach ,it's working fine,datas are inserted in DB table properly.
            foreach (var id in ids)
            {
                if (id.Contains(";"))
                {
                    var IdInfo = id.Split(';');
                    if (IdInfo[0] != null)
                    {
                        typeId = Info[0];
                    }

                    if (IdInfo.Length >= 2)
                    { type = IdInfo[1]; }

                    if (IdInfo.Length >= 3)
                    { value = IdInfo[2]; }

                    if (IdInfo.Length >= 4)
                    { isValid = IdInfo[3]=="T" ? true:false; } // T=true
                }
                else
                {
                    return;
                }
                DBInsertMethod("someuniqueId", typeId, type, value, isValid);
            }

我用Parrallel.ForEach替换了同一段代码

Parallel.ForEach ( ids,(id)=>
            {
                if (id.Contains(";"))
                {
                    var IdInfo = id.Split(';');
                    if (IdInfo[0] != null)
                    {
                        typeId = Info[0];
                    }

                    if (IdInfo.Length >= 2)
                    { type = IdInfo[1]; }

                    if (IdInfo.Length >= 3)
                    { value = IdInfo[2]; }

                    if (IdInfo.Length >= 4)
                    { isValid = IdInfo[3]=="T" ? true:false; } // T=true
                }
                else
                {
                    return;
                }
                DBInsertMethod("someuniqueId", typeId, type, value, isValid);
            });

现在同一行在表格中插入三次。感谢任何建议/建议。

Answer 1

所以你有这些变量：

string typeId, type, value;
bool isValid;

如果我运行此foreach，则任何时候都会访问typeId一次。

foreach (var item in list)
{
    typeId = item.id;
}

现在，如果我将其并行，那么任何时候都会有N个线程访问typeId。 N个线程可以在任何时间更新它。

Parallel.ForEach(list, item =>
{
    typeId = item.id;
});

将临时变量移动到ForEach中，以便将它们限定为匿名函数：

Parallel.ForEach(list, item =>
{
    string typeId, type, value;
    bool isValid;
    typeId = item.id;
});

Answer 2

实际上，即使Paralle.Foreach创建了多个线程来处理并行处理，一些变量也会在其范围之外声明，因此它们是唯一的。尝试在lambda中声明它们：

begin
  for cur_r in (select id1, id2 from t1) loop
    sp_test (cur_r.id1, cur_r.id2);
  end loop;
end;

Parallel.ForEach没有按预期工作C＃

2 个答案: