来自重载<<运算符 - C++的奇怪链接器错误

Question

我写了一个小的Timer类来检查代码的性能.源代码可以在这里找到http://pastebin.com/i1PX2VPN(稍后我将重点介绍重要的部分)

现在我遇到了一个非常奇怪的错误:上面链接的代码位于文件Timer.hpp中.我在main.cpp中使用这个文件,它工作得很好.但是,我也想在另一个源文件中使用它,然后,只要我添加一个

  #include <Timer.hpp> 

进入下一个源文件,我收到以下链接器错误:

  ld: duplicate symbol operator<<(std::basic_ostream<char, std::char_traits<char> >&, Timer const&) in /var/folders/XZ/XZ93KWBqG0SR1aCVpTCVQE+++TI/-Tmp-//ccyaHyyU.o and /var/folders/XZ/XZ93KWBqG0SR1aCVpTCVQE+++TI/-Tmp-//ccETAaTw.o
  collect2: ld return 1 as exit status
  make: *** [debugd] Error 1

功能是

 std::ostream& operator<<(std::ostream& os, const Timer& t) {
         return os << std::scientific << std::setw(8)  << std::setprecision(3)
                            << t.timeTotal_ << " \t " << t.timeMin_ << " \t " <<          t.timeMax_;
   }

没有什么太花哨了......再说一遍:只要它只包含在我的main.cpp中就可以了

显然,这似乎是运算符的多个定义(？)的问题.但我无法理解为什么或如何解决它.我使用include-guard来保存它不被包含多次.

我已经检查过C++中是否有其他Timer类 - 我从未听说过的库,显然不是这种情况(重命名类不会改变任何东西)...

在Timer.hpp中,还定义了另一个类,也有自己的运算符<< overloaded.我不知道这是否重要,我想我应该提一下

所以...我希望转发你的想法,谢谢

Answer 1

您应该将您的函数标记为inline,因为它是一个全局函数,其定义包含在多个翻译单元中.

    inline std::ostream& operator<<(std::ostream& os, const Timer& t) {
//  ^^^^^^
         return os << std::scientific << std::setw(8)  << std::setprecision(3)
               << t.timeTotal_ << " \t " << t.timeMin_ << " \t " 
               << t.timeMax_;
    }

或者,您可以将函数的声明保留在标头中,并将定义放在一个.cpp文件中.

如果您想知道为什么包含警卫不会从多个符号定义中保存您,您可能会发现StackOverflow上的此问答很有用.