2017-02-17 7 views
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]; 
     } 
    } 
} 
+1

「new」と「double **」を使用する決定につながったプロセスは何ですか?再訪する価値はある。 –

+0

生の配列とポインタを使用すると、コードの読み込みやエラーが発生しにくくなります。そして、あなたは 'delete []'をすべて欠いています。幸いにも、生の配列の代わりに 'std :: vector'を使うことができます。それは 'new []'がするすべてのことをより良い、より安全な方法で行います。 –

+0

同じようにエラーが起こりやすい 'atoi'関数の代わりに' std :: stoi'も使うべきです。 –

答えて

1

これ:

A[i] = new double(aColCount); 

にすべきである:

A[i] = new double[aColCount]; 

と同様の他の場所。あなたのコードはdouble doubleを割り当て、aColCountの値で初期化します。

関連する問題