std::accumulate使用的一个小细节

今天使用std::accumulate模板函数的时候出现了一个错误,特此记录一下。

#include <iostream>
#include <numeric>

int main()
{
    int LevelColRow[][3] =
    {
        1,  2,     1,
        2,  4,     2,
        3,  8,     4,
        4,  16,    8,
        5,  32,    16,
        6,  64,    32,
        7,  128,   64,
        8,  256,   128,
        9,  512,   256,
        10, 1024,  512,
        11, 2048,  1024,
        12, 4096,  2048,
        13, 8192,  4096,
        14, 16384, 8192,
        15, 32768, 16384,
        16, 65536, 32768 /*,
        17, 131072, 65536/*,
        18, 262144, 131072,
        19, 524288, 262144*/
    };
    double xx = std::accumulate( std::begin( LevelColRow ), std::end( LevelColRow ),
                     0, [] (double val, int lcr[3]) { return(val + double(lcr[1]) * double(lcr[2]) ); } );
    double xx2 = 0;
    for ( auto x : LevelColRow )
    {
        xx2 += double(x[1]) * double(x[2]);
    }
    std::cout << (int64_t) xx << std::endl;
    std::cout << (int64_t) xx2 << std::endl;
    std::cout << "Hello, world!\n";
}

这个代码是用于求这个行列组总共有多少个格子的,但是算出的结果总是为负数。
经过排查,当格子数大于2^31个时候,就出现问题了,这就应该是计算的结果实际是int32的。
查看accumulate(http://en.cppreference.com/w/cpp/algorithm/accumulate)的声明如下:

template< class InputIt, class T >
T accumulate( InputIt first, InputIt last, T init );

template< class InputIt, class T, class BinaryOperation >
T accumulate( InputIt first, InputIt last, T init,
              BinaryOperation op );

那么原因就很简单了,因为我在使用的时候,第三个参数写的是0(默认为int),改为0.0就可以了。

  • 0
    点赞
  • 0
    收藏
    觉得还不错? 一键收藏
  • 0
    评论

“相关推荐”对你有帮助么?

  • 非常没帮助
  • 没帮助
  • 一般
  • 有帮助
  • 非常有帮助
提交
评论
添加红包

请填写红包祝福语或标题

红包个数最小为10个

红包金额最低5元

当前余额3.43前往充值 >
需支付:10.00
成就一亿技术人!
领取后你会自动成为博主和红包主的粉丝 规则
hope_wisdom
发出的红包
实付
使用余额支付
点击重新获取
扫码支付
钱包余额 0

抵扣说明:

1.余额是钱包充值的虚拟货币,按照1:1的比例进行支付金额的抵扣。
2.余额无法直接购买下载,可以购买VIP、付费专栏及课程。

余额充值