Некорректная работа класса

[InKo1]

функция length() не работает внутри копирующего конструктора, некорректно работает оператор +

#include<iostream>

using namespace std;

class veryLongInt
    {
        public:
            veryLongInt();
            veryLongInt(int x);
            veryLongInt(const veryLongInt& s);
            int length() const;
            void out() const
                {
                    for(int i = 0; i < 20; i++)
                        cout << m[i];
                }
            int operator[](int i) const
                {
                    return m[i];
                }
            int& operator[](int i)
                {
                    return m[i];
                }
            veryLongInt operator+(veryLongInt& s);
        private:
            int k;
            int m[19];
            void init()
                {
                    for(int i = 0; i < 20; i++)
                        m[i] = 0;
                    k = 0;
                }
    };

veryLongInt::veryLongInt()
    {
        init();
    }

veryLongInt::veryLongInt(int x)
    {
        init();
        int i = 19;
        do
            {
                m[i] += x % 10;
                i--;
                k++;
            } while((x /= 10) > 0);
    }

veryLongInt::veryLongInt(const veryLongInt& s)
    {
        init();
        k = s.length(); // возвращает 0
        for(int i = 19 - k + 1; i < 20; i++)
            m[i] = s[i];
    }

int veryLongInt::length() const
    {
        return k;
    }

veryLongInt veryLongInt::operator+(veryLongInt& s)
    {
        veryLongInt _new;
        int buf = 0;
        for(int i = 19; i >= 0; i--)
            {
                if(buf != 0)
                    {
                        _new[i] += buf;
                        buf = 0;
                    }
                _new[i] += s[i] + this -> m[i];
                if(_new[i] > 9)
                    {
                        buf = _new[i] - 10;
                        _new[i] -= 10;
                    }
            }
        return _new;
    }

int main()
    {
        veryLongInt x(32);
        veryLongInt y(12);
       // veryLongInt z = x + y; всё прекрасно работает
/* veryLongInt z;
z = x + y; перестаёт работать
cout << x[19]; должно вывести 2, а выводит 0, в вышестоящем случае выводит 2
*/
        return 0;
    }

[netrino]

У Вас в operator+ не вычисляется k для _new. То есть оно остаётся равным нулю.

[InKo1]

Я это знаю, поэтому пока никак не использую k в программе, проблема никак не в k

[netrino]

k здесь имеет непосредственное отношение к Вашей проблеме, взгляните

veryLongInt::veryLongInt(const veryLongInt& s)
    {
        init();
        k = s.length(); // возвращает 0
        for(int i = 19 - k + 1; i < 20; i++)
            m[i] = s[i];
    }

Так как k == 0, а начальное i равно 19 - 0 + 1 == 20, то цикл не выполнится ни единого разу, вследствие чего данные скопированы не будут. В случае

veryLongInt z = x + y;

работает, потому что, благодаря оптимизациям компилятора, копирующий конструктор в этом куске кода не вызывается, но в таком случае:

veryLongInt z;
z = x + y;

может быть вызван копирующий конструктор

[InKo1]

Блин, как сразу не дошло, спасибо большое

[InKo1]

функция length() работет некорректно даже в таком случае:

veryLongInt x(32);
veryLongInt y(x);

[netrino]

А у Вас массив недостаточных размеров для используемых индексов (так как массив размером 19, то максимальный индекс - 18). Измените размер массива на 20. По всей видимости, init y, вылезая за пределы массива m, затирает первое поле (int k) переменного x.

Надо заметить, что Ваш копирующий конструктор излишен в данном случае, стандартный справится ничуть не хуже (даже лучше). Тем не менее, если впоследствии планируется добавить под управление экземпляра класса внешние данные (например, выделенные в куче), то помимо копирующего конструктора необходимо также определить копирующее присваивание

veryLongInt &operator= (const veryLongInt& s)  {
// ...
}

[InKo1]

Спасибо, всё работает