必须在线程之间共享数据。哪种是最好的同步方法?
锁定是更好的方法还是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++;
}
}
}
}
答案 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");
}
那应该有点整洁。
我没有确切地看到您在代码中共享的数据。也许你可以详细说明一下?