改善自定义比较代码质量

Question

编写此比较器的更好方法？

Comparator<JSONObject> comparator = (op1,op2) -> {
         Integer id1 = Integer.valueOf(String.valueOf(op1.get(field)));
         Integer id2 = Integer.valueOf(String.valueOf(op2.get(field)));
         return id2.compareTo(id1);
         };

有可能避免两个valueOf？

Answer 1

使用 Comparator.comparingInt

您应该能够执行以下操作：

Comparator<JSONObject> comparator = 
    Comparator.comparingInt(obj -> Integer.valueOf(String.valueOf(obj.get(field))))
    .reversed();

getInt

检查JSONObject的API以查看是否有getInt方法或类似方法，然后您应该能够直接使用它而不是包装valueOf调用，例如：

Comparator<JSONObject> comparator = Comparator.<JSONObject>comparingInt(obj -> obj.getInt(field))
    .reversed();

返回类型.get

另外，您应该研究obj.get(field)返回哪种类型。如果幸运的话，它可以像打字一样简单：

Comparator<JSONObject> comparator = 
    Comparator.<JSONObject>comparingInt(obj -> (Integer) obj.get(field))
    .reversed();

返回类型是String

如果您总是从obj.get产生的Strings，则应该能够直接使用Integer.valueOf，而不是调用String.valueOf

Comparator<JSONObject> comparator = 
    Comparator.<JSONObject>comparingInt(obj -> Integer.valueOf(obj.get(field)))
    .reversed();

根据API，您可能仍需要打字到String

Comparator<JSONObject> comparator = 
    Comparator.<JSONObject>comparingInt(obj -> Integer.valueOf((String) obj.get(field)))
    .reversed();

Answer 2

是否可以避免两个值？

这取决于您是否试图获得的field的类型，即：是int还是string，还是

如果是保证的int，您可以简单地执行此操作

Comparator<JSONObject> comparator = (op1,op2) -> {
     return Integer.valueOf(op1.get(field)).compareTo(Integer.valueOf(op2.get(field));
};

上升顺序

如果值是string，则可以执行此操作

Comparator<JSONObject> comparator = (op1,op2) -> {
     return Integer.parseInt((String)op1.get(field)) - Integer.parseInt((String)op2.get(field));
};