Eine Variable scheint den Zugriff aufzuheben, wenn ich sie in einem dynamischen Array (C++) verwende.

Question

Hier sind wir wieder einmal gute Leute im Internet.

Dies ist der Code Ich verwende:

//This is what is in the header file 
int *myArr[]; // A two-dimensional array representing holding the matrix data 

//This is what is in the definition file 
Matrix::Matrix(int n, int m) 
{ 
    myRows = n; 
    myColumns = m; 
    initialize(); 
} 

void Matrix::initialize() 
{ 
    *myArr = new int[myRows]; 

    for (int i=0; i < 3; i++)//Only set to 3 since myRows is acting crazy 
    { 
     myArr[i] = new int[myColumns]; 
    } 
} 

Aus irgendeinem Grund, wenn ich myRows Variable verwenden, um die myArr Array zu erstellen, scheint es nur um den Wert zu stoppen verweisen sie auf, bevor zeigt wurde.

Zum Beispiel gebe ich ihm den Wert 3 und nachdem der * myArr = new int [myRows] ausgeführt wurde ändert es den Wert von myRows zu 9834496, was ich nicht verstehe.

Stimmt die "neue" Variable oder etwas davon ab? Oder mache ich etwas falsch?

Oh, und da dies ein Schulpraxisprojekt ist (also werde ich dich nicht beschuldigen, wenn du nicht antwortest), würde ich eine Antwort lieber über Arbeitscode bevorzugen, damit ich wissen könnte, was ich falsch für zukünftige Projekte gemacht habe.

Answer 1

int *myArr[]; 

Das ist falsch! Sie müssen dem Compiler auch die Größe Ihres Zeigerarrays mitteilen. Wie wäre es, wenn Sie int a[] deklarieren. Sie weisen den Compiler an, ein Array von int unbekannter Größe zu erstellen, das in C++ nicht erlaubt ist. Deshalb können Sie das nicht tun.

Ich würde vorschlagen, Sie, dies zu tun:

int **myArr; 

void Matrix::initialize() 
{ 
    myArr = new int*[myRows]; //note int* here! 

    for (int i=0; i < myRows; i++) 
    { 
     myArr[i] = new int[myColumns]; 
    } 
} 

Das jetzt funktionieren sollte.

Answer 2

Sie sollten std :: vector <> verwenden. Es befasst sich mit allen Problemen der Speicherzuweisung und Freigabe. Und das ohne Bugs.

Und dann konzentrieren Sie sich auf die wirklichen Ziele Ihres Algorithmus. Nicht auf Speicherverwaltung :-)

typedef std::vector<int> Ints; 
typedef std::vector<Ints> Matrix; 
Matrix myArray; 

Answer 3

Versuchen Sie ersetzen:

*myArr = new int[myRows]; 

von

myArr = new int*[myRows]; 

Answer 4

Ich bin mir nicht sicher, ob Sie Projekt sind erfordert, dass Sie Multi-Level-Zeiger zu verwenden, Wenn Sie sich diesem Problem nicht anders nähern können, behandeln Sie einfach das mehrdimensionale Array als ein großes, flaches Array.

Das bedeutet, wenn Sie das Ende einer Zeile erreichen, wäre der Index danach das erste Element der nächsten Zeile. So könnte der Code aussehen:

// In this approach the double pointer int** 
// is replaced with just a simple int* 
int *myArr; 

// Here's your Matrix ctor. Note the use of the initializer list 
Matrix::Matrix(int n, int m) : myRows(n), myColumns(m) 
{ 
    initialize(); 
} 

void Matrix::initialize() 
{ 
    myArr = new int[myRows * myColumns]; 

    /* This loop is no longer needed since we're allocating 
     one big chunk at once. 
    for (int i=0; i < 3; i++)//Only set to 3 since myRows is acting crazy 
    { 
     myArr[i] = new int[myColumns]; 
    } 
    */ 
} 

// To retrieve stuff from your array 
// you would do something like this: 
int Matrix::operator() (const int x, const int y) 
{ 
    return myArr[x * myRows + y]; 
}