Início / coding / Perguntas / 77294364
Accepted
user1402866
Asked: 2023-10-15 04:19:28 +0800 CST

Chamada de função incorreta ao usar um std::function como comparador

  • 772

Estou usando lambdas em vez de uma classe comparadora para meu conjunto, mas estou tendo problemas.

Eu tenho algo assim:

class Processor {
    
private:
    const function<bool(std::string, std::string)> _lenComp = [](string a, string b) { return a != b && a.size() >= b.size(); };
    set<string, decltype(_lenComp)> _inputSet;

// ...
    
public:
    Processor() : _inputSet() {
    }

    void addLine(string line) {
        _inputSet.insert(line);
    }

Quando crio uma Processorinstância e chamo addLine2 vezes, recebo uma exceção de chamada de função incorreta. Como inicializo corretamente esta classe?

c++

2 respostas

  1. Best Answer

    decltype(_lenComp)é function<bool(std::string, std::string)>que está vazio std::function, então você recebe uma chamada ruim ao chamá-lo.

    o que você queria era o tipo de lambda armazenado nele, você deveria definir isso com static constexpr auto, pois não é permitido saber o tipo de lambda.

    class Processor {
    
private:
    static constexpr auto _lenComp = [](string a, string b) { return a != b && a.size() >= b.size(); };
    set<string, decltype(_lenComp)> _inputSet;
};
    • 3

  2. Para expandir a outra resposta: nesta linha o tipo de lambda e o tipo de _lenCompnão são iguais:

    const function<bool(std::string, std::string)> _lenComp = [](string a, string b) { return a != b && a.size() >= b.size(); };

    std::function<bool(string, string)>é um functor wrapper genérico que pode conter qualquer tipo de função que retorne um bool e receba duas strings, seja um lambda, uma função regular ou um método estático. Você pode reatribuir a implementação real que esse contêiner encapsula.

    O que você fez foi criar um lambda que possui um tipo opaco (oculto) e armazená-lo em uma instância do wrapper. Então você passou o tipo do wrapper para o std::setmodelo.

    Quando _inputSeté construído, ele não tem conhecimento do seu lambda. Ele construirá uma nova instância do tipo de comparador fornecido, que no seu caso é um arquivo std::function. E um padrão construído std::functionestá vazio.

    Outra correção para o problema acima, se você quiser manter, std::functionseria passar a instância do comparador que você já possui para o conjunto quando ele for construído, assim:

    // ...

    const function<bool(std::string, std::string)> _lenComp = [](string a, string b) { return a != b && a.size() >= b.size(); };
    set<string, decltype(_lenComp)> _inputSet;

public:
    Processor(): _inputSet({}, _lenComp) {}

// ...
    • 2

relate perguntas