Pointer auf Struct

[Ingo (Gast)]

Hallo,

ich habe folgende Struct in der main.h:

struct Values_s{

  struct{
    struct{
      volatile uint16_t min,max;
    }Extrema, MittlereMaximalamplitude, MittlereZweiteAbleitung ;
  }Referenzphase, Vergleichphase;

  uint16_t Measurecounter;
  struct{
    uint16_t ProzentualerVortschritt;
  }Anzeige;
};

in der main.c dann:

volatile struct Values_s Daten;

dann in einer anderen C-Datei eine Funktion, die etwas mit den Variablen 
in der Struct machen soll:

void Get_Maxima ( uint16_t *DataPtr )
{
   /* Mach was mit den Daten */
  DataPtr->Measurecounter = 0;
}

Ich bekomme aber folgenden Fehler wenn ich die Structadresse übergebe:

Get_Maxima( &Daten );

 error: request for member 'Measurecounter' in something not a structure 
or union

Warum bzw. was mache ich falsch? ARM-GCC / CoIDE


Ingo

[Sepp (Gast)]

void Get_Maxima ( uint16_t *DataPtr )

DataPtr ist ein uint16_t. Da gibts keine Elemente!

[Bitflüsterer (Gast)]

>Warum bzw. was mache ich falsch?

Du hast den Parameter als Zeiger auf ein vorzeichenloses 16 Bit integer 
deklariert. Nicht aber als Zeiger auf eine Struktur namens Values_s.

An dieser Stelle scheint mir, der mit recht so beliebte Hinweis auf ein 
C-Buch, angebracht zu sein.

[Ingo (Gast)]

Sepp schrieb:
> DataPtr ist ein uint16_t. Da gibts keine Elemente!

void Get_Maxima ( struct Values_s *DataPtr )

wirft diesen Fehler:
 error: dereferencing pointer to incomplete type

[Bitflüsterer (Gast)]

Du solltest ohnehin schon bei der Zeile:

Get_Maxima( &Daten );

eine Warnung erhalten haben. Im Gegensatz zu VHDL sind Warnungen in C so 
gut wie niemals zu ignorieren.

[Ingo (Gast)]

bei dieser Konstellation:

DataPtr->Measurecounter = 0;

[Bitflüsterer (Gast)]

Ingo schrieb:
> Sepp schrieb:
>> DataPtr ist ein uint16_t. Da gibts keine Elemente!
>
>

> void Get_Maxima ( struct Values_s *DataPtr )
>

>
> wirft diesen Fehler:
>  error: dereferencing pointer to incomplete type

Schön. Was steht denn da? Das der Typ nicht komplett ist! Da fehlt also 
in dieser Datei entweder die Strukturdeklaration oder das include der 
Datei in der die Strukturdeklaration enthalten ist.

[Karl H. (kbuchegg)]

Ingo schrieb:
> Sepp schrieb:
>> DataPtr ist ein uint16_t. Da gibts keine Elemente!
>
>

> void Get_Maxima ( struct Values_s *DataPtr )
>

>
> wirft diesen Fehler:
>  error: dereferencing pointer to incomplete type

Dann musst dann eben in diesem Teil auch das Header File inkludieren, in 
dem drinnen steht, wie die Struktur Values_s aussieht. Woher soll denn 
der COmpiler das sonst wissen? Aus den Fingern saugen?

Merke: Jedes C-File wird für sich selbst übersetzt, ohne Ansehen von 
anderen C-Files.
Was in einem anderen C-File drinnen steht, interessiert den Compiler 
schlicht und ergreifend nicht. Jedes C-File hat für sich selbst 
vollständig zu sein.

[Ingo (Gast)]

warning: 'struct Values_s' declared inside parameter list [enabled by 
default]

[Bitflüsterer (Gast)]

Ingo schrieb:
> warning: 'struct Values_s' declared inside parameter list [enabled by
> default]

Sollen wir raten oder ist das ein Alexandriner, oder was? :-)

[Sepp (Gast)]

Probiers mal so:

*.h:

typedef struct
{

  struct{
    struct{
      volatile uint16_t min,max;
    }Extrema, MittlereMaximalamplitude, MittlereZweiteAbleitung ;
  }Referenzphase, Vergleichphase;

  uint16_t Measurecounter;
  struct{
    uint16_t ProzentualerVortschritt;
  }Anzeige;
} Values_s;

*.c:

volatile Values_s Daten;

void Get_Maxima (Values_s *dataptr)
{
   /* Mach was mit den Daten */
  dataptr->Measurecounter = 0;
}

[Karl H. (kbuchegg)]

Ingo schrieb:
> warning: 'struct Values_s' declared inside parameter list [enabled by
> default]

Irgendwann wirst auch du nicht drum herum kommen, dir mal 2 Fragen zu 
stellen
* was will mir der Compiler mit dieser Fehlermeldung sagen? Was steht in 
der Fehlermeldung? Und wie lässt sich das mit den C Regeln in Einklang 
bringen?
* Warum zum Teufel hab ich eigentlich nicht längst ein C-Buch 
durchgearbeitet anstatt mich hier durch Rumraten zum Hampelmann zu 
machen?


Data.h

#ifndef DATA_H_INCLUDED
#define DATA_H_INCLUDED

struct Values_s {
  ...
};

#endif

Function.h

#ifndef FUNCTION_H_INCLUDED
#define FUNCTION_H_INCLUDED

#include "Data.h"
void SetData( struct Values_s * data );

#endif

main.c

#include "Data.h"
#include "Function.h"

int main()
{
  struct Values_s myData;

  SetData( &myData );
}

Function.c

#include "Function.h"

void SetData( struct Values_s * data )
{
  data-> .....
}

Jedes C File ist in dem Sinne vollständig, dass alle Defintionen und 
Deklarationen in jedem C-File bekannt sind. Und durch die Includes ist 
sicher gestellt, dass sie auch in allen C Files konsistent gleich sind.
Man kann jedes C-File für sich hernehmen und von jedem Begriff, der 
darin vorkommt und der kein C-Schlüsselwort ist, exakt sagen, wofür 
dieser Begriff steht bzw. welche 'innere Struktur' er hat (im Falle der 
struct). Und zwar NUR durch betrachten dieses einen C-Files in seiner 
vollständigen Form (also alle include bereits aufgelöst)


NB:
In Function.h würde es theoretisch auch eine Forward-Deklaration der 
struct tun. Es schadet aber auch nichts, wenn Function.h ganz einfach 
Data.h includiert. Die Regel ist ganz einfach: Verwendet ein C oder H 
File etwas, das in einem andern H-File definiert wird, dann wird von 
diesem C oder H File genau dieses andere H File auch includiert. 
Function.h verwendet etwas (nämlich den Namen der Struktur) aus Data.h, 
ergo hat es den include durchzuführen.

[Bitflüsterer (Gast)]

Na gut. Dann nicht. Ich denke ohnhin nur, dass ich C kann bis Karl Heinz 
kommt.

[Ingo (Gast)]

Danke Karl-Heinz,

deine Lösung hat geholfen, ich muss gestehen das ich K&R aber nichts 
dazu gefunden habe...


Ingo

[Bitflüsterer (Gast)]

Naja. Was man alles falsch machen kann, steht in so einem Buch nicht 
d'rin. Das würde wohl leicht ein paar Tonnen wiegen.
Aber von unvollständigen und vorwärts-Deklarationen steht schon was 
darin.

[Ingo (Gast)]

Was mich aufs Eis geführt hat war offensichtlich die Struct Definition / 
redefinition im Funktionsprototyp.

[A. H. (ah8)]

Mal eine Frage am Rande:

Sepp schrieb:
> Probiers mal so:
>
> *.h:
>

typedef struct
{

  struct{
    struct{
      volatile uint16_t min,max;
    }Extrema, MittlereMaximalamplitude, MittlereZweiteAbleitung ;
  }Referenzphase, Vergleichphase;

  uint16_t Measurecounter;
  struct{
    uint16_t ProzentualerVortschritt;
  }Anzeige;
} Values_s;

>
> *.c:

volatile Values_s Daten;

Ein volatile sollte hier doch überflüssig sein, oder?

[test (Gast)]

@A.H.:
Ja, das bei der Definition der Struktur ist Unsinn. Vor allem wenn er es 
danach an eine Funktion übergibt, die als Parameter keine volatile 
struct-Pointer hat.
Da sollte ebenfalls ne Warnung kommen.

@Ingo:
Kompilier deinen ganzen Kram mal zumindest mit "-Wall -pedantic" da 
sieht die Welt ot schn ganz anders aus.