верны ли эти функции msec‹-›timeval?

У меня есть ошибка в этой программе, и я постоянно возвращаюсь к этим двум функциям, но мне они кажутся правильными. Что-то не так?

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;
}

c timeval
person Scott    schedule 29.01.2010    source источник

Ответы (2)


arrow_upward
2
arrow_downward

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 цента :-)

person Fuzz    schedule 29.01.2010

arrow_upward
3
arrow_downward

Ваша первая функция округляет в меньшую сторону, так что 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;

(Вышеизложенное предполагает, что вы не имеете дело с отрицательными значениями времени - если да, то все становится сложнее).

person caf    schedule 29.01.2010