1. Zuhause
  2. Frage und Antwort
  3. Eine weitere Vergleichsmethode, die den Vertrag verletzt

Eine weitere Vergleichsmethode, die den Vertrag verletzt

2016-03-04 5 views
5

Ich habe diese Klasse bekommt:Eine weitere Vergleichsmethode, die den Vertrag verletzt

class Column implements Comparable<Column> { 
    private final float startX; 
    private final float endX; 

    public Column(float startX, float endX) { 
    this.startX = startX; 
    this.endX = endX; 
    } 

    public boolean isSameColumn(Column c) { 
    return c.startX <= this.startX && this.startX < c.endX || this.startX <= c.startX && c.startX < this.endX; 
    } 

    @Override 
    public int hashCode() { 
    return Objects.hash(startX, endX); 
    } 

    @Override 
    public boolean equals(Object obj) { 
    if (obj == null) return false; 
    if (getClass() != obj.getClass()) return false; 
    final Column other = (Column) obj; 
    return this.startX == other.startX && this.endX == other.endX; 
    } 

    @Override 
    public int compareTo(Column o) { 
    return isSameColumn(o) ? 0 : Float.compare(this.startX, o.startX); 
    } 
}

Es scheint mir, dass die compareTo Methode mit dem Vergleicher Vertrag erfüllt, auch wenn es nicht im Einklang mit equals ist (absichtlich) - das sollte kein Problem nach the javadoc sein:

Es ist in der Regel der Fall, aber nicht unbedingt erforderlich, dass (compare(x, y)==0) == (x.equals(y))

Allerdings bekomme ich manchmal:

java.lang.IllegalArgumentException: Comparison method violates its general contract!

Zum Beispiel der folgende Code löst eine Ausnahme (auch auf ideone - es ist ein bisschen lang, aber das meiste davon ist Testdaten).

Beachten Sie auch, dass das Ausführen des sorted() vor distinct() im Stream das Problem löst.

public static void main (String[] args) { 
    String points = "54.199997, 88.399216, 135.2, 250.09616, 517.1, 544.6972, 54.199997, 88.399216, 108.2, 120.79874, 135.2, 168.19669, 178.7, 207.49712, " + 
      "135.2, 168.19669, 370.1, 391.69785, 108.2, 120.79874, 135.2, 189.49884, 135.2, 180.7966, 370.1, 391.69785, 108.2, 115.39928, 135.2, 215.59856, " + 
      "135.2, 172.69742, 377.9, 391.6986, 517.1, 544.6972, 54.199997, 88.399216, 108.2, 120.79874, 135.2, 212.59627, 135.2, 172.69742, 370.1, 391.69785, " + 
      "108.2, 120.79874, 135.2, 212.59627, 135.2, 172.69742, 374.0, 391.69824, 108.2, 120.79874, 135.2, 212.59627, 135.2, 172.69742, 374.0, 391.69824, " + 
      "108.2, 115.39928, 135.2, 215.59856, 135.2, 172.69742, 377.9, 391.6986, 517.1, 544.6972, 54.199997, 88.399216, 108.2, 120.79874, 135.2, 189.49884, " + 
      "135.2, 180.7966, 374.0, 391.69824, 517.1, 544.6972, 54.199997, 88.399216, 108.2, 120.79874, 135.2, 212.59627, 135.2, 172.69742, 374.0, 391.69824, " + 
      "108.2, 120.79874, 135.2, 217.09616, 135.2, 170.59763, 377.9, 391.6986, 108.2, 120.79874, 135.2, 208.99689, 135.2, 163.39717, 374.0, 391.69824, " + 
      "108.2, 120.79874, 135.2, 214.6982, 135.2, 172.69742, 374.0, 391.69824, 108.2, 120.79874, 135.2, 189.49884, 135.2, 180.7966, 374.0, 391.69824, " + 
      "135.2, 250.09616, 517.1, 544.6972, 108.2, 120.79874, 135.2, 198.79779, 135.2, 187.69778, 377.9, 391.6986, 108.2, 115.39928, 135.2, 215.59856, " + 
      "135.2, 172.69742, 377.9, 391.6986, 517.1, 544.6972, 54.199997, 88.399216, 108.2, 121.69865, 135.2, 191.89688, 135.2, 214.39604, 370.1, 391.69785, " + 
      "108.2, 120.79874, 135.2, 217.39865, 135.2, 175.69711, 374.0, 391.69824, 517.1, 544.6972, 54.199997, 88.399216, 108.2, 120.79874, 135.2, 219.49155, " + 
      "135.2, 163.39717, 374.0, 391.69824, 108.2, 120.79874, 135.2, 201.19598, 135.2, 172.69742, 374.0, 391.69824, 108.2, 120.79874, 135.2, 218.59897, " + 
      "135.2, 189.49573, 370.1, 391.69785, 108.2, 120.79874, 135.2, 218.59897, 135.2, 189.49573, 370.1, 391.69785, 108.2, 120.79874, 135.2, 209.5979, " + 
      "135.2, 163.39717, 370.1, 391.69785, 108.2, 120.79874, 135.2, 174.19609, 135.2, 172.69742, 370.1, 391.69785, 108.2, 120.79874, 135.2, 174.19609, " + 
      "135.2, 172.69742, 374.0, 391.69824, 108.2, 120.79874, 135.2, 189.49884, 135.2, 180.7966, 374.0, 391.69824, 108.2, 120.79874, 135.2, 214.39862, " + 
      "135.2, 178.9956, 374.0, 391.69824, 108.2, 120.79874, 135.2, 161.59735, 135.2, 196.39772, 374.0, 391.69824, 517.1, 544.6972, 54.199997, 88.399216, " + 
      "108.2, 115.39928, 135.2, 215.59856, 135.2, 172.69742, 377.9, 391.6986, 108.2, 115.39928, 135.2, 215.59856, 135.2, 172.69742, 377.9, 391.6986, " + 
      "108.2, 120.79874, 135.2, 195.1961, 135.2, 208.6958, 135.2, 178.0957, 374.0, 391.69824, 517.1, 544.6972, 54.199997, 88.399216, 108.2, 118.99892, " + 
      "135.2, 144.1991, 374.0, 391.69824, 517.1, 544.6972, 54.199997, 88.399216, 108.2, 118.99892, 135.2, 193.09724, 374.0, 391.69824, 108.2, 120.79874, " + 
      "135.2, 185.59799, 135.2, 189.19801, 377.9, 391.6986, 108.2, 120.79874, 135.2, 202.997, 135.2, 172.69742, 374.0, 391.69824, 108.2, 120.79874, " + 
      "135.2, 217.69742, 135.2, 163.39717, 374.0, 391.69824, 135.2, 250.09616, 517.1, 544.6972, 108.2, 120.79874, 135.2, 187.69786, 135.2, 172.69742, " + 
      "370.1, 391.69785, 108.2, 115.39928, 135.2, 215.59856, 135.2, 172.69742, 377.9, 391.6986, 108.2, 120.79874, 135.2, 208.99646, 135.2, 182.8964, " + 
      "374.0, 391.69824, 108.2, 120.79874, 135.2, 214.6982, 135.2, 172.69742, 374.0, 391.69824, 108.2, 120.79874, 135.2, 178.69823, 135.2, 163.39717, " + 
      "374.0, 391.69824, 108.2, 120.79874, 135.2, 178.69844, 135.2, 172.69742, 370.1, 391.69785, 108.2, 120.79874, 135.2, 178.69844, 135.2, 172.69742, " + 
      "377.9, 391.6986, 108.2, 120.79874, 135.2, 221.29475, 135.2, 163.39717, 374.0, 391.69824, 108.2, 120.79874, 135.2, 221.29475, 135.2, 163.39717, " + 
      "370.1, 391.69785, 524.7, 553.7995, 54.199997, 88.399216, 108.2, 121.69865, 135.2, 191.89688, 135.2, 214.39604, 440.6, 468.19724, 108.2, 125.59826, " + 
      "135.2, 179.2979, 187.7, 210.49771, 135.2, 208.69688, 370.1, 391.69785, 108.2, 120.79874, 135.2, 178.69844, 135.2, 172.69742, 374.0, 391.69824, " + 
      "517.1, 544.6972, 54.199997, 88.399216, 108.2, 121.69865, 135.2, 191.89688, 135.2, 214.39604, 436.7, 468.19687, 108.2, 118.698944, 135.2, " + 
      "196.99606, 135.2, 172.99727, 360.2, 391.69687, 108.2, 120.79874, 135.2, 201.19598, 135.2, 172.69742, 374.0, 391.69824, 108.2, 120.79874, 135.2, " + 
      "209.5979, 135.2, 163.39717, 370.1, 391.69785, 108.2, 120.79874, 135.2, 189.49884, 135.2, 180.7966, 374.0, 391.69824, 108.2, 120.79874, 135.2, " + 
      "208.69637, 135.2, 175.39758, 377.9, 391.6986, 108.2, 115.39928, 135.2, 215.59856, 135.2, 172.69742, 377.9, 391.6986, 108.2, 115.39928, 135.2, " + 
      "215.59856, 135.2, 172.69742, 377.9, 391.6986, 517.1, 544.6972, 54.199997, 88.399216, 108.2, 121.69865, 135.2, 191.89688, 135.2, 214.39604, 436.7, " + 
      "468.19687, 108.2, 118.698944, 135.2, 208.09802, 135.2, 162.79724, 364.1, 391.69724, 135.2, 250.09616, 517.1, 544.6972, 108.2, 120.79874, 135.2, " + 
      "178.0957, 135.2, 163.39717, 374.0, 391.69824, 108.2, 120.79874, 135.2, 185.59691, 135.2, 163.39717, 370.1, 391.69785, 108.2, 115.39928, 135.2, " + 
      "178.69844, 135.2, 172.69742, 374.0, 391.69824, 517.1, 544.6972, 54.199997, 88.399216, 108.2, 125.59826, 135.2, 179.2979, 187.7, 210.49771, 135.2, " + 
      "213.79517, 370.1, 391.69785, 517.1, 544.6972, 54.199997, 88.399216, 108.2, 120.79874, 135.2, 214.6982, 135.2, 172.69742, 370.1, 391.69785, 108.2, " + 
      "120.79874, 135.2, 212.59627, 135.2, 172.69742, 374.0, 391.69824, 108.2, 120.79874, 135.2, 180.79544, 135.2, 167.29678, 374.0, 391.69824, 108.2, " + 
      "120.79874, 135.2, 165.49696, 135.2, 167.29678, 374.0, 391.69824, 108.2, 120.79874, 135.2, 165.49696, 135.2, 167.29678, 374.0, 391.69824, 108.2, " + 
      "115.39928, 135.2, 215.59856, 135.2, 172.69742, 377.9, 391.6986, 108.2, 115.39928, 135.2, 215.59856, 135.2, 172.69742, 377.9, 391.6986, 108.2, " + 
      "115.39928, 135.2, 190.69446, 135.2, 190.69894, 377.9, 391.6986, 108.2, 115.39928, 135.2, 165.49861, 135.2, 195.19905, 377.9, 391.6986, 517.1, " + 
      "544.6972, 54.199997, 88.399216, 108.2, 118.99892, 135.2, 205.99895, 135.2, 195.79625, 374.0, 391.69824, 517.1, 544.6972, 54.199997, 88.399216, " + 
      "108.2, 118.99892, 135.2, 201.49861, 374.0, 391.69824, 108.2, 118.698944, 135.2, 201.79861, 135.2, 183.19757, 364.1, 391.69724, 517.1, 544.6972, " + 
      "54.199997, 87.499214, 108.2, 125.59826, 135.2, 182.2976, 190.7, 214.39763, 135.2, 215.897, 370.1, 391.69785, 517.1, 544.6972, 54.199997, " + 
      "87.499214, 108.2, 118.99892, 135.2, 211.99535, 370.1, 391.69785, 108.2, 121.69865, 135.2, 191.89688, 135.2, 214.39604, 364.1, 391.69724, 108.2, " + 
      "121.69865, 135.2, 191.89688, 135.2, 214.39604, 370.1, 391.69785, 108.2, 125.59826, 135.2, 222.19763, 135.2, 215.2975, 370.1, 391.69785, 108.2, " + 
      "115.39928, 135.2, 199.99884, 135.2, 193.99792, 377.9, 391.6986, 108.2, 115.39928, 135.2, 175.69751, 135.2, 166.39688, 374.0, 391.69824, 135.2, " + 
      "250.09616, 523.1, 544.6978, 108.2, 115.39928, 135.2, 198.797, 135.2, 171.1964, 374.0, 391.69824, 108.2, 115.39928, 135.2, 198.797, 135.2, " + 
      "171.1964, 377.9, 391.6986, 108.2, 115.39928, 135.2, 198.797, 135.2, 171.1964, 377.9, 391.6986, 523.1, 544.6978, 54.199997, 87.499214, 108.2, " + 
      "118.698944, 135.2, 174.19894, 135.2, 161.59735, 370.1, 391.69785, 108.2, 120.79874, 135.2, 178.69844, 135.2, 180.7966, 377.9, 391.6986, 108.2, " + 
      "120.79874, 135.2, 189.49884, 135.2, 180.7966, 370.1, 391.69785, 108.2, 115.39928, 135.2, 160.99742, 135.2, 195.79776, 377.9, 391.6986, 523.1, " + 
      "544.6978, 54.199997, 87.499214, 108.2, 120.79874, 135.2, 217.69742, 135.2, 163.39717, 370.1, 391.69785, 108.2, 120.79874, 135.2, 212.59627, " + 
      "135.2, 172.69742, 374.0, 391.69824, 108.2, 120.79874, 135.2, 187.09769, 135.2, 163.39717, 377.9, 391.6986, 108.2, 115.39928, 135.2, 215.59856, " + 
      "135.2, 172.69742, 377.9, 391.6986, 108.2, 120.79874, 135.2, 195.1961, 135.2, 209.29948, 135.2, 166.09691, 135.2, 173.89784, 135.2, 200.29836, " + 
      "370.1, 391.69785, 108.2, 118.698944, 135.2, 181.39537, 135.2, 193.09871, 377.9, 391.6986, 527.0, 544.69824, 54.199997, 87.499214, 108.2, " + 
      "125.59826, 135.2, 179.2979, 187.7, 211.39763, 135.2, 208.69688, 370.1, 391.69785, 108.2, 120.79874, 135.2, 212.59627, 135.2, 172.69742, 374.0, " + 
      "391.69824, 524.7, 553.7995, 54.199997, 87.499214, 108.2, 120.79874, 135.2, 216.79735, 135.2, 163.39717, 374.0, 391.69824, 108.2, 121.69865, " + 
      "135.2, 191.89688, 135.2, 214.39604, 440.6, 468.19724, 517.1, 544.6972"; 
    String[] pointss = points.split(", "); 
    List<Column> columns = new ArrayList<>(); 
    for (int i = 0; i < pointss.length; i += 2) { 
    columns.add(new Column(Float.parseFloat(pointss[i]), Float.parseFloat(pointss[i+1]))); 
    } 
    List<Column> columnsWithOverlap = columns.stream() 
      .distinct() 
      .sorted() 
      .collect(toList()); 
    System.out.println(columnsWithOverlap); 
}

Quelle

2016-03-04 assylias

+2

Sind Sie sicher, dass dies etwas mit Gleichen zu tun hat und vergleichen? http://stackoverflow.com/questions/8327514/comparison-method-violates-its-general-contract Welche Schritt durch Ihren Code und finden Sie die 2 Punkte, die dazu führen, dass die Ausnahme zu werfen. – Falmarri

+0

Ist die tatsächliche Ausnahme "Vergleichsmethode verletzt seinen allgemeinen Vertrag!"? Bitte könnten Sie, dass in der Frage :) –

+0

@AndyTurner – assylias

Antwort

8

Ich glaube nicht, Ihr Komparator ist transitiv, das heißt columnA(1, 4), columnB(3, 7), columnC(6, 9). In diesem Fall columnA.compareTo(columnB) = 0 und columnB.compareTo(columnC) = 0, jedoch columnA.compareTo(columnC) < 0.

Warum schlägt es fehl, wenn sorted() vor distinct() erscheint, wer weiß, was passiert, wenn compareTo() nicht transitiv ist?

Quelle

2016-03-04 23:10:35

+1

Wahrscheinlich tritt der fehlgeschlagene Vergleich nie auf, wenn die Liste nicht eindeutig ist, weil die zwei Werte, die zum Fehlschlagen führen, in der ursprünglichen Liste im Beispiel niemals nebeneinander liegen. – reegnz

+1

@reegnz Das könnte die Erklärung sein, aber was ist, wenn der Stream parallel ist? –

+0

Afaik, die parallele Sortierung ist deterministisch in einer Art "teile und erobere" mit Gabelzinken, so dass sie das nicht beeinflussen sollte. Aber ich bin wahrscheinlich nicht die richtige Person, um das zu beantworten. : P – reegnz

1

Ok, ich denke, ich hätte das Problem gefunden, aber nicht genau, wie es zu lösen ist. Das Problem ist, dass der Komparator nicht transitiv ist. Andere haben dies auch bemerkt, und ich tat es auch in den Kommentaren der Frage.

Wenn Sie tauschen die Werte des Float.compare aus diesem:

return isSameColumn(o) ? 0 : Float.compare(this.startX, o.startX);

dazu:

return isSameColumn(o) ? 0 : Float.compare(o.startX, this.startX);

die Art funktioniert ohne Probleme, aber das gibt eine umgekehrte Reihenfolge.

Jetzt ist der Grund noch nicht ganz klar für mich, aber ich hoffe, das gibt Ihnen den Anstoss in die richtige Richtung.

Quelle

2016-03-05 00:28:55 reegnz

+0

Was mir auch seltsam ist, ist, dass die Umkehrung der Sortierreihenfolge das Problem zurückbringt sort (Collections.reverseOrder()) – reegnz

+2

Auch unter Berücksichtigung der Transitivität ist es zu einfach, ein Beispiel zu bringen, das zeigt, dass der Komparator nicht transitiv ist. siehe die Antwort von Federico. – reegnz

 Verwandte Themen

  • Keine verwandten Themen^_^