Struktelement in Funktion ändern

[Max F. (mexxe)]

Hallo Gemeinde,
Ich habe ein Problem mit meinem Programm. Ich möchte einem Element eines 
Structs innerhalb einer Funktion ein Feld übergeben und dieses ausgeben 
lassen. Eigentlich arbeite ich da schon mit Feldern, weshalb ich den 
Fehler nicht sehe...Kann mir da jemand behilflich sein?

struct node {                       // Zur Übung ist bisher nur ein Element
  char* vorname;              // drin, weitere sollen aber folgen
};


void eingabe_vorname(node *temp) {
  char *f1, *f2, c;
  int pos = 0;

  
  f1 = new char [100];

  c = getchar();

  while (c != '\n') {
    f1[pos] = c;
    pos++;
    c = getchar();
  }

  f1[pos] = '\0';
  f2 = new char[pos];
  
  pos = 0;

  while (f1[pos] != '\0') {        
    f2[pos] = f1[pos];
    pos++;
  }

  delete [] f1;
  temp->vorname = f2;      // Hier vermute ich den Fehler
  pos = 0;
  delete [] f2;

/*  while (*temp->vorname[pos] != '\0') {         // Funktioniert nicht
    putchar(*temp->vorname[pos]);
    pos++;
  }
*/

}


int main () {
        int pos = 0;
  node *temp;
  temp = new node;
  printf("Bitte geben Sie hier Ihren Vornamen ein: ");
  eingabe_vorname(temp);

  while (temp->vorname[pos] != '\0') {    // Funktioniert nicht
    putchar(temp->vorname[pos]);
    pos++;
  }

  delete [] temp;

  return 0;
}

[Hans Ulli K. (Gast)]

Max F. schrieb:
>

> void eingabe_vorname(node *temp) {
>   char *f1, *f2, c;
>   int pos = 0;
> ...
>   delete [] f1;
>   temp->vorname = f2;
>   pos = 0;
>   delete [] f2;
> ...
>

Fällt dir was auf ??

[Peter II (Gast)]

temp->vorname = f2;
  pos = 0;
  delete [] f2;

du gibst ja gleich den Speicher wieder frei.

Generell solltest du ein paar mehr string funtionen verwenden.

[Max F. (mexxe)]

Peter II schrieb:
>

> temp->vorname = f2;
>   pos = 0;
>   delete [] f2;
> 
>

>
>
> du gibst ja gleich den Speicher wieder frei.
>
> Generell solltest du ein paar mehr string funtionen verwenden.

Also gebe ich den Speicher frei, wo meine Elemente drin stehen...Er kann 
also keine Elemente ausgeben, die logischerweise ja nicht da sind, oder 
sehe ich das falsch?

Edit fragt: Warum bekomme ich da eigentlich keinen 
Speicherzugriffsfehler?

Viel wichtiger: Wenn ich dann den Speicher mittels

delete [] temp;

wieder freigebe, wird dann auch der Speicher für f2 freigegeben? Und 
wenn nein, wie mache ich das?
Meine ursprüngliche Aufgabe ist ein Feld von (Stru ct-)Zeigern zu 
initialisieren, die dem Nullzeiger zugewiesen sind oder eben auf den 
struct node zeigen.
Ich frage deshalb, weil ich in einer Teilaufgabe den Struct auch wieder 
löschen muss und dem Zeiger die 0 zuweisen muss und da Bedenken zwecks 
dem angeforderten Speicher für f2 habe.

[Arc N. (arc)]

Max F. schrieb:
> Edit fragt: Warum bekomme ich da eigentlich keinen
> Speicherzugriffsfehler?

Solange nichts anderes mit dem Speicher angestellt wird, "funktioniert" 
das...
Nennt sich allerdings Use-After-Free oder Dangling-Pointer und ist eine 
klassische Sicherheitslücke 
https://en.wikipedia.org/wiki/Dangling_pointer
von denen noch ein paar andere im Quelltext schlummern...

> Viel wichtiger: Wenn ich dann den Speicher mittels
>

> delete [] temp;
>

> wieder freigebe, wird dann auch der Speicher für f2 freigegeben?

Nein.

> Und wenn nein, wie mache ich das?

Erst delete [] temp->vorname, dann delete [] temp oder im Destructor von 
node (Anhand der Mischung ist nicht so ganz eindeutig, ob das jetzt C 
oder C++ werden soll und ob und wenn ja welche Funktionen aus den 
dazugehörigen Libs verwendet werden sollen/dürfen)

[Max F. (mexxe)]

Arc N. schrieb:
> Max F. schrieb:
>> Edit fragt: Warum bekomme ich da eigentlich keinen
>> Speicherzugriffsfehler?
>
> Solange nichts anderes mit dem Speicher angestellt wird, "funktioniert"
> das...
> Nennt sich allerdings Use-After-Free oder Dangling-Pointer und ist eine
> klassische Sicherheitslücke
> https://en.wikipedia.org/wiki/Dangling_pointer
> von denen noch ein paar andere im Quelltext schlummern...
>
>> Viel wichtiger: Wenn ich dann den Speicher mittels
>>

>> delete [] temp;
>>

>> wieder freigebe, wird dann auch der Speicher für f2 freigegeben?
>
> Nein.
>
>> Und wenn nein, wie mache ich das?
>
> Erst delete [] temp->vorname, dann delete [] temp oder im Destructor von
> node (Anhand der Mischung ist nicht so ganz eindeutig, ob das jetzt C
> oder C++ werden soll und ob und wenn ja welche Funktionen aus den
> dazugehörigen Libs verwendet werden sollen/dürfen)

Es soll C++ werden ;)
Danke, hab es jetzt auch verstanden! NEW liefert ja nur einen Zeiger und 
wenn ich den Zeiger jetzt temp->vorname zuordne, kann ich auch darüber 
den Speicher freigeben...!  :)
Ich darf lediglich stdio.h benutzen^^

[Hans Ulli K. (Gast)]

Max F. schrieb:
> Danke, hab es jetzt auch verstanden! NEW liefert ja nur einen Zeiger und
> wenn ich den Zeiger jetzt temp->vorname zuordne, kann ich auch darüber
> den Speicher freigeben...!  :)

Du kannst den Zeiger so oft kopieren wie du willst.

Das Problem bei Zeigern ist, das die Kopien nicht wissen, das der 
original Speicherbereich nicht mehr existiert !!

Ich glaube es gab mal die Idee das es mit einem "Zeiger auf Zeiger" zu 
lösen. In C++ ginge das mit einer Referenz.
Aber wenn diese(r) wieder "aufgelöst" wird, kommt man zum obigen Problem 
wieder zurück.

[Arc N. (arc)]

Hans Ulli K. schrieb:
> Ich glaube es gab mal die Idee das es mit einem "Zeiger auf Zeiger" zu
> lösen.

unique_ptr<>, shared_ptr<>, weak_ptr<> und Konsorten in neueren 
C++-Varianten. In Rust ist es sauberer gelöst, aber auch komplizierter 
(Ownership, Borrowing und Lifetimes)

> In C++ ginge das mit einer Referenz.

Genauso gut...

char* str = new char[100];
char& str_ref = *str;
...
delete [] str;
...
return str_ref;

[Rolf Magnus (Gast)]

Max F. schrieb:
> (Anhand der Mischung ist nicht so ganz eindeutig, ob das jetzt C
> oder C++ werden soll und ob und wenn ja welche Funktionen aus den
> dazugehörigen Libs verwendet werden sollen/dürfen)
>
> Es soll C++ werden ;)

Der Code ist allerdings komplett im C-Stil gehalten. Bis auf die 
Verwendung von new und delete statt malloc und free ist das reiner 
C-Code.

> Ich darf lediglich stdio.h benutzen^^

Ist nicht unbedingt sehr sinnvoll in C++.

[Daniel A. (daniel-a)]

Arc N. schrieb:
>> In C++ ginge das mit einer Referenz.
>
> Genauso gut...char* str = new char[100];
> char& str_ref = *str;
> ...
> delete [] str;
> ...
> return str_ref;

NEEEIIN! So referenziert man nur das erste char, und nach dem Delete ist 
das Referenzierte char auchnoch freigegeben! Referenzen sind genau wie 
zeiger, abgesehen davon, dass sie sich nur auf ein Element beziehen, und 
man sie nur über Umwege auf 0 Zeigen lassen kann.