Funktionszeiger in Menüstruktur

[Ulf (Gast)]

Hallo!

In Guttenbergscher Anlehnung an die hier
Beitrag "Menüstruktur für Grafikdisplay"
geposteten Erkenntnisse will ich eine Menüstruktur zusammenbasteln, die 
mit einem PIC18f4550 und einem Graphik-Touchdisplay läuft.

#define off 0
#define on 1

void menu(void);
void leer(void);
//typedef void (*action)();

/*===================================================*/
//   Buttonberechnung:
//   StartPosition ist immer Textposition
//   Rahmen errechnet sich aus Textposition
//   Rahmengröße immer gleich, max. 11 Zeichen, 104x24 Pixel
//    
/*===================================================*/


struct Button
{
 char Text[12];      // angezeigter Button-Name max. 11 Zeichen
  unsigned startx,starty;  // Startpunkt des Buttons als Textposition
  unsigned Active;    // Nur aktive B. werden genutzt
  void (*action)(void);    // Zeiger auf Funktion beim Drücken
}button[10];

struct Button buttons[] =   // hier sind alle Buttons gespeichert
{ 
  { "12345678901",26, 17, on,leer },
  { "ABC        ",26, 20, on,leer },
  { "Werte      ",26, 23, on,leer },
  { "Reset      ",26, 26, on ,leer},
};

#define NrAllButtons ( sizeof( buttons ) / sizeof( buttons[0] ) )

void ShowActiveButtons() //malt die Buttons
{
 unsigned i;
 for( i = 0; i < NrAllButtons; ++i ) {
  if( buttons[i].Active ) {
   GLCD_Rectangle(buttons[i].startx*8,buttons[i].starty*8,
    (buttons[i].startx*8)+104, (  buttons[i].starty*8)+24 );
   GLCD_TextGoTo(( buttons[i].startx)+1, (buttons[i].starty)+1);  
   GLCD_WriteText(buttons[i].Text );
  }
 }
}

/********************************************************************
* Function:        imButton
*  
*  true, wenn eingegebene Koordinaten 
*  innerhalb von button[i] liegen.
 *******************************************************************/
unsigned char imButton(unsigned int txpos, unsigned int typos,unsigned char nr)
{
 if( buttons[nr].Active )
  if(((buttons[nr].startx<txpos)&& (txpos<(buttons[nr].startx+104)))&&
      (buttons[nr].starty<typos)&& (typos<(buttons[nr].starty*+24 )))
    return(1);
  else return(0);
}

/********************************************************************
* Function:        void leer(void)
*  
* eine der Menüfunktionen
*     
 *******************************************************************/
void leer(void){
  GLCD_TextGoTo(13, 27);
  GLCD_WriteTextP("ertappt!");
}


/********************************************************************
 * Function:        void menuHandler(void)
*
*  muß in main.c zyklisch aufgerufen werden....
*  
*  liest die Touch- Koordinaten aus
*  und ruft die entsprechenden Funktionen auf.
 *******************************************************************/
void menuHandler(void){
 unsigned char i;
 touch_init();//schaltet die Ports
  if Finger{//Touch gedrückt? (X-Leitungen mit Y verbunden)
   for(i = 0; i < NrAllButtons; i++){          
    if((imButton(Touch_x()/8, Touch_y()/8, i))!=0){
     button[i].action(); //---->Dieser Aufruf führt zum Absturz
     GLCD_TextGoTo(3, 26);GLCD_WriteText(buttons[i].Text );
//zeigt den gedrückten Buttontext noch mal an---nur zum Testen-, muß wieder raus!!--
   }
  }
  GLCD_SetPixel(Touch_x(),Touch_y(), 1);//nur zur Kontrolle, wo gedrückt wurde, muß wieder raus!!
 }
 monitor();//zeigt allerlei Variablen an
}

Zur Verzweiflung bringt mich der Aufruf von
button[i].action();
welcher jedesmal zum Absturz und Reset führt. Läßt man statt dessen 
einfach nur den Namen des Buttons anzeigen, läuft alles wie gewollt, nur 
eben der Funktionszeiger scheint irgendwo ins Nirvana zu weisen...
Sieht jemand einen Fehler?


Dank im Voraus!

ulf.
als lausig laienhafter Autodidakt

[Klaus W. (mfgkw)]

Gibt es auch einen Absturz, wenn du statt button[i].action() direkt 
leer() aufrufst?

[Ben j. (scarab)]

Setze mal einen Breakpoint in der Zeile und siehe nach was in action 
steht.

Ich persönlich prüfe Funktionszeiger immer ob sie ungleich 0 sind bevor 
ich sie Aufrufe, so viel Zeit muss sein.

[A. S. (telekatz)]

Ulf schrieb:
> Zur Verzweiflung bringt mich der Aufruf von
> button[i].action();

Du hast zwei Arrays definiert, button und buttons. Initialisiert wurde 
aber nur das Array buttons.

[Ulf (Gast)]

>Du hast zwei Arrays definiert, button und buttons. Initialisiert wurde
>aber nur das Array buttons.
... das ja verschiedene Varianten von button speichert.
Aber Heureka! Das hat mich auf Umwegen doch auf den Tippfehler gebracht:

button[i].action(); muß heißen:
buttons[i].action();

Man sollte eben differenziertere Namen vergeben und noch öfter alles neu 
tippen...

Danke nochmals!!!

ulf.
nicht mehr verzweifelt.