2017-02-17 5 views
1

Ich habe begonnen, C++ zu lernen, und ich habe bereits ein Problem. Mein Programm wird 4 Argumente von der Kommandozeile nehmen. Diese Argumente werden die Dimensionen von zwei mehrdimensionalen Arrays sein. ./myprogram 2 2 2 2Segmentierungsfehler beim Versuch, in einem zweiten multidimensionalen Array zu lesen

Dies ist, wie I, die Elemente für das erste 2x2-Array eingeben:

1 2 3 4

Dann gebe ich die Elemente für das zweite Array: 5

Dann erhalte ich einen Fehler in das Terminal mit der Nummer Segmentation fault (core dumped)

Hier ist der Code, der von der Befehlszeile und Reifen liest s in den Array-Elementen zu lesen:

#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

Welcher Prozess führte zu der Entscheidung, 'new' und' double ** 'zu verwenden? Es lohnt sich, es erneut zu besuchen. –

+0

Ihre Verwendung von unformatierten Arrays und Zeigern macht Ihren Code schwer lesbar und fehleranfällig. Und dir fehlen alle 'delete []' s. Glücklicherweise können Sie statt roher Arrays 'std :: vector' verwenden. Es tut alles, was "neu" tut, aber auf eine bessere und sicherere Art und Weise. –

+0

Sie sollten statt der ebenfalls fehleranfälligen 'atoi'-Funktion auch' std :: stoi' verwenden. –

Antwort

1

Dieser:

A[i] = new double(aColCount); 

sollte sein:

A[i] = new double[aColCount]; 

und ähnliche anderswo. Ihr Code weist ein einzelnes double zu und initialisiert es mit dem Wert aColCount.

Verwandte Themen