2

我的 C++ 程序（使用矩阵处理一些计算，请参阅下面的文件）崩溃并显示以下消息：

*** 检测到 glibc *** ./matrix: munmap_chunk(): 无效指针: 0x08bfd068 ***

其次是回溯和内存映射。当我第一次在 Matrix 类中调用 set-Method 时会发生这种情况 - 但我不知道我做错了什么......非常感谢每一个帮助（以及改进我的代码的一般提示）！

数组.cpp

#include <iostream>
#include <stdlib.h>
#include <string.h>
#include "Array.h"

using namespace std;

Array::Array(){
    Array(10);
}

Array::Array(int size){
    data = new int[size];
    memset(data, 0, sizeof(data));
    length = size;
}

Array::~Array(){
    delete [] data;
}

void Array::set(int pos, int value){
    if(pos < 0) pos = 0;
    if(pos >= length) pos = length-1;
    *(data + pos) = value;
}

int Array::get(int pos){
    if(pos < 0) pos = 0;
    if(pos >= length) pos = length-1;
    return *(data + pos);
}

void Array::print(){
    for(int i = 0; i < length; i++){
        cout << *(data + i) << "\t";
    }
    cout << endl;
    return;
}

/*works only for arrays of length 9*/
int Array::find_max(int data[]){
    int max = data[0];

    for(int i = 1; i < 9; i++){
        if(data[i] > max) max = data[i];
    }

    return max;
}

矩阵.h

#ifndef MATRIX_H
#define MATRIX_H
#include "Array.h"

class Matrix{

    private:
        Array * data;
        int height;
        int width;

    public:
        Matrix();
        ...
};

#endif

矩阵.cpp

#include <iostream>
#include <stdlib.h>
#include <string.h>
#include "Matrix.h"

using namespace std;

Matrix::Matrix(){
    Matrix(10, 10);
}

Matrix::Matrix(int h, int w){
    height = h;
    width = w;

    data = new Array(height);

    for(int i = 0; i < height; i++){
        *(data + i) = *(new Array(width));
    }

}

Matrix::~Matrix(){
    for(int i = 0; i < height; i++){
        Array * row = (data + i);
        delete row;
    }

    delete data;
}

void Matrix::set(int h, int w, int value){
    Array row = *(data + h);
    row.set(w, value);
}

...

主文件

#include <iostream>
#include <stdlib.h>
#include <string.h>
#include "Array.h"
#include "Matrix.h"

using namespace std;

int main(int argc, char** argv){
    if(argc != 3){
        cout << "usage: " << argv[0] << " <m> x <n>" << endl;
        exit(-1);
    }

    int m = atoi(argv[1]);
    int n = atoi(argv[2]);

    Matrix * myMatrix = new Matrix(m, n);

    /*fill matrix randomly*/
    int guess, minus;
    srand(time(NULL));

    for(int r = 0; r < m; r++){
        for(int c = 0; c < n; c++){
            guess = rand() % 1001;
            minus = rand() % 2;

            if(minus == 0) guess *= -1;
            std::cout << " set " << c << ", " << r << " " << guess << std::endl;
            myMatrix->set(r, c, guess);

        }
    }

    ...

    delete myMatrix;
    ...

    return 0;
}
4

1 回答 1

3 
Matrix::Matrix(){
    Matrix(10, 10);
}

这不是你认为的那样。它只是创建一个临时矩阵并丢弃它。这意味着正在构造的矩阵的高度、宽度和数据没有在那里初始化。

我认为您的意图是做与Matrix::Matrix(int h, int w). 但是，您不能那样做。

如果你有C++11支持，你可以使用委托构造来做同样的事情。

Matrix::Matrix() :  // <- Notice the colon
    Matrix(10, 10)
{
}

如果您没有 C++11 支持，您必须手动完成

Matrix::Matrix(){
    height = 10;
    width = 10;

    data = new Array(height);

    for(int i = 0; i < height; i++){
        *(data + i) = *(new Array(width));
    }

}

或者您可以创建另一个执行常见操作的成员函数并从两个构造函数中调用它。

另一个问题

Array row = *(data + h);
row.set(w, value);

您正在将数组行复制到一个名为row. 您没有复制构造函数。所以直接复制指针的值。当函数返回row被破坏时，意味着所指向的内存row.data也是deleted如此，这意味着矩阵中的数据也会受到影响，因为它指向的是同一个东西。见三法则。

应该

Array* row = (data + h);
row->set(w, value);
于 2013-05-05T20:22:17.817 回答