[Dxlib]関数ポインタを引数にもつ関数を作成したい

Question

前提・実現したいこと

関数ポインタを引数に持つ関数を作成したい。

実装したいものは、以下のように関数の入った箱（MethodBox）とそれを読むリーダーがあり、リーダーは読み込んだ関数をキューへ保存していき、一番右端のMethodBoxを読み終わったら、順次キューに入った関数を実行していくようなシステムを作りたい。

発生している問題・エラーメッセージ

C2664'MethodBox::MethodBox(int,int,int,int,const char *,Method::DRAW)': 引数 6 を '_Ty' から 'Method::DRAW' へ変換できません。	
DxLibTemplate2017　C:\Program Files (x86)\Microsoft Visual Studio\2019\Community\VC\Tools\MSVC\14.28.29910\include\xutility	

該当のソースコード

関数ポインタの定義　Method.h

#pragma once
#include <DxLib.h>

class Method
{	
public:
	Method() = default;
	~Method() = default;

	typedef void (*DRAW) (); //円を描画する関数のポインタ

	void RedCircle()
	{
		DrawCircle(100, 100, 10, GetColor(255, 0, 0));
	}

	void BlueCircle()
	{
		DrawCircle(100, 100, 10, GetColor(0, 0, 255));
	}
};

関数を持つ箱　MethodBox.h

#pragma once

#include "Task.h"
#include "Method.h"

class MethodBox : public Task //このように継承するとMethodBoxを使用する関数からもTaskの公開メンバをいじれる
{
private:
	int m_x, m_y, m_sizeX, m_sizeY; //箱の位置・大きさ
	const char* m_str; //箱に書かれている文字
	Method::DRAW m_fp; //読まれると実行される関数のポインタ

public:
	MethodBox(int _x, int _y, int _sizeX, int _sizeY, const char* _str, Method::DRAW _fp);
	~MethodBox() = default;

	bool update() override;
	void draw() const;
	void move(); //D&D
};

Method.cpp

#include "MethodBox.h"
#include "Define.h"
#include "DxLib.h"

MethodBox::MethodBox(int _x, int _y, int _sizeX, int _sizeY, const char* _str, Method::DRAW _fp)
{
	m_x = _x;
	m_y = _y;
	m_sizeX = _sizeX;
	m_sizeY = _sizeY;

	m_str = _str;
	m_fp = _fp;
}

bool MethodBox::update()
{
	move();
	return true;
}

void MethodBox::draw() const
{
	int width = m_x + GetFontSize(); //文字のx位置
	int height = m_y + (m_sizeY / 2) - GetFontSize()/2; //文字のy位置

	DrawBox(m_x, m_y, m_x + m_sizeX, m_y + m_sizeY, GetColor(255, 255, 255), true); //四角形の描画
	DrawFormatString(width, height, GetColor(0,0,0), m_str); //文字の描画
}

//D&D処理
void MethodBox::move()
{
	//左クリックされていたら
	if (GetMouseInput() & MOUSE_INPUT_LEFT)
	{
		int mousePosX, mousePosY;

		GetMousePoint(&mousePosX, &mousePosY); //マウスの座標を取得する

		//クリックした座標がMethodBoxの範囲内であれば、
		if 
		(
			(mousePosX >= m_x) && (mousePosX <= (m_x + m_sizeX)) //X範囲
			&&
			(mousePosY >= m_y) && (mousePosY <= (m_y + m_sizeY)) //Y範囲
		)
			while (GetMouseInput() & MOUSE_INPUT_LEFT) //左クリックされている間、
			{
				GetMousePoint(&mousePosX, &mousePosY);

				//マウスカーソルが画面内であれば
				if(mousePosX > 0 && mousePosX < Define::WIN_W && mousePosY > 0 && mousePosY < Define::WIN_H)
				{
					//MethodBoxの座標を変更
					m_x = mousePosX - (m_sizeX / 2);
					m_y = mousePosY - (m_sizeY / 2);
				}
				
			}		
	}
}

MethodBox生成　GameScene.cpp

#include "GameScene.h"
#include <DxLib.h>

using namespace std;

//このシーンで使用するオブジェクトの生成
GameScene::GameScene()
{
    m_field = make_shared<Field>();
    m_reader = make_shared<Reader>();
    m_methodBox = make_shared<MethodBox>(200,200,130,50,"赤い丸を表示", &Method::RedCircle);
}

//現在のシーンのフレーム処理
bool GameScene::update()
{
    m_field->update();
    m_reader->update();
    m_methodBox->update();
    return true;
}

//現在のシーンの描画処理
void GameScene::draw() const
{
    m_field->draw();
    m_reader->draw();
    m_methodBox->draw();
}

試したこと

Typedefの使い方などを調べなおした。
関数のポインタを使用する -C++プログラミング

補足情報（FW/ツールのバージョンなど）

windows10 VisualStudio2019
プログラミング初心者です。足りない情報などあれば追記いたしますのでご連絡ください。

Comments

[episteme]

MethodBox のコンストラクタ(MethodBox::MethodBox)を呼んでるのはどこですか?

[THETA97556919]

すぐに追記いたします！すみません！

[int32_t]

提示されているコードでは Method::RedCircle() は this に依存していないので static を付けられますが、MethodBox に渡したい関数はすべて static メンバ関数か非メンバ関数にできますか?

[THETA97556919]

すみません、本当に初心者すぎて「Method::RedCircle() は this に依存していないので static を付けられます」のところが理解できてません。軽く教えていただけることはできますか？
少し調べてみましたが、グローバル関数のようなものとして捉えても大丈夫ですか？

[int32_t]

すべてを詳細に説明するのは面倒なので質問を変えると、「Methodクラスにデータメンバを足して、Method::RedCircle()からそのデータメンバを参照する予定はありますか？」です。

現状のコードだと「static void RedCircle()」にすれば解決するでしょうけど、もし上記質問の答えが「はい」なら別の解決方法が必要です。

[THETA97556919]

Methodクラスのメンバ変数に参照する関数は今のところ作る予定はないです！
おっしゃる通りstaticをつけるとうまく動きました！！詳しい解決法の提示ありがとうございましたm(_ _)m
なぜstatic必要なのか、どのような不具合だったのかはまた自分で調べて理解してみます。

Answer 1

現状だと Method::RedCircle と Method::DRAW は型が異なるので代入できません。Method::RedCircle を呼ぶだめには Method 型のインスタンスが必要なのです。

コメントによるとMethod::RedCircleからデータメンバを使う予定はないとのことなので、このメンバ関数をstaticにしてしまえば良いです。

Method.h:

    static void RedCircle()
    {

---

そもそも関数ポインタを避けるのが今どきのC++の流儀です。typedef void (*DRAW) ();の代わりに

    using DRAW = std::function<void(void)>;

これで関数ポインタもメンバ関数もラムダ式も対応できるようになります。
(メンバ関数を代入するにはstd::bindかラムダ式の助けが必要)

例:

    m_methodBox = make_shared<MethodBox>(200,200,130,50,"赤い丸を表示", []() {
        DrawCircle(100, 100, 10, GetColor(255, 0, 0));
    });

Answer 2

Method::RedCircle()をメンバ関数から静的メンバ関数にすることで解決した。

Method.h (一部抜粋)

static void RedCircle()
    {
        DrawCircle(100, 100, 10, GetColor(255, 0, 0));
    }

static void BlueCircle()
    {
        DrawCircle(100, 100, 10, GetColor(0, 0, 255));
    }

Answer 3

m_methodBox = make_shared<MethodBox>(200,200,130,50,"赤い丸を表示", &Method::RedCircle);

あ、こりゃダメです。

Method::DRAW は void (*)() であるのに対し
&Method::RedCircle は void (Method::*)() です。 両者に互換性はありません。