线程同步

时间:2011-09-08 05:11:21

标签: c# thread-synchronization

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

锁定是更好的方法还是Mutex?

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++;
            }
        }
    }
}

1 个答案:

答案 0 :(得分:5)

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

在继续使用主线程之前,您正在使用Thread.Join等待线程完成。没关系。

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

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");
}

那应该有点整洁。

我没有确切地看到您在代码中共享的数据。也许你可以详细说明一下?

相关问题