排序位置：比较方法违反其总合同

Question

我有一个类 LocationComparator，它看起来像这样：

public class LocationComparator implements Comparator<MyLocation> {

    double mLatitude, mLongitude;

    public LocationComparator(double baseLat, double baseLon){
        mLatitude = baseLat;
        mLongitude = baseLon;
    }

    @Override
    public int compare(MyLocation o, MyLocation o2) {
        float[] result1 = new float[3];
        android.location.Location.distanceBetween(mLatitude, mLongitude, o.latitude, o.longitude, result1);
        Float distance1 = result1[0];           

        float[] result2 = new float[3];
        android.location.Location.distanceBetween(mLatitude, mLongitude, o2.latitude, o2.longitude, result2);
        Float distance2 = result2[0];           

        return distance1.compareTo(distance2);
    }    
}

接下来，我有<代码>列表

Collections.sort(locations, new LocationComparator(baseLat, baseLon));

其中baseLat和baseLon是当前位置的纬度和经度。有时，代码会抛出异常 java.lang.IllegalArgumentException： 比较方法违反了其通用协定！

我知道，如果比较方法不满足传递条件，则会抛出此异常。我在这里找到了一些信息：比较方法违反了其总合同！我理解为什么在引用的问题中抛出异常，但我仍然不知道为什么在我的代码中抛出异常。

你能帮我一下吗？谢了。

更新

MyLoction的列表用于ListView的适配器。如果有新的GPS位置，则对ListView中的数据进行排序。代码如下所示：

@Override
public void onLocationChanged(Location location) {
    super.onLocationChanged(location);
    sortAdapter(new LatLng(location.getLatitude(), location.getLongitude()));
    mIsSortedByLocation = true;   
    }
}

和sort适配器（latLng latLng）方法在这里：

private void sortAdapter(LatLng latLng) {
    if (mAdapter == null || latLng == null) {
        return;
    }
    List<MyLocation> list = Arrays.asList(mAdapter.getItems());
    Collections.sort(locations, new LocationComparator(latLng.latitude, latLng.longitude));
    mAdapter.setItems(mMyLocations = (MyLocation[]) list.toArray());
}

更新 2

数据来自服务器。首先，数据存储到数据库中，然后，有一个我的内容提供程序和加载器。在加载完成（加载器




Answer 1

匿名用户

在不了解代码细节的情况下，我会假设您的问题来自并发。

List<MyLocation> list = Arrays.asList(mAdapter.getItems());
Collections.sort(locations, new LocationComparator(latLng.latitude, latLng.longitude));

在您的示例中，它要么是打字错误，要么位置引用确实填充在方法范围之外。所以，我假设您正在从服务器加载它。或者很可能是您正在从后台线程修改Adapter的内容。

避免更改主线程使用的后台线程的数据。因此，您必须在使用集合之前对其进行同步，或者只是将结果从后台传递给主线程< code >处理程序:

List<MyLocation> locations;

void myBackgroundTask() {
    List<MyLocation> result = getLocationsFromServer();
    runOnUiThread(() -> { locations = result }); // lambda or Runnable
}