Flutter：有没有更好的方法来写下面的代码？

如何解决Flutter：有没有更好的方法来写下面的代码？

这是我尝试编写的 Flutter 代码……但由于我是编码新手，因此我几乎不需要代码方面的帮助，例如我可以编写以下代码的任何较短版本？或者这是最好的？这是代码......如果你有更好的命名约定，你也可以帮助;)

splitvar = 从 PHP-sql 数据库获取的逗号分隔数组字符串

  static haversine() async {
    String splitvar = await Latlong1.latlong();
    List<String> splittedString = splitvar.split(",");
    int j = splittedString.length;
    List<List<double>> moblatlong = List();
    splittedString.removeLast();

    List<double> splittedInt = splittedString.map(double.parse).toList();

    for (int i = 0; i < j - 2; i = i + 3) {
      List<double> row = List();
      row.add(splittedInt[i]);
      row.add(splittedInt[i + 1]);
      row.add(splittedInt[i + 2]);
      moblatlong.add(row);
    }
    print(moblatlong);

    var usersInRange = new List();
    final lat1 = x.894165;
    final lon1 = y.0544283;

    for (int i = 0; i < moblatlong.length; i++) {
      var p = 0.017453292519943295; // Math.PI / 180
      var a = 0.5 -
          cos((moblatlong[i][2] - lat1) * p) / 2 +
          cos(lat1 * p) *
              cos(moblatlong[i][2] * p) *
              (1 - cos((moblatlong[i][1] - lon1) * p)) /
              2;

      var k = 12742 * asin(sqrt(a));
      print(k);
      if (k <= 3) {
        usersInRange.add(moblatlong[i][0].ceil());
        // newlong.add(moblatlong[i]);
      }
    }
    print(usersInRange);
  }

  //add haversine formula and check the radius
  //filter out mobile numbers
  //
  //moblatlong = moblatlong[nth user][1]//Longitude = mobllatlong[nth user][2]

}```

解决方法

这看起来不错，但既然你发布了这个，我会挑剔一些可以缩短的东西。

1. 您为列表的长度创建一个变量：int j = splittedString.length;，然后稍后在您的 for 循环中引用它。变量 j 相当晦涩，我什至不会创建该变量。我只想在您的 for 循环中使用 splittedString.length，因为它可以消除一些混乱。

row.add(splittedInt[i]);
row.add(splittedInt[i + 1]);
row.add(splittedInt[i + 2]);

可以简化为：

row.addAll([splittedInt[i],splittedInt[i + 1],splittedInt[i + 2]])

3. var usersInRange = new List(); 你永远不会重新分配 userInRange 只是简单地改变它，这样你就可以将它更改为 final 并且构造 new List() 可以缩短为 [] .最终结果如下：final usersInRange = [];.

4. var p = 0.017453292519943295; // Math.PI / 180 我喜欢你在评论中展示那个数字是什么，但我还是会把它改成 final 并在你的 for 循环之外而不是在你的 for 循环内初始化它仅仅是因为如果列表很长，它会节省性能，而不是 Dart 编译器每次循环运行时都必须为 p 初始化空间，它只在开始时执行一次。

5. // newlong.add(moblatlong[i]); 永远不要注释掉代码（或者如果您之后很快删除它）是一个好习惯。使用 Git 之类的工具，这是不必要的，并且随着时间的推移您的代码会发生变化，这会很快导致代码混乱。

这只是一些事情，但我希望它有所帮助。总体来说非常不错的代码。