linter规则与零安全

Question

最近，我用flutter_linter替换了过时的迂腐包，现在我收到了数百条这样的警告：

Avoid using `forEach` with a function literal.

它与呼叫有关

如果您有非空列表，这听起来不错，但是如果代码如下所示呢：

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

怎么改呢？所以在重构后不应该很难看。

Answer 1

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

第一个解决方案可能不是最有效的方法，因为我们正在正确地创建一个新的list对象：

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的对象使用扩展方法。原因是扩展方法实际上不是我们扩展的类的一部分，而是编译器自动插入调用的静态方法。这里的解决方法是添加一个类型广播。