linter 规则和 null 安全

linter rule and null safety

我最近用 flutter_linter 替换了已弃用的迂腐包 现在我有数百个这样的警告:

Avoid using `forEach` with a function literal.

avoid_function_literals_in_foreach_calls

有关

如果您有不可为空的列表,这听起来不错,但如果代码如下所示:

GetIt<SomeService>().someBehaviorSubject.value?.data?.list?.forEach((f) {
  //
});

怎么改?所以重构后应该不难看了

有趣的问题。我真的没有很好的答案,但这是我目前能想到的最好的答案。

第一个解决方案可能不是最有效的方法,因为我们正确地创建了一个新的列表对象:

void main() {
  final Map<String, Iterable<int>> map = {};

  for (final value in [...?map['test']]) {
    // Do something with value
  }
}

效率更高但时间更长的是:

void main() {
  final Map<String, Iterable<int>> map = {};

  for (final value in map['test'] ?? const <int>[]) {
    // Do something with value
  }
}

最好的解决方案可能是在 Iterable<T>? 上添加扩展方法,如下所示:

void main() {
  final Map<String, Iterable<int>> map = {};

  for (final value in map['test'].emptyIfNull) {
    // Do something with value
  }
}

extension EmptyIfNullIterableExtension<T> on Iterable<T>? {
  Iterable<T> get emptyIfNull => this ?? Iterable<T>.empty();
}

这是允许的,因为扩展方法是静态解析的,并且通过在可空类型上添加方法,我们实际上甚至可以在 null.

上进行调用

但这也意味着,如果最终结果是 dynamic,则此解决方案将不起作用,因为我们不能对类型为 dynamic 的对象使用扩展方法。原因是扩展方法实际上不是我们扩展的 class 的一部分,而是编译器自动插入调用的静态方法。此处的解决方法是添加类型转换。