Goの無名関数内で使用する変数がnil pointerになる

Question

前提

golang 1.18

実現したいこと

無名関数を変数に代入し、特定のエラーの時のみ再実行したい。

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

再実行した時に、無名関数内で使用している変数の値がnil pointerになります。

該当のソースコード

func (us *Interactor) CreateTrs(ctx context.Context, trs *transaction.Transaction) (*transaction.Transaction, apperror.AppError) {
	var err apperror.AppError

	// begin db transaction
	f := func() apperror.AppError {
		err := us.tx.WithTx(ctx, func(ctx context.Context) apperror.AppError {
			fmt.Println("-------------------------")
			fmt.Println(trs)
			fmt.Println("-------------------------")
			if err = us.transactionService.ConfirmNotExceedAmountLimit(ctx, trs.UserID, trs.Amount); err != nil {
				return err
			}

			trs, err = us.transactionRepository.Save(ctx, trs)
			if err != nil {
				return err
			}

			return nil
		})
		return err
	}
	err = f()
	if errors.Is(特定のエラー)) {
		err = f()
	}
	if err != nil {
		return nil, err
	}

	return trs, nil
}

errors.Isがtrueになった場合 f()が再試行されるとこまではいいのですが、 trsがnilになり nil pointerになってしまいます。

fmt.Println("-------------------------")
fmt.Println(trs)
fmt.Println("-------------------------")

// output
-------------------------
<nil>
-------------------------

試したこと

無名関数の代入前にnilになる変数をコピーすれば nilにならない事を確認しました。

ttt := trs
f := func() apperror.AppError {
// tttを使う

上記で一応エラーは回避できたのですが、これが良い方法なのか、中で何が起こっているのかがわかっておりません。
知見がございましたらアドバイス頂けると幸いです。

Answer 1

見慣れない書き方かつ未知のライブラリを参照しているので把握しきれていないですが、
疑似コードを書いてみるとちゃんとnilにならずに動作しました。

https://go.dev/play/p/xe0Pvj3PvbE

おそらくリトライ時「us.transactionRepository.Save」でエラーが出ていないでしょうか？
（つまりerrがnot-nilでtrsにnilが代入されている？）

無名関数の外側の変数を参照するだけならいいのですが、
無名関数の外側の変数を上書きするのは挙動が読みにくくなるのでお勧めしません。
（あまり使いまわすメリットもないと思います）

Comments

[Shika_Tech]

playgroundまで作って頂きありがとうございます。

実際には以下のようなコードに近いです。
https://go.dev/play/p/nWJHg6XDO4k

> おそらくリトライ時「us.transactionRepository.Save」でエラーが出ていないでしょうか？
（つまりerrがnot-nilでtrsにnilが代入されている？）

まさにその通りで Saveでerrが返り trs がnilとなっている場合に リトライ処理をすると
trs.UserID などで trsのフィールドを見る時に nil pointer になっておりました。

ありがとうございました！