以下是一些永久生成GUID的代码。我写这篇文章是为了了解线程。在它中你会发现我已经锁定了我生成GUID并将它们排队的地方,即使ConcurrentQueue是线程安全的。这是因为我的实际代码需要使用NHibernate,因此我必须确保只有一个线程可以填充队列。
当我在任务管理器中监视此代码时,我注意到该进程将线程数从18(在我的机器上)减少到14但不少。这是因为我的代码不好吗?
如果他们认为合适,有人可以重构吗?我喜欢更短的代码。
class Program
{
ConcurrentNewsBreaker Breaker;
static void Main(string[] args)
{
new Program().Execute();
Console.Read();
}
public void Execute()
{
Breaker = new ConcurrentNewsBreaker();
QueueSome();
}
public void QueueSome()
{
ThreadPool.QueueUserWorkItem(DoExecute);
}
public void DoExecute(Object State)
{
String Id = Breaker.Pop();
Console.WriteLine(String.Format("- {0} {1}", Thread.CurrentThread.ManagedThreadId, Breaker.Pop()));
if (Breaker.Any())
QueueSome();
else
Console.WriteLine(String.Format("- {0} XXXX ", Thread.CurrentThread.ManagedThreadId));
}
}
public class ConcurrentNewsBreaker
{
static readonly Object LockObject = new Object();
ConcurrentQueue<String> Store = new ConcurrentQueue<String>();
public String Pop()
{
String Result = null;
if (Any())
Store.TryDequeue(out Result);
return Result;
}
public Boolean Any()
{
if (!Store.Any())
{
Task FillTask = new Task(FillupTheQueue, Store);
FillTask.Start();
FillTask.Wait();
}
return Store.Any();
}
private void FillupTheQueue(Object StoreObject)
{
ConcurrentQueue<String> Store = StoreObject as ConcurrentQueue<String>;
lock(LockObject)
{
for(Int32 i = 0; i < 100; i++)
Store.Enqueue(Guid.NewGuid().ToString());
}
}
}
答案 0 :(得分:2)
您正在使用.NET的ThreadPool,因此.NET / Windows根据等待处理的工作量来管理线程数。
答案 1 :(得分:1)
我在Task中监视此代码 经理,我注意到这个过程下降了 18的线程数(在我的 机器)到14但不低于。这是 因为我的代码不好?
这并不表示存在问题。 14仍然很高,除非你有一个16核cpu。
线程池将尝试调整并尽可能少地运行线程。
当线程数 up 时,您应该开始担心。