Вопрос

Я пытаюсь использовать статистику “rusage” в своей программе, чтобы получить данные, аналогичные данным время инструмент.Однако я почти уверен, что делаю что-то не так.Значения кажутся примерно правильными, но иногда могут быть немного странными.Я не нашел хороших ресурсов в Интернете.Кто-нибудь знает, как сделать это лучше?

Извините за длинный код.

class StopWatch {
public:
    void start() {
        getrusage(RUSAGE_SELF, &m_begin);
        gettimeofday(&m_tmbegin, 0);
    }

    void stop() {
        getrusage(RUSAGE_SELF, &m_end);
        gettimeofday(&m_tmend, 0);
        timeval_sub(m_end.ru_utime, m_begin.ru_utime, m_diff.ru_utime);
        timeval_sub(m_end.ru_stime, m_begin.ru_stime, m_diff.ru_stime);
        timeval_sub(m_tmend, m_tmbegin, m_tmdiff);
    }

    void printf(std::ostream& out) const {
        using namespace std;

        timeval const& utime = m_diff.ru_utime;
        timeval const& stime = m_diff.ru_stime;

        format_time(out, utime);
        out << "u ";
        format_time(out, stime);
        out << "s ";
        format_time(out, m_tmdiff);
    }

private:
    rusage m_begin;
    rusage m_end;
    rusage m_diff;
    timeval m_tmbegin;
    timeval m_tmend;
    timeval m_tmdiff;

    static void timeval_add(timeval const& a, timeval const& b, timeval& ret) {
        ret.tv_usec = a.tv_usec + b.tv_usec;
        ret.tv_sec = a.tv_sec + b.tv_sec;
        if (ret.tv_usec > 999999) {
            ret.tv_usec -= 1000000;
            ++ret.tv_sec;
        }
    }

    static void timeval_sub(timeval const& a, timeval const& b, timeval& ret) {
        ret.tv_usec = a.tv_usec - b.tv_usec;
        ret.tv_sec = a.tv_sec - b.tv_sec;
        if (a.tv_usec < b.tv_usec) {
            ret.tv_usec += 1000000;
            --ret.tv_sec;
        }
    }

    static void format_time(std::ostream& out, timeval const& tv) {
        using namespace std;
        long usec = tv.tv_usec;
        while (usec >= 1000)
            usec /= 10;
        out << tv.tv_sec << '.' << setw(3) << setfill('0') << usec;
    }
}; // class StopWatch
Это было полезно?

Решение

Какова цель:

while (usec >= 1000)
    usec /= 10;

Я так понимаю, что вам нужны самые значимые три цифры usec;в таком случае, самый простой способ, который я могу придумать, - это разделить usec на 1000 и покончить с этим.

Тестовые примеры:

  • 999999 ⇒ 999
  • 99999 ⇒ 999 (должно быть 099)
  • 9999 ⇒ 999 (должно быть 009)
  • 999 ⇒ 999 (должно быть 000)

Другие советы

Я думаю, что, вероятно, где-то в вашем составе sec и usec есть ошибка.Я не могу точно сказать, что именно, не зная, какие ошибки вы видите.Грубое предположение состоит в том, что usec никогда не может быть > 999999, поэтому вы полагаетесь на переполнение, чтобы знать, когда регулировать sec.Это также может быть просто проблема с вашим форматом вывода длительности.

В любом случае.Почему бы не сохранить компоненты utime и stime в виде секунд с плавающей запятой, вместо того чтобы пытаться создать свой собственный rusage на выходе?Я почти уверен, что следующее даст вам нужные секунды.

static int timeval_diff_ms(timeval const& end, timeval const& start) {
    int micro_seconds = (end.tv_sec  - start.tv_sec) * 1000000 
        + end.tv_usec - start.tv_usec;

    return micro_seconds;
}

static float timeval_diff(timeval const& end, timeval const& start) {
    return (timeval_diff_ms(end, start)/1000000.0f);
}

Если вы хотите разложить это обратно на rusage, вы всегда можете использовать int-div и modulo.

@Крис:

Я так понимаю, что вам нужны самые значимые три цифры usec

ДА.Общее количество цифр в usec варьируется, и я хочу сократить их ниже 1000.Например, если usec=1000, я хочу получить результат 100 (а не 1, как вы предлагаете).Поэтому я не могу просто разделить на 1000.

Лицензировано под: CC-BY-SA с атрибуция
Не связан с StackOverflow
scroll top