为几个字段实现compareTo方法

我想比较基于5元组的两个对象：srcAddr，dstAddr，srcPort，dstPort，protocol

这是我有的：

public class Flows implements Serializable, Comparable { String srcAddr, dstAddr, srcPort, dstPort, protocol; public int compareTo(Flows arg0) { if(this.srcAddr == arg0.srcAddr && this.dstAddr == arg0.dstAddr && this.srcPort == arg0.srcPort && this.dstPort == arg0.dstPort && this.protocol == arg0.protocol) return 0; } 

}

但它不起作用。 它说无法比较两个字符串。 任何人都可以帮我知道是什么问题？ 谢谢。

Answer 1

编译器/代码检查器警告您将String值与==进行比较几乎总是错误的。

但修复它并不会真正有用，因为你的代码没有像正确实现的compareTo方法那样做。

您的Flows类的compareTo直接实现将是：

 public int compareTo(Flows other) { int res = this.srcAddr.compareTo(other.srcAddr); if (res != 0) { return res; } res = this.dstAddr.compareTo(other.dstAddr); if (res != 0) { return res; } res = this.srcPort.compareTo(other.srcPort); if (res != 0) { return res; } res = this.dstPort.compareTo(other.dstPort); if (res != 0) { return res; } return this.protocol.compareTo(other.protocol); } 

这假设字段永远不为空。 如果是，则编写一个safeCompare(String, String)方法，该方法处理空值并将其应用于每个字段，如上所述。

编辑

鉴于您正在定义compareTo您还应该声明equals和hashCode与它们一致。 否则某些收集方法可能会表现不正确。

编辑2

您在关于如何覆盖compareTo方法的注释中提到的编译器错误发生是因为int compareTo(Flow flow)方法实际上实现了Comparable的compareTo方法。 如果要将Flow声明为实现原始接口类型Comparable则需要签名

 public int compareTo(Object obj) { Flow flow = (Flow) obj; ... 

但更好的解决方案是将类声明更改为：

 public class Flows implements Serializable, Comparable { ... 

Answer 2

尝试：

 @Override public int compareTo(final Flows that) { return ComparisonChain.start(). compare(this.srcAddr, that.srcAddr). compare(this.dstAddr, that.dstAddr). compare(this.srcPort, that.srcPort). compare(this.dstPort, that.dstPort). compare(this.protocol, that.protocol). result(); } 

需要番石榴

Answer 3

使用string.equals()而不是==。

Answer 4

你也可以这样写，我在我的项目中这样做了

 public int compareTo(Flows arg0) { int comp1, comp2, comp3, comp4; comp1 = this.srcAddr.compareTo(arg0.srcAddr); comp2 = this.dstAddr.compareTo(arg0.dstAddr); comp3 = this.srcPort.compareTo(arg0.srcPort); comp4 = this.protocol.compareTo(arg0.protocol); if (comp1 == 0 && comp2 == 0 && comp3 == 0 && comp4 == 0) { return 0; } else { if (comp1 != 0) return comp1; else { if (comp2 != 0) return comp2; else { if (comp3 != 0) return comp3; else { if (comp4 != 0) return comp4; else return 0; } } } } }