ISRs in Include / C-File ausgliedern

In deinem global.h gehören nur DEKLARATIONEN rein, also das 
Bekanntmachen von Variablen. Die DEFINITIONEN, also das Anlegen im 
Speicher, gehören in ein noch nicht existentes global.c

Und dann müssen dein globalen Variablen in globals.h als extern 
definiert werden. Das klappts auch mit den ausgelagerten ISRs.

globals.h

extern volatile uint16_t key_state;
extern volatile uint16_t key_press;



globals.c

volatile uint16_t key_state;
volatile uint16_t key_press;

Eine "global.h" ist grundsätzlich eine schlechte Idee. Pack die 
Definitionen der globalen Variablen in die c-Dateien der Module und die 
Deklaration (mit extern) in die dazugehörige h-Datei. Dann brauchst Du 
keine global.h.

Die ISRs gehören in der Form auch nicht in eine eigene Datei. Wenn Du 
eine ISR für beide Module (Display und Tasten) nutzen willst, wäre es 
sinnvoller, von der gemeinsamen ISR aus zwei Funktionen aufzurufen. Die 
ISR ist dann nur zwei Zeilen lang und kann in main.c:

1

ISR(TIMER2_OVF_vect)

2

{

3

  keys_isr_handler();

4

  display_isr_handler();

5

}

Die eigentliche Funktionalität (keys_isr_handler und 
display_isr_handler) kommt in die entsprechenden c-Dateien, wo auch die 
Modulvariablen liegen, die sie nutzen. Dann brauchst Du die globalen 
Variablen überhaupt nicht.

Grundsätzlich: Modularer Code bedeutet lose Kopplung, starke Bindung. 
Sprich: Nicht alles auf so viele Dateien wie möglich aufteilen, sondern 
das zusammen lassen, was zusammen gehört. Alles was mit den Tasten zu 
tun hat, kommt in keys.c/h, alles was mit dem Display zu tun hat in 
display.c/h. Diese Module müssen so weit wie möglich eigenständig sein 
und keine gegenseitigen Abhängigkeiten, vor allem keine zum 
Hauptprogramm haben. Nur so kann man diese Module sinnvoll in anderen 
Programmen wiederverwenden.