Problem mit struct bei einer Funktion

[Tobias B. (horschtx)]

Hallo Zusammen,

bestimmt kann mir hier jemand helfen. Ich stehe gerade auf dem Schlauch 
und komme nicht weiter.

Ich versuche gerade im Atmel Studio ein Menü zu Programmieren. Habe mir 
auch schon einige Tip´s hier geholt.
Jetzt versuche ich gerade ein struct an eine Funktion zu übergeben. Das 
will aber nicht klappen. Es kommen immer irgendwelche Compiler fehler 
dabei raus und ich finde den Fehler nicht.
Das ganze wird unter dem AVR-Studio Programmiert und soll auf einem 
XMega laufen.
Der Grundsätzlich Aufbau funktioniert auch, ich wollte nur den Code 
einfacher gestalten und ein paar Funktionen einsparen.

Ich mach jetzt einfach mal ausschnitte aus dem Programm. Da es schon 
ziemlich viel Code geworden ist hoffe ich mal das das so reicht um zu 
helfen. Wenn mehr benötigt wird sollte das aber kein Problem sein ;)

Ziel des ganzen ist es eigentlich nur aus einer Funktion her verschieden 
Menüs anzuzeigen.

Das ist mein Protoyp für die Funktion:

void draw_menu (uint8_t, struct MenuEntry );

Das ist das struct dazu:

struct MenuEntry
{
  char Text_Zeile1[16];
  char Text_Zeile2[18];
  MenuFnct Function;
  };

Der Aufruf:

draw_menu(quelle_posi,QuelleMenu);

Das die Funktion:

void draw_menu(uint8_t x, struct MenuEntry menu[])
{
  lcd_clear();
  lcd_setcursor(0,0);
  lcd_string(menu[x].Text_Zeile1);
  lcd_setcursor(0,1);
  lcd_string(menu[x].Text_Zeile2);
}

Wenn ich es so versuch zu Compilieren kommt einmal der Fehler:

Error  4  type of formal parameter 2 is incomplete

und dann noch

Error  6  conflicting types for 'draw_menu'

Vielleicht kann mir ja jemand vom "Schlauch" runterhelfen?

Vielen Dank schon einmal im voraus.

Schöner Gruß
Tobias

[NSA (Gast)]

In welcher Zeile zeigt der Compiler den Error?
Am besten den ganzen Sourcecode Posten (als Anhang!)

[Karl H. (kbuchegg)]

Tobias B. schrieb:

> Das ist mein Protoyp für die Funktion:
>

void draw_menu (uint8_t, struct MenuEntry );

Aha.
Hier sagst du: Die Funktion kriegt als 2. Argument 1 einzelnes Objekt


>

void draw_menu(uint8_t x, struct MenuEntry menu[])
>

und hier sagst du, die Funktion kriegt ein ganzes Array von Objekten 
(d.h. um genau zu sein sagst du hier eigentlich die Funktoin kriegt 
einen Pointer auf ein Array)

Der springende Punkt ist aber: Ja, was denn nun? Ein einzelnes Objekt 
oder ein Array?


Es ist doch wirklich ganz simpel.
SChreib deine Funktion

void draw_menu(uint8_t x, struct MenuEntry menu[])
{
  lcd_clear();
...
  lcd_string(menu[x].Text_Zeile2);
}

nimm die Zeile mit der Funktionsdefinition mittels Strg+C in den Copy 
Buffer

void draw_menu(uint8_t x, struct MenuEntry menu[])

geh an die Stelle, an der du den Protoypen für die Funktion haben willst 
(Header File), setze dort die Zeile mittles Strg+V ein und hänge einen 
';' hinten drann

void draw_menu(uint8_t x, struct MenuEntry menu[]);

und fertig ist dein Protoyp.

Und wenn der Compiler wieder die Funktionsdefinition anmäkelt, dann 
siehst du dir nächstes mal die Datentypen aller Argument (inklusive 
Array-[]) genauer an.

[Falk B. (falk)]

Man sollte besser einen Typ definieren, das ist in der Praxis DEUTLICH 
günstiger zu handhaben.

typedef struct 
{
  char Text_Zeile1[16];
  char Text_Zeile2[18];
  MenuFnct Function;
  } MenuEntry_t;

MenuEntry_t TopMenuEntry;

void draw_menu (uint8_t, MenuEntry_t *menu);

void draw_menu(uint8_t x, MenuEntry_t *menu)
{
  lcd_clear();
  lcd_setcursor(0,0);
  lcd_string(menu[x].Text_Zeile1);
  lcd_setcursor(0,1);
  lcd_string(menu[x].Text_Zeile2);
}

[Tobias B. (horschtx)]

Falk Brunner schrieb:
> Man sollte besser einen Typ definieren, das ist in der Praxis DEUTLICH
> günstiger zu handhaben.

Danke schön, das hat geholfen

[Rolf M. (rmagnus)]

Falk Brunner schrieb:
> Man sollte besser einen Typ definieren, das ist in der Praxis DEUTLICH
> günstiger zu handhaben.

Also so DEUTLICH finde ich das nicht und hab kein Problem damit, structs 
ohne typedef zu nutzen.

[Dirk B. (dirkb2)]

Wenn man die Definitionszeile der Funktion auch zur Deklaration nimmt, 
bekommt man (bei sinnvollen Varablennamen) auch gleich eine 
Kurzdokumentation.