Funktionszeiger in C

[nj (Gast)]

Hallo zusammen und noch ein frohes Neues. :)

Ich wollte mich mal ein bisschen mit Menüprogrammierung beschäftigen und 
hab mir mal die Seite zu Gemüte geführt:
http://www.mikrocontroller.net/articles/Funktionszeiger_in_C

( Unten der Beitrag: Menüs mit Funktionszeigern )

erste Frage:  In der Funktion DoMenu... Warum soll Auswahl mit 1 
subtrahiert werden.? Ergibt für mich keinen Sinn.

Zweite Frage: Ist es überhaupt sinnvoll, die Tasterabfrage so und an 
diesen Ort zu schreiben.?

Dritte Frage: Wenn ich nun ein Taster betätige, ruft es die Funktion 
auf. Das funktioniert schon mal.
Mein Problem ist jetzt, dass die Funktion DoMenu weiter ausgeführt wird, 
und die Hauptmenüstruktur weiter angezeigt wird. In meinen Funktionen 
wie:
void HandleEdit (int arg)
kann ich es aber nicht löschen auf dem LCD. Ist auch logisch dass er es 
so macht. Aber wie kann ich es ändern.?

danke schon mal

mit freundlichem Gruß

[Sebastian V. (sebi_s)]

nj schrieb:
> erste Frage:  In der Funktion DoMenu... Warum soll Auswahl mit 1
> subtrahiert werden.? Ergibt für mich keinen Sinn.

Weil der erste Menüpunkt den Wert 1 hat aber Arrays in C das erste 
Element den Index 0 hat.

nj schrieb:
> Zweite Frage: Ist es überhaupt sinnvoll, die Tasterabfrage so und an
> diesen Ort zu schreiben.?

Prinzipiell schon. Ich würde aber nicht 5mal fast den gleichen Code 
schreiben sondern das über eine Schleife lösen. Etwa so:

int taster[] = {1<<T1, 1<<T2, 1<<T3, 1<<T4, 1<<T5};
Auswahl = -1;
while(Auswahl == -1)
{
  for(int i = 0; i < 5; ++i)
  {
    if(get_key_short(taster[i]))
      Auswahl = i;
  }
}

nj schrieb:
> Dritte Frage: Wenn ich nun ein Taster betätige, ruft es die Funktion
> auf. Das funktioniert schon mal.
> Mein Problem ist jetzt, dass die Funktion DoMenu weiter ausgeführt wird,
> und die Hauptmenüstruktur weiter angezeigt wird. In meinen Funktionen
> wie:
> void HandleEdit (int arg)
> kann ich es aber nicht löschen auf dem LCD. Ist auch logisch dass er es
> so macht. Aber wie kann ich es ändern.?

Die Frage ist mir nicht ganz klar. Wenn du nicht willst das die 
Menüstruktur danach weiter angezeigt wird musst du dafür sorgen, dass 
die Funktion HandleEdit erst returned wenn die komplette Aktion 
abgeschlossen ist und zum Menü zurückgekehrt werden soll.

[nj (Gast)]

Sebastian V. O. schrieb:
> Ich würde aber nicht 5mal fast den gleichen Code
> schreiben

ja :D das war bloß zum probieren.

...

jetzt hatte ich ein Denkfehler. Stimmt, ich muss eine eine neue Struktur 
bilden

[nj (Gast)]

Ich habe mal noch eine andere Frage. Dürfte auch zum Thema Menü 
erstellen passen.

Durch Tasterbetätigung will ich nun Ausgänge setzen und rücksetzen. 
Zusätzlich auch auf dem LCD anzeigen, ob AN oder nicht. Funktioniert 
auch alles. Mein Problem ist jetzt, wenn ich aus diesem Menüpunkt 
rausgeh, später diesen wieder öffne, dann sind zwar die Ausgänge noch 
gesetzt(logisch, weil sie ja nie zurückgesetzt worden), aber den Zustand 
zeigt es nicht mehr an, ob AN oder nicht.

Was macht man da am besten.?

Man könnte jetzt für jede Spannung noch eine Variable erstellen(die es 
speichert) und dann beim Öffnen des Menüpunktes abfragen.

Aber das sollte doch einfacher und mit weniger Code gehen oder.?

danke

void Spannungen (int arg){
  ClearScreen();
  LCDSoftText("Spannungen",0,0,BLACK);
  LCDSoftText("5V",0,16,BLACK);
  LCDSoftText("12V",0,24,BLACK);
  LCDSoftText("24V",0,32,BLACK);
  LCDSoftText("0...12V",0,40,BLACK);
  LCDSoftText("0...12V",0,48,BLACK);
  DrawLine(45,16,45,54,BLACK);
  DrawLine(65,40,65,54,BLACK);
  while(1){
    int taster[] = {1<<T1, 1<<T2, 1<<T3, 1<<T4, 1<<T5};
    for(int i = 0; i < 5; ++i){
      if(get_key_short(taster[i])){
        LCDSoftText(puffer,110,4,BLACK);
        LCDSoftText("AN",47,(i*8)+16,BLACK);
        //PORTD |= (i);
      }        
      if (get_key_long(taster[i])){
        LCDSoftText("    ",i*26,56,BLACK);
        //PORTD &= ~ (i);
      }
    }  
    tasterback();
  }
}