消除非受检警告

2018-07-26  本文已影响0人  没走过的二丁目

使用泛型编程时,会遇到许多编译器警告,如:非受检强制转化警告,非受检方法调用警告,非受检普通数组创建警告,非受检转换警告。当你越来越熟悉泛型之后,遇到的警告也会越来越少,但是不要期待从一开始用泛型编写代码就可以正确的进行编译。
PS:非受检警告就是代码上黄色的感叹号
有许多非受检警告很容易消除,例如:

 ArrayList c = new ArrayList();
  c.add("d");

 ArrayList<String> strList = new ArrayList<>();
        strList.add("d");

这个非受检警告只要在ArrayList后边加入对应泛型并在之后的代码中应用这个变量就可消除了。

       有些警告比较难以消除。本章节要介绍这种警告的示例。当你遇到需要进行一番思考的警告时,要坚持住!要尽可能的消除每一个非受检的警告。如果,消除了所有的警告,就可以确保代码类型是安全的,这是一件很好的事情。这意味着不会在运行期间出现ClassCastException异常,你会更加自信自己的程序可以实现预期的功能。

       如果无法消除警告,同时可以证明该代码类型是安全的,(只有在这种情况下才)可以使用一个Supppress Warnings(“unchecked”)注解来禁止警告。如果在禁止警告之前没有先证实代码类型是安全的,那就只是给你自己一种错误的安全感而已。代码在编译的时候可能没有出现任何警告,但是他在运行期间仍会抛出ClassCastException异常。但是如果忽略(而不禁止)明知道是安全的非受检警告,那么当新出现一条真正有问题的警告时,你也不会注意到,新出现的警告就会淹没在所有的错误警告中
       SupppressWarnings注解可以用任何颗粒度的级别中,从单独的局部变量声明,到整个类都可以,应该始终在尽可能小的范围中使用SupppressWarnings注解,他通常是个变量声明,或者 非常简短的方式或者构造器。永远不要在整个类上使用这样的注解,这么做可能会掩盖了重要的警告。 如果你发现自己在长度不止一行的方法或者构造器中使用了这样的注解,可以将它移到一个局部的变量的声明中。虽然你必须声明一个新的局部变量,不过这么做还是值得的。例如:考虑ArrayList类当中的toArray方法

public <T> T[] toArray(T[] var1) {
        if (var1.length < this.size) {
            return (T[])Arrays.copyOf(this.elementData, this.size, var1.getClass());//出现警告
        } else {
            System.arraycopy(this.elementData, 0, var1, 0, this.size);
            if (var1.length > this.size) {
                var1[this.size] = null;
            }

            return var1;
        }
    }

如上代码中会出现这样的警告found:object[ ] ,required T[] ,将suppress warnings 注解放到return上是非法的,因为它不是一个声明,你可以试着将注解放到整个方法上,但是在实践中千万不要这么做,而是应该声明一个局部变量来保存返回值,并注解其声明

 public <T> T[] toArray(T[] var1) {
        if (var1.length < this.size) {
//this cast is correct because the array we are creating
//is the same type as the one passed in ,which is T[]
            @suppress warnings("unchekced") T[] result =
                    (T[]) Arrays.copyOf(this.elementData, this.size, var1.getClass());
            return result;
        } else {
            System.arraycopy(this.elementData, 0, var1, 0, this.size);
            if (var1.length > this.size) {
                var1[this.size] = null;
            }

            return var1;
        }
    }

每当使用@SuppressWarnings("unchecked")注解时,都要添加一条注释,说明为什么这么做是安全的。这样可以帮助他人理解代码,更重要的是,可以尽量减少其他人修改代码后导致计算不安全的概率。如果你觉得这种注释很难编写,就要多加思考,最终你会发现非受检操作是非常不安全的。

    总而言之,非受检的警告很重要,不要忽略他们。每一条警告都表示可能在运行时抛出ClassCastException异常。要尽最大的努力消除这些警告。如果无法消除非受检警告,同时可以证明引起警告代码的类型安全的,就可以在尽可能小的范围内,用@SuppressWarnings 注解禁止警告,要用注释把禁止该警告的原因记录下来。

总结:
1、要尽可能地消除每一个非受检警告。
2、如果无法消除警告,同时可以证明引起警告的代码是类型安全的,(只有这种情况下 才)可以用一个@SuppressWarnings("unchecked")注释来禁止这条警告。
3、应该始终在尽可能小的范围中使用SuppressWarnings注释。
4、每当使用SuppressWarnings注释时,都要添加一条注释,说明为什么这么做是安全的。

上一篇下一篇

猜你喜欢

热点阅读