多线程会提高 WCF 服务中方法的性能吗？

Question

我有一个托管在 IIS6 中的 WCF 服务。该方法的重要部分如下所示：

public MyUser[] GetUsers(string appName, string[] names)
{
    List<User> users = new List<User>();
    foreach (string user in names)
    {
      MembershipUser mu = this.ADAMProvider.GetUser(user, false);  //Unmanaged call to AzMan
      if (mu != null)
      {
        users.Add(MyUser.CreateFrom(mu);
      }
    }
    return users.ToArray();
}

当使用大量用户名数组（超过 100 个左右）调用该方法时，该方法的性能非常差。返回可能需要一分多钟的时间。另外，如果该方法被多个客户端同时调用，则会超时。我什至看到它导致应用程序池崩溃。请注意，在循环中正在调用 AzMan。 AzMan 是非托管 COM 组件。

为了提高性能，我正在考虑采用多线程方法。 .NET 4 不是一个选项，因此 Parallel.For 不是一个选项，但在 3.5 中执行等效操作是。

我的问题是创建一堆线程（然后在返回之前等待所有线程）实际上会提高性能吗？在 IIS6 托管的 WCF 服务中执行此操作是否存在危险？

Answer 1

首先，我指出 COM 组件可能会出现问题，具体取决于其单元状态。单线程单元对象只能在一个线程中运行。 STA 对象上会发生自动编组操作，该操作将有效地序列化对其的所有调用，因此无论您如何努力，可能都无法从中获得任何并行化。即使它是一个 MTA 对象，如果 GetUser 方法未设计为线程安全的，也可能会出现问题。

但假设这都不是问题¹，我会使用ThreadPool，而不是创建一堆线程来执行此操作。它可能是这样的。

public MyUser[] GetUsers(string appName, string[] names) 
{ 
  int count = 1; // Holds the number of pending work items.
  var finished = new ManualResetEvent(false); // Used to wait for all work items to complete.
  var users = new List<User>(); 
  foreach (string user in names) 
  {
    Interlocked.Increment(ref count); // Indicate that we have another work item.
    ThreadPool.QueueUserWorkItem(
      (state) =>
      { 
        try
        {
          MembershipUser mu = this.ADAMProvider.GetUser(user, false); 
          if (mu != null) 
          { 
            lock (users)
            {
              users.Add(MyUser.CreateFrom(mu); 
            }
          } 
        }
        finally
        {
          // Signal the event if this is the last work item.
          if (Interlocked.Decrement(ref count) == 0) finished.Set();
        }
      });
  } 
  // Signal the event if this is the last work item.
  if (Interlocked.Decrement(ref count) == 0) finished.Set();
  // Wait for all work items to complete.
  finished.WaitOne();
  return users.ToArray(); 
}

关于我上面使用的模式，一个令人困惑的事情是它将主线程（对工作进行排队的线程）视为另一个工作项。这就是为什么您会在循环末尾看到检查和信号代码。如果没有它，Set 和 WaitOne 调用之间可能会出现非常微妙的竞争条件。

顺便说一句，我应该指出，TPL 在 .NET 3.5 中作为

¹_{我怀疑我提到的两个问题之一将在现实中发挥作用。}

Answer 2

通常，我会说这可能会有所帮助 - 但是，这会令人担忧：

此外，如果多个客户端同时调用此方法，则会超时。我什至看到它导致应用程序池崩溃。请注意，在循环中正在调用 AzMan。 AzMan 是非托管 COM 组件。

这听起来像“AzMan”组件不是线程安全的。如果是这种情况，则无法有效地多线程该例程，因为它的大部分时间都花在该例程上。

但是，如果该例程是线程安全的，并且不共享状态，则多线程可能会提高性能。然而，这取决于很多其他问题，包括机器本身的工作负载（如果所有核心都得到很好的利用，可能没有帮助）等。