C言語で関数ポインタ登録がうまくいかない

Question

inputFunc.input1～4に処理a～cをsetに合わせて登録したいのですが
setInputFunc関数で引数func(inputFunc.input1～4)に処理a～cののアドレスがセットされると思ったのですが、セットされません。
何か問題あるでしょうか？
ちなみに

inputFunc.input1 = a;

とした場合正常に関数aのアドレスがセットされています。

static struct{
	void	(*input1)(int input);	/* 入力1に対して設定されている処理 */
	void	(*input2)(int input);	/* 入力2に対して設定されている処理 */
	void	(*input3)(int input);	/* 入力3に対して設定されている処理 */
	void	(*input4)(int input);	/* 入力4に対して設定されている処理 */
}inputFunc;				/* 実行処理登録構造体 */

/* 実処理 */
static void a(int input){}
static void b(int input){}
static void c(int input){}

/* 実行処理登録 */
static void setInputFunc(void (*func)(int), int set)
{
	switch(set){
		case 0:
			func = a;
			break;
		case 1:
			func = b;
			break;
		case 2:
			func = c;
			break;
	}
}

static void main(void)
{
	setInputFunc(inputFunc.input1, 0);
	setInputFunc(inputFunc.input2, 1);
	setInputFunc(inputFunc.input3, 2);
}

Answer 1

こんにちは。

この辺、難しいですよね。

ポインタも変数の一種です。以下は以前書いた図なのですが、aPointerはint型変数へのポインタです。この図のようにメモリが割り当てられて、その中にポイント先のアドレスが記録されます。

関数ポインタの場合も同じです。
inputFunc.input1等もポインタ変数と考えると良いです。

setInputFunc()関数でinputFunc.input1へ値を設定するのですから、inputFunc.input1へのポインタを渡す必要があります。
ですので、呼び出しは下記のように書きます。

setInputFunc(&(inputFunc.input1), 0);

受け取り側も対応が必要です。
ポインタへのポインタは**となりますので、下記です。

static void setInputFunc(void (**func)(int), int set)

そして、funcは「input1等のポインタ変数」へのポインタですから、input1をアクセスするには*をつける必要があるので、*funcで値を設定すればOKです。

Comments

[luckyclock]

詳しい回答ありがとうございます。
ポインタへのポインタは普段あまり使わないのではまってしまいましたが、説明をみてよくわかりました。

Answer 2

setInputFunc関数内でしかfuncの値が変わっていないからです。
func( int x)などで呼出元の値が変わらないのと同様です。

以下のようにポインタ変数で渡すと意図する動作になるかと思います。

typedef void (*FUNC_PTR)(int);  // 関数ポインタ型をtypedef
static struct{
    FUNC_PTR input1;
    // 以下同様
}inputFunc;                /* 実行処理登録構造体 */

static void setInputFunc( FUNC_PTR *p_func, int set) // ポインタ変数で渡す
{
    // 略
    *p_func = a;
}
static void main(void)
{
    setInputFunc( &(inputFunc.input1), 0);
    // 以下同様

Comments

[luckyclock]

回答ありがとうございます。
投稿時間が一番早かったのでベストアンサーとさせていただきました。
意図する動作になりました。

[can110]

投稿２番目のようです^^; BA変更しもらってかまいません

Answer 3

intの値を書き換えるにはint * pで引数を受け取って*P = 100;と代入しないといけないのと同様、ポインタを書き換えるにはポインタのポインタを引数とする必要があります。

static void setInputFunc(void (**func)(int), int set) {
  // 1例のみに省略
  *func = a;
}

int main(void)
{
    setInputFunc(&inputFunc.input1, 0);
    setInputFunc(&inputFunc.input2, 1);
    setInputFunc(&inputFunc.input3, 2);
}

なお、mainは外部とリンクする必要がありますので、staticとしては動かないはずです。

Comments

[luckyclock]

回答ありがとうございます。
回答いただいたコードでの動作確認しました。