Variable nur innerhalb einer Header Datei einsehbar

[Merlo (Gast)]

Hallo,

wie mache ich dass, das eine Variable nur von ISR angepasst werden kann. 
Am besten soll die in der main.c überhaupt nicht bekannt sein, sondern 
nur über die Anfrage der ms() als Rückgabewert bekannt gegeben werden.

Beispiel:

// main.c
#include "time.h"

void main()
{
time_init();
while(1) { /*...*/ }
}

// time.h
volatile uint16_t _ms;

ISR(/* ... */)
{
_ms++;
}

uint16_t ms(void) { return _ms; }

[Noch einer (Gast)]

Die ISR und die ms() in einen eigene Datei packen?

[Peter II (Gast)]

volatile uint16_t _ms;

einfach nicht in die Header Datei schreiben, sondern nur in die time.c

[Merlo (Gast)]

Hab jetzt nur die h Datei gemacht, da es sich um einen nur kleinen 
Zusatz handelt. Gibt es innerhalb der h-Datei eine Möglichkeit, 
ansonsten mache ich die c Datei und gut ist ;)

[Fabian O. (xfr)]

Bei der Anweisung

#include "time.h"

macht der Preprozessor nichts anderes, als den ganzen Inhalt von time.h 
an dieser Stelle in main.c einzufügen. Erst danach sieht der Compiler 
den Code. Es macht hier also keinen Unterschied, ob etwas in main.c oder 
in time.h steht, denn es ist für den Compiler nicht zu unterscheiden.

Du bekommst allerdings ein Problem, wenn Du time.h in eine zweite 
C-Datei (z.B. modul2.c) inkludierst. Dann steht da nämlich nochmal die 
Definition der Variable, der Funktion und der ISR. Darüber wird sich der 
Linker beschweren, denn jede Funktion darf nur einmal definiert werden.

Der richtige Weg ist, aus Deiner time.h eine time.c zu machen und eine 
neue time.h zu erstellen, dir nur Deklarationen enthält, die ein anderes 
Modul (main.c) kennen muss, um die Funktionalität nutzen zu können.

// time.h
uint16_t time_ms(void);

// time.c
static volatile uint16_t ms;

ISR(/* ... */)
{
ms++;
}

uint16_t time_ms(void) { return ms; }

Ich habe noch zwei Änderungen vorgenommen:

1. Die Variable "_ms" static gemacht (und das Unterstrich-Prefix 
entfernt, das ist nämlich für die C-Standardbibliothek reserviert). 
Dadurch kann sie nur von Funktionen aus time.c verwendet werden. Von 
main.c aus ist sie nicht sichtbar. Das war ja Deine ursprüngliche Frage.

2. Der Funktion "ms" das Prefix "time_" gegeben. Somit kann man sofort 
erkennen, zu welchem Modul diese Funktion gehört. Gewöhn Dir am besten 
gleich an, dass alle Funktionen, die in einer H-Datei aufgeführt sind, 
das gleiche Prefix haben. Ansonsten wirst Du schnell den Überblick 
verlierten und die Gefahr von unbeabsichtigten Namenkollisionen steigt.

[Thomas E. (thomase)]

Fabian O. schrieb:
> Du bekommst allerdings ein Problem, wenn Du time.h in eine zweite
> C-Datei (z.B. modul2.c) inkludierst. Dann steht da nämlich nochmal die
> Definition der Variable, der Funktion und der ISR. Darüber wird sich der
> Linker beschweren, denn jede Funktion darf nur einmal definiert werden.

Deswegen macht man immer sowas:

//time.h

#ifndef TIME_H
#define TIME_H

//Inhalt der Headerdatei

#endif /*TIME_H*/

mfg.

[Fabian O. (xfr)]

Das hilft in dem Fall nichts, denn jede C-Datei wird unabhängig von 
anderen C-Dateien kompiliert. Ist ja auch gut so, sonst ließe sich so 
eine H-Datei nur in eine einzige C-Datei einbinden und wäre in den 
anderen wirkungslos.

Diese Includeguards haben den Sinn zu verhindern, dass der Inhalt einer 
H-Datei mehrmals in die gleiche C-Datei inkludiert wird, wenn die 
H-Datei noch von (mehreren) anderen H-Dateien inkludiert wird. In 
ungünstigen Konstellationen könnte das sogar zu Endlosschleifen führen.