ich habe ein kleines Problem und suche immer noch des Rätsels Lösung. Und zwar erstelle ich vier Arrays aus einer Matrix mat[r][c] mit r für die Reihe und C für die Spalte. Jedes Array soll dann die Spalte dieser Matrix in sich haben. Jedoch wirft mir der Abruf dieses Arrays nur die letzte eingetragene Spalte aus.

Zeig das mal am Code:

void Matrix::sort_matrix(){
for (int r = 0 ; r < row ; r++){
SpalteA[r]= mat[r][0];
//cout << A[r]<< " " << mat[r][0]<<"\n";
}
for (int s = 0 ; s < row ; s++){
SpalteB[s]= mat[s][1];
//cout << B[s]<< " " << mat[s][1]<<"\n";
}
for (int t = 0 ; t < row ; t++){
SpalteC[t]= mat[t][2];
//cout << C[t]<< " " << mat[t][2]<<"\n";
}
for (int v = 0 ; v < row ; v++)
cout << SpalteA[v] << " ";
cout << "\n";
for (int u = 0 ; u < row ; u++){
SpalteD[u]= mat[u][3];
//cout << D[u]<< " " << mat[u][3]<<"\n";
}
}

So wenn jetzt die Spalte A nach der dritten Zuweisung abrufe, zeigt er mir aber die SpalteC. Ich denke, das irgendetwas mit der Zuweisung falsch läuft!

Kann mir jemand helfen und mir meinen Fehler aufdecken?!

Gruss und Dank Jimmie

Matrix(int r = 0, int c = 0) {
row = r;
col = c;
mat = new unsigned long* [row];
for(int j = 0; j < row ; j++)
mat[j] = new unsigned long[col];
for(int i = 0; i < row ; i++)
for(int j = 0; j < col ; j++)
mat[i][j] = 0;
}

Matrix(const string &path) {
int q=0;
ifstream fin(path.c_str());
ifstream infile(path.c_str());
if (!fin) // File Error
cout << " Sorry the file doesnot exist ";
else {
while (infile>>iwert) q++;
//cout<< q;
infile.close();
row=q/4;
col=4;
if (path == "helnum.txt"){
mat = new unsigned long* [row];
for(int j = 0; j < row ; j++)
mat[j] = new unsigned long[row];
for (int c = 0 ; c < col ; c++)
for (int r = 0 ; r < row ; r++)
fin >> mat[r][c];
fin.close();
} else {
mat = new unsigned long* [row];
for(int j = 0; j < row ; j++)
mat[j] = new unsigned long[col];
for (int r = 0 ; r < row ; r++)
for (int c = 0 ; c < col ; c++)
fin >> mat[r][c];
fin.close();
}
}
}

und Spalte A-D unsigned long SpalteA[], ..Spalte D[];

und Spalte A-D unsigned long SpalteA[], ..Spalte D[];

Wie groß?

nur wenn ich diese Matrix zuweise funzt es nicht

?

hast du irgendwo sowas stehen wie

Matrix a(5,7);
...//mach irgendwas mit a
Matrix b = a;

?

dann zeig doch mal bitte deinen copy ctor.

Die Matrix wird aus einer Datei gelesen:
Beispiel:
17 395 412 987
17 395 412 1019
678 395 567 352
678 395 567 353

Die Matrix beinhaltet dann die Werte in mat[r][c]
wenn ich das cout << A[r]<< " " << mat[r][0]<<"\n"; ausführe direkt hinter der Zuweisung, erhalte ich auch die richtigen Angaben.
Beispiel
A[0]=17 mat[0][0]=17
A[1]=17 mat[1][0]=17
A[2]=678 mat[2][0]=678
and so on..

Die Werte scheinen aber nachdem ich SpalteB auffülle, auch die Werte aus der SpalteA zu überschreiben.
Denn danach ist SpalteA mit 395,395, 395... etc gefüllt.
Deswegen denke ich ist es was mit der Zuordnung.

also erstmal ist mat row*row groß, was so wohl nicht gewollt ist. eher col*row.

zweitens iterierst du in der schleife als erstes über row und dann über col. in dem else zweig iterierst du als erstes über col und dann über row. ich würde das mal konsistent machen.

was es damit auf sich?

ifstream fin(path.c_str()); 
ifstream infile(path.c_str());

warum zweimal?

was machst du hier?

while (infile>>iwert) q++;

ok du zählst die einträge, aber warum verwirfst du dann die alten werte anstatt sie in einem vector unterzubringen? um sie dann zb. schneller in die matrix packen zu können.

warum arbeitest du nicht mit vectoren? wäre doch einfacher oder nicht?

Die Matrix ist ja eigentlich nur eindimensional, so habe ich sie konstruiert. Und so funktioniert sie auch.

Warum ich keine Vektoren nehme? Hmm.. meinst du, das die Zuweisungsprobleme sich dann auflösen werden? Oder meintest du, das ich die Matrix als Vektor umschreiben soll?? Das ist leider nicht so einfach, der Code ist leider nicht so klein, das das so auf die schnelle geht!

Kann es mal mit den Vektoren versuchen, denke aber nicht, das dadurch das Zuweisungsproblem sich verändern wird....

naja, ein vektor ist im allgemeinem etwas einfacher zu handhaben als ein array und der overhead ist wirklich gering.
aber ich vertstehe das man keine lust hat seinen code umzuschreiben, wenn er noch nicht richtig funzt..aber du könntest es ja bei einem refactoring machen..

hast du mit nem debugger in mat reingeschaut? ansosnten zeig doch mal den code wo du die matrix verwendest..ggf. finden wir da den fehler..

undefiniert, packe da so viel Werte rein, wie nötig, da die Matrix immer unterschiedlich gross ist.
Hat auch bei anderen Arrays funktioniert, nur wenn ich diese Matrix zuweise funzt es nicht!

Und wo setzt du die Größe fest? (bei "normalen" Arrays muß die Größe bereits zur Compilezeit angegeben werden - und eine Definition long Spalte[]; ohne Größenangabe ist eigentlich nicht zulässig)