【问题标题】:Error "bad function call" when passing lambda to STL set将 lambda 传递给 STL 集时出现“错误的函数调用”错误
【发布时间】:2019-10-26 22:10:08
【问题描述】:

我正在尝试将 lambda 作为排序标准传递给一组 int 类型的索引。我使用 unordered_map 将索引映射到它们的值,并将其按值传递给 lambda,它接受两个索引并返回 bool。

代码可以编译,但在运行时出现“错误的函数调用”错误。

我已经使用 std::function 类型包装器来声明 lambda,而不是使用 decltype() 会出错。

代码:

#include <iostream>
#include <unordered_map>
#include <functional>
#include <set>
#include <vector>

using namespace std;

int main()
{
    int T;
    cin >> T;

    while (T--)
    {
        int N;
        cin >> N;

        unordered_map<int, int> M;
        function<bool(int, int)> l = [M](int i1, int i2)->bool { return M.at(i1) > M.at(i2); };
        set<int, function<bool(int, int)>> S;
        for (int i = 1; i <= N; ++i)
        {
            int p;
            cin >> p;

            M[i] = p;
            S.insert(i);
        }

        vector<vector<int>> Adj(N + 1);
        for (int i = 1; i <= N - 1; ++i)
        {
            int u, v;
            cin >> u >> v;

            Adj[u].push_back(v);
            Adj[v].push_back(u);
        }

        int maximum = 0;
        for (int i = 1; i <= N; ++i)
        {
            S.erase(i);
            for (int e : Adj[i])
                S.erase(e);
            cout << *S.cbegin() << ' ';
            for (int e : Adj[i])
                S.insert(e);
            S.insert(i);
        }
        cout << endl;
    }
}

输入:

1
6
5 10 15 20 25 30
1 3
2 3
3 4
4 5
4 6

我在 try-catch 块中有这段代码,当我输入第二个值(输入中的 10)时,它正在打印“错误的函数调用”

【问题讨论】:

  • 请尝试创建minimal reproducible example 向我们展示。您显示的代码不使用 lambda。或者也许这的问题?你没有告诉集合使用你的 lambda,而是尝试使用一些未指定的函数?
  • 您是否尝试通过这样做为std::set 设置谓词:set&lt;int, function&lt;bool(int, int)&gt;&gt; S;?如果是这样,我认为语法不正确。
  • 您正在捕获一张空地图。在其中查找内容是个坏主意。
  • 还要注意[M]“通过复制捕获”,这意味着对包含函数中的M 的更改不会对lambda 中使用的M 产生任何影响。看起来您希望 [&amp;M] “通过引用捕获”。 (但要确保std::function 的任何副本都不会比M 寿命更长!)
  • 在使用 std::function 之前,我使用 auto 并使用 decltype(l) 传递集合中的类型,但它没有编译,所以我尝试了这个并编译了。我应该上传整个代码吗?

标签: c++ lambda stl set


【解决方案1】:

我在 try-catch 块中有这段代码,它正在打印“错误的函数调用”

当您尝试调用默认初始化的std::function 时会发生这种情况:

std::bad_function_call 如果 *this 不存储可调用函数目标,即 !*this == true。

std::function&gt;::operator()的文档中提供

当您初始化对象时,您不会将仿函数传递给std::set 构造函数,因为您使用std::set 的默认ctor:

set<int, function<bool(int, int)>> S; 

所以应该改用这个:

set<int, function<bool(int, int)>> S( l ); 

并且您的 lambda 应该通过引用捕获 M,因为您稍后对 M 所做的任何更改都不会反映在此 lambda 中,因为它按值捕获 M

【讨论】:

  • 我上传了完整的代码,但我不明白你在说什么,你能详细解释一下吗?编辑:我尝试在 S 的构造函数中传递 l 但仍然是同样的错误。
  • @lucieon 你需要告诉集合 S 使用你的 lambda。
  • 有什么特别不明白的地方?您使用默认 ctor 创建您的集合S,默认初始化仿函数,然后您会得到异常。我向你展示了你应该修复的那条线。
  • @Someprogrammerdude 好的,我忘记通过引用传递 M,它现在正在工作。谢谢大家!
  • 请不要在您询问和回答后更改您的代码,这会使您的回答无效
猜你喜欢
  • 1970-01-01
  • 2021-04-13
  • 1970-01-01
  • 2014-12-10
  • 1970-01-01
  • 2020-05-09
  • 1970-01-01
  • 1970-01-01
  • 1970-01-01
相关资源
最近更新 更多