将实例属性传递给异步任务中的静态方法

Question

代码如下：

namespace ConsoleApp2
{
    using System;
    using System.Collections.Generic;
    using System.Linq;
    using System.Threading.Tasks;
    class Program
    {
        static async Task Main(string[] args)
        {
            var split = new SplitService();

            var tasks = new List<Task>();
            for (var nI = 0; nI < 10; nI++)
            {
                var fathers = new List<Father> { new Father { Id = Guid.NewGuid() } };

                var task = new Task (() => split.Split(fathers));
                tasks.Add(task);
            };

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

            Console.ReadKey();
        }
    }

    public class SplitService
    {
        public IEnumerable<Father> Split(List<Father> fathers)
        {
            this.FatherProperties = fathers.GetFatherValues();

            this.RecalculateProperties(fathers);

            return fathers;
        }

        public List<FatherProperties> FatherProperties { get; private set; } = new List<FatherProperties>();

        public void RecalculateProperties(List<Father> fathers)
        {
            fathers.Update(this.FatherProperties);
        }
    }

    public static class FatherExtensions
    {
        public static List<FatherProperties> GetFatherValues(this List<Father> fathers)
        {
            return new List<FatherProperties>
            {
                new FatherProperties
                {
                    FatherId = fathers.FirstOrDefault().Id
                }
            };
        }

        public static void Update(this List<Father> fathers, List<FatherProperties> properties)
        {
            foreach (var father in fathers)
            {
                var match =
                    ( 
                     from value in properties
                     where value.FatherId == father.Id
                     select new
                     {
                         father.Id
                     }).SingleOrDefault();

                if (match == null)
                {
                    Console.WriteLine("Error");
                }
                else
                {
                    Console.WriteLine(match.Id); 
                }
            }
        }
    }

    public class Father
    {
        public Guid Id { get; set; }
    }

    public class FatherProperties
    {
        public Guid FatherId { get; set; }
    }
}

给出错误是多个线程正在运行。 任何人都可以帮我解释原因吗？ 这条线是：

fathers.Update(this.FatherProperties);

对此的更改：

fathers.Update(father.GetFatherValues());

该代码有效。

这与物业的访问有关吗？ 我不明白为什么。 我试图阅读许多网站，但仍然找不到此错误的原因。

谢谢您的帮助。

Answer 1

您正在为所有任务使用相同的SplitService实例。 在里面你正在修改它的成员FatherProperties 。

显然，这就是你有这种行为的原因。

为每个任务创建一个服务：

for (var nI = 0; nI < 10; nI++)
{
    var split = new SplitService(); // <-- a dedicated service for each

    var fathers = new List<Father> { new Father { Id = Guid.NewGuid() } };
    var task = new Task (() => split.Split(fathers));
    tasks.Add(task);
};

Answer 2

第一：过度使用扩展方法使代码很难阅读。
第二：您正在使用比较的集合。 您只使用一个 SplitService 及其父属性，并且在每次调用 split.Split(fathers) 时都会更改它们。 由于您是多线程进行的，因此您可以在一个线程中进行比较，而另一个则只是在其中放置一个新的父亲属性集合。 再次使用 GetFatherValues() 时，您将避免这种行为，因此不会出现错误。

将拆分服务的创建放入迭代中。

static void Main(string[] args)
    {
        var tasks = new List<Task>();
        for (var nI = 0; nI < 100; nI++)
        {
            var fathers = new List<Father> { new Father { Id = Guid.NewGuid() } };
            var split = new SplitService();
            var task = new Task(() => split.Split(fathers));
            tasks.Add(task);
        };

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

        Console.ReadKey();
    }