Question

我正在尝试使用以下方法对arraylist进行排序。我希望美国成为第一，其次是英国，等等。但是，这似乎并没有做任何事情。我做错了什么？

    orderedTerritories.sort((o1, o2) -> {
        if (o1.getCode().equals("US*"))
            return 1;
        else if (o2.getCode().equals("US*"))
            return 0;
        else if (o1.getCode().contains("UK") && o1.getContains() != null)
            return 1;
        else if (o2.getCode().contains("UK") && o2.getContains() != null)
            return 0;
        else if (o1.getCode().equals("DE*"))
            return 1;
        else if (o2.getCode().equals("DE*"))
            return 0;
        else if (o1.getCode().equals("JP"))
            return 1;
        else if (o2.getCode().equals("JP"))
            return 0;
        else if (o1.getCode().equals("IN"))
            return 1;
        else if (o2.getCode().equals("IN"))
            return 0;
        else return 1;
    });

Answer 1

当o1.getCode（）小于o2.getCode（）时，您的排序逻辑不会检查两个代码是否相同，并且还返回0而不是负值。如果不这样做，你就会违反比较合同。

根据您的排序逻辑，您不能简单地依赖国家/地区代码的字符串表示进行排序。要缓解它，您需要为每个国家/地区定义自定义排序值。我假设您无法修改获取代码方法以返回Enum或某些自定义对象。在这种情况下，您可以在国家/地区代码和订单之间定义地图，并使用该订单值进行排序。这是一个例子：

其中一条评论已经暗示了类似的方法。

     private static Map<String, Interger> codeToOrderMap = new HashMap<>();

    static{
     codeToOrderMap.put("US*", 0);
     codeToOrderMap.put("UK", 1);
     codeToOrderMap.put("DE*", 2);
     codeToOrderMap.put("JP", 3);
     codeToOrderMap.put("IN", 4);
    }

    orderedTerritories.sort((o1, o2) ->
codeToOrderMap.get( o1.getCode() ).compareTo( codeToOrderMap.get( o2.getCode() );

如果不使用上述方法或自定义代码，您将不得不编写一个非常冗长的比较器。

Answer 2

我倾向于同意Lew Bloch，因为你应该使用枚举来使用更面向对象的方法。

我不理解您对Lew Bloch发表的评论的回应。你说，＆＃34;因为我无法对其进行硬编码。难道没有办法在java中进行简单的排序吗？＆＃34;但是您已经对当前代码中的值进行了硬编码。例如，您的代码中包含以下内容：

o1.getCode().equals("US*")在本声明中，＆＃34; US *＆＃34;是硬编码的。

以下是仅使用美国和英国代码的枚举方法示例：

import java.util.List;
import java.util.ArrayList;
import java.util.Collections;

public class Example implements Comparable<Example>
{
  public enum CountryCode{
    US,UK
  }

  public static void main(String[] args)
  {
    List<Example> codeList = new ArrayList<>();
    codeList.add(new Example(CountryCode.UK));
    codeList.add(new Example(CountryCode.US));

    System.out.println("Before sort: "+codeList);
    Collections.sort(codeList);
    System.out.println("After sort: "+codeList);

  }

  private CountryCode countryCode;

  public Example(CountryCode code){
    this.countryCode = code;
  }

  public CountryCode getCountryCode(){
    return countryCode;
  }

  public int compareTo(Example other){
    return this.countryCode.compareTo(other.getCountryCode());
  }

  public String toString(){
    return "Example: "+countryCode.toString();
  }
}

由于枚举按此顺序包含美国和英国，并且Example类推迟实现compareTo的枚举，因此List Example的排序方式为国家/地区代码为US的Example个对象位于国家/地区代码为UK的对象之前。

<强>输出：

Before sort: [Example: UK, Example: US]
After sort: [Example: US, Example: UK]

最终产品：我不清楚＆＃34; *＆＃34;用于表示代码值。如果您没有指定要求，则此方法可能无法按预期工作。

Answer 3

谢谢，我考虑了你们对以下解决方案所说的内容，其中有效：

       List<Territory> orderedTerritories = new LinkedList<>();
       List<Integer> integerValues = new ArrayList<>();
       Map<Integer, Territory> intToTerritoryMap = new HashMap<>();
       int i = 5;
       for (Territory territory : finalSet) {
            int index = 5;
            if (territory.getCode().equals("US*"))
                index = 0;
            else if (territory.getCode().contains("UK") && territory.getContains() != null)
                index = 1;
            else if (territory.getCode().equals("DE*"))
                index = 2;
            else if (territory.getCode().equals("JP"))
                index = 3;
            else if (territory.getCode().equals("IN"))
                index = 4;
            if (index < 5) {
                intToTerritoryMap.put(index, territory);
                integerValues.add(index);
            } else {
                intToTerritoryMap.put(i, territory);
                integerValues.add(i++);
            }
        }
        Collections.sort(integerValues);

        integerValues.forEach(j -> {
            orderedTerritories.add(intToTerritoryMap.get(j));
        });

        return orderedTerritories;

数字中的冗长和硬编码。我希望有一种更简单的方法。

内联对象比较Java

3 个答案: