сбой охранников кода - PullRequest
Новая
       30

сбой охранников кода

1 голос
/ 24 июля 2011

Возьмите эти файлы:

хиджры 

#ifndef A_H
#define A_H

char EL[] = "el";
#endif

a.cpp 

#include "a.h"

b.h 

#ifndef B_H
#define B_H

#include "a.h"

#endif

b.cpp 

#include "b.h"

main.cpp 

#include "b.h"
#include "a.h"

int main() { }

Это только пример, но у меня действительно такая проблема: 

g++ -c a.cpp
g++ -c b.cpp
g++ -c main.cpp
g++ -o main main.o a.o b.o


a.o:(.data+0x0): multiple definition of `EL'
main.o:(.data+0x0): first defined here
b.o:(.data+0x0): multiple definition of `EL'
main.o:(.data+0x0): first defined here
collect2: ld returned 1 exit status

почему и как решить?

1 Ответ

8 голосов
/ 24 июля 2011

Включенные охранники не защищают вас от определения объекта несколько раз, если вы включаете определение в несколько единиц перевода!

В качестве решения никогда не define вещи в заголовках, но только объявляют их: 

// header
extern char EL[2];

// TU
#include "header.h"
char EL[2] = "el";

// Other consumer
#include "header.h";
// can now use EL

(Конечно, есть исключения; например, определения классов хороши (но определения функций-членов класса не являются (но встроенными являются)) - будьте осторожны.)

Я должен добавить, что в качестве альтернативы вы можете сказать static в своем заголовочном файле, чтобы сделать определение частным для каждого TU: 

// header
static char EL[] = "EL";  // every TU gets a copy

(В C ++ 0x вы не можете использовать объекты статической связи в качестве параметров шаблона.)

...