malloc-Anomalie

[Metalmind]

Hallo Leute,

Ich habe einen merkwürdigen Fehler den ich mir nicht erklären kann, mein Code:

int *SerienNr2;

SerienNr2 = (int*) malloc(laenge*sizeof(int));

for(i=0;i<laenge;i++){

SerienNr2=SerienNr;

}

if(((strcmp(verfahren,"Bubblesort"))==0)&&((strcmp(kriterium,"Seriennummer"))==0)){ //Bubblesort für Seriennummer

Bubblesort(SerienNr2);

}

int *TeileNr2;

TeileNr2 = (int*) malloc(laenge*sizeof(int));

for(i=0;i<laenge;i++){

TeileNr2=TeileNr;

}

if(((strcmp(verfahren,"Bubblesort"))==0)&&((strcmp(kriterium,"Teilenummer"))==0)){ //Bubblesort für Seriennummer

Bubblesort(TeileNr2);

}

Die Ausgabe:

printf("%d %d %s %d %d %d %d\n",Nr,SerienNr,&Bezeichnung,TeileNr, Lagerbestand, SerienNr2, TeileNr2);

}

Soweit funktioniert alles, wenn ich nun aber die beiden Teile:

TeileNr2 = (int*) malloc(laenge*sizeof(int));

for(i=0;i<laenge;i++){

TeileNr2=TeileNr;

}

SerienNr2 = (int*) malloc(laenge*sizeof(int));

for(i=0;i<laenge;i++){

SerienNr2=SerienNr;

}

in die If-Funktionen (vor das "Bubblesort...") schreibe, damit der Speicher nur belegt wird falls dies wirklich nötig ist, bekomme ich keinen Compiler Fehler, aber das Programm stürzt ab sobald ich bei der Ausgabe angekommen bin.

Es funktioniert auch, wenn ich einfach die Ausgabe ",TeileNr" entferne.

Kann sich das irgendjemand hier erklären? Ich habe das jetzt wie oben gezeigt gestaltet, allerdings ist das einfach nur eine Hilfslösung, die, soweit ich es verstanden habe, viel zu viel Speicher frisst.

[Klotzkopp]

Du gibst SerienNr2 und TeileNr2 immer mit aus, auch wenn du dafür keinen Speicher reserviert hast. Das kann ja nicht funktionieren.

Du solltest übrigens deinen Code ordentlich einrücken und beim Posten hier im Forum in [CODE]Code-Tags

Und noch eins: Den Rückgabewert von malloc soll man nicht casten. Es ist überflüssig und kann unter bestimmten Bedingungen Fehler verdecken.

SerienNr2 = malloc(laenge*sizeof(int)); // ohne (int*)

einschließen, damit die Einrückung nicht verlorengeht.

[Metalmind]

Wenn ich also die Ausgabe der Variablen auch an ein If-Anweisung hänge, so dass sie nur ausgegeben wird wenn sie auch gebraucht wird müsste es trotzdem funktionieren?

Warum hat das Programm den Fehler eigendlich nur bei TeileNr2 und nicht auch bei der Ausgabe von SerienNr bemängelt?

Danke für deine Hilfe, und ab sofort werde ich mein Code in die Tags schreiben, das mit dem einrücken hat mich selbst schon gestört.

Gruß

Metalmind

[Klotzkopp]

Wenn ich also die Ausgabe der Variablen auch an ein If-Anweisung hänge, so dass sie nur ausgegeben wird wenn sie auch gebraucht wird müsste es trotzdem funktionieren?

Trotz was?

Warum hat das Programm den Fehler eigendlich nur bei TeileNr2 und nicht auch bei der Ausgabe von SerienNr bemängelt?

Das ist das, was der Standard "undefiniertes Verhalten" nennt. C und C++ sind voll davon. Im Prinzip kann alles passieren, wenn du solche Fehler machst. Kann sein, dass es abstürzt, kann sein, dass falsche Werte angezeigt werden, kann sein, dass das Programm trotzdem richtig läuft, kann sein, dass deine Festplatte formatiert wird (). Du kannst dich also nicht darauf verlassen, dass solcher Code abstürzt.