Menü mit Funktionspointern

Diese Seite verwendet Cookies. Durch die Nutzung unserer Seite erklären Sie sich damit einverstanden, dass wir Cookies setzen. Weitere Informationen

  • Menü mit Funktionspointern

    Hallo!
    Ich habe gerade versucht ein Menü mit Funktionspointern zu erstellen.
    Nun bekomme ich aber vom LCC-Compiler eine Fehlermeldung bzgl. "acess violation". Denke das es an einem Pointer liegt, der ins Nirvana zeigt, allerdings wüsste ich nicht welcher das sein sollte!?
    Hat von euch jemand eine Ahnung woran es liegen könnte?

    Viele Grüße Maik

    PS: Hier der Code:

    Quellcode

    1. //Includes
    2. #include <stdio.h>
    4. //------------------Defines------------------
    5. #define BOOL char
    6. #define FALSE 0
    7. #define TRUE (!FALSE)
    9. //Number of menu entry
    10. #define STATE_MAINMENU 10
    11. #define STATE_SUB1MENU 20
    12. #define STATE_SUB1SUB1MENU 21
    13. #define STATE_SUB2MENU 30
    15. //Key numbers
    16. #define KEY_NULL 0
    17. #define KEY_ENTER 1
    18. #define KEY_VORWAERTS 2
    19. #define KEY_RUECKWAERTS 3
    21. //------------------Vars------------------
    22. unsigned char state;
    24. //Text for the entries
    25. char ITEM_MAINMENU[] = "Hauptmenue\n";
    26. char ITEM_SUB1MENU[] = "1. Untermenue\n";
    27. char ITEM_SUB1SUB1MENU[] = "1. Untermenue im Untermenue\n";
    28. char ITEM_SUB2MENU[] = "2. Untermenue\n";
    30. //---------Function prototypes------------
    31. unsigned char StateMachine(char state, unsigned char stimuli);
    32. char Submenu1(char input);
    33. char Submenu2(char input);
    34. char Sub1submenu1(char input);
    36. typedef struct
    37. {
    38. unsigned char state;
    39. unsigned char input;
    40. unsigned char nextstate;
    41. } MENU_NEXTSTATE;
    44. typedef struct
    45. {
    46. unsigned char state;
    47. char *pText;
    48. char (*pFunc)(char input);
    49. } MENU_STATE;
    52. MENU_NEXTSTATE menu_nextstate[] = {
    53. // STATE INPUT NEXT STATE
    54. {STATE_MAINMENU, KEY_ENTER, STATE_SUB1MENU},
    55. {STATE_SUB1MENU, KEY_ENTER, STATE_SUB1SUB1MENU},
    56. {STATE_SUB1MENU, KEY_VORWAERTS, STATE_SUB2MENU},
    57. {STATE_SUB1MENU, KEY_RUECKWAERTS, STATE_MAINMENU},
    58. {STATE_SUB2MENU, KEY_RUECKWAERTS, STATE_SUB1MENU},
    59. {STATE_SUB1SUB1MENU, KEY_RUECKWAERTS, STATE_SUB1MENU},
    60. {0, 0, 0},
    61. };
    64. MENU_STATE menu_state[] = {
    65. // STATE STATE TEXT STATE_FUNC
    66. {STATE_MAINMENU, ITEM_MAINMENU, NULL},
    67. {STATE_SUB1MENU, ITEM_SUB1MENU, Submenu1},
    68. {STATE_SUB1SUB1MENU, ITEM_SUB1SUB1MENU, Sub1submenu1},
    69. {STATE_SUB2MENU, ITEM_SUB2MENU, Submenu2},
    70. {0, NULL, NULL},
    71. };
    74. void main(void)
    76. {
    78. unsigned char nextstate;
    79. static char *statetext;
    80. char (*pStateFunc)(char);
    81. char input=0;
    82. char i;
    84. // Initial state variables
    85. state = nextstate = STATE_MAINMENU;
    86. statetext = ITEM_MAINMENU;
    87. pStateFunc = NULL;
    90. for (;;) // Main loop
    91. {
    93. // Plain menu text
    94. if (statetext)
    95. {
    96. printf(statetext);
    98. statetext = NULL;
    99. }
    102. scanf( "%d", &input);
    105. if (pStateFunc)
    106. {
    107. // When in this state, we must call the state function
    108. nextstate = pStateFunc(input);
    111. }
    112. else if (input != KEY_NULL)
    113. {
    114. // Plain menu, clock the state machine
    115. nextstate = StateMachine(state, input);
    116. }
    118. if (nextstate != state)
    119. {
    120. state = nextstate;
    121. for (i=0; menu_state[i].state; i++)
    122. {
    123. if (menu_state[i].state == state)
    124. {
    125. statetext = menu_state[i].pText;
    126. pStateFunc = menu_state[i].pFunc;
    127. break;
    128. }
    130. }
    131. }
    132. }
    136. }
    139. unsigned char StateMachine(char state, unsigned char stimuli)
    140. {
    141. unsigned char nextstate = state; // Default stay in same state
    142. unsigned char i;
    144. for (i=0; menu_nextstate[i].state; i++)
    145. {
    146. if (menu_nextstate[i].state == state && menu_nextstate[i].input == stimuli)
    147. {
    148. // This is the one!
    149. nextstate = menu_nextstate[i].nextstate;
    150. break;
    151. }
    152. }
    154. return nextstate;
    155. }
    159. char Submenu1(char input){
    161. static char enter = 1;
    163. if(enter)
    164. {
    165. enter = 0;
    167. printf("la le lu Untermenue 1");
    168. }
    169. else if (input == KEY_RUECKWAERTS)
    170. {
    171. enter = 1;
    172. return STATE_MAINMENU;
    173. }
    175. return STATE_SUB1MENU;
    178. }
    181. char Submenu2(char input)
    182. {
    184. static char enter = 1;
    186. if(enter)
    187. {
    188. enter = 0;
    190. printf("lu lu lu Untermenu 2");
    191. }
    192. else if (input == KEY_RUECKWAERTS)
    193. {
    194. enter = 1;
    195. return STATE_SUB1MENU;
    196. }
    198. return STATE_SUB2MENU;
    201. }
    203. char Sub1submenu1(char input)
    204. {
    207. static char enter = 1;
    209. if(enter)
    210. {
    211. enter = 0;
    213. printf("lala le: Unter Untermenu");
    214. }
    215. else if (input == KEY_RUECKWAERTS)
    216. {
    217. enter = 1;
    218. return STATE_SUB1MENU;
    219. }
    221. return STATE_SUB1SUB1MENU;
    223. }
    Alles anzeigen

  • Ein bisschen kaotisch dein Programm. Kommt der Fehler vom Compiler oder vom Debugger btw. vom erstellten Programm selbst? Ich vermute letzeres.

    Hab mal dein Programm compiliert und getestet, es läuft, aber ich komme nur ins "la le lu Untermenue 1" rein der rest ist anscheinend nicht zugänglich. Kriege auch keinen Fehler.


    for (i=0; menu_nextstate.state; i++)

    Weist du was die Schleife macht?
    Sie wird einmal auf jedenfall ausgeführt und dann wird immer geschaut ob [i]menu_nextstate[i].state != NULL     ist, dies kann zu unerwarteten Fehlern führen.


    Quellcode

    1. if (pStateFunc)
    2. {
    3. // When in this state, we must call the state function
    4. nextstate = pStateFunc(input);
    5. }
    6. else
    7. if (input != KEY_NULL)
    8. {
    9. // Plain menu, clock the state machine
    10. nextstate = StateMachine(state, input);
    11. }
    Alles anzeigen


    irgendetwas stimmt an dieser Stelle nicht, die Funktion StateMachine wird nur im Hauptmenu aufgerufen, und somit ist die Auswahl nur im Haputmenu möglich ....
    Ich würd dir mal empfehlen deine Struktur zu überarbeiten und dem Benutzer eine Menuführungsbeschreibung geben bezüglich der Eingaben.

    PS: Denk nicht so komplitziert bei der Struktur, einfach ist oft besser.

    Mfg Rushh0ur

  • Hi Rushh0ur!
    Besten Dank für die Antwort.
    Ja, der Fehler kam wohl vom Compiler!
    Das der Code nicht das Gelbe vom Ei ist war mir klar.
    War nur zum testen (Quick and dirty).
    Habe das Programm ein wenig umgeschrieben (ja, auch die Forschleife :) habe ich ergänzt ), ein paar Datentypen verändert, usw.
    Und nun läuft das ganze, zwar mit einem anderen Compiler, aber immerhin und Fehlerfrei!


    Nochmals besten Dank für die Hilfe,

    Viele Grüße Maik