Wir wollten eine "debug.h" schreiben mit Precompiler-definitionen. Das ganze soll uns ein debuggen in unserem Programm ermöglichen (können keinen anderen funktionalen Debugger finden, da unser Proggi ein Betriebssystem ist und im bochs emuliert wird) welches von einer einzigen Datei aus steuerbar ist.

Die Datei schaut nun wie folgt aus:

#ifndef debug_h
#define debug_h

// include/debug.h
// This defines a couple of cpp macros for debugging output.
// output can easily be switched of by clearing respective 
// macros at the bottom of this file. 
// written for our CoStubs implementation by Christian Bendele (2006)

// makes writing possible on the current output Channel used by main()

#include "io/PrintStream.h"
#include "device/CgaChannel.h"
#include "device/CgaAttr.h"

extern PrintStream out;
extern CgaChannel cga;
extern CgaAttr oldAttr;

// for general debugging output

#define _DPRINT(name,a) cga.getAttr(oldAttr); \
                        cga.setAttr(CgaAttr(CgaAttr::Yellow, CgaAttr::Black, false)); \
                        out.print("                              "); \
                        out.print(name); \
                        out.print(" = "); \
                        out.print(a); \
                        out.print("\n"); \
                        cga.setAttr(oldAttr);

// for function hints and more verbose debugging output
#define _DFPRINT(a)     cga.getAttr(oldAttr); \
                        cga.setAttr(CgaAttr(CgaAttr::Green, CgaAttr::Black, false)); \
                        out.print("                              "); \
                        out.print(a); \
                        cga.setAttr(oldAttr);

// for special output prints (please just one element)
#define _DSPRINT(a)     cga.getAttr(oldAttr); \
                        cga.setAttr(CgaAttr(CgaAttr::Magenta, CgaAttr::Black, false)); \
                        out.print(a); \
                        out.print("\n"); \
                        cga.setAttr(oldAttr);

// uncomment the following lines to disable parts of the debugging output

#define _DPRINT(name,a)
#define _DFPRINT(a)
#define _DSPRINT(a)

#endif

Die oben als extern definierten Variablen werden in unserer main() deklariert und hier genutzt. Die ersten beiden werden sowieso bei uns in der main() benötigt, daher können diese beruhigt hier als "extern" deklariert werden. Lediglich die "CgaAttr oldAttr" Variable wird nur hier benötigt (bzw. später dann in allen Dateien, denen wir das debuggen ermöglichen wollen). Wenn wir nun diese Variable nicht als extern deklarieren (also direkt hier in der header), dann bekommen wir beim Compilieren eine Menge Fehlermeldungen über eine "Mehrfachdeklaration der Variable oldAttr".

Dieser Variablenname kommt in keiner der anderen Dateien vor und eigentlich dachte ich, diese Mehrfachdeklaration mit dem "#ifndef debug_h" übergangen zu haben. Aber anscheinend stört es dennoch.

Darf ich denn soetwas überhaupt machen, globale Variablen hier in einer separaten Header Datei deklarieren, die dann überall dort nutzbar sind, wenn diese Header Datei über <#include "debug.h"> eingebunden wurde?

Die ganze Datei funktioniert, sobald ich diese Variable oldAttr in unserer main.cc (bzw. sogar noch ausserhalb des Anweisungsblocks von main.cc, als globale Variable) deklariert habe und hier in der debug.h via unserem "extern" darauf zugreife. Dann kann ich problemlos compilieren und es funktioniert auch entsprechend.

Eine Idee?

mfg Maddin

Daraus folgt die Antwort auf Deine Frage, nämlich dass das keine besonders gute Idee ist, eine Variable in einem Header vollständig zu definieren.

Vorschlag: Fügt Eurem Projekt doch eine debug.c hinzu wo solche Variablen definiert werden, und im Header bleibt dann die extern-Deklaration.

Aber trotzdem danke für den Vorschlag mit der debug.c ... wenns nicht anders geht, werd ich das wohl auch so machen

mfg maddin