Visual Studio C++ 0xC0000005错误排查：矩阵类内存访问违规

阿华AIGC实验室

2026-5-15

排查你的C++矩阵类内存访问错误（0xC0000005）

嘿，咱们来拆解下你的C++矩阵类为啥会触发0xC0000005访问违规错误。根源很明确：你在两个核心函数里混淆了遍历逻辑，把元素值当成了索引/行号来用，导致访问了vector的非法内存地址。下面一步步说问题和修复方案：

1. `operator+`函数的索引误用

先看这段加法重载的代码：

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;
}

这里的for (auto& i : values)是在遍历values容器里的实际元素值，比如你的测试矩阵里有-1、-4这类负数元素。当i=-1时，result.values[-1]就是在访问vector的负索引，这直接就触发内存访问违规了——vector根本不允许这种操作。

修复方法：

改成按索引遍历容器，而不是元素值：

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);
    // 用索引遍历，idx从0到values.size()-1
    for (size_t idx = 0; idx < values.size(); ++idx) {
        result.values[idx] = this->values[idx] + rhs.values[idx];
    }
    return result;
}

2. `show`函数的行遍历逻辑错误

触发错误的核心就在show函数里：

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]; 
            delimiter = ",";
        }
        std::cout << std::endl;
    }
}

同样的坑：for (auto& i : values)遍历的是矩阵里的每个元素值，不是你需要的行号。比如当i=-1时，i * 3 + j会得到负数，直接越界访问内存，这就是你看到的0xC0000005错误的直接原因。

修复方法：

应该遍历行号（从0到rows-1），然后计算每行对应的元素索引：

void matrix::show() {
    int cols = values.size() / rows; // 先算出列数
    string delimiter = "";
    // 遍历行号，从第0行到第rows-1行
    for (int row = 0; row < rows; ++row) {
        delimiter = "";
        for (int col = 0; col < cols; ++col) {
            // 正确的索引计算：行号*列数 + 列号
            cout << delimiter << values[row * cols + col];
            delimiter = ",";
        }
        std::cout << std::endl;
    }
}

3. 几个额外的优化建议

去掉构造函数的冗余代码：你的构造函数里初始化列表已经完成了values和rows的赋值，函数体里的重复赋值完全没必要，还可以用std::move避免vector的拷贝，提升性能：
```
matrix::matrix(vector<int> v, int r) : values(std::move(v)), rows(r) {
    // 删掉这里的 values = v; 和 rows = r;
}
```

优化运算符重载的效率：你现在的operator+=是先调用operator+创建临时对象，再赋值给当前对象，其实可以直接在当前对象上修改，效率更高，还能复用代码：

matrix& matrix::operator += (const matrix& rhs) {
    if (rows != rhs.rows || values.size() != rhs.values.size()) {
        throw std::length_error("Matrices shapes mismatch");
    }
    for (size_t idx = 0; idx < values.size(); ++idx) {
        values[idx] += rhs.values[idx];
    }
    return *this;
}
// 然后让operator+复用operator+=，代码更简洁
matrix matrix::operator + (const matrix& rhs) {
    matrix result = *this;
    result += rhs;
    return result;
}