关于Dart规则use_build_context_synchronously的代码逻辑疑问

1. 修改后的代码中Navigator.of(context).pop()是否可能不执行？ #

是的，这条语句确实有可能不执行。当异步操作（比如示例中的Future.delayed）完成时，如果当前组件已经被从组件树中移除（即context.mounted返回false），代码会通过if (!context.mounted) return;提前退出，pop()自然不会被执行。

2. 为什么改变原有逻辑的写法更优？ #

先看原来的“坏代码”存在的问题：

// BAD CODE
void onButtonTapped(BuildContext context) async {
  await Future.delayed(const Duration(seconds: 1));
  Navigator.of(context).pop();
}

异步操作等待的过程中，用户可能已经触发了其他操作（比如返回上一页、退出当前页面），导致当前组件被销毁。这时context对应的组件已经不在组件树中，调用Navigator.of(context)会直接抛出状态错误，导致App崩溃。

而修改后的代码：

// GOOD CODE
void onButtonTapped() async {
  await Future.delayed(const Duration(seconds: 1));

  if (!context.mounted) return;
  Navigator.of(context).pop();
}

虽然跳过了组件已卸载时的pop()操作，但这是合理且必要的：

• 当组件已经被卸载，执行pop()本身没有任何意义——当前页面已经不存在，再执行返回操作完全多余；
• 更关键的是，这种写法彻底避免了因使用已失效的context而引发的崩溃，保证了App的稳定性。

简单来说，原代码的逻辑是“不管组件死活，异步完成就执行pop”，这会带来崩溃风险；修改后的逻辑是“组件还在才执行pop，不在就直接跳过”，既避免了错误，也符合实际业务场景——组件都没了，执行pop也没用。

内容的提问来源于stack exchange，提问作者Searene