5

我有这个非常简单的函数，我有一些值需要计算但只计算一次，最好的时间是在编译时。这些值仅在此函数中很重要。这是对 constexpr 的一个很好的使用，还是我应该将它们声明为静态 const？

ps 我知道性能差异微乎其微，但我想以“正确的”c++11 方式来做。

void MainWindow::UpdateDateTimes()
{
// for some dumb reason DateTime only has add seconds method
    // so we have to calculate the seconds per hour and the number of hours
    // we do this with static constant values so that the calculations
    // only happen once.
    static constexpr const int secsPerHour = 60 * 60;
    static constexpr const int cdtOffsetHours = -5;
    static constexpr const int edtOffsetHours = -4;
    static constexpr const int cetOffsetHours = 2;
    static constexpr const int cdtOffsetSecs = secsPerHour * cdtOffsetHours;
    static constexpr const int edtOffsetSecs = secsPerHour * edtOffsetHours;
    static constexpr const int cetOffsetSecs = secsPerHour * cetOffsetHours;

    QDateTime time( QDateTime::currentDateTimeUtc() );

    ui->mTimeLocal->setDateTime( time.toLocalTime() );

    ui->mTimeCDT->setDateTime( time.addSecs( cdtOffsetSecs ) );
    ui->mTimeEDT->setDateTime( time.addSecs( edtOffsetSecs ) );
    ui->mTimeCET->setDateTime( time.addSecs( cetOffsetSecs ) );
}
4

1 回答 1

7

您的使用很好，即使不是有点冗长。在这种情况下constexpr和const意思完全一样。任何一个（甚至两者）都可以。

Fwiw，std::chrono::hours::period::num将是另一种指定方式60*60（如果您想炫耀一些 C++11 信誉:-)）。

或者实际上你可以这样做：

void MainWindow::UpdateDateTimes()
{
    constexpr std::chrono::seconds cdtOffsetSecs = std::chrono::hours(-5);
    constexpr std::chrono::seconds edtOffsetSecs = std::chrono::hours(-4);
    constexpr std::chrono::seconds cetOffsetSecs = std::chrono::hours(2);

    QDateTime time( QDateTime::currentDateTimeUtc() );

    ui->mTimeLocal->setDateTime( time.toLocalTime() );

    ui->mTimeCDT->setDateTime( time.addSecs( cdtOffsetSecs.count() ) );
    ui->mTimeEDT->setDateTime( time.addSecs( edtOffsetSecs.count() ) );
    ui->mTimeCET->setDateTime( time.addSecs( cetOffsetSecs.count() ) );
}

我也很想放弃static. 在我的系统上，无论有没有static. 这一切都发生在编译时，因此不需要static“仅一次”初始化语义。

更新

为了清楚起见，我将原始示例编辑为：

void f(int);

void UpdateDateTimes()
{
    constexpr std::chrono::seconds cdtOffsetSecs = std::chrono::hours(-5);
    constexpr std::chrono::seconds edtOffsetSecs = std::chrono::hours(-4);
    constexpr std::chrono::seconds cetOffsetSecs = std::chrono::hours(2);

    f(cdtOffsetSecs.count());
}

使用 clang++ 和 libc++ 使用 -O1 （几乎没有启用优化）编译它，程序集是：

    .globl  __Z15UpdateDateTimesv
    .align  4, 0x90
__Z15UpdateDateTimesv:                  ## @_Z15UpdateDateTimesv
    .cfi_startproc
## BB#0:
    pushq   %rbp
Ltmp2:
    .cfi_def_cfa_offset 16
Ltmp3:
    .cfi_offset %rbp, -16
    movq    %rsp, %rbp
Ltmp4:
    .cfi_def_cfa_register %rbp
    movl    $-18000, %edi           ## imm = 0xFFFFFFFFFFFFB9B0
    popq    %rbp
    jmp __Z1fi                  ## TAILCALL
    .cfi_endproc

然后我用相同的设置编译了这个程序：

void UpdateDateTimes2()
{
    f(-18000);
}

生成的程序集是：

    .globl  __Z16UpdateDateTimes2v
    .align  4, 0x90
__Z16UpdateDateTimes2v:                 ## @_Z16UpdateDateTimes2v
    .cfi_startproc
## BB#0:
    pushq   %rbp
Ltmp7:
    .cfi_def_cfa_offset 16
Ltmp8:
    .cfi_offset %rbp, -16
    movq    %rsp, %rbp
Ltmp9:
    .cfi_def_cfa_register %rbp
    movl    $-18000, %edi           ## imm = 0xFFFFFFFFFFFFB9B0
    popq    %rbp
    jmp __Z1fi                  ## TAILCALL
    .cfi_endproc

所以恕我直言，这几乎是一个人得到免费午餐的时候了。:-)

于 2013-04-09T22:04:01.217 回答