Doppelpointer: Speicherplatz freimachen mit free, nur wie?

miller_m

Bashar schrieb:

free prüft intern selbst, ob der übergebene Zeiger ein Nullpointer ist. Zitat aus dem C-Standard: «If ptr is a null pointer, no
action occurs.»

wusste ich nicht

Bashar schrieb:

Das Nullsetzen des Zeigers danach ist Geschmackssache, IMHO unnötig, da es Fehler verschleiern kann.

in wiefern ?

Bashar

Doppeltes freigeben wird so nicht gefunden, dürfte aber ziemlich sicher auf einen Logikfehler im Programm zurückzuführen sein. Also ist es besser, in der Entwicklungszeit im Fall des Falles einen Absturz zu riskieren und dann den Fehler zu beseitigen, als ihn ewig unerkannt mitzuschleppen.

ga_un

Weiter Unklarheit:

Kann ich den array so in einen neuen reinkopieren?

NeuArrayDoppelpointer = (int**)malloc(sizeof(int*)*2);
memcpy(NeuArrayDoppelpointer,array,sizeof(int)*10);

oder brauch ich ne Schleife?

for(int i=0;i<2;++i){
  memcpy(NeuArrayDoppelpointer[i],array[i],sizeof(int)*10);
}

Shade Of Mine

miller_m schrieb:

Bashar schrieb:

Das Nullsetzen des Zeigers danach ist Geschmackssache, IMHO unnötig, da es Fehler verschleiern kann.

in wiefern ?

Wenn du einen gelöschten Zeiger nochmals löscht, hast du einen Fehler im Programmablauf. Wenn du den Zeiger auf auf 0 gesetzt hast, merkst du es nicht. Wenn du es nicht auf 0 gesetzt hast, bekommst du ne Access Violation (oder etwas ähnliches) - du bemerkst den Fehler also und kannst ihn beseitigen. Und da ein Fehler selten alleine kommt, wirst du neben dem fehlerhaften free() sicher auch noch andere Sachen entdecken die dann nicht ganz so sein sollten.

Shade Of Mine

ga_un schrieb:

Kann ich den array so in einen neuen reinkopieren?

NeuArrayDoppelpointer = (int**)malloc(sizeof(int*)*2);
memcpy(NeuArrayDoppelpointer,array,sizeof(int)*10);

oder brauch ich ne Schleife?

for(int i=0;i<2;++i){
  memcpy(NeuArrayDoppelpointer[i],array[i],sizeof(int)*10);
}

Du hast nicht genug speicher... NeuArrayDoppelpointer hat nur Speicher für 2 int* du willst aber zwei int[10] Arrays reinkopieren - das kann nicht gut gehen.

Beide Codestücke sind somit falsch.

ga_un

Aber so paßt's, oder???

for(int i=0;i<2;++i){
  NeuArrayDoppelpointer[i] = (int*) calloc(10,sizeof(int));
  memcpy(NeuArrayDoppelpointer[i],array[i],sizeof(int)*10);
}

miller_m

Shade Of Mine schrieb:

miller_m schrieb:

Bashar schrieb:

Das Nullsetzen des Zeigers danach ist Geschmackssache, IMHO unnötig, da es Fehler verschleiern kann.

in wiefern ?

Wenn du einen gelöschten Zeiger nochmals löscht, hast du einen Fehler im Programmablauf. Wenn du den Zeiger auf auf 0 gesetzt hast, merkst du es nicht. Wenn du es nicht auf 0 gesetzt hast, bekommst du ne Access Violation (oder etwas ähnliches) - du bemerkst den Fehler also und kannst ihn beseitigen. Und da ein Fehler selten alleine kommt, wirst du neben dem fehlerhaften free() sicher auch noch andere Sachen entdecken die dann nicht ganz so sein sollten.

somit wäre es vollkommen ausreichend "free(array);" zuschreiben. ?!?

Shade Of Mine

ga_un schrieb:

Aber so paßt's, oder???

for(int i=0;i<2;++i){
  NeuArrayDoppelpointer[i] = (int*) calloc(10,sizeof(int));
  memcpy(NeuArrayDoppelpointer[i],array[i],sizeof(int)*10);
}

ja, allerdings wäre malloc statt calloc besser - da du den Inhalt ja nicht 0 setzen musst, da er ja sowieso gleich überschrieben wird.

@miller_m:
ja, free(array); finde ich schöner als
if(array) { free(array); array=0; }

Die Schleife für array[i] muss natürlich trotzdem drinnen bleiben...

miller_m

@shade of mine und bashar
ich danke euch, wieder was gelernt.

k1ro

vielleicht ne blöde Zwischenfrage,
aber gehört das nich eher in ANSI C ?