android - 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相当晦涩，我什至不会创建那个变量。我只会用 splittedString.length在你的 for 循环中，因为它清除了一些困惑。

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 初始化空间，这将节省性能。每次循环运行时，它只在开始时执行一次。

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

这只是一些事情，但我希望它有所帮助。整体非常好的代码。