等待计时器未等待指定时间段

Question

我正在创建一个手动等待计时器，如下所示

m_hTimer = CreateWaitableTimer(NULL, true, NULL);

并在函数中使用它

HRESULT ClassA::InduceSleep(UINT32 uiMiliSeconds)
{
    if (m_hTimer)
    {
        LARGE_INTEGER liDueTime;
        liDueTime.QuadPart = (uiMiliSeconds) * (-10000) * (1LL);
        if (!SetWaitableTimer(m_hTimer, &liDueTime, 0, NULL, NULL, 0))
        {
            Log("SetWaitableTimer failed GLE[%d]", GetLastError());
            goto exit;
        }

        // Wait for the timer.

        if (WaitForSingleObject(m_hTimer, INFINITE) != WAIT_OBJECT_0)
        {
            Log("WaitForSingleObject failed GLE[%d]", GetLastError());
        }
        return S_OK;
    }
exit:    
    Sleep(uiMiliSeconds);
    return S_OK;
}

我观察到，在延迟 10 秒（或 5 秒）调用 InduceSleep() 时，waitforsingleobject 会立即返回 WAIT_OBJECT_0 而没有任何延迟，因此会立即发出计时器信号。文档中提到 setwaitable 计时器停止并重新激活计时器，因此它不应处于信号状态，并且应仅在给定时间后发出信号。我在这里错过了什么？

Answer 1

这是一个简单的错误：

uiMiliSeconds 属于 UINT32 类型，即无符号类型。

这让

(uiMiliSeconds) * (-10000)

无符号乘法，即-10000 在乘法之前转换为unsigned。 （我同意，类型提升有时是一个棘手的话题。）

我在一个最小的示例中尝试了这个：

#include <iostream>

int main()
{
  uint32_t uiMilliseconds = 10 * 1000;
  std::cout << uiMilliseconds * -10000 << '\n';
  std::cout << (int)uiMilliseconds * -10000 << '\n';
  return 0;
}

输出：

4194967296
-100000000

Live Demo on coliru

因此，解决方案是在乘法之前将uiMiliseconds 简单地转换为有符号整数（就像我在第二个输出行中所做的那样）。

故事的其余部分对 OP 来说可能是显而易见的。来自 MSDN 关于SetWaitableTimer function：

lpDueTime

计时器状态设置为已发出信号的时间，以 100 纳秒为间隔。使用 FILETIME 结构描述的格式。正值表示绝对时间。请务必使用基于 UTC 的绝对时间，因为系统内部使用基于 UTC 的时间。负值表示相对时间。

因此，错误的计算时间似乎产生了一个已经过去的值。这是SetWaitableTimer(m_hTimer, &liDueTime, 0, NULL, NULL, 0) 立即返回的唯一合理解释。 （OP 检查了错误，但没有识别出来。）

---

为了使其“防弹”，我建议

 uiMiliSeconds * -10000LL

考虑到例如VC 对int 使用 32 位（即使对于 x64 也是如此）。因此，由于类型提升，任何UINT32 值都将扩展为相应的long long，而没有溢出的危险。

---

RbMm 抱怨为MiliSeconds 输入UINT32 可能不是那么幸运，因为它只涵盖了可以设置的时间子范围。在 OP 的情况下，这个子范围可能就足够了。否则，可以考虑输入UINT64。