Flutter: 子Widgetに渡したコールバック関数が実行されない問題

Question

Flutterの学習をしています。
親Widgetで実行する関数をコールバック関数として渡し、子Widgetの方で実行させる練習をしています。

スクリーンには値を入力するTextFieldとTextButtonのカスタムWidgetCustomTextButtonがあります。
TextFieldの値をprintするコールバック関数をCustomTextButtonに渡して、onPressedで実行したいのですが実行されませんでした。

import 'package:flutter/material.dart';

class Test extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    String? inputtedText;

    return MaterialApp(
      home: Scaffold(
        appBar: AppBar(
          title: const Text('test'),
        ),
        body: Column(
          children: [
            TextField(
              onChanged: (value) {
                inputtedText = value;
              },
            ),
            const SizedBox(
              height: 20,
            ),

            /// テキストフィールドの値を出力するコールバックを渡す。
            CustomTextButton(callBack: () {
              print(inputtedText ?? 'empty');
            })
          ],
        ),
      ),
    );
  }
}



class CustomTextButton extends StatelessWidget {
  const CustomTextButton({Key? key, required this.callBack}) : super(key: key);

  final Function callBack;

  @override
  Widget build(BuildContext context) {
    return TextButton(
      /// プレス時に実行されない。
      onPressed: () {
        callBack;
      },
      child: Container(
          color: Colors.lightBlueAccent,
          padding: const EdgeInsets.all(10.0),
          child: const Text(
            'print',
            style: TextStyle(color: Colors.white),
          )),
    );
  }
}

デバックしたところ、CustomTextButtonのプロパティcallBackの返り値がnullになっていました。

これは返り値を持たない関数が渡されたのでnullになったという解釈なのか、うまく渡されていないのかどちらなのかがわかりませんでした。

エラー発生してないため何が原因なのかがわかりません。

ご助力願います。

Answer 1

惜しかったですね、

onPressed: () {
 callBack();
},

括弧の２文字足りませんでした。カッコを付けないと「実行」はされないとイメージしてください。callBack.call()と書くことも一応できます。ただcallBackと書いても代入も処理も何もしていないコードなので何も起きません。コンパイルエラーにはならないと思いますが、何も起きていない式なのでコンパイラの最適化によってむしろ削除されているかもしれません。

ちなみに下記のように書くのをおすすめします。

final Function callBack;

これを

final VoidCallback? callBack;

に変えてみてください。そうすると、TextButtonのonPressedと型が一致します。でも仕様を見ると結局typedef VoidCallback = void Function();と定義されているので同じことですね。

そうすると

onPressed: () {
  callBack;
},

これを

onPressed: callBack,

このように書けます。すっきりしました。もちろんタップ時にコールバックがコールされます。
（記憶から書いていて未検証なのでどこか間違っていたらすみません。）

Comments

[massanmesu]

ありがとうございます。うまく動作しました。
ですが、コールバックの型を`Function`から`VoidCallback`に変えても、実行時には()は必要でした。

```
final VoidCallback? callBack;
...
onPressed: callBack(),
```

簡易ドキュメントを見ても、返り値は`void Function()`であることは間違い無いのですが、仕様が変わったんでしょうかね？

[hiroshihorie]

実行時には必ず()が必要です。

パラメータとして渡す場合は
onPressed: callBack(),
ではなくて
onPressed: callBack,
と書きます。

つまり
onPressed: () {
callBack();
},
と書くか
onPressed: callBack,
と書くかの２択です。