LCD Userinterface

@ Peter (Gast)

>Ich habe nur das Gefühl das das ganze irgendwie suboptimal gelöst ist.

Ja, so viele _delay_ms in einem Programm können nicht gut sein. Warum 
taktest du deinen uC nicht ei nfach mit 50 Hz Netzfrequenz?
Sowas macht man per Timer und Interrupt üer ein Flag.

>Es würde mich freuen wenn ich jemand mal den Codeschnipsel anschauen
>würd und mir sagen könnte ob es wirklich sehr krude gelöst ist.

Jain. es gibt wie immer Verbesserungsmöglichkeiten.

>    lcd_gotoxy(0,0);
>    lcd_puts("   Select Program");
>    lcd_gotoxy(1,1);
>    lcd_puts("FastDraw  Stopwatch");
>    lcd_gotoxy(1,2);
>    lcd_puts("IPSC      PPC");
>    lcd_gotoxy(1,3);
>    lcd_puts("Chrono    Duel");

Diese vielen konstanten String brauchen so doppelt Speicher, im Flash 
und im RAM. Mit der Funktion PSTR() un eine kleine Anpassung von 
lcd_puts() kann man die konstanten Strings in den Flash verschieben, 
siehe Doku der libc zum Themas sprintf().

>      lcd_gotoxy(0,1);
>      lcd_puts(">");
>      break;

Hier reicht es EINMAL ein lcd_puts(">"); NACH dem switch zu machen, 
spart Programmspeicher. Genauso in main.

  while (1) {

>    // wenn runter taste gedrückt auswahl erhöhen
>    if ( button_active(down) && active_programm < 5) {
>      active_programm++;
>      lcd_draw_ui(3);
>      _delay_ms(100);
>    }

MfG
Falk

@  Peter (Gast)

>Da ich gern ein repeat beim gedrückt halten und eine erkennung von zwei
>gleichzeitigen Tastendrücken als Metataste haben möchte.

Lässt sich alles problemlos mit einem Timer machen.

MFG
Falk