函数将向量中存储的大整数相加的问题

时间:2018-10-30 15:32:01

标签: c++ algorithm addition multiprecision

因此,我决定编写自己的多精度数据类型。我编写了一个简单的函数,将添加在vector<uint_fast8_t>中的大量数字相加。

vector<uint_fast8_t> Add(vector<uint_fast8_t > x, vector<uint_fast8_t > y){
    unsigned int x_size = x.size() / sizeof(uint_fast8_t);
    unsigned int y_size = y.size() / sizeof(uint_fast8_t);
    unsigned int res_size{};
    if(x_size>y_size){
        res_size = x_size;
        y.insert(y.end(),uint_fast8_t(0), res_size-y_size);
    } else{
        res_size = x_size;
        x.insert(x.end(),uint_fast8_t(0), res_size-x_size);
    }
    reverse(x.begin(), x.end());
    reverse(y.begin(), y.end());
    vector<uint_fast8_t > res(res_size, 0);
    for(unsigned int i = 0; i < res_size; ++i){
        uint_fast8_t curr = res[i] + x[i] + y[i];
        if(curr >= 10){
            if(i==res_size){
                res.push_back(uint_fast8_t(1));
            } else{
                res[i+1] = uint_fast8_t(1);
            }
            res[i] = curr - uint_fast8_t(10);
        } else{
            res[i] = curr;
        }
    }
    reverse(res.begin(), res.end());
    return res;
}

问题 此功能仅适用于0到10000000(10000000vector<uint_fast8_t>{1,0,0,0,0,0,0,0})的数字。对于更大的数字,结果是疯狂的。例如,它吐出10000000000 + 123 + = 1012300000123。为什么会这样呢? 编辑1 ,有人问我有关该部门x.size() / sizeof(uint_fast8_t)的信息。据我所知,它返回对象的大小(以字节为单位)。我将其除以uint_fast8_t的大小,以获得向量中的元素数。看来运作良好。也许我误会了一些东西。

2 个答案:

答案 0 :(得分:1)

使用std::vector::resizestd::transform和适当的函数对象,可以更简单地表示这一点。

using Multiprecision = std::vector<uint_fast8_t>;

Multiprecision Add(Multiprecision x, Multiprecision y){

    auto common_size = std::max(x.size(), y.size());
    x.resize(common_size);
    y.resize(common_size);
    Multiprecision res(common_size);

    bool carry = false;
    std::transform(x.begin(), x.end(), y.begin(), res.begin(), [&carry](uint_fast8_t x, uint_fast8_t y){ 
        uint_fast8_t value = x + y + carry; 
        carry = (value >= 10);
        return value - (carry * 10);
    });

    return res;
}

答案 1 :(得分:0)

好,我已修复

感谢您指出我的代码中的一些错误。我是一个初学者,所以对我来说很重要。在您的帮助下,我设法解决了该问题。我认为问题在于预先确定向量的大小。这是工作代码:(我认为insert函数可以插入许多零而无需for循环。我会检查一下是否可以替换它)

vector<uint_fast8_t> Add(vector<uint_fast8_t > x, vector<uint_fast8_t > y) {
unsigned long x_size = x.size();
unsigned long y_size = y.size();
unsigned long res_size{};
uint_fast8_t curr = 0;
vector<uint_fast8_t > res{};

if(x_size>y_size){
    res_size = x_size;
    for(unsigned int i = 0; i < res_size - y_size; ++i){
        y.insert(y.begin(), uint_fast8_t(0));
    }
} else{
    res_size = y_size;
    for(unsigned int j = 0; j < res_size - x_size; ++j){
        x.insert(x.begin(), uint_fast8_t(0));
    }
}

reverse(x.begin(), x.end());
reverse(y.begin(), y.end());

for(unsigned int k = 0; k < res_size; ++k){
    curr += x[k] + y[k];
    if(curr >= 10){
        res.push_back(curr - uint_fast8_t(10));
        curr = 1;
        if(k == res_size -1){
            res.push_back(curr);
        }
    } else{
        res.push_back(curr);
        curr = 0;
    }
}

reverse(res.begin(), res.end());

return res;

}