在Flutter的build方法内声明函数是否合理？是否属于不良实践？

你给出的实现代码如下：

@override
Widget build(BuildContext context) {

  void doRegister() async {
    ...
  }

  return Scaffold(
    ...
  );
}

这种在build方法内部声明业务函数的做法属于不推荐的开发实践，存在多处隐患，通常建议将这类函数移至build方法外部，作为State类的成员方法存在。

存在的核心问题 #

• 不必要的性能开销：build方法会在状态更新、页面依赖（如主题、MediaQuery）变化时被频繁触发，每次执行都会重新创建一个全新的doRegister函数实例，会增加垃圾回收的压力，高频触发场景下会积累不必要的性能损耗。
• 容易出现闭包逻辑隐患：如果内部函数引用了build作用域内的临时变量，且函数被作为回调传递给子组件，子组件会持有旧build生成的函数实例，很可能出现读取到过时变量值的异常逻辑，排查成本极高。
• 代码可维护性差：大量内部函数堆积在build方法中会让代码结构臃肿，业务逻辑和视图构建逻辑耦合在一起，后续迭代、排查问题的成本会明显升高。

优化建议 #

• 所有包含业务逻辑的命名函数，都建议抽到State类中作为成员方法，需要用到context时可以直接使用State类自带的context属性，不需要依赖build的作用域。
• 仅当回调逻辑非常简短、仅作用于当前视图节点、不会向外传递时，可以在build中直接写匿名回调（例如onPressed: () => Navigator.pop(context)），这种场景的性能影响可以忽略。

内容的提问来源于stack exchange，提问作者Kasymbek R. Tashbaev