Problem mit struct und void pointer

[Niklaus S. (niklaus_s)]

Hallo,

Ich steh da irgendwie auf dem Schlauch.

Im folgenden Code, habe da mal die relevanten Teile zusammengeschnitten, 
habe ich das Problem dass die 2 letzten Anweisungen einfach nicht 
klappen wollen, irgendwie funktioniert es nicht das struct das ich im 
void * data hinterlegt habe anzusprechen.

Wäre schön wenn mir da jemand auf die Sprünge helfen könnte.

Nik

devices.h

typedef struct Sensors_t
{
  uint8_t id;
  uint16_t lastUpdate;
  void * data;
}Sensors;

typedef struct Leistung_t
{
  int vcc;
  long int ID;
  int Watt;
  long int kWh;
  long int kWhd;
} Leistung ; 

typedef struct Klima_t
{
  int vcc;
  long int ID;
  int Grad;
  int Rel;
} Klima ;

main.c

...
static uint8_t leistung_pt = 0; // Zeiger auf das letzte element im array ‘leistung’

Sensoren sensors[20]; // Array von Allen Sensoren
Leistung leistung[4];     // Array von Leisungssensoren
Klima   klima[6];           // Array von Klimasensoren

int8_s = get_sensor_index((uint8_t) id, sensors); // Hole den index auf sensors[?] anhand der id
 
sensors[s].data = &leistung[leistung_pt]; // Weise dem void * data die Addr von leistung[x] zu

Leistung * data = (Leistung*) sensors[s].data; // Initialisiere den ‘Arbeistpointer’ 
...
data->Watt=watt; //
printf("%u\r\n", data->Watt);

[Max (Gast)]

void-pointer musst du casten! (Leistung*)

[Max (Gast)]

sensors[s].data = (Leistung*)&leistung[leistung_pt];

[Karl H. (kbuchegg)]

Verrätst du uns auch noch, was man sich unter
'funktioniert irgendwie nicht'
vorstellen soll.

Denn eines muss dir klar sein: Nur weil du den Compiler zwingst, die 
Schnauze zu halten, indem du den Pointer niedercastest, wird nicht 
magisch aus einer Klimastruktur, deren Adresse du in data abgelegt hast, 
eine Leistung Datenstruktur.

Ich würde jedem empfehlen um void Pointer einen Bogen zu machen. 
Meistens ist das keine gute Lösung.

In deiner Sensors Datestruktur brauchst du sowieso noch einen Member, 
der dir sagt, ob du es jetzt mit einem Leistungssensor oder einem 
Klimasensor zu tun hast. So etwas wie ein Typflag

typedef struct Leistung_t
{
  int vcc;
  long int ID;
  int Watt;
  long int kWh;
  long int kWhd;
} Leistung ; 

typedef struct Klima_t
{
  int vcc;
  long int ID;
  int Grad;
  int Rel;
} Klima ; 

typedef struct Sensors_t
{
  uint8_t  id;
  uint16_t lastUpdate;

  uint8_t  sensorTyp;
  union
  {
    Leistung* pLeist;
    Klima*    pKlima;
  } data;
}Sensors;

die beiden Pointer liegen durch die union übereinander. Entweder ist der 
Pointer data.pLeist gültig, oder der Pointer data.pKlima ist gültig. Der 
Member sensorTyp (den du natürlich beim Aufbau der Daten korrekt 
besetzen musst), verrät dir dann, welcher der richtige ist, so dass du 
zb alle Sensoren im Array in einer einzigen Schleife abhandeln kannst.

#define SENSOR_UNKNOWN  0
#define SENSOR_KLIMA    1
#define SENSOR_LEISTUNG 2

...
  nrSensors = 0;

...

  sensors[nrSensors].senssorTyp = SENSOR_KLIMA;
  sensors[nrSensors].data.pKlima = klima[0];
  nrSensors++;

  sensors[nrSensors].sensorTyp = SENSOR_LEISTUNG;
  sensors[nrSensors].data.pLeistung = leistung[0];
  nrSensors++;


   ...
  for( i = 0; i < nrSensors; i++ )
  {
    if( sensors[i].sensorTyp == SENSOR_KLIMA )
    {
       //ist ein Klima Sensor
      mach_was_mit( sensors[i].data.pKlima );
    }
    else if( sensors[i].sensorTyp == SENSOR_LEISTUNG )
    {
       // ist ein Leistungssensor
       mach_was_anderes_mit( sensors[i].data.pLeistung );
    }
    else
    {
      // unbekannter Sensortyp, mit dem kann man nichts anfagen
      // Weißt auf einen programmfehler hin, wenn das Programm jemals
      // hier her kommt.
    }
  }

...

void mach_was_mit( Klima* pKlima )
{
  ...
}

void mac_was_anderes_mit( Leistung* pLeistung )
{
  ...
}

[Niklaus S. (niklaus_s)]

Hallo miteinander,

Herzlichen Dank für die schnellen Antworten.

@Max
Leider bin ich mit dir nicht ganz einverstanden.
du sagst:

 sensors[s].data = (Leistung*)&leistung[leistung_pt];

Da leistung[leistung_pt] bereits ein Pointer auf Leistung ist, ist das 
doppelt gemoppelt, richtig wäre dann eher:

sensors[s].data = (void *)&leistung[leistung_pt];

da ich ja den Pointer in den .data reinschreiben will und der ein (void 
*) ist.
Das casting muss ich aber eigentlich nur beim Zugreifen manchen, und 
nicht unbedingt beim Pointer zuweisen.

@Heinz
Das mit der 'union' ist zwar das gleiche was ich gemacht habe, aber von 
der Typenprüfung etwas sicherer. Werde das mal ausprobieren.

Das Problem äussert sich darin, dass der Atmega328 neu startet wenn ich 
die zwei letzten Zeilen einfüge. Entweder schreibe ich ausversehen in 
ein Register oder springe sonst an eine 'falsche' Addresse. - oder - ich 
habe ein Problem mit dem Stack, da ich einige Interrupts in Betrieb 
habe, und da vielleicht was überschrieben wird.

Gruss
 Nik

[Mark B. (markbrandis)]

Karl Heinz schrieb:
> Ich würde jedem empfehlen um void Pointer einen Bogen zu machen.
> Meistens ist das keine gute Lösung.

Hm, was wenn man ein Bridge Pattern in C implementieren will? Oder wenn 
man flexibel auf Daten zugreifen will, die von verschiedenem Typ sein 
können, und erst zur Laufzeit feststeht (z.B. durch Benutzereingabe) was 
man denn haben will?

Oder wenn man eine Lib schreibt und der Benutzer sich nicht auf Details 
der  Implementierung verlassen soll, weswegen ihm diese nicht bekannt 
sein sollen.

[Karl H. (kbuchegg)]

Wer weiß was sowas ist, kann dann auch schon mit void Pointern umgehen. 
Der fragt aber dann auch nicht hier im Forum nach.

Das ein Thomas Bubendorfer die Eiger Nordwand ohne Seil durchklettern 
kann ist kein Argument dafür, dass jeder Hamburger Flachlandtiroler mit 
nichts anderem als Sandalen in die Berge gehen kann.

Hier geht es um den Aufbau einer einfachen Datenstruktur, die man gut 
ohne void Pointer implementieren kann. So wie 95% aller anderen 
Datenstrukturen oder Pattern.

[Mark B. (markbrandis)]

Okay. Wer Interesse hat, hier noch ein schönes Beispiel:

http://www.qnx.com/developers/articles/article_302_2.html

[Karl H. (kbuchegg)]

Niklaus S. schrieb:

> Das Problem äussert sich darin, dass der Atmega328 neu startet wenn ich
> die zwei letzten Zeilen einfüge. Entweder schreibe ich ausversehen in
> ein Register oder springe sonst an eine 'falsche' Addresse. - oder - ich
> habe ein Problem mit dem Stack, da ich einige Interrupts in Betrieb
> habe, und da vielleicht was überschrieben wird.

Schön.
Dann  brauchen wir das konmplette Programm und nicht nur ein paar 
Ausschnitte.
Denn das Problem sitzt dann nicht dort, wo du es vermutest. Du siehst 
nur die Symptome. Die sind aber nicht das eigentliche Problem.

[Oliver (Gast)]

void * mögen zwar ihre Tücken haben, aber bis darauf, daß das oben 
gezeigte Programm unvollständig ist und Tippfehler enthält, und daher 
gar nicht kompilieren würde, ist das, was da mit den Pointern gemacht 
wird, prinzipiell richtig.

Wenn der Prozessor abstürzt, liegt es wohl daran, daß das 
Originalprogramm ganz anders aussieht, oder eben an anderen Teilen des 
Programms.

Oliver

[Niklaus S. (niklaus_s)]

Hallo Karl Heinz,

Das ist mir jetzt auch klar. Ich wollte nur noch eine andere Meinung 
dazu hören ob ich da was übersehen habe bei der Implementation der 
Datenstruktur, weil ja nach dem Einfügen der letzten Zeilen der uC 
restartete.

Ich kann auch gerne das ganze Program senden, aber da es halt auf einen 
Atemea328 mit RFM12 zugeschnitten ist wird das nicht viel bringen. Ich 
werde wohl selbst nochmals weitersuchen müssen.

Gruss
 Nik