Нет жизнеспособного перегруженного '=' для назначения функции std :: function в качестве функции-члена

Question

Я пытаюсь подключить функцию-член до обратного вызова std::function. Тип обратного вызова:

std::function<void(Touch*, Event*)> 

Моя функция член:

void Game::onTouchEnded(Touch* touch, Event* event) 

При попытке назначить функцию обратного вызова, используя следующие:

listener->onTouchEnded = bind(&Game::onTouchEnded, this); 

я получаю ошибку No viable overloaded '=' с несколькими кандидатами , Вот подробности того, что я считаю, наиболее уместными:

Candidate function not viable: no known conversion from 
'__bind<void (Game::*)(cocos2d::Touch *, cocos2d::Event *), Game *>' 
to 
'std::__1::function<void (cocos2d::Touch *, cocos2d::Event *)>' 
for 1st argument` 

Я попробовал несколько различных Bind() меры, но я не уверен, что ошибка говорит мне. Типы кажутся правильными с точки зрения параметров и возвращаемого значения, поэтому я не уверен, почему он не примет его?

Answer 1

Попробуйте

listener->onTouchEnded = bind(&Game::onTouchEnded, this, std::placeholders::_1, std::placeholders::_2); 

или попробовать функцию лямбда

listener->onTouchEnded = [this](Touch* touch, Event* event){ this->onTouchEnded(touch, event); }; 

Answer 2

bind должно быть сказано, что делать с параметрами, передаваемыми в возвращаемого значения. По умолчанию он просто отбрасывает их.

Когда вы передаете его std::function, он пытается набрать его. std::function передает Touch*, Event* на номер bind. Возвращаемое значение bind отбрасывает их и вызывает (this->*&Game::onTouchEnded)(), как вы просили. Это недействительный вызов, и вы получаете сообщение об ошибке.

Простым способом исправить это является добавление заполнителей, которые говорят «что делать с аргументами для возвращаемого значения привязки». bind рассматривает функции-члены как функции, которые принимают дополнительный аргумент this. Вы хотите:

(this->*&Game::onTouchEnded)(_1, _2) 

что соответствует:

std::bind(&Game::onTouchEnded, this, _1, _2) 

---

Теперь это все хорошо, но на самом деле вы должны избегать std::bind. Полностью понять это сложно: у него есть странные угловые случаи, которые могут укусить вас или заставить вас бесполезно стирать шрифт, когда вы называете его рекурсивно.

Вместо этого используйте лямбда.В С ++ 11 он выглядит следующим образом:

[this](Touch* touch, Event* event){return this->onTouchEnded(touch, event);} 

В С ++ 14, альтернатива:

[this](auto&&args)->decltype(auto){return this->onTouchEnded(decltype(args)(args)...);} 

или даже:

template<class T, class M> 
auto bind_method(T* t, M* m) { 
    return [t, m](auto&&...args)->decltype(auto){ 
    return (t->*m)(decltype(args)(args)...); 
    }; 
} 

, которые могут быть использованы следующим образом:

bind_method(this, &Game::onTouchEnded) 

и является заменой на ваш звонок bind. Здесь мы автоматически переадресуем аргументы методу, который мы вызываем, вместо того, чтобы явно указывать их.