0

我在库（alibrary.lib）中有一个头文件。该库是一个静态库 (.lib)，它正确链接到 exe。

现在，我有一个类：Vector3d。

class Vector3d 
{
    void amethod()
    {
       blah
    }

};

Vector3d cross(const Vector3d &v0, const Vector3d &v1) 
{
      float x,y,z;

      x = v0.y*v1.z-v0.z*v1.y;
      y = v0.z*v1.x-v0.x*v1.z;
      z = v0.x*v1.y-v0.y*v1.x;

     return Vector3d(x,y,z);

}

Vector3d 在头文件 (Vector3d .h) 中声明和定义。在类声明之后，我使用了交叉函数。

lib compile 是文件，但是当它链接到单元测试 exe 时，我收到此错误：

 flywindow.obj :error LNK2005: "class Vector3d __cdecl cross(class Vector3d const &,class Vector3d const &)" (?cross@@YA?AVVector3d@@ABV1@0@Z) already defined in fly.obj

有任何想法吗？

谢谢

4

2 回答 2

2

如果您定义一个自由（不是类的成员）函数，它必须在单独编译的 .cpp 文件中定义，或者在标头中定义并标记为内联。因此，在您的情况下，您可以通过以下方式对其进行编译：

inline Vector3d cross(const Vector3d &v0, const Vector3d &v1) {
      float x,y,z;

      x = v0.y*v1.z-v0.z*v1.y;
      y = v0.z*v1.x-v0.x*v1.z;
      z = v0.x*v1.y-v0.y*v1.x;

     return Vector3d(x,y,z);

}

该错误是因为您在标题中定义了函数，但没有将其标记为内联。如果您现在将该头文件包含在两个分别编译的文件中，则链接器在尝试链接已编译的目标文件时将抛出错误，因为它会看到一个交叉函数被定义了两次。

它无需显式地为类的成员函数内联即可工作，因为在类定义中定义的成员函数是隐式内联的。

然而，在头文件中定义函数通常不是一个好主意。如果您的函数将依赖于其他类型而不仅仅是向量（在您的情况下，恕我直言，这很好，但它当然值得商榷 - 有些人不喜欢它），那么您将需要包含这些类型的标题。这将不必要地膨胀标题中间接包含的代码。相反，在这些情况下，您只需将函数声明放在标题中：

Vector3d cross(const Vector3d &v0, const Vector3d &v1);

但是在单独编译的 .cpp 文件中定义它。当然，应该删除内联。

让我添加一个定义和声明的小列表，只是为了帮助清楚地了解声明和定义对函数和类的含义。请注意，每个定义也是一个声明，但不是相反：

// class _declaration_ of boo
class boo;

// class _definition_ of foo.
class foo {
    // member function _declaration_ of bar
    void bar();

    // member function _definition_ of baz
    void baz() { }
};

// function _definition_ of fuzz
inline void fuzz() { }

// function _declaration_ of fezz
void fezz();
于 2009-02-01T04:14:32.330 回答
1

最可能的解释是您的包含文件中有代码（特别是交叉的定义），并且您的包含文件被两个源文件包含，因此是双重定义。

头文件中应该有声明，而不是定义。声明（表示存在某种东西）是诸如 typedef、类声明、枚举等之类的东西。

定义（对存在的事物赋予意义）是诸如函数、变量定义等之类的事物。

您的交叉函数应在头文件中声明：

Vector3d cross(const Vector3d &v0, const Vector3d &v1);

但在单独的源文件中定义。

于 2009-02-01T04:12:09.823 回答