Question

调试断言失败！

程序：   档案：c：\ program files（x86）\ microsoft visual studio \ 2017 \ community \ vc \ tools \ msvc \ 14.15.26726 \ include \ vector   线：1742

表达：向量下标超出范围

有关程序如何引起断言的信息   失败，请参见有关断言的Visual C ++文档。

（按“重试”以调试应用程序）   Sandbox.exe触发了一个断点。

我正在尝试为另一个项目编写一个变量维度（如1类型以包含任何n×m矩阵）矩阵库，因此我想创建和修改“ 2D”数字数组。为此，我使用了img个中的std::vector个。

我了解该错误是指我尝试访问的索引超出范围。即尝试访问3x1数组的value [5] [8]。查看调试信息，Multiply函数会标记错误，并且问题本身可能与“ 2D向量” std::vector和data的分配有关，这可能是构造函数的错误，对吗？（但是，矩阵最初是正确创建的。.？）

这是我的代码（忽略缺少干净的nameapces / typedefs-重构时我会整理所有内容，我喜欢这样保留它，以便我知道所有内容在哪里/什么）

// Maths.h

out.data

// Maths.cpp

#pragma once
#include "ckpch.h" // C/C++ standard libraries
#include "Core.h" // API  (__declspec(im/export) = CK_API)

namespace Maths {

    struct CK_API Matrix {

        int Rows, Cols;

        std::vector<std::vector<float>> data;

        Matrix(int rows, int cols);

        Matrix(int rows, int cols, float *values);

        ~Matrix();

        Matrix Multiply(const Matrix& mat) const ;
        friend CK_API Matrix operator*(Matrix& left, const Matrix& right);

        friend CK_API std::ostream& operator<<(std::ostream& os, const Matrix& mat);
    };

}

// SandboxApp.cpp

 #include "ckpch.h"
 #include "Maths.h"
 #include "Log.h" // Loads log info

namespace Maths {

    Matrix::Matrix(int rows, int cols) {
        Rows = rows;
        Cols = cols;

        std::vector<std::vector<float>> data(rows, std::vector<float>(cols, 0.0f));
        data.resize(rows, std::vector<float>(cols, 0.0f));

    }

    Matrix::Matrix(int rows, int cols, float* values) {

        Rows = rows;
        Cols = cols;
        std::vector<std::vector<float>> data(rows, std::vector<float>(cols, 0.0f));
        data.resize(rows, std::vector<float>(cols, 0.0f));
        for (int i = 0; i < rows; i++) {
            for (int j = 0; j < cols; j++) {
                data[i][j] = values[j + i * cols];
            }
        }

    }

    Matrix::~Matrix() {
        this->data.clear();

    }

    Matrix Matrix::Multiply(const Matrix& mat) const {
        int inR1 = this->Rows; // Matrix 1 Rows
        int inR2 = mat.Rows;   // Matrix 2 Rows
        int inC1 = this->Cols; // Matrix 1 Columns 
        int inC2 = mat.Cols;   // Matrix 2 Columns

        // (n x m) * (n' x m') --> (n x m')

        int outR = this->Rows;
        int outC = mat.Cols;

        Matrix out = Matrix(outR, outC);

        if (this->Cols == mat.Rows) {
            for (int i = 0; i < inR1; i++) {
                for (int j = 0; j < inC2; j++) {
                    float sum = 0.0f;
                    for (int off = 0; off < inR1; off++) {
                        sum += this->data[off][j] * mat.data[i][off];
                    }
                    out.data[i][j] = sum;
                }
            }
            return out;
        } else {
            CK_WARN("Matrix 1 Column and Matrix 2 Row dimension mismatch! ({0} =/= {1})", Cols, mat.Rows);
        }
    }

    Matrix operator*(Matrix& left, const Matrix& right) { return left.Multiply(right); }

    std::ostream& operator<<(std::ostream& os, const Matrix& mat){

        os << mat.Rows << " x " << mat.Cols << " - Matrix: " << std::endl;

        for (int i = 0; i < mat.Rows; i++) {
            for (int j = 0; j < mat.Cols; j++) {
                os << mat.data[i][j] << ", ";
            }
            os << std::endl;
        }
        return os;
    }
}

由于调用了函数/方法等，我认为所有相关的代码都很有意义-我删除了其他所有内容（函数的重载，加法，减法等）

Answer 1

此行

std::vector<std::vector<float>> data(rows, std::vector<float>(cols, 0.0f));

创建一个名为data的新局部变量，该变量与您的属性Matrix :: data

同名

这称为阴影，非常糟糕，因为您对此本地所做的任何事情都不会触及该属性。

一个不错的编译器会警告您。

您能建议最好的解决方法吗？会只是删除那一行吗？

是的

如果使用std :: vector :: push_back（）代替赋值，则无需调整大小。对于非常大的数据集，这会有点慢。

std :: vector大小未设置且数据分配不正确->下标范围错误

1 个答案: