Java Comparator、Comparable 和 TreeSet.contains

标签 java comparator comparable treeset

<分区>

编辑 - 下面稍微简化的示例(更简单但我不确定它是否包含所需行为的所有元素)

下面是一个代码片段,它代表了我试图用 Comparator 做的一件事。我希望 contains 在第二次调用 doRSM 时返回 true:

package comparisonTest;

import java.util.ArrayList;
import java.util.Collection;
import java.util.Comparator;
import java.util.HashMap;
import java.util.Iterator;
import java.util.Map;
import java.util.PriorityQueue;
import java.util.SortedSet;
import java.util.TreeSet;

public class ComparisonTest {

    private static class ArbitraryItem {
        String node;
        ArbitraryItem(String node) {
            this.node = node;
        }
        public String getNode() {
            return node;
        }
    }

    private static final Map<String, ResultSet> idToArbitraryItems = new HashMap<>();


    private static class ArbitraryItemComparable implements Comparable<ArbitraryItemComparable> {
        ArbitraryItem item;
        Comparator<ArbitraryItemComparable> c;
        ArbitraryItemComparable(ArbitraryItem item, Comparator<ArbitraryItemComparable> c) {
            this.item = item;
            this.c = c;
        }
        @Override
        public boolean equals(Object o) {
            if (!(o instanceof ArbitraryItemComparable)) {
                System.out.println("not a ArbitraryItemComparable="+o);
                return false;
            }
            boolean eq = this.c.compare(this, ((ArbitraryItemComparable)o)) == 0;
            System.out.println("   equality val="+eq);
            return eq;
        }
        @Override
        public int compareTo(ArbitraryItemComparable o) {
            int compVal = this.c.compare(this, ((ArbitraryItemComparable)o));
            System.out.println("   comparison val="+compVal);
            return compVal;
        }

    }

    private static class ResultSet {
        SortedSet<ArbitraryItemComparable> usedResults;
        String node;
        Comparator<ArbitraryItemComparable> comparator;
        ResultSet(String node, Comparator<ArbitraryItemComparable> comparator) {
            this.usedResults = new TreeSet<ArbitraryItemComparable>();
            this.node = node;
            this.comparator = comparator;
        }
    }

    static private void doRSM(Collection <ArbitraryItem> foobar, ResultSet set, int max, boolean close, String id) {
        Iterator<ArbitraryItem> items = foobar.iterator();
        for (;items.hasNext();) {
            ArbitraryItem item = (ArbitraryItem) items.next();
            ArbitraryItemComparable itemComparable = new ArbitraryItemComparable(item, set.comparator);
            System.out.println("*** looking at node "+itemComparable.item.getNode()+"***, sur size="+set.usedResults.size());
            if (!set.usedResults.contains(itemComparable)); {
                System.out.println("*** node "+itemComparable.item.getNode()+" not in usedResults");
            }
            set.usedResults.add(itemComparable);
        }
    }

    public static void main(String [] args)
    {
        Collection<ArbitraryItem> items = new ArrayList<>();
        for (int i = 0; i < 3; i++) {
            items.add(new ArbitraryItem(""+i));
        }

        Comparator<ArbitraryItemComparable> comparator = new Comparator<ArbitraryItemComparable>() {
            @Override
            public int compare(ArbitraryItemComparable o1, ArbitraryItemComparable o2) {
                // this is where the magic needs to happen!!
                System.out.println("calling compare: o1 node="+o1.item.getNode()+" "+o1.item.getNode().hashCode()+" o2 node="+o2.item.getNode()+" "+o2.item.getNode().hashCode());
                return o1.item.getNode().hashCode() - o2.item.getNode().hashCode();
            }
            @Override
            public boolean equals(Object o) {
                System.out.println("why is this called?");
                return false;
            }
        };
        ResultSet set = new ResultSet("3", comparator);
        idToArbitraryItems.put("q", set);
        doRSM(items, set, 1000, false, "q");
        doRSM(items, set, 1000, false, "q");
    }
}

但是,日志显示:

*** looking at node 0***, sur size=0
*** node 0 not in usedResults
calling compare: o1 node=0 48 o2 node=0 48
   comparison val=0
*** looking at node 1***, sur size=1
calling compare: o1 node=1 49 o2 node=0 48
   comparison val=1
*** node 1 not in usedResults
calling compare: o1 node=1 49 o2 node=0 48
   comparison val=1
*** looking at node 2***, sur size=2
calling compare: o1 node=2 50 o2 node=0 48
   comparison val=2
calling compare: o1 node=2 50 o2 node=1 49
   comparison val=1
*** node 2 not in usedResults
calling compare: o1 node=2 50 o2 node=0 48
   comparison val=2
calling compare: o1 node=2 50 o2 node=1 49
   comparison val=1
*** looking at node 0***, sur size=3
calling compare: o1 node=0 48 o2 node=1 49
   comparison val=-1
calling compare: o1 node=0 48 o2 node=0 48
   comparison val=0
*** node 0 not in usedResults
calling compare: o1 node=0 48 o2 node=1 49
   comparison val=-1
calling compare: o1 node=0 48 o2 node=0 48
   comparison val=0
*** looking at node 1***, sur size=3
calling compare: o1 node=1 49 o2 node=1 49
   comparison val=0
*** node 1 not in usedResults
calling compare: o1 node=1 49 o2 node=1 49
   comparison val=0
*** looking at node 2***, sur size=3
calling compare: o1 node=2 50 o2 node=1 49
   comparison val=1
calling compare: o1 node=2 50 o2 node=2 50
   comparison val=0
*** node 2 not in usedResults
calling compare: o1 node=2 50 o2 node=1 49
   comparison val=1
calling compare: o1 node=2 50 o2 node=2 50
   comparison val=0

违规行从这里开始:

*** looking at node 0***, sur size=3
calling compare: o1 node=0 48 o2 node=1 49
   comparison val=-1
calling compare: o1 node=0 48 o2 node=0 48
   comparison val=0
*** node 0 not in usedResults

表示比较值为 0(表示它们相等),但仍在调用添加代码,表示列表并未表示它包含该值。为什么?我认为返回 0compareTo 意味着相等?此外,为什么永远不会调用 equalsTreeSet 文档说明 包含:

Returns true if this set contains the specified element. More formally, returns true if and only if this set contains an element e such that (o==null ? e==null : o.equals(e)).

这会让我相信 ArbitraryItemComparable.equals 会被调用,但事实并非如此?关于如何让 usedResults.contains 函数返回 true 的任何想法?

最佳答案

你有这个:

...
if (!set.usedResults.contains(itemComparable)); { // NOTE THE SEMICOLON!!!!

这基本上意味着“忽略此条件并无条件地执行以下 block ”。

关于Java Comparator、Comparable 和 TreeSet.contains,我们在Stack Overflow上找到一个类似的问题: https://stackoverflow.com/questions/35155463/

相关文章:

java - 在某些情况下,TreeSet Comparator 无法删除重复项?

c++ - C++中最小堆的比较器

java - JButton 文本和图像图标被省略号截断

java - 设置AsyncTask调用其他方法

java - 如何使用 Java Comparator 使用两个参数对 ArrayList 进行排序?

java - 使用可比较的排序字符串属性

java - 使用泛型创建返回较大值的 max 函数

java - 如何将 dateTime 类型转换为 gregorianCalendar

java - 第一次运行时如何在同一帧打开文本文件?

java - 使用可比较的java进行排序