1

我对 C++ 编码很陌生，所以如果这个问题有一个非常明显的简单答案，我深表歉意。我正在尝试创建一个从 -1 到 1 的随机数矩阵。这是我正在使用的两个函数：

#include <iostream>
#include "matrix_fill_random.cpp"
using namespace std;

int main(int argc, int argv[]){

int n1, n2, n3;

if (argc != 4) {
    cerr << "This program requires 3 argument!" <<endl;
    return 1;
}
else{
    n1 = argv[1];
    n2 = argv[2];
    n3 = argv[3];

    double** a;

    matrix_fill_random(n1, n2, a);

    return 0;
}
}

和

#include <iostream>
using namespace std;

int matrix_fill_random(int n1, int n2, double** a){
for (int i=0; i<n1; i++){
    for (int j=0; j<n2; j++){
        double num = rand() % 2 - 1;
        a[i][j]=num;
    }
}
return 0;
}

最终，我试图创建两个矩阵，然后将它们相乘，因此 n1、n2 和 n3 代表两个矩阵的行和列，但这现在还不是很重要。我认为错误可能在于我如何声明变量或将它们传递给其他函数，但我不完全确定。

我觉得如果我能理解创建其中一个矩阵的原理，那么这将转化为我需要使用的其他函数。

4

4 回答 4

1 
double** a;

您还没有为指针分配内存，因此每次使用取消引用它时都会得到未定义的行为operator []。

a您应该在通过函数之前分配一次...

double** a = new double*[n1];

并再次在函数的 for 循环中：

for (int i = 0; i < n1; i++)
{
    a[i] = new double[n2];
    for (int j = 0; j < n2; j++)
    {
        double num = rand() % 2 - 1;
        a[i][j] = num;
    }
}

delete[]但是一旦你完成使用它，不要忘记指针。您也应该delete在 for 循环内分配其他指针。

当然，这一切都可以通过使用来避免std::vector。这是您安装了标准库的程序：

std::vector<std::vector<double>> a(n1, std::vector<double>(n2));

int matrix_fill_random(std::vector<std::vector<double>> a)
{
    for (int i = 0; i < a.size(); ++i)
    {
        for (int j = 0; j < a[i].size(); ++j)
        {
            double num = rand() % 2 - 1;
            a[i][j] = num;
        }
    }
    return 0;
}
于 2013-09-29T22:15:05.603 回答
1

为 分配内存a。您还没有为 a 分配内存。将声明double** a;更改为

  double** a = new double[n1];

并按如下方式更改循环

  for (int i = 0; i < n1; i++)
  {
    //Every row will be of size = number of columns.
    a[i] = new double[n2];
    for (int j = 0; j < n2; j++)
    {
        double num = rand() % 2 - 1;
        a[i][j] = num;
    }
  }
于 2013-09-29T22:16:28.740 回答
0 
double** a;
matrix_fill_random(n1, n2, a);

将未初始化的指针传递a给函数，该函数尝试初始化二维数组的元素：

a[i][j]=num;

它调用未定义的行为。最简单的解决方案是分配一个足够大的内存块来保存矩阵：

double* a = new double[rows * cols];
matrix_fill_random(n1, n2, a);
delete[] a;
a = NULL;

...

// accessing element a[i][j] in the function's body:
a[i*cols + j] = num;

但最合理的解决方案是使用std::vector而不是 C 样式的数组。

于 2013-09-29T22:13:40.603 回答
0

在将内存a传递给matrix_fill_random(). 

double** a = new double[n1];

由于您使用的是 C++，因此您应该考虑使用向量或其他模板容器。

于 2013-09-29T22:19:43.497 回答