cocos2dxのコールバック関数を関数に書く方法

Question

void Title::MenuCreate(int posX,int posY,std::string _string,呼び出したい関数)←ここにコールバック関数を入れたい
{
//画面の幅を取る
Size visibleSize = Director::getInstance()->getVisibleSize();
Vec2 origin = Director::getInstance()->getVisibleOrigin();
// ボタンの枠の生成
Sprite* buttonframe = Sprite::create();
buttonframe->setTextureRect(Rect(0, 0, 110, 60));
buttonframe->setColor(Color3B::BLACK);
buttonframe->setPosition(Point(x, y));
this->addChild(buttonframe);
//ボタンの作成 Start
Sprite* startbutton = Sprite::create();
startbutton->setTextureRect(Rect(0, 0, 100, 50));
startbutton->setColor(Color3B::WHITE);
startbutton->setPosition(Point(x, y));
this->addChild(startbutton);
//ボタンの文字
auto label = Label::createWithSystemFont(_string, "arial", 24);
label->setColor(Color3B::RED);
auto buttonEvent = MenuItemLabel::create(label, CC_CALLBACK_1(呼び出したい関数, this));
//メニューの作成 menuの中にbuttonEventを入れる
Menu* menu = Menu::create(buttonEvent, NULL);
//pMenuを画面中央に配置
menu->setPosition(x,y);
this->addChild(menu);
}
//////////////////////////////////////////////////////////////////////////
ボタンを作る関数を作ってみようと思ったのですが
コールバック関数の中身を引数に入れる方法がイマイチ分かりません。

一応わかりやすいかと思って全文書きましたが太字の部分だけで言いたいことはわかると思います。

あと厚かましいかと思いますがもう少しプログラム良い書き方あったり、
わかりやすい方法があったらご指摘してくれたら嬉しいです

Answer 1

MenuCreateの引数についてはtoki_tdさんの形で正しいと思います。
呼び出したい関数の引数（この場合ClickButtonの引数）の数がCC_CALLBACK_1の数字と一致していないのでは？
見たところ引数を取らないようなので、CC_CALLBACK_0を用いて以下のコードで試したらきちんと動きました。

{
//呼び出し側
	this->MenuCreate(100, 100, "TEST", CC_CALLBACK_0(HelloWorld::ClickButton, this));
}
void HelloWorld::ClickButton(){}

Answer 2

未解決ですが、違う方法を模索します

Answer 3

MenuItemLabel::createメソッドの定義は以下のようになっています．

static MenuItemLabel * create(Node*label, const ccMenuCallback& callback);

このccMenuCallbackがコールバック関数の型になりますが，その定義は以下のようになっています．

typedef std::function<void(Ref*)> ccMenuCallback;

つまり，Refポインタを1つ引数に取る関数オブジェクトです．
マクロを使用せずに書くならば，以下のようになるでしょう．

MenuItemLabel::create(
    label,
    std::bind(&HogeScene::callbackMethod, this, std::placeholders::_1)
);

このstd::bindをマクロにしたものがCC_CALLBACK_Nになり，定義は以下のようになっています．

#define CC_CALLBACK_0(__selector__,__target__, ...) std::bind(&__selector__,__target__, ##__VA_ARGS__)
#define CC_CALLBACK_1(__selector__,__target__, ...) std::bind(&__selector__,__target__, std::placeholders::_1, ##__VA_ARGS__)
#define CC_CALLBACK_2(__selector__,__target__, ...) std::bind(&__selector__,__target__, std::placeholders::_1, std::placeholders::_2, ##__VA_ARGS__)
#define CC_CALLBACK_3(__selector__,__target__, ...) std::bind(&__selector__,__target__, std::placeholders::_1, std::placeholders::_2, std::placeholders::_3, ##__VA_ARGS__)

ですので，CC_CALLBACK_1を使用する方法で間違っていないはずなのですが，
うまく行かいないようでしたら，一度コールバック関数の方の定義を見なおしてみると良いでしょう．
先ほどの例ですと，コールバック関数の定義は以下のようになるはずです．

void HogeScene::callbackMethod(Ref* pSender);

Answer 4

CC_CALLBACK_1はbindをラップしただけでMenuItemLabel::createのコールバック(ccMenuCallback)はfunctionそのものです。
ですので呼び出し側でCC_CALLBACK_1で渡せばいいかと。

void Title::MenuCreate(int posX,int posY,std::string _string, const ccMenuCallback& callback)
{
  ....
  auto buttonEvent = MenuItemLabel::create(label, callback); 
  ....
}


呼び出し側
this->MenuCreate(x, y, text, CC_CALLBACK_1(&Title::menuItemClicked, this));

Comments

[sekaizyu]

出来ませんでした・・・
this->MenuCreate(x, y, text, CC_CALLBACK_1(&Title::ClickButton, this));
呼び出し側のCC_CALLBACKでエラーが出てます