线程同步

Question

数据必须在线程之间共享。哪种是最好的同步方法？

锁是更好的方法还是互斥锁？

namespace ConsoleApplication8
{
    class Data
    {
        public int X
        {
            set;
            get;
        }

        public int Y
        {
            get;
            set;
        }

        public int result;
    }

    class Program
    {
        static int a;
        private static object aLock = new object();

        static void Main(string[] args)
        {
            ParameterizedThreadStart aStart = new ParameterizedThreadStart(Addition);
            Thread aThread = new Thread(aStart);
            Data aData = new Data();
            aData.X = 10;
            aData.Y = 20;

            Thread aThread2 = new Thread(aStart);
            aThread2.Start();


            aThread.Start(aData);
            aThread.Join();
            aThread2.Join();
            Console.WriteLine("End of the program");
        }

        static void Addition(object data)
        {
            var a = data as Data;
            var b = a.X + a.Y;
            a.result = b;

            Console.WriteLine(a.result);
            Thread.Sleep(1000);
            Console.WriteLine("End of thread");
            updateValue();
        }

        static void updateValue()
        {
            lock (aLock)
            {
                a++;
            }
        }
    }
}

Answer 1

你有两个“同步”线程的地方。

在继续主线程之前，您正在使用Thread.Join等待线程完成。这很好。

您还使用了一个锁来确保一次只有一个线程递增计数器变量a。这也很好，但可以改进。在System.Threading中有一个名为Interlocked的类，它可以以线程安全的方式为您执行增量。

Interlocked.Increment(ref a);

您的代码确实在两个地方使用了变量a -在Addition内部，您有一个局部变量a，它隐藏了静态外部变量a。我想这只是个巧合。

另一个问题是您的Addition方法接受object作为参数。我理解为什么它不接受Data作为参数，因为ParameterizedThreadStart需要一个对象。有一个更好的方法来解决这个问题。

请尝试以下代码：

private static int __counter = 0;

public class Data
{
    public int X { set; get; }
    public int Y { set; get; }
    public int Result { set; get; }
}

private void Addition(Data data)
{
    data.Result = data.X + data.Y;
    Interlocked.Increment(ref __counter);

    Thread.Sleep(1000);

    Console.WriteLine(data.Result);
    Console.WriteLine("End of thread");
}

现在Main可以写成：

void Main()
{
    ParameterizedThreadStart pts = o => Addition(o as Data);

    var t1 = new Thread(pts);
    var t2 = new Thread(pts);

    t1.Start(new Data { X = 10, Y = 20 });
    t2.Start(new Data { X = 13, Y = 42 });

    t1.Join();
    t2.Join();

    Console.WriteLine(__counter);
    Console.WriteLine("End of the program");
}

这应该会更整洁一点。

我看不到您在代码中确切地共享了什么数据。也许你可以详细说明一下？