在特定时间触发BackgroundWorker的正确方式？Windows服务数据过期检查疑问

你的这种实现方式不正确，主要存在以下几个问题：

• CPU资源浪费：无限循环中没有任何延迟，线程会持续高频检查时间，导致CPU占用率飙升，完全没必要。
• 重复执行风险：即使到了14:37:00，由于循环执行速度极快，同一秒内会多次触发if条件，导致业务逻辑被重复执行。
• 逻辑运算符误用：代码中用了按位与&，虽然这里逻辑结果没问题，但判断条件应该用逻辑与&&，更符合C#的代码规范。

正确的实现方式 #

应该通过计算下一次执行的时间间隔，使用Task.Delay让线程进入等待状态，避免空转，同时确保到点仅执行一次业务逻辑。示例代码如下：

protected override async Task ExecuteAsync(CancellationToken stoppingToken)
{
    while (!stoppingToken.IsCancellationRequested)
    {
        // 计算今天14:37:00的时间点
        var targetTime = DateTime.Today.AddHours(14).AddMinutes(37);
        // 如果当前时间已经过了今天的目标点，就取明天的同一时间
        if (DateTime.Now > targetTime)
        {
            targetTime = targetTime.AddDays(1);
        }
        // 计算需要等待的时间间隔
        var delayTime = targetTime - DateTime.Now;
        
        // 等待到目标时间，同时响应取消信号
        await Task.Delay(delayTime, stoppingToken);
        
        // 确认未被取消后执行业务逻辑
        if (!stoppingToken.IsCancellationRequested)
        {
            // do something...
        }
    }
}

额外说明 #

• 使用Task.Delay后，线程会在等待期间释放CPU资源，大幅降低服务的资源消耗。
• 通过判断当前时间是否已过当日目标点，自动切换到次日的执行时间，保证每天只执行一次。
• 每次等待后检查CancellationToken，确保服务停止时能及时终止任务。

内容的提问来源于stack exchange，提问作者Shawn Chen