Flutter：避免将“forEach”与函数文字一起使用

Question

大家好，这是我的整个方法：

Future<void> init() async {
    FirebaseAuth.instance.userChanges().listen((user) {
      if (user != null) {
        _loginState = ApplicationLoginState.loggedIn;
        _guestBookSubscription = FirebaseFirestore.instance
            .collection('guestbook')
            .orderBy('timestamp', descending: true)
            .limit(3)
            .snapshots()
            .listen((snapshot) {
          _guestBookMessages = [];
          snapshot.docs.forEach((document) {
            _guestBookMessages.add(
              GuestBookMessage(
                name: document.data()['name'] as String,
                message: document.data()['text'] as String,
              ),
            );
          });
          notifyListeners();
        });
      } else {
        _loginState = ApplicationLoginState.loggedOut;
        _guestBookMessages = [];
        _guestBookSubscription?.cancel();
      }
      notifyListeners();
    });
  }

dart 抱怨的部分是：

snapshot.docs.forEach((document) {
            _guestBookMessages.add(
              GuestBookMessage(
                name: document.data()['name'] as String,
                message: document.data()['text'] as String,
              ),
            );
          });

我怎样才能改变这个方法而不破坏整个功能？我只是在寻找一种让 dart 快乐的方法。我提前感谢您的帮助。

Answer 1

避免将 forEach 与函数文字一起使用。

避免：

snapshot.docs.forEach((document) {
  ...
});

好的：

for (var document in snapshot.docs) {
  // Rest of your code
}

Answer 2

使用类似的data.forEach(function); 例子：

void main() async {
  List<int> list = [1, 2, 3, 4, 5];

  //recommended
  list.forEach(showSquareNumbers);

  //not recommended
  list.forEach((int number) => showSquareNumbers(number));
  list.forEach((int number) {
    showSquareNumbers(number);
  });
}

void showSquareNumbers(int data) {
  print("$data * $data = ${data * data}");
}

这是我的意见。

我认为这forEach似乎比for循环更复杂，并且forEach不能使用continueand break（return可用，但使用时不会发生任何事情return）。

void test(List data) {
  data.forEach((element) {
    print(element);
    if(element) return;
  });

  for (var element in data) {
    print(element);
    if(element) continue;
    if(element) break;
    if(element) return;
  }
}

我认为当您的循环看起来像下面的代码时，我们应该使用 thefor而不是循环，因为循环比我所说的有更多选项。forEachforEachforforEach

  data.forEach((element) {
    print(element)
  });
  //or
  data.forEach((element) => print(element));

我认为forEach循环用于短代码（易于理解），当您想做一些事情时，您不关心像此代码这样的结果（使用 with Function(dynamic)）。

void test(List data) {
  void showInTerminal(e) {
    print("data is $e");
  }
  data.forEach(showInTerminal);
  
  Function(dynamic) function = showInTerminal;
  data.forEach(function);
}

确保数据类型和函数（类型）相同。

//error
void test(List<Map> data) {
  void showInTerminal(String e) {
    print("data is $e");
  }
  data.forEach(showInTerminal);
}

//success
void test(List<Map> data) {
  void showInTerminal(Map e) {
    print("data is $e");
  }
  data.forEach(showInTerminal);
}

我这样编码。我认为这很容易阅读。

void test() {
  dataList.forEach(removeData);
  fileList.forEach(removeFile);
}