14

在我的一个课程中，我尝试使用std::priority queue指定的 lambda 进行比较：

#pragma once
#include <queue>
#include <vector>

auto compare = [] (const int &a, const int &b) { return a > b; };
class foo
{
public:
    foo() {  };
    ~foo() {  };
    int bar();
private:
    std::priority_queue< int, std::vector<int>, decltype(compare)> pq;
};

我的程序编译完美，直到我添加一个.cpp文件来伴随标题：

#include "foo.h"

int foo::bar()
{
    return 0;
}

这一次，我的编译器产生了一个错误：

>main.obj : error LNK2005: "class <lambda> compare" (?compare@@3V<lambda>@@A) already defined in foo.obj

.cpp如果我的头文件包含 lambda，为什么我不能创建随附文件？

编译器：Visual Studio 2012

我的main.cpp：

#include "foo.h"

int main(){
    return 0;
}
4

2 回答 2

12

正如@Rapptz 建议的那样，

const auto compare = [] (const int &a, const int &b) { return a > b; };

解决了这个问题。为什么？

内部与外部联系。默认情况下auto，, likeint具有外部链接。那么如何：

int j = 5;

foo.h稍后将包含在throws foo.cppa

错误 2 错误 LNK2005: "int j" (?j@@3HA) 已在 Header.obj 中定义

（对比 2013 年）

但是，默认情况下const将链接设为内部，这意味着它只能在一个翻译单元中访问，从而避免了该问题。

于 2014-06-01T12:52:45.773 回答
1

由于某种原因，我无法复制此问题。不过，我正在 VS2010 上尝试这个 - 不确定这是否有所作为。事实上，我尝试将您的标头包含在两个源文件中，它可以编译、链接并运行良好。

也就是说，您是否要考虑使用std::function. 这样您就可以在 cpp 代码中定义 lambda，并且无论出于何种原因，它都不会被多次定义。（顺便说一句，从哪里来foo.obj？你有另一个包含这个头文件的源文件吗？）。

富.h：

#pragma once
#include <queue>
#include <vector>
#include <functional>

typedef std::function<bool (int, int) > comptype;
//auto compare = [] (const int &a, const int &b) { return a > b; };
class foo
{
public:
    foo() {  };
    ~foo() {  };
    int bar();

private:
    std::priority_queue< int, std::vector<int>, comptype> pq;
};

然后稍后在 cpp 中包含并定义 lambda，并在创建时将其pq传递给构造函数。

foo.cpp：

auto compare = [] (const int &a, const int &b) { return a > b; };

foo::foo():pq(compare){}

这样，您就不会巧妙地多次定义该函数。

于 2013-08-07T23:22:39.920 回答