struct Vergleich

[Stefan T. (Gast)]

Abend, ich habe Probleme mit dem Vergleich von struct. Kann mir jemand 
helfen? index wird nicht gefunden, somit auch 255.

typedef struct {
  uint8_t deviceID;
  int16_t measuredTime;
  int16_t actuelTime;
  bool active;        
} shutter_t;

shutter_t poolShutter[30];

// Such den Index wo das device drinsteht
uint8_t poolShutterIndex(uint8_t device) {
  uint8_t i;
  for(i=0; i<30; i++) {
    if(poolShutter[i].deviceID == device)
      return i;
  }
  return 255;
}

Eintragung:

shutter_t new_shutter_device;
new_shutter_device.deviceID = 0x01;
new_shutter_device.measuredTime = 0;
new_shutter_device.actuelTime = 0;
new_shutter_device.active = false;
      
poolShutter[index] = new_shutter_device;

Vergleich:

uint8_t index = poolShutterIndex(0x01);  // suche im poolShutter den Index
if(index != 255) {
   if(poolShutter[index].deviceID == 0x01) {
      poolShutter[index].measuredTime = rxbuf[1];
      poolShutter[index].actuelTime = 0;
      poolShutter[index].active = true;          
   }
}

[Helfer (Gast)]

> poolShutter[index] = new_shutter_device;

Kopiert nicht.

Du hast zwei Möglichkeiten

1) Tatsächlich kopieren z.B. mit memcpy(), Frisst aber ziemlich Speicher

2) poolShutter[] als Feld von Zeigern anlegen und in den Feldelementen 
die Adressen der tatsächlichen Structs eintragen

shutter_t * poolShutter[30];
...
poolShutter[index] = &new_shutter_device;
...
uint8_t index = poolShutterIndex(0x01);  // suche im poolShutter den Index
...
if(index != 255) {
   if(poolShutter[index]->deviceID == 0x01) {
      poolShutter[index]->measuredTime = rxbuf[1];
      poolShutter[index]->actuelTime = 0;
      poolShutter[index]->active = true;          
   }
}
...
uint8_t poolShutterIndex(uint8_t device) {
  uint8_t i;
  for(i=0; i<30; i++) {
    if(poolShutter[i]->deviceID == device)
      return i;
  }
  return 255;
}

[Karl H. (kbuchegg)]

Helfer schrieb:
>> poolShutter[index] = new_shutter_device;
>
> Kopiert nicht.

Doch. Das kopiert schon.
Strukturen können zugewiesen werden.

Zeig mal ein wenig mehr Code

[Alex (Gast)]

Hi,

das Problem liegt in folgender Zeile:

"poolShutter[index] = new_shutter_device;"

hier versuchst du die Adresse von "new_shutter_device" nach 
"poolShutter[index]", dass wird so nicht funktionieren.

Um einen neuen Eintrag zu erstellen (kopieren) musst du den Inhalt 
deiner Struktur "new_shutter_device" in dein Array "poolShutter[index" 
kopieren.

Geht zum Beispiel so:

memcyp (poolShutter[index], new_shutter_device, sizeof (shutter_t));

Wichtig: Bei sizeof unbedingt den typedef angeben, nicht die Struktur 
ansich, da du sonst nur die Länge des Pointers kopieren würdest.

Gruß Alex

[Karl H. (kbuchegg)]

Alex schrieb:
> Hi,
>
> das Problem liegt in folgender Zeile:
>
> "poolShutter[index] = new_shutter_device;"
>
> hier versuchst du die Adresse von "new_shutter_device" nach
> "poolShutter[index]",

Ist doch gar nicht wahr.

Was er macht ist völlig legal.

Aber mich würde mal der Wert von index bei der Zuweisung interessieren. 
Mich würde auch interessieren, ob er sich irgendwo mit lokalen Variablen 
ein Schnippchen geschlagen hat. Mich würde interessieren, ob er sein 
Array einigermassen vernünftig vorinitialisiert hat.

Das Problem steckt irgendwo in den Teilen, die er nicht gezeigt hat.

> Geht zum Beispiel so:
>
> memcyp (poolShutter[index], new_shutter_device, sizeof (shutter_t));

Genau das macht die Zuweisung ganz von alleine

[Klaus W. (mfgkw)]

Karl Heinz Buchegger schrieb:
>> Geht zum Beispiel so:
>>
>> memcyp (poolShutter[index], new_shutter_device, sizeof (shutter_t));
>
> Genau das macht die Zuweisung ganz von alleine

nicht nur, daß die Zuweisung das in der Tat schafft, wäre die
memcpy-Variante auch noch so falsch, wie es nur geht.
- es heißt memcpy, nicht irgendwie ähnlich
- statt poolShutter[index] müsste man &poolShutter[index] schreiben
- entsprechend statt new_shutter_device muß es &new_shutter_device
  heißen

[Stefan T. (Gast)]

Das ist gut zu hören, dass der Fehler schonmal nicht bei struct ist. Die 
poolshutter initialisiere ich garnicht. sollte man es mit poolShutter[0] 
= 0; initialisieren?

[Karl H. (kbuchegg)]

Stefan T. schrieb:
> Das ist gut zu hören, dass der Fehler schonmal nicht bei struct ist. Die
> poolshutter initialisiere ich garnicht. sollte man es mit poolShutter[0]
> = 0; initialisieren?

Das kommt drauf an.
Wenn es sich um eine globales Array handelt, dann ist es bereits 
komplett mit 0-Byte zur Welt gekommen.

[Stefan T. (Gast)]

also ich hab jetzt den index komplett auf 0 umgestellt, da ich im moment 
nur mit einem gerät experementiere. ich habe festgestellt, dass

new_shutter_device.deviceID = 0x01;
poolShutter[0] = new_shutter_device;


poolShutter[0].deviceID

0 ist.
Also ich schreibe 0x01 rein aber es ist nur 0x00 vorhanden. Die Werte 
werden also nicht gespeichert.

[Karl H. (kbuchegg)]

Poste endlich deinen kompletten Code!

Dein Problem liegt irgendwo anders.

[Tom (Gast)]

Dein Code funktioniert bei mir wie erwartet. Die unerwartete Ausgabe

ID = 0x00  index =   1

kommt daher, dass alle Einträge von poolShutter[] am Anfang  --  wie von 
Karl Heinz Buchegger erwähnt -- mit Nullen gefüllt sind und die erste 
Fundstelle der ID 0x00 das erste nicht veränderte Element im Array (hier 
bei index==1) ist.

[Stefan T. (Gast)]

Ok Vielen Dank.

Hab nochmal alles kontrolliert und festgestellt, dass ich das device 
nicht abspeichere aufgrund von falschen variablen. Deshalb war das 
Device auch immer 0x00. Fehlersuche ohne uart oder lcd display ist halt 
etwas schwieriger.