Question

我正在尝试编译一个包含类头的main函数。 main函数以及类cxx文件使用名为“SayHello”的辅助函数。但是，如果我在main和类cxx文件中包含辅助文件“Hello.h”，则会出现多重定义错误。我在下面举了一些例子。你能救我吗？

干杯，安德烈亚斯

Main.cxx：

#include "Class.h"
#include "Hello.h"

int main(int argc, char **argv){
    SayHello();
    return 0;
}

Class.h

#ifndef CLASS_H
#define CLASS_H

class Class {
public:
    Class();
    ~Class();
};

#endif

Class.cxx

#include "Class.h"
// #include "Hello.h" //with this it breaks!

Class::Class(){
    // SayHello(); //with this it breaks!
}

Hello.h

#ifndef HELLO_C
#define HELLO_C

#include <iostream>

void SayHello(){
  std::cout<<"hello!"<<std::endl;
}

#endif

我的makefile：

# Compiler
CXX           = $(shell root-config --cxx)

# Compiler flags
ROOTCFLAGS   := $(shell root-config --cflags)
ROOTLIBS     := $(shell root-config --libs)
CFLAGS        = -Wall ${ROOTCFLAGS} ${INCLUDE_PATH}
LFLAGS        = -O3 ${ROOTLIBS} -lHistPainter

# Targets
EXE = Main
OBJS = Main.o Class.o

# Processing
# -------------------------------------
all: ${OBJS} ${EXE}
    @echo "Done"

${EXE}: ${OBJS}
    @echo "Making executable $(notdir $@)"
    ${CXX} ${CFLAGS} ${OBJS} ${LFLAGS} -o $@

${EXE}.o: ./${EXE}.cxx
    @echo "Compiling $(notdir $<)"
    ${CXX} $(CFLAGS) -c $< -o $@ 

%.o: ./%.cxx ./%.h
    @echo "Compiling $(notdir $<)"
    ${CXX} $(CFLAGS) -c $< -o $@ 

clean:
    @echo "Cleaning"
    @rm -f ./Main
    @rm -f ./Main_cxx.so
    @rm -f ./Main_cxx.d
    @rm -f ./*.o
    @rm -f ./*ACLiC_dict_rdict.pcm

Answer 1

您将SayHello函数的定义放在头文件中。这是错误的。

由于您将此头文件包含在两个不同的文件中，因此该定义在两个不同的文件中编译两次，并且您的链接器最终会在最后看到两个相同函数的定义。

这就是你的头文件Hello.h应该只包含函数原型的原因：

#ifndef HELLO_C
#define HELLO_C

void SayHello();

#endif

虽然定义应该在单独的Hello.cxx文件中：

#include <iostream>
#include "Hello.h"

void SayHello(){
  std::cout<<"hello!"<<std::endl;
}

当然，不要忘记将这个新文件添加到Makefile：

OBJS = Main.o Class.o Hello.o

更新

正如Rene所述，您还可以通过在标头中添加inline关键字来简单地将您的函数声明为内联。它将使您的编译器在每次调用此函数时复制整个函数内容，而不是执行函数调用。

这通常用于非常短的功能，通常是getter和setter功能。由于您的函数只是一个“Hello”打印，因此将其作为内联函数可能并不是一个坏习惯。

但要小心：代码中过多的内联函数会降低编译速度，并增加包含依赖性。例如，如果您内联SayHello函数，则包含"Hello.h"的任何文件也应包含<iostream>。

包含在main和class体中返回多个定义错误

1 个答案:

更新