struct Vergleich

Stefan T. Gast 28.03.2011 21:19 #2122929

Lesenswert?

•

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;          

Antwort Zitieren Bearbeiten Thread verschieben Thread sperren Anmeldepflicht aktivieren Anpinnen Thread löschen Thread mit anderem zusammenführen Beitrag melden

Helfer Gast 28.03.2011 21:34 #2122953

Lesenswert?

•

▲
▼

> 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;

Antwort Zitieren Bearbeiten Löschen Beitrag melden

Karl H. (kbuchegg) 28.03.2011 21:44 #2122978

Lesenswert?

•

▲
▼

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

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

Zeig mal ein wenig mehr Code

Antwort Zitieren Bearbeiten Löschen Beitrag melden

Alex Gast 28.03.2011 21:45 #2122980

Lesenswert?

•

▲
▼

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

Antwort Zitieren Bearbeiten Löschen Beitrag melden

Karl H. (kbuchegg) 28.03.2011 21:47 #2122991

Lesenswert?

•

▲
▼

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

Antwort Zitieren Bearbeiten Löschen Beitrag melden

Klaus W. (mfgkw) 28.03.2011 22:02 #2123014

Lesenswert?

•

▲
▼

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

Antwort Zitieren Bearbeiten Löschen Beitrag melden

Stefan T. Gast 28.03.2011 22:05 #2123019

Lesenswert?

•

▲
▼

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?

Antwort Zitieren Bearbeiten Löschen Beitrag melden

Karl H. (kbuchegg) 28.03.2011 22:22 #2123055

Lesenswert?

•

▲
▼

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.

Antwort Zitieren Bearbeiten Löschen Beitrag melden

Stefan T. Gast 28.03.2011 23:04 #2123131

Lesenswert?

•

▲
▼

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.

Antwort Zitieren Bearbeiten Löschen Beitrag melden

Karl H. (kbuchegg) 29.03.2011 00:00 #2123186

Lesenswert?

•

▲
▼

Poste endlich deinen kompletten Code!

Dein Problem liegt irgendwo anders.

Antwort Zitieren Bearbeiten Löschen Beitrag melden

Tom Gast 29.03.2011 00:57 #2123228

Lesenswert?

•

▲
▼

Dein Code funktioniert bei mir wie erwartet. Die unerwartete Ausgabe

1	`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.

Angehängte Dateien:

ttt.c (1,27 KB) | Codeansicht

Antwort Zitieren Bearbeiten Löschen Beitrag melden

Stefan T. Gast 29.03.2011 14:33 #2123825

Lesenswert?

•

▲
▼

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.

Antwort Zitieren Bearbeiten Löschen Beitrag melden

struct Vergleich

Antwort schreiben