Java ArrayBlockingQueue 设计

Question

我刚刚在研究 JDK 1.6 时发现ArrayBlockingQueue - 构造函数调用了公共(public)可重写方法之一!我认为这对于 API 来说是一种不好的做法。

 public ArrayBlockingQueue(int capacity, boolean fair,
                          Collection<? extends E> c) {
    this(capacity, fair);
    if (capacity < c.size())
        throw new IllegalArgumentException();

    for (Iterator<? extends E> it = c.iterator(); it.hasNext();)
        add(it.next()); // -> surprise:  add is public 
}

我实际上是在尝试扩展 ArrayBlockingQueue ，并添加一些状态并覆盖 add() ，我很快就得到了 java.lang.NullPointerException 当我调用 super(capacity,fair,col) 构造函数时。我在这里缺少一些设计概念吗？

Answer 1

这并不是什么不寻常的事情；本质上，您可以将其视为行的单行语法

ArrayBlockingQueue<E> queue = new ArrayBlockingQueue<E>(capacity, fair);
queue.addAll(c);

这是可重写的。调用可重写的方法通常不是一个好主意，但在这里它是完全合理的。

但是...也就是说，从 java.util 或 java.util.concurrent 扩展任何集合都是一种邪恶。您应该装饰它们，而不是直接扩展它们。