1
私はC++を学び始めましたが、問題は既にあります。私のプログラムはコマンドラインから4つの引数を受け取ります。これらの引数は、2つの多次元配列の次元になります。 ./myprogram 2 2 2 2
セカンダリの多次元配列を読み込もうとするとセグメンテーションフォルトが発生する
これは私が最初の2x2アレイの要素を入力する方法です。
1 2 3 4
その後、私は二番目の配列のための要素を入力します。 5
その後、私はエラーを取得します端末の説明Segmentation fault (core dumped)
ここでは、コマンドラインとタイヤから読み取るコードです配列要素に読み取るための:
#include <iostream>
#include <stdlib.h>
using namespace std;
void readArr(int, int, double**);
int main(int argc, char* argv[]) {
int aRowCount = atoi(argv[1]);
int aColCount = atoi(argv[2]);
int bRowCount = atoi(argv[3]);
int bColCount = atoi(argv[4]);
std::cout << "Input accepted" << endl;
if(aColCount != bRowCount) {
std::cerr << "Col. Count of the first must match Row. Count of the second matrix." << endl;
return 1;
}
double **A = new double*[aRowCount];
for(int i = 0; i < aRowCount; i++){
A[i] = new double(aColCount);
}
std::cout << "allocating A" << endl;
double **B = new double*[bRowCount];
for(int j = 0; j < bRowCount; j++){
A[j] = new double(bColCount);
}
std::cout << "allocating B" << endl;
double **C = new double*[aRowCount];
for(int k = 0; k < aRowCount; k++) {
C[k] = new double(bColCount);
}
std::cout << "Reading in A" << endl;
readArr(aRowCount, aColCount, A);
std::cout << "Reading in B" << endl;
readArr(bRowCount, bColCount, B);
return 0;
}
void readArr(int rowCount, int colCount, double **array) {
for(int i = 0; i < rowCount; i++) {
for(int j = 0; j < colCount; j++) {
std::cin >> array[i][j];
}
}
}
「new」と「double **」を使用する決定につながったプロセスは何ですか?再訪する価値はある。 –
生の配列とポインタを使用すると、コードの読み込みやエラーが発生しにくくなります。そして、あなたは 'delete []'をすべて欠いています。幸いにも、生の配列の代わりに 'std :: vector'を使うことができます。それは 'new []'がするすべてのことをより良い、より安全な方法で行います。 –
同じようにエラーが起こりやすい 'atoi'関数の代わりに' std :: stoi'も使うべきです。 –