LCD-Menü - Compilerwarnungen

[PcVirus]

Hi,
ich habe mir ein kleines Menü für mein LCD Programmiert. Funktionieren tut es auch. Allerdings bekomme ich noch einige Compilerwarnungen.

Hier der Code:


//Definitionen (in der Funktion showMenu() )
#define HAUPT 0
#define SUB 1
uint8_t cursor = 0; //Zeilenposiition
uint8_t menu_pos = HAUPT; //Enthält die aktuelle Position im Menü (Haupt oder SUB)
uint8_t akt_cat = 0; //Wird für das SUB Menü benötigt, um die gewählt haupkategorie zu erkennen
char * main_cat[4];
char * sub_cat[4][4];
uint8_t status = 0; //Wenn 1, dann wird die while schleife unterbrochen

main_cat[0] = "---Hauptmenu---";
main_cat[1] = "Beep";
main_cat[2] = "Nix";
main_cat[3] = "Nix2";
main_cat[4] = "";

sub_cat[1][0] = "Zurueck";
sub_cat[1][1] = "Lang-kurz-Lang";
sub_cat[1][2] = "kurz-kurz-kurz";
sub_cat[1][3] = "kein Anschluss";

//Aufruf (aus der Funktion showMenu() )
redraw(&cursor, &akt_cat, &menu_pos, &main_cat, &status);

//Funktion redraw()

//Zeigt das Hauptmenü erneut an
void redraw(uint8_t *_cursor, uint8_t *_akt_cat, uint8_t *_menu_pos, char* *_main_cat, uint8_t *_status){
#define HAUPT 0
#define SUB 1
lcd_clrscr();
*_cursor=0;
*_akt_cat=0;
*_menu_pos=HAUPT;
for(uint8_t i=0; i<4; i++){
lcd_gotoxy(0,i);
lcd_puts(_main_cat[i]);
}
*_status=0;
}

Warnungen:


main.c: In function 'showMenu':
main.c:287: warning: passing argument 4 of 'redraw' from incompatible pointer type
main.c:293: warning: passing argument 4 of 'redraw' from incompatible pointer type
main.c:299: warning: passing argument 4 of 'redraw' from incompatible pointer type
main.c:305: warning: passing argument 4 of 'redraw' from incompatible pointer type
main.c:308: warning: passing argument 4 of 'redraw' from incompatible pointer type
main.c:317: warning: passing argument 4 of 'redraw' from incompatible pointer type
main.c:324: warning: passing argument 4 of 'redraw' from incompatible pointer type


Wie kann ich die noch beheben?

MfG Jörn

[McJenso]

Hallo Jörn,

da gibt es einige Sache in dem Code.


char * main_cat[4];

Hiermit reservierst du 4 Byte für 4 Zeiger auf char.



main_cat[0] = "---Hauptmenu---";
main_cat[1] = "Beep";
main_cat[2] = "Nix";
main_cat[3] = "Nix2";
main_cat[4] = "";

Das passt nie und nimmer in vier Byte :shock:

Du suchst:


char main_cat[5][16] ={"---Hauptmenu---",
"Beep", ...

""};

Das gleich gilt natürlich auch für sub_cat.
Die 5 steht für 5 Elemente und nicht für den größten Index. Die 16 Bytes werden gebraucht, da ein String immer mit einer 0 abschließt.

Jetzt musst du noch Zeiger beim Funktionsaufruf übergeben.
foo (&main_cat[1][0]);
Zeigt auf den Anfang von Beep.
Wenn du jetzt noch in der Deklaration der Funktion einen einfachen Zeiger auf char angibst, sieht das bestimmt schon viel besser aus.


Gruß

Jens

[PcVirus]

Ich habe das mit den Variablen mal geändert. Ich bekomme aber immernoch die Warnungen. Die gehen auch weg wenn ich als aufruf z.B. "main_cat[1][0]"
wähle. Aber dann funktioniert ja meine redraw Funktion nicht mehr, da dort ja alle Einträge von main_cat durchgegangen werden.



//Definitionen
#define HAUPT 0
#define SUB 1
uint8_t cursor = 0; //Zeilenposiition
uint8_t menu_pos = HAUPT; //Enthält die aktuelle Position im Menü (Haupt oder SUB)
uint8_t akt_cat = 0; //Wird für das SUB Menü benötigt, um die gewählt haupkategorie zu erkennen
char main_cat[4][16]; //Maximal 15zeichen lang
char sub_cat[4][4][28]; //Max 27 Zeichen
uint8_t status = 0; //Wenn 1, dann wird die while schleife unterbrochen

strcpy(main_cat[0], "---Hauptmenu---");
strcpy(main_cat[1], "Beep");
strcpy(main_cat[2], "Nix");
strcpy(main_cat[3], "Nix2");
strcpy(main_cat[4],"");

strcpy(sub_cat[1][0], "Zurueck");
strcpy(sub_cat[1][1], "Lang-kurz-Lang");
strcpy(sub_cat[1][2], "kurz-kurz-kurz");
strcpy(sub_cat[1][3], "kein Anschluss");

//Aufruf
redraw(&cursor, &akt_cat, &menu_pos, &main_cat, &status);

//Redraw
void redraw(uint8_t *_cursor, uint8_t *_akt_cat, uint8_t *_menu_pos, char *_main_cat, uint8_t *_status){
#define HAUPT 0
#define SUB 1
lcd_clrscr();
*_cursor=0;
*_akt_cat=0;
*_menu_pos=HAUPT;
for(uint8_t i=0; i<4; i++){
lcd_gotoxy(0,i);
lcd_puts(_main_cat[i]);
}
*_status=0;
}

Fehler:


main.c: In function 'showMenu':
main.c:286: warning: passing argument 4 of 'redraw' from incompatible pointer type
main.c:292: warning: passing argument 4 of 'redraw' from incompatible pointer type
main.c:298: warning: passing argument 4 of 'redraw' from incompatible pointer type
main.c:304: warning: passing argument 4 of 'redraw' from incompatible pointer type
main.c:307: warning: passing argument 4 of 'redraw' from incompatible pointer type
main.c:316: warning: passing argument 4 of 'redraw' from incompatible pointer type
main.c: In function 'redraw':
main.c:346: warning: passing argument 1 of 'lcd_puts' makes pointer from integer without a cast


Mit dem Code wird das Menü nichteinmal erneut angezeigt(Funktion redraw). In der Funktion habe ich ja auch nicht den gesamten Inhalt von main_cat. Wie kann ich das Problem lösen ohne globale Variablen zu verwenden?

MfG Jörn

EDIT:
Wohl die einfachste Lösung


if(redraw==1){
lcd_clrscr();
cursor=0;
akt_cat=0;
menu_pos=HAUPT;
for(uint8_t i=0; i<4; i++){
lcd_gotoxy(0,i);
lcd_puts(main_cat[i]);
}
status=0;
redraw=0;
}
Wo sonst redraw(....) aufgerufen wurde, wird einfach redraw auf 1 gesetzt und beim nächsten Schleifendurchlauf wird die if-Abfrage durchlaufen.

Ob es mit einer Pointerlösung auch geht interessiert mich trotzdem noch.

[McJenso]

Hallo,



redraw(&cursor, &akt_cat, &menu_pos, (char*)main_cat, &status);


Gruß

Jens