这些 msec<->timeval 函数正确吗？

Question

我的程序有一个错误，我不断地回到这两个函数，但它们对我来说看起来很正确。这里有什么问题吗？

long visual_time_get_msec(VisTime *time_)
{
    visual_log_return_val_if_fail(time_ != NULL, 0);

    return time_->tv_sec * 1000 + time_->tv_usec / 1000;
}


int visual_time_set_from_msec(VisTime *time_, long msec)
{
    visual_log_return_val_if_fail(time_ != NULL, -VISUAL_ERROR_TIME_NULL);


    long sec = msec / 1000;
    long usec = 0;

    visual_time_set(time_, sec, usec);

    return VISUAL_OK;
}

Answer 1

您的第一个函数是向下舍入的，因此 1.000999 秒四舍五入为 1000 毫秒，而不是 1001 毫秒。要解决这个问题（使其四舍五入到最接近的毫秒），您可以这样做：

long visual_time_get_msec(VisTime *time_)
{
    visual_log_return_val_if_fail(time_ != NULL, 0);

    return time_->tv_sec * 1000 + (time_->tv_usec + 500) / 1000;
}

Fuzz 已经指出了你的第二个例子中的截断 - 我唯一要补充的是，你可以使用模运算符稍微简化它：（

long sec = msec / 1000;
long usec = (msec % 1000) * 1000;

以上所有假设你没有处理负时间值- 如果你是，事情会变得更加复杂）。

Answer 2

Visual_time_set_from_msec 看起来不正确...

如果有人调用 Visual_time_set_from_msec(time, 999)，那么你的结构将被设置为零，而不是 999,000us。

你应该做的是：

// Calculate number of seconds
long sec = msec / 1000; 
// Calculate remainding microseconds after number of seconds is taken in to account
long usec = (msec - 1000*sec) * 1000;

这实际上取决于你的输入，但这就是我的 2 美分:-)