Проблема с функцией, которая добавляет большие целые числа, хранящиеся в векторах - PullRequest
Новая
       91

Проблема с функцией, которая добавляет большие целые числа, хранящиеся в векторах

0 голосов
/ 30 октября 2018

Итак, я решил написать свой собственный тип данных 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 (10000000 равно vector<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 голосов
/ 31 октября 2018

Это можно выразить гораздо проще, используя std::vector::resize, std::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;
}
0 голосов
/ 31 октября 2018

Хорошо, я исправил это

Спасибо, что указали на некоторые ошибки в моем коде. Я начинающий, поэтому это важно для меня. С вашей помощью мне удалось решить проблему. Я думаю, что проблема заключалась в предопределении вектора. Вот рабочий код: (Я думаю, что функция вставки может вставить много нулей без цикла 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;

}

...