我怎么能在这里得到“已经定义”的链接器错误？

Question

我第一次尝试在 C++ 中进行单元测试，但我已经很多年没有使用过 C++（我目前主要是 C# 编码器）。看来我正在做一个正确的猪耳朵 - 我希望有人能引导我回到正义的道路上。我才刚刚开始，我真的很想使用可能的最佳实践来实施这些测试，所以欢迎任何和所有 cmets，即使我目前最担心我的链接器错误。

所以，我有一个整体解决方案“Technorabble”，包含子项目“CalibrationTool”和“CalibrationToolUnitTests”。

CalibrationTool 有一个 MathUtils.h 文件：

#ifndef __math_utils__
#define __math_utils__

#include "stdafx.h"
#include <vector>

namespace Technorabble
{
    namespace CalibrationTool
    {
        double GetDoubleVectorAverage(std::vector<double> v)
        {
            double cumulativeValue = 0;
            for(std::vector<double>::iterator iter = v.begin(); iter != v.end(); ++iter) 
            {
                cumulativeValue += *iter;
            }
            return cumulativeValue / v.size();
        }
    }; // end namespace CalibrationTool
};  // end namespace Technorabble

#endif // !__math_utils__

（但没有 .cpp 文件，因为我遇到了各种（有点相似）问题，让我的模板函数正常工作 - 所以我最终定义了内联）。

继续进行单元测试项目，我有一个 main.cpp：

#include "MathUtilsTest.h"

void RunMathUtilsTests();

int main()
{
    RunMathUtilsTests();

    // Other class tests will go here when I have things to test
}

void RunMathUtilsTests()
{
    MathUtilsTest* mathUtilsTest = new MathUtilsTest();
    mathUtilsTest->RunTests();
    delete mathUtilsTest;
}

最后，MathUtilsTest 类的标头和 cpp，同样非常简单：

.h:

#ifndef __MATH_UTILS_TEST__
#define __MATH_UTILS_TEST__

#include "CalibrationToolUnitTestsLogging.h"
#include "..\CalibrationTool\MathUtils.h"

class MathUtilsTest
{
public:
    MathUtilsTest();
    ~MathUtilsTest();

     bool RunTests();

private:    
    bool GetDoubleVectorAverageTest();

}; // end class MathUtilsTest

#endif

.cpp:

#include "MathUtilsTest.h"
#include <sstream>

bool MathUtilsTest::RunTests()
{
    return GetDoubleVectorAverageTest();

}

MathUtilsTest::~MathUtilsTest()
{

}

MathUtilsTest::MathUtilsTest()
{

}

bool MathUtilsTest::GetDoubleVectorAverageTest()
{
    bool passed = true;
    std::vector<double> values;
    for (int i = 1; i < 23; i++)
    {
        values.push_back(i);
    }
    // vector becomes: 1, 2, 3, 4, .....20, 21, 22.  Average is 11.5
    double expectedAverage = 11.5;
    double calculatedAverage = Technorabble::CalibrationTool::GetDoubleVectorAverage(values);
    if (calculatedAverage != expectedAverage)
    {
        std::ostringstream s;
        s << calculatedAverage;
        std::string avgString = s.str();
        CalibrationToolUnitTestsLogging::Write("Failed MathUtilsTest.GetDoubleVectorAverageTest:  " + avgString);
        passed = false;
    }
    else
    {
        CalibrationToolUnitTestsLogging::Write("Passed MathUtilsTest.GetDoubleVectorAverageTest");
    }

    return passed;
}

这一切对我来说似乎都很好，我正在使用#ifndef 等保护我的标题。但我仍然收到以下错误：

1) 错误 LNK1169：找到一个或多个多重定义符号

2) 错误 LNK2005: "double __cdecl Technorabble::CalibrationTool::GetDoubleVectorAverage(class std::vector >)" (?GetDoubleVectorAverage@CalibrationTool@Technorabble@@YANV?$vector@NV?$allocator@N@std@ @@std@@@Z) 已在 main.obj C:_SVN\Technorabble\Windows Software\CalibrationToolUnitTests\MathUtilsTest.obj 中定义

这怎么可能？谁能发现哪里出了问题？

Answer 1

标头中定义的函数应标记为inline:

inline double GetDoubleVectorAverage(std::vector<double> v)
{
}

如果超过几行，请考虑将其移至实现文件。

pragmas 或包含守卫不能防止多重定义。

请注意，您应该通过 const 引用而不是按值传递 v。

Answer 2

您正在标题中定义一个函数GetDoubleVectorAverage。这意味着它将在包含该标题的每个翻译单元（即每个源文件）中定义。如果您的程序包含多个这样的翻译单元，那么您将有多个定义 - 这是不允许的。

解决方案是：

• 在函数定义中添加inline，放宽这个规则，允许多个相同的定义；或
• 将函数定义移动到源文件中，并且只在头文件中声明。

我用#ifndef 保护我的标题

这只会防止标题在同一个翻译单元中被多次包含。它不会阻止包含多个单元。

此外，您不应该使用 reserved name 之类的 __math_utils__ 作为标头保护，即使互联网上充斥着这样做的狡猾代码示例。

在让我的模板函数正常工作时遇到了各种（有些相似的）问题

模板通常需要在头文件中定义，以使定义在使用时可用。函数模板隐含为inline，但普通函数（如这个）不是。