Effective-java 3 中文翻译系列 (Item 27

文章也上传到

github

（欢迎关注，欢迎大神提点。）

---

ITEM 27 消除 unchecked（未检查） 警告

当你使用范型的时候，可能会经常看到各种各样的编译器警告：未检查方法调用警告、未检查变量类型警告、未检查转换警告等等。很多未检查警告很容易被消除。例如，假如你写了这样的代码：

Set<Lark> exaltation = new HashSet();

此时编译器会警告你：

Venery.java:4: warning: [unchecked] unchecked conversion
Set<Lark> exaltation = new HashSet();
                ^
required: Set<Lark>
found: HashSet

要消除这里的警告，可以使用钻石符号（<>）,编译器可以自己推断出真实的类型（这里是Lark）：

Set<Lark> exaltation = new HashSet<>();

当你遇到一些很难消除的警告时，持之以恒，尽可能的将其解决！当你消除了所有的警告时，好处是就可以确定你的代码是类型安全的。意味着在运行时你不会遇到ClassCastException的异常。

当你遇到你不能消除的警告时，如果你能保证被警告的代码类型是什么，就可以使用（也只能使用这种方式）@SuppressWarnings(“unchecked”）注解来消除警告。但是当你不能确保其类型的时候，就不要使用这个注解消除警告，因为即使你在编译期间消除了警告，在运行时也可能会报出ClassCastException的异常。

SuppressWarnings注解可以用在声明任何对象时，例如声明一个变量、方法、结构体或类等。但是永远要保证尽可能小范围的使用SuppressWarnings注解。不要对整个类使用SuppressWarnings，因为这样可能会掩盖掉严重的警告。

如果你发现你要使用SuppressWarnings注解在一个对象（方法或结构体等）上长度超过了一行，那么你可以通过移动这个对象成为一个本地的变量解决。虽然这样需要声明一个本地变量，但是这是值得的。例如，ArrayList中的toArray方法：

    public <T> T[] toArray(T[] a) {
        if (a.length < size)
            // Make a new array of a's runtime type, but my contents:
            return (T[]) Arrays.copyOf(elementData, size, a.getClass());
        System.arraycopy(elementData, 0, a, 0, size);
        if (a.length > size)
            a[size] = null;
        return a;
    }

这个方法会报出警告：

ArrayList.java:305: warning: [unchecked] unchecked cast
return (T[]) Arrays.copyOf(elements, size, a.getClass());
                    ^
required: T[]
found: Object[]

在android平台上是将整个方法添加了@SuppressWarnings("unchecked")，但是尽量不要这样做。你可以缩小注解的范围像这样：

public <T> T[] toArray(T[] a) {
    if (a.length < size) {
        // This cast is correct because the array we're creating
        // is of the same type as the one passed in, which is T[].
        @SuppressWarnings("unchecked") T[] result = 
        (T[])Arrays.copyOf(elements, size, a.getClass());
        return result;
    }
    System.arraycopy(elements, 0, a, 0, size);
    if (a.length > size)
        a[size] = null;
    return a;
}

每次当你使用@SuppressWarnings("unchecked")注解时，都应该添加注释说明为什么这么做是安全的。这样做不仅可以使其他人更容易读懂，而且防止别人随意更改代码。