Über #include und #define

[chri (Gast)]

Hallo zusammen,

ich habe ein Problem mit #include und #define.

//file: display.h
#include<stdio.h>

void display(int a);

//file: display.cpp
#include "display.h"

void display(int a)
{
  printf("a = %d\n", a);
}

//file: control.h
#ifdef MAIN
extern int a;
#else
int a;
#endif

void control();

//file: control.cpp
#include "control.h"

void control()
{
  a = 10;
}

//file: test.cpp
#define MAIN

#include "control.h"
#include "display.h"

#undef MAIN

int main()
{
  control();
  display(a);

  return 0;
}

Feher: "int a" ist bereits in control.obj definiert. Mindestens ein 
mehrfach definiertes Symbol gefunden.

Kann jemand mir erzählen warum?

Vielen Dank

Gruß

[Tobi H. (tobi-)]

In Header-Dateien nie Speicher für Variablen reservieren, sondern 
immer nur den Namen bekannt machen, d.h:

In control.h immer nur

extern int a;

In control.cpp dann

int a;

Damit können alle Programmteile, die control.h einbinden auf die 
Variable a in der Datei control.cpp zugreifen. Damit ist die Krücke 
mittels des defines überhaupt nicht mehr nötig.

[Stefan E. (sternst)]

> Kann jemand mir erzählen warum?

Weil das hier
> #ifdef MAIN
> extern int a;
> #else
> int a;
> #endif
genau verkehrt herum ist.

[I_ H. (i_h)]

Eine weitere wichtige Sache: Jeden Header musst(solltest) du mit

#ifndef SYMBOLNAME
#define SYMBOLNAME

beginnen, und entsprechend ganz unten mit #endif abschließen. Als 
Symbolnamen nimmt man was für den Dateinamen eindeutiges, zB. DISPLAY_H, 
_DISPLAY_H oder sowas.

Hintergrund: Wenn Datei a.h b.h inkludiert, die wiederrum a.h 
inkludiert, dreht sich die Sache im Kreis. Der Makrocode am Anfang sorgt 
dafür, dass jeder Header nur genau einmal eingebunden wird.


Also in deinem Fall zB:

//file: control.h
#ifndef CONTROL_H
#define CONTROL_H

extern int a;

void control();

#endif