Ошибка Visual Studio C++ 0xC0000005: спутана с выделением памяти

Question

Я Java-разработчик, но теперь мне нужна библиотека C++, и я не очень опытен в этом языке. В частности, меня всегда смущают указатели, ссылки и распределение памяти. Это, я думаю, и есть причина, по которой я получаю ошибку в разрабатываемом классе матрицы.

Основной код:

#include "stdafx.h"
#include "matrix.cpp"

void matrixtest();

int main()
{
    matrixtest();
    system("pause");
    return 0;
}

void matrixtest()
{
    // let's try 3x3 matrices
    static const int arr1[] = {1, 2, 1, -1, 1, 2, 2, 3, -4};
    static const int arr2[] = {0, 2, 2, 1, -1, 0, 3, 2, -2};

    vector<int> values1(arr1, arr1 + sizeof(arr1) / sizeof(arr1[0]));
    vector<int> values2(arr2, arr2 + sizeof(arr2) / sizeof(arr2[0]));

    matrix A(values1, 3);
    matrix B(values2, 3);

    matrix sum = A + B;
    sum.show();

    matrix diff = A - B;
    diff.show();

    matrix prod = A * B;
    prod.show();
}

matrix.cpp интересный код:

matrix::matrix(vector<int> v, int r) : values(v), rows(r) {
    values = v;
    rows = r;
}

// [...]

matrix& matrix::operator += (const matrix& rhs) {
    matrix result = (*this) + rhs;
    (*this) = result;
    return *this;
}

matrix matrix::operator + (const matrix& rhs) {
    if (rows != rhs.rows || values.size() != rhs.values.size()) {
        throw std::length_error("Matrices shapes mismatch");
    }
    matrix result(values, rows);
    for (auto& i : values) {
        result.values[i] = this->values[i] + rhs.values[i];
    }
    return result;
}

// [...]

void matrix::show() {
    string delimiter = "";
    for (auto& i : values) {
        delimiter = "";
        for (auto j = 0; j < values.size()/rows; j++) {
            cout << delimiter << values[i * values.size()/rows + j];  // this is the line giving the error
            delimiter = ",";
        }
        std::cout << std::endl;
    }
}

полный matrix.hpp файл:

#ifndef matrix_hpp
#define matrix_hpp

class matrix {
    private:
        std::vector<int> values;        // size is found by values.size()
        int rows;                       // columns is values.size()/rows

    public:
        matrix(vector<int>, int); // base ctor.
        matrix(const matrix& rhs); // copy ctor.
        matrix& operator=(const matrix& rhs); // assign. ctor.
        ~matrix(); // dtor.
        int& operator () (int row, int column);
        const int& operator () (int row, int column) const;     
        matrix operator + (int scalar) const;
        matrix operator - (int scalar) const;
        matrix operator * (int scalar) const;
        matrix& operator += (int scalar);
        matrix& operator -= (int scalar);
        matrix& operator *= (int scalar);

        matrix operator + (const matrix&);
        matrix operator - (const matrix&);
        matrix operator * (const matrix&);
        matrix& operator += (const matrix&);
        matrix& operator *= (const matrix&);

        // should be private ??
        void reshape(int newRows, int newColumns);
        void show(); //used for dev. only
        void range(int start, int defaultStep = 1);
        void fill(int value);
        void randint(int lowerBound, int upperBound);
};

#endif /* CMatrix_hpp */

Этот класс основан на примере, приведенном в примере матрицы.

В сообщении об ошибке указано "0xC0000005: место чтения нарушения доступа 0x5820A694". Поэтому я предполагаю, что распределение памяти неправильное и / или существует массив out of bounds, и / или я работаю с операторами '&'.

Изменить: я получаю следующий след:

• это 0x00dffe24 {значения ={размер =9 } строк =3 } матрица *

Итак, матрица существует, но по какой-то причине я получаю ошибку.

Answer 1

Цикл for, который вы используете, перед тем, который вызывает ошибку

for (auto& i : values)

является основанным на диапазоне для цикла. При этом вы получите значения, присутствующие в векторе (values).

Но на основе написанной вами логики вы хотите получить индекс, который представляет строку, с которой вы работаете. Вы должны пойти для нормального цикла.

for(int i =0; i<rows; ++i)