8

现在，我有：

    public <T> T[] toArray(T[] old) {
        T[] arr = Arrays.copyOf(old, old.length + size());
        int i = old.length;
        for(E obj : this) {
            arr[i] = old.getClass().getComponentType().cast(obj);
            ++i;
        }
        return arr;
    }

（请注意，这不符合 axtavt 指出的合同。）

我在哪里得到这个警告：

Type safety: Unchecked cast from capture#2-of ? to T

这仍然是实现它的最佳/最直接的方法吗？我可以在没有警告的情况下以某种方式对其进行编码吗？否则我将如何实施它？

编辑：我目前的解决方案。首先，我真的不想自己有这样的警告toArray。因此，我编写了这些小辅助函数（阅读此处以进一步讨论这些）：

@SuppressWarnings("unchecked") static <T> Class<? extends T> classOf(T obj) {
    return (Class<? extends T>) obj.getClass();
}

@SuppressWarnings("unchecked") static <T> Class<? extends T> classOf(T[] array) {
    return (Class<? extends T>) array.getClass().getComponentType();
}

@SuppressWarnings("unchecked") static <T> T[] newArray(Class<T> clazz, int size) {
    return (T[]) Array.newInstance(clazz, size);
}

现在，我的toArray实现看起来像：

    public <T> T[] toArray(T[] array) { 
        int size = size();
        if (array.length < size) { 
            array = newArray(classOf(array), size);
        } else if (array.length > size) {
            array[size] = null;
        }

        int i = 0;
        for (E e : this) {
            array[i] = classOf(array).cast(e);
            i++;
        }
        return array;
    }
4

2 回答 2

8

这仍然是实现它的最佳/最直接的方法吗？否则我将如何实施它？

乔什·布洛赫不是这样的。看看源代码AbstractCollection#toArray()。这是 JDK 1.6.0_22 的相关摘录。

public <T> T[] toArray(T[] a) {
    // Estimate size of array; be prepared to see more or fewer elements
    int size = size();
    T[] r = a.length >= size 
        ? a 
        : (T[]) Array.newInstance(a.getClass().getComponentType(), size);
    Iterator<E> it = iterator();

    for (int i = 0; i < r.length; i++) {
        if (!it.hasNext()) { // fewer elements than expected
            if (a != r)
                return Arrays.copyOf(r, i);
            r[i] = null; // null-terminate
            return r;
        }
        r[i] = (T) it.next();
    }
    return it.hasNext() ? finishToArray(r, it) : r;
}

源代码src.zip在 JDK 的文件中可用。AbstractCollection您可以将它集成到任何像 Eclipse/IDEA/Netbeans 这样的体面的 IDE 中，以便在打开类时可以看到它。

我可以在没有警告的情况下以某种方式对其进行编码吗？

不@SuppressWarnings("unchecked")，如果它困扰您，请使用。

也就是说，如果可能的话，我建议您扩展AbstractCollection而不是实现Collection，以便您至少拥有已经为您实现的基本功能。

于 2010-10-24T23:10:15.713 回答
5

首先，如果它应该是 的实现Collection.toArray()，则它不遵循约定 - 您不应该将旧元素保留在数组中（请参阅javadoc）。

正确的实现如下所示：

public <T> T[] toArray(T[] array) { 
    int size = size();
    if (array.length < size) { 
        // If array is too small, allocate the new one with the same component type
        array = Array.newInstance(array.getClass().getComponentType(), size);
    } else if (array.length > size) {
        // If array is to large, set the first unassigned element to null
        array[size] = null;
    }

    int i = 0;
    for (E e: this) {
        // No need for checked cast - ArrayStoreException will be thrown 
        // if types are incompatible, just as required
        array[i] = (T) e;
        i++;
    }
    return array;
}
于 2010-10-24T23:27:51.443 回答