Windows 服务内存使用量增加

Question

我有一个 Windows 服务正在执行 3 个预定的作业。首先是向员工发送电子邮件，了解他们的轮班时间。第二个是获取 Active Directory 信息并将其保存到本地数据库。最后一个是将活动目录照片保存到文件目录。

每个作业都在单独的线程上完成。计时器每 45 秒计时一次。除了服务的内存使用量稳步增加外，一切都运行良好。

您知道是什么原因造成的吗？

Thread[] thread;

protected override void OnStart(string[] args)
{
    timer = new System.Timers.Timer();
    timer.AutoReset = true;
    timer.Enabled = true;
    timer.Interval = 1000 * 45;
    timer.Start();
    timer.Elapsed += Timer_Elapsed;
    servisList = new List<IService>() { new ShiftNotification(), new     ActiveDirectoryService(), new DirectoryPhotos() };
    thread = new Thread[servisList.Count];
}

private void Timer_Elapsed(object sender, System.Timers.ElapsedEventArgs e)
{
    try
    {
        for (int i = 0; i < servisList.Count; i++)
        {
            if (thread[i] == null)
            {
                if (TimeControl.CheckTime(DateTime.Now, servisList.ElementAt(i).sProps))
                {
                    thread[i] = new System.Threading.Thread(new System.Threading.ThreadStart(servisList.ElementAt(i).NotifyUsers));
                    thread[i].Start();
                }
            }
            else
            {
                if (thread[i].ThreadState != System.Threading.ThreadState.Running)
                {
                    if (TimeControl.CheckTime(DateTime.Now, servisList.ElementAt(i).sProps))
                    {
                        thread[i] = new System.Threading.Thread(new System.Threading.ThreadStart(servisList.ElementAt(i).NotifyUsers));
                        thread[i].Start();
                    }
                }
            }
        }
    }
    catch (Exception ex)
    {
    }
}

Answer 1

只是对此的一些想法

1. ActiveDirectory 和SmcpClient（如果您不将其用于电子邮件，请纠正我）的使用意味着您正在处理非托管资源，您是否正确处理了这些类？您可能应该将客户端变量设置为静态字段，并在程序结束时处理它们。

2. 这些行我不清楚：

   if (thread[i].ThreadState != System.Threading.ThreadState.Running)
   {
       if (TimeControl.CheckTime(DateTime.Now, servisList.ElementAt(i).sProps))
       {
           thread[i] = new System.Threading.Thread(new System.Threading.ThreadStart(servisList.ElementAt(i).NotifyUsers));
           thread[i].Start();
       }
   }
   

   Thread 也是非托管资源，因此，由于它正在实现 IDisposable，您应该在为数组条目分配新值之前处理前一个：thread[i].Dispose();

3. 次要你为什么要使用全名，因为你已经添加了using System.Threading？

Answer 2

AFAICT，如果没有关于您在 NotifyUsers 方法中所做的更具体的信息，就不可能回答这个问题。

但即使在研究之前，我也会先用这个重新审视你所拥有的线程生成逻辑

if (thread[i] == null)
{
   // (code omitted)
}
else
{
   // (code omitted)
}

我的意思是，首先尝试弄清楚您的服务是否最终会产生过多的工作线程（这是我的疯狂猜测，先验）。

更具体地说，一旦/每次您使用 System.Threading.Thread，请始终将race condition 视为最先/最可能需要注意的问题之一。

多线程绝非易事。但是plenty of resources 可以帮助您完成工作。

'希望这会有所帮助。