时间戳

首页 > 解决方案 > Flutter：有没有更好的方法来编写以下代码？

android - 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]

}```

标签： androidfunctionflutterdartcoding-style

解决方案

这看起来像是很好的代码，但是自从您发布此代码后，我将挑剔一些可以缩短的内容。

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

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

  2. var p = 0.017453292519943295; // Math.PI / 180我喜欢您在评论中显示该数字是什么，但我再次将其更改为final并在您的 for 循环之外初始化它而不是在您的 for 循环中，因为如果您的列表很长而不是 Dart，它将节省性能编译器必须在每次循环运行时初始化空间，p它只在开始时执行一次。

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

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

推荐阅读