1

我的 MultiSet 类的一些方法存在一些问题。这是一个测试器，MultiSet 类应该得到输出：“Succes！” 如果它工作正常。这是测试仪：

public class MultiSetTest {

    public static void main(String[] args) {
    MultiSet<String> a = new MultiSet<String>();
    MultiSet<String> b = new MultiSet<String>();

    a.add("Foo");
    a.add("Bar");
    a.add("Foo");
    System.out.println("a:" + a); // test toString

    b.add("Bar");
    b.add("Foo");
    b.add("Bar");
    b.add("Foo");
    System.out.println("b:" + b);

    assert !a.equals(b) : "Failed test 1!"; // test equals
    assert b.remove("Bar") : "Failed test 2!"; // test remove
    assert a.equals(b) : "Failed test 3!";
    for(String s : a) { // test iterator
        assert b.remove(s) : "Failed test 4!";
    }
    assert b.size() == 0 : "Failed test 5!";

    Set<String> baseSet = new HashSet<String>(a);
    assert baseSet.size()==2 : "Failed test 6!";  

    b = new MultiSet<String>(a);
    assert a.equals(b) : "Failed test 7!";

    try {
        assert false;
        System.out.println("Please enable assertions!");
    }
    catch(AssertionError e) {
        System.out.println("Success!");
    }
    }
}

还有我的 Multiset 类：

public class MultiSet<E> extends AbstractCollection<E>
{
    private int size = 0;
    private Map<E, Integer> values = new HashMap<E, Integer>();

    public MultiSet()
    {

    }

    public MultiSet(Collection<E> c)
    {
        addAll(c);
    }

    public boolean add()
    {
        return false;
    }

    public boolean remove()
    {
        return false;
    }

    public Iterator<E> iterator()
    {
        return new Iterator<E>()
        {
            private Iterator<E> iterator = values.keySet().iterator();
            private int remaining = 0;
            private E current = null;

            public boolean hasNext()
            {
                return remaining > 0 || iterator.hasNext();
            }

            public E next()
            {
                if (remaining == 0)
                {
                    current = iterator.next();
                    remaining = values.get(current);
                }
                remaining--;
                return current;
            }
            public void remove()
            {
                throw new UnsupportedOperationException();
            }
        };
    }

        public boolean equals(Object object)
        {
            if (this == object) return true;
            if (this == null) return false;
            if (this.getClass() != object.getClass()) return false;
            MultiSet<E> o = (MultiSet<E>) object;
            return o.values.equals(values);
        }


        public int hashCode()
        {
            return values.hashCode()*163 + new Integer(size).hashCode()*389;
        }

        public String toString()
        {
            String res = "";
            for (E e : values.keySet());
                    //res = ???;
            return getClass().getName() + res;
        }

        public int size()
        {
            return size;
        }
    }

也许如果您可以在添加或删除的过程中帮助我，那么我可能可以解决另一个问题。

此外，我的 equals 似乎无法正常工作，我不确定如何在 String toString 处计算“res”。不要介意我的退货声明，稍后我会加上一些括号等以使其看起来不错。

谢谢您的帮助。// 克里斯

4

1 回答 1

2

为什么不使用经过良好测试的 Google Guavas 的 Multiset而不是重新发明轮子呢？您可以选择多种实现之一：

  • ConcurrentHashMultiset，
  • 枚举多集，
  • 转发多集，
  • HashMultiset，
  • 不可变多集，
  • LinkedHashMultiset,
  • 树多集

什么应该涵盖您的用例，或者 - 如果您真的想要 - 自己实现 Multiset 接口，查看默认实现的来源。

编辑：

您的实现违反了接口合同-Collecion您不能返回false. add(E e)阅读收藏文档：

boolean add(E e)

参数：

e- 要确保在此集合中存在的元素

回报：

true如果此集合因调用而更改

抛出：

UnsupportedOperationException- 如果此集合不支持添加操作

如果您想使用只读 Multiset 使用ImmutableMultiset（更具体地说是ImmutableMultiset.copyOf(Iterable)）或实现Multiset 的接口add(E e)方法 throwing UnsupportedOperationException。

于 2011-12-09T14:52:22.710 回答