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

Question

这是我尝试编写的 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]

}```

Answer 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]])

var usersInRange = new List(); 你永远不会重新分配 userInRange 只是简单地改变它所以你可以将它更改为 final 并且构造 new List() 可以缩短为 []。最终结果如下所示：final usersInRange = [];.
var p = 0.017453292519943295; // Math.PI / 180 我喜欢你在评论中显示那个数字是什么，但我还是会把它改成 final 并在你的 for 循环之外初始化它而不是在你的for循环中只是因为如果你的列表很长它会节省性能而不是每次循环运行时Dart编译器都必须为p初始化space它只在开始时执行一次。
// newlong.add(moblatlong[i]); 最好永远不要注释掉代码（或者如果您之后很快删除它）。使用像 Git 这样的工具，这是不必要的，并且随着您的代码随着时间的推移而变化，很快就会导致代码混乱。

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

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

Flutter: Is there a better way to write the following code?

android

coding-style

function

dart

flutter