Java错误：比较方法违反其一般约定

Java错误：比较方法违反其一般约定

异常消息实际上是描述性的。这里所指的合同是传递：如果

A > B

和

B > C

那么对于任意的

A，B和C：A > C

。我用纸和铅笔检查了一下，你的代码似乎有几个孔：

if (card1.getRarity() < card2.getRarity()) {  return 1;

-1

如果你不返回

card1.getRarity() > card2.getRarity()

。

if (card1.getId() == card2.getId()) {  //...}return -1;

-1

如果id不相等，则返回。你应该返回-1或1根据哪个ID更大。

看看这个。除了更具可读性之外，我认为它实际上应该可以工作：

if (card1.getSet() > card2.getSet()) {    return 1;}if (card1.getSet() < card2.getSet()) {    return -1;};if (card1.getRarity() < card2.getRarity()) {    return 1;}if (card1.getRarity() > card2.getRarity()) {    return -1;}if (card1.getId() > card2.getId()) {    return 1;}if (card1.getId() < card2.getId()) {    return -1;}return cardType - item.getCardType();  //watch out for overflow!