C Struct Pointer in Funktion "swappen" ?

[AVRli (Gast)]

Hallo Community,

ich möchte ganz gerne im Hauptprogramm über eine Funktion zwei Zeiger 
untereinander tauschen. Das gelingt mir nicht. Was habe ich übersehen?

...
typedef struct
{
    unsigned char fid;
    unsigned char mount;
    unsigned char name[8];
} MY_device;
...
...
void SwapDevices(MY_device *dev_a, MY_device *dev_b)
{
  OWI_device *dev_s;
  
  *dev_s = *dev_a;
  *dev_a = *dev_b;
  *dev_b = *dev_s;  
}
...
...
int main (void) {
    MY_device *dvo, *dvi;
...
...
...
    SwapDevices(dvo, dvi);
}

Vor dem Aufruf von "SwapDevice" stimmen die Adressen von dvi und dvo 
nach dem Aufruf sind sie leider nicht getauscht.

Ich bitte Euch hiermit um Hilfe.
AVRli

[Stefan K. (stefan64)]

Du musst Pointer auf Pointer übergeben, damit die Funktion Deine Pointer 
swappen kann:

void SwapDevices(MY_device **dev_a, MY_device **dev_b)
{
...
}

int main (void) {
    MY_device *dvo, *dvi;
...
...
...
    SwapDevices(&dvo, &dvi);
}

Gruß, Stefan

[Stefan K. (stefan64)]

https://www.eskimo.com/~scs/cclass/int/sx8.html

Gruß, Stefan

[Rainer B. (katastrophenheinz)]

Der normale Parameterübergabemechanismus in C ist "call by value", d.h. 
so wie du das machst, übergibst du Kopien deiner Zeiger, die lokal 
vertauscht werden, ausserhalb der Funktion jedoch ( eben weil es Kopien 
sind ) unverändert sind. Wenn du "call by reference" haben willst, dann 
musst Zeiger auf die Originale übergeben, also Zeiger auf Zeiger:
[/c]
void SwapDevices(MY_device **dev_a, MY_device **dev_b)
{
  OWI_device *dev_s;

  dev_s = *dev_a;
  *dev_a = *dev_b;
  *dev_b = dev_s;
}
...
...
int main (void) {
    MY_device *dvo, *dvi;
...
...
...
    SwapDevices(&dvo, &dvi);
}

[Decius (Gast)]

oder einfach:

void SwapDevices(MY_device *dev_a, MY_device *dev_b)
{
  OWI_device *dev_s;

  dev_s = dev_a;
  dev_a = dev_b;
  dev_b = dev_s;
}

[Peter II (Gast)]

Decius schrieb:
> oder einfach:

bestimmt nicht.

[Decius (Gast)]

Bei genauerer Überlegung ist es wahrscheinlich doch so wie Rainer es 
geschrieben hat. Denn bei call by reference wird eine adresse übergeben, 
deren Inhalt dann geändert werden kann.

Bei:

void SwapDevices(MY_device *dev_a, MY_device *dev_b)
{
}

wird zwar eine Adresse übergeben, aber es soll nicht der Inhalt auf 
dieser Adresse sonder die Adresse selbst geändert werden. Also ist wohl 
doch ein Zeiger auf einem Zeigern notwendig:

void SwapDevices(MY_device **dev_a, MY_device **dev_b)
{
}

[Decius (Gast)]

Sorry. War etwas langsamer als PeterII.

[AVRli (Gast)]

@all
Danke für Eure Antworten!

Rainer B. schrieb:
> Wenn du "call by reference" haben willst, dann
> musst Zeiger auf die Originale übergeben, also Zeiger auf Zeiger:

Danke Rainer! That's is! :-D
Zeiger auf Zeiger - da wäre ich im Leben nicht drauf gekommen!

Und die Swap Funktion habe ich auch übernommen, ich hab echt eine 
"Zeiger-Blockade" ob ich die noch ablegen kann. :-/

[Zephyrinus Z. Zeiger (Gast)]

Decius schrieb:
> wird zwar eine Adresse übergeben, aber es soll nicht der Inhalt auf
> dieser Adresse sonder die Adresse selbst geändert werden. Also ist wohl
> doch ein Zeiger auf einem Zeigern notwendig:

Ja, man braucht eben immer mindestens eine Indirektion mehr, wenn man 
den Wert selbst verändern will und nicht bloß seine Kopie: Wert -> 
Zeiger, Zeiger ("Adresswert") -> Zeiger auf Zeiger, Zeiger auf Zeiger -> 
Zeiger auf Zeiger auf Zeiger, ...

[Nop (Gast)]

AVRli schrieb:
> Zeiger auf Zeiger - da wäre ich im Leben nicht drauf gekommen!

Dann ist es bis zum Dreisterne-Programmierer ja noch ein weiter Weg. ;-)