手动更新 for 循环迭代器是不好的做法吗？

Question

我有一个程序可以遍历车辆坐标列表，然后对它们执行其他功能。我只是在寻找不太靠近的成对连续坐标。如果车辆长时间静止，则可能会有数千个连续坐标相互重叠。

一旦找到我的两个坐标，我就会手动更新我的迭代器 (i=j-1)，这样我就不必重复检查成千上万个坐标。我的问题是，这是好的做法吗？还是有可行的替代方案？我在网上找不到任何这样的例子。我看到了“继续”语句，但这似乎需要额外的 if 语句，并且手动更新更“优雅”。

int LatLngSize = latLngList.size();
for (int i = 0; i < LatLngSize; i++) {
    j = i + 1;
    validPoints = true;
    if (LatLngSize > j) {
        latLng1.setCoordinate(latLngList.get(i));
        latLng2.setCoordinate(latLngList.get(j));
        consecutivePointDistance = latLng1.distance(latLng2);
    }
    while (consecutivePointDistance < 0.05) {
        j++;
        if (LatLngSize > j) {
            latLng2.setCoordinate(latLngList.get(j));
            consecutivePointDistance = latLng1.distance(latLng2);
            i = j - 1; // This is the potential offender.
        } else {
            validPoints = false;
            consecutivePointDistance = 100;
        }   
    }

    //Do stuff with my latlng1 and latlng2
}

Answer 1

是的。这是不好的做法。它会编译并运行得很好，但在回答你的问题时，这是不好的做法。

如果您想手动更新迭代器，请使用while 循环。

您可以使用for 循环执行的任何操作也可以使用while 循环执行。选择实际实现的循环是可读性的问题。在for 循环中，人们期望迭代器在更新语句中更新，并且只在更新语句中更新。

如果您不知道如何重写循环以使迭代器仅在更新语句中更新，请考虑以下内容：

for(int i=0; i<someValue; ++i /*i also updated at line 18*/)

仅作为示例。这仍然比重写代码更糟糕，因此它不会在第 18 行（或任何行）更新，但比在第 18 行更新它并且不在更新语句中留下注释要好得多。

或者，对于这个答案的每个 cmets，试试这个：

for(int i=0; i<someValue; /*see body*/) {
    //do stuff
    //update i
    //do stuff
}

现在在这种情况下，您的更新语句是完全空的，因此即使没有 /*see body*/ 注释，维护您的代码的任何人都已经知道 i 必须在正文中的某个位置进行修改。只需将 ++i 添加为 for 循环的最后一行，即可重新创建更新语句中的 ++i 行为。

Answer 2

是的。这是一个不好的做法。不要这样做。如果我在维护你的代码，我明白了

for (int i = 0; i < latLngSize; i++)

我认为这意味着i 一次又一次地从 0 增加到 latLngSize。我不会进一步寻找i 更改的其他实例。所以，更改i 后果自负，前提是您不介意招致所有未来开发者的愤怒。

Answer 3

这不是你想要的吗？

int size = latLngList.size();
for (int i = 0; i + 1 < size; i++) {
    latLng1.setCoordinate(latLngList.get(i));
    latLng2.setCoordinate(latLngList.get(i+1));
    if(latLng1.distance(latLng2) >= 0.05) {
        //Do stuff with my latlng1 and latlng2
    }
}