Golang コールバック関数のテストが通らない

Question

解決したいこと

初歩的な質問ですみません。golangでユーザー定義した関数を引数に取る関数のテストを書いているのですが、main_test.goで書いた値が、関数の引数に代入されません。これはなぜそうなってしまうのでしょうか？

###main.go

package main

import "fmt"

// debugツールで、a,bの値を見ると、a=0,b=0になっている
// var a,b intを消すと、undeclared name: ~t怒られる
func hoge(x int, f func(a, b, x int) int) int {
	var a,b int
	return f(a,b,x)
}


func fuga(a, b, x int) int {
	return a*x + b
}

func main() {
	ret := hoge(2, fuga)
	fmt.Println(ret)
}

###main_test.go

package main

import (
	"testing"
)

func TestLinearFunction(t *testing.T) {
	tests := []struct {
		a int
		b int
		x int
		expected int
	}{
		{1,1,1,2},{2,1,1,3},{1,2,1,3},{1,1,2,3},
		{2,2,1,4},{2,1,2,5},{1,2,2,4},{2,2,2,6},
	}

	for _, test := range tests {
		result := hoge(test.x,fuga)
		if result != test.expected {
			t.Errorf("input: a=%d b=%d x=%d, expected:%d, got:%d",test.a,test.b,test.x,test.expected,result)
		}
	}
}

test結果

--- FAIL: TestLinearFunction (0.00s)
input: a=1 b=1 x=1, expected:2, got:0
input: a=1 b=1 x=1, expected:3, got:0
input: a=1 b=1 x=1, expected:3, got:0
input: a=1 b=1 x=1, expected:3, got:0

・・・以後も取得するのは「0」

実際にPASSしたコード

main.go

package main

import "fmt"

func hoge(a, b, x int, f func(int, int, int) int) int {
	return f(a, b, x)
}

func fuga(a, b, x int) int {
	return a*x + b
}
// 一例
func main() {
	ret := hoge(1,2,3, fuga)
	fmt.Println(ret)
}

main_test.go

package main

import (
	"testing"
)

func TestLinearFunction(t *testing.T) {
	tests := []struct {
		a int
		b int
		x int
		expected int
	}{
		{1,1,1,2},{2,1,1,3},{1,2,1,3},{1,1,2,3},
		{2,2,1,4},{2,1,2,5},{1,2,2,4},{2,2,2,6},
	}

	for _, test := range tests {
		result := hoge(test.a,test.b,test.x,fuga)
		if result != test.expected {
			t.Errorf("input: a=%d b=%d x=%d, expected:%d, got:%d",test.a,test.b,test.x,test.expected,result)
		}
	}
}

Answer 1

hogeの引数fの型宣言は関数型でa,b,xの引数が書かれていますがこれは
関数型宣言のための仮引数です。

func hoge(x int, f func(a, b, x int) int)

fを実際に呼ぶときのパラメータは別途必要です。

fの型宣言におけるa, b, xは仮引数といってただの「わかりやすいタグ」なので省略できます。
どこかで、テストデータのa,bと勘違いしているのだと思います。
仮引数を省略しても別の名前にしてもコンパイルは通るのでお試しください。
以下のように書いても良いわけです。

func hoge(x int, f func(int, int, int) int) {
    ...
}

return f(a,b,x)のxは実引数xが渡されているのでエラーにはなりませんが、
a,bは実引数がないのでエラーになります。

var a,b intという宣言はゼロ値で初期化されたa,bが作られるだけです。
テストデータのtestsにある値のaやbをhogeやfugaに渡している引数がないのでこのままの
現状のhogeの実装では期待したテスト結果にはなりません。

hogeの実装は以下のようになっている必要があると思います。

func hoge(a, b, x int, f func(int, int, int) int) int {
    return f(a, b, x)
}

Comments

[ikutyan46]

「何を渡して、何を期待するのか」自分でも考え直し、無事にテストコードもPASSして理解できていることを確認できました。nobonoboさん、仮引数・実引数の関係など詳細なご説明、ありがとうございました！