Question

所以我在文件cerus.h中写了一小组日志记录功能。该文件的内容如下所示。它包含在main.cpp，model.cpp，engine.cpp和camera.cpp中。可以看出，我有警卫，所以我不确定为什么我会收到这个错误：

$ make

的输出

jed@ArchPC:~/glPlayground$ make
g++ -std=c++11 -c model.cpp -o bin/model.o
g++ -std=c++11 -c tiny_obj_loader.cc -o bin/tinyobj.o
g++ -std=c++11 -c camera.cpp -o bin/camera.o
g++ -g -std=c++11 -o main bin/main.o bin/engine.o bin/tinyobj.o bin/model.o  bin/camera.o -lGL -lGLU -lglut -lSOIL -lGLEW -lglfw
bin/engine.o: In function `LOG(char const*)':
engine.cpp:(.text+0x0): multiple definition of `LOG(char const*)'
bin/main.o:main.cpp:(.text+0x0): first defined here
bin/engine.o: In function `LOGERR(char const*)':
engine.cpp:(.text+0x3d): multiple definition of `LOGERR(char const*)'
bin/main.o:main.cpp:(.text+0x3d): first defined here
bin/model.o: In function `LOG(char const*)':
model.cpp:(.text+0x0): multiple definition of `LOG(char const*)'
bin/main.o:main.cpp:(.text+0x0): first defined here
bin/model.o: In function `LOGERR(char const*)':
model.cpp:(.text+0x3d): multiple definition of `LOGERR(char const*)'
bin/main.o:main.cpp:(.text+0x3d): first defined here
bin/camera.o: In function `LOG(char const*)':
camera.cpp:(.text+0x0): multiple definition of `LOG(char const*)'
bin/main.o:main.cpp:(.text+0x0): first defined here
bin/camera.o: In function `LOGERR(char const*)':
camera.cpp:(.text+0x3d): multiple definition of `LOGERR(char const*)'
bin/main.o:main.cpp:(.text+0x3d): first defined here
collect2: error: ld returned 1 exit status
Makefile:4: recipe for target 'main' failed
make: *** [main] Error 1

cerus.h

#ifndef CERUS_H
#define CERUS_H
#include <iostream>
//Need to add Windows and Mac Includes here

// Linux Include Statements

void LOG(const char* str){
    std::cout << "[INFO]" << str << "\n";
}
void LOGERR(const char* str){
    std::cout << "[ERROR]" << str << "\n";
}

#include <GL/glew.h>
#include <GL/gl.h>
#include <GL/glu.h>
#include <GL/glut.h>

#include <GLFW/glfw3.h>

#endif

生成文件

all: main

main: bin/main.o bin/engine.o bin/model.o bin/tinyobj.o bin/camera.o cerus.h
    g++ -g -std=c++11 -o main bin/main.o bin/engine.o bin/tinyobj.o  bin/model.o  bin/camera.o -lGL -lGLU -lglut -lSOIL -lGLEW -lglfw

bin/main.o: main.cpp cerus.h
    g++ -std=c++11 -c main.cpp -o bin/main.o

bin/engine.o: engine.cpp engine.h cerus.h
    g++ -std=c++11 -c engine.cpp -o bin/engine.o

bin/tinyobj.o: tiny_obj_loader.cc tiny_obj_loader.h cerus.h
    g++ -std=c++11 -c tiny_obj_loader.cc -o bin/tinyobj.o

bin/model.o: model.cpp model.h cerus.h
    g++ -std=c++11 -c model.cpp -o bin/model.o

bin/camera.o: camera.cpp camera.h cerus.h
    g++ -std=c++11 -c camera.cpp -o bin/camera.o

clean:
    rm -f bin/*.o main

如果有人能向我解释为什么我的警卫没有抓住这个，我会非常感谢你的帮助。

编辑：通过添加名为cerus.cpp的文件并在其中定义我的日志记录功能而不是cerus.h

来解决此问题

Answer 1

这种保护措施是为了避免在相同的翻译单元中声明或定义事物。

它不会对不同翻译单元（即多个源文件）中的多重定义产生任何影响。

在这种情况下，您应该将函数LOG和LOGERR的定义移动到另一个.cpp文件，并将函数的声明放在头文件中。

Answer 2

Guards没有做错任何事，他们只是保护你的声明/内联/模板。这些定义是真正的问题。如果您的cpp中有内联函数，请将它们放在标题中，对于模板也是如此。不要包含cpp文件。看不到你的代码，但大多数情况都是如此。

功能的多重定义，为什么不守护这个？

2 个答案: