Effective Java (方法) « 关注Java

公告:上个空间遭黑客攻击,刚恢复换了空间,数据全部丢失,我也尽力挽回了部分链接,请各位交换链接的博友,看到后留言给我。

Effective Java (方法)

SyndContentImpl.value=

三十八、检查参数的有效性:

      绝大多数方法和构造器对于传递给它们的参数值都会有些限制。比如,索引值必须大于等于0,且不能超过其最大值,对象不能为null等。这样就可以在导致错误的源头将错误捕获,从而避免了该错误被延续到今后的某一时刻再被引发,这样就是加大了错误追查的难度。就如同编译期能够报出的错误总比在运行时才发现要更好一些。事实上,我们不仅仅需要在函数的内部开始出进行这些通用的参数有效性检查,还需要在函数的文档中给予明确的说明,如在参数非法的情况下,会抛出那些异常,或导致函数返回哪些错误值等,见如下代码示例:

 1     /** 2      * Returns a BigInteger whose value is(this mod m). This method 3      * differs from the remainder method in that it always returns a 4      * non-negative BigInteger. 5      * @param m the modulus, which must be positive. 6      * @return this mod m. 7      * @throws ArithmeticException if m is less than or equal to 0. 8 */ 9      public BigInteger mod(BigInteger m) {10          if (m.signum() <= 0)11              throw new ArithmeticException("Modulus <= 0: " + m);12          ... //Do the computation.13      }

       是不是我们为所有的方法均需要做出这样的有效性检查呢?对于未被导出的方法,如包方法等,你可以控制这个方法将在哪些情况下被调用,因此这时可以使用断言来帮助进行参数的有效性检查,如:

1      private static void sort(long a[],int offset,int length) {2          assert(a != null);3          assert(offset >= 0 && offset <= a.length);4          assert(length >= 0 && length <= a.length - offset);5          ... //Do the computation6      }

       和通用的检查方式不同,断言在其条件为真时,无论外部包得客户端如何使用它。断言都将抛出AssertionError。它们之间的另一个差异在于如果断言没有起到作用,即-ea命令行参数没有传递给java解释器,断言将不会有任何开销,这样我们就可以在调试期间加入该命令行参数,在发布时去掉该命令行选项,而我们的代码则不需要任何改动。
       需要强调的是,对于有些函数的参数,其在当前函数内并不使用,而是留给该类其他函数内部使用的,比较明显的就是类的构造函数,构造函数中的很多参数都不一样用于构造器内,只是在构造的时候进行有些赋值操作,而这些参数的真正使用者是该类的其他函数,对于这种情况,我们就更需要在构造的时候进行参数的有效性检查,否则一旦将该问题释放到域函数的时候,再追查该问题的根源,将不得不付出更大的代价和更多的调试时间。
       对该条目的说法确实存在着一种例外情况,在有些情况下有效性检查工作的开销是非常大的,或者根本不切实际,因为这些检查已经隐含在计算过程中完成了,如Collections.sort(List),容器中对象的所有比较操作均在该函数执行时完成,一旦比较操作失败将会抛出ClassCastException异常。因此对于sort来讲,如果我们提前做出有效性检查将是毫无意义的。
    
三十九、必要时进行保护性拷贝:

      如果你的对象没有做很好的隔离,那么对于调用者而言,则有机会破坏该对象的内部约束条件,因此我们需要保护性的设计程序。该破坏行为一般由两种情况引起,首先就是恶心的破坏,再有就是调用者无意识的误用,这两种条件下均有可能给你的类带来一定的破坏性,见如下代码:

 1     public final class Period { 2         private final Date start; 3         private final Date end; 4         public Period(Date start,Date end) { 5             if (start.compareTo(end) > 0) { 6                 throw new IllegalArgumentException(start + "After " + end); 7             this.start = start; 8             this.end = end; 9         }10         public Date start() {11             return start;12         }13         public Date end() {14             return end;15         }16     }

      从表面上看,该类的实现确实对约束性的条件进行了验证,然而由于Date类本身是可变了,因此很容易违反这个约束,见如下代码:

1     public void testPeriod() {2         Date start = new Date();3         Date end = new Date();4         Period p = new Period(start,end);5         end.setYear(78);  //该修改将直接影响Period内部的end对象。6     }

      为了避免这样的攻击,我们需要对Period的构造函数进行相应的修改,即对每个可变参数进行保护性拷贝。

1     public Period(Date start,Date end) {2         this.start = new Date(start.getTime());3         this.end = new Date(end.getTime());4         if (start.compareTo(end) > 0) {5             throw new IllegalArgumentException(start + "After " + end);6     }

      需要说明的是,保护性拷贝是在坚持参数有效性之前进行的,并且有效性检查是针对拷贝之后的对象,而不是针对原始对象的。这主要是为了避免在this.start = new Date(start.getTime())到if (start.compareTo(end) > 0)这个时间窗口内,参数start和end可能会被其他线程修改。
      现在构造函数已经安全了,后面我们需要用同样的方式继续修改另外两个对象访问函数。

1     public Date start() {2         return new Date(start.getTime());3     }4     public Date end() {5         return new Date(end.getTime());6     }

      经过这一番修改之后,Period成为了不可变类,其内部的“周期的起始时间不能落后于结束时间”约束条件也不会再被破坏。
      参数的保护性拷贝并不仅仅针对不可变类。每当编写方法或者构造器时,如果它要允许客户提供的对象进入到内部数据结构中,则有必要考虑一下,客户提供的对象进入到内部数据结构中,则有必要考虑一下,客户提供的对象是否有可能是可变的。如果是,就要考虑你的类是否能够容忍对象进入数据结构之后发生变化。如果答案是否定的,就必须对该对象进行保护性拷贝,并且让拷贝之后的对象而不是原始对象进入到数据结构中。例如,如果你正在考虑使用有客户提供的对象引用作为内部Set实例的元素,或者作为内部Map实例的键(Key),就应该意识到,如果这个对象在插入之后再被修改,Set或者Map的约束条件就会遭到破坏。
    
四十一、谨慎重载:

      见下面一个函数重载的例子:

 1     public class CollectionClassfier { 2         public static String classify(Set<?> s) { 3             return "Set"; 4         } 5         public static String classify(List<?> l) { 6             return "List"; 7         } 8         public static String classify(Collection<?> c) { 9             return "Unknown collection";10         }11         public static void main(String[] args) {12             Collection<?>[] collections = {13                 new HashSet<String>(),14                 new ArrayList<BigInteger>(),15                 new HashMap<String,String>().values()16             };17             for (Collection<?> c : collections)18                 System.out.println(classify(c));19         }20     }

      这里你可能会期望程序打印出
      //Set
      //List
      //Unknown Collection
      然而实际上却不是这样,输出的结果是3个”Unknown Collection”。为什么会是这样呢?因为函数重载后,需要调用哪个函数是在编译期决定的,这不同于多态的运行时动态绑定。针对此种情形,该条目给出了一个修正的方法,如下:

1     public static String classify(Collection<?> c) {2         return c instanceof Set ? "Set" : c instanceof List 3             ? "List" : "Unknown Collection";4     }

      和override不同,重载机制不会像override那样规范,并且每次都能得到期望的结果。因此在使用时需要非常谨慎,否则一旦出了问题,就会需要更多的时间去调试。该条目给出以下几种尽量不要使用重载的情形:
      1.    函数的参数中包含可变参数;
      2.    当函数参数数目相同时,你无法准确的确定哪一个方法该被调用时;
      3.    在Java 1.5 之后,需要对自动装箱机制保持警惕。
      我们先简单说一下第二种情形。比如两个重载函数均有一个参数,其中一个是整型,另一个是Collection<?>,对于这种情况,int和Collection<?>之间没有任何关联,也无法在两者之间做任何的类型转换,否则将会抛出ClassCastException的异常,因此对于这种函数重载,我们是可以准确确定的。反之,如果两个参数分别是int和short,他们之间的差异就不是这么明显。
      对于第三种情形,该条目给出了一个非常典型的用例代码,如下:

 1     public class SetList { 2         public static void main(String[] args) { 3             Set<Integer> s = new TreeSet<Integer>(); 4             List<Integer> l = new ArrayList<Integer>(); 5             for (int i = -3; i < 3; ++i) { 6                 s.add(i); 7                 l.add(i); 8             } 9             for (int i = 0; i < 3; ++i) {10                 s.remove(i);11                 l.remove(i);12             }13             System.out.println(s + " " + l);14         }15     }

      在执行该段代码前,我们期望的结果是Set和List集合中大于等于的元素均被移除出容器,然而在执行后却发现事实并非如此,其结果为:
      [-3,-2,-1] [-2,0,2]
      这个结果和我们的期望还是有很大差异的,为什么Set中的元素是正确的,而List则不是,是什么导致了这一结果的发生呢?下面给出具体的解释:
      1. s.remove(i)调用的是Set中的remove(E),这里的E表示Integer,Java的编译器会将i自动装箱到Integer中,因此我们得到了想要的结果。
      2. l.remove(i)实际调用的是List中的remove(int index)重载方法,而该方法的行为是删除集合中指定索引的元素。这里分别对应第0个,第1个和第2个。
      为了解决这个问题,我们需要让List明确的知道,我们需要调用的是remove(E)重载函数,而不是其他的,这样我们就需要对原有代码进行如下的修改:

 1     public class SetList { 2         public static void main(String[] args) { 3             Set<Integer> s = new TreeSet<Integer>(); 4             List<Integer> l = new ArrayList<Integer>(); 5             for (int i = -3; i < 3; ++i) { 6                 s.add(i); 7                 l.add(i); 8             } 9             for (int i = 0; i < 3; ++i) {10                 s.remove(i);11                 l.remove((Integer)i); //or remove(Integer.valueOf(i));12             }13             System.out.println(s + " " + l);14         }15     }

      该条目还介绍了一种实现函数重载,同时又尽可能避免上述错误发生的方式。即其中的一个重载函数,在其内部通过一定的转换逻辑转换之后,再通过转换后的参数类型调用其他的重载函数,从而确保即便使用者在使用过程中出现重载误用的情况,也因两者可以得到相同的结果而规避了潜在错误的发生。

四十二、慎用可变参数:

      可变参数方法接受0个或者多个指定类型的参数。可变参数机制通过先创建一个数组,数组的大小为在调用位置所传递的参数数量,然后将参数值传到数组中,最后将数组传递给方法,如:

1     static int sum(int...args) {2         int sum = 0;3         for (int arg : args)4             sum += arg;5         retrun sum;6     }

      上面的方法可以正常的工作,但是在有的时候,我们可能需要至少一个或者多个某种类型参数的方法,如:

 1     static int min(int...args) { 2         if (args.length == 0) 3             throw new IllegalArgumentException("Too few arguments."); 4         int min = args[0]; 5         for (int i = 0; i < args.length; ++i) { 6             if (args[i] < min) 7                 min = args[i]; 8         } 9         return min;10     }

      对于上面的代码主要存在两个问题,一是如果调用者没有传递参数是,该函数将会在运行时抛出异常,而不是在编译期报错。另一个问题是这样的写法也是非常不美观的,函数内部必须做参数的数量验证,不仅如此,这也影响了效率。将编译期可以完成的事情推到了运行期。下面提供了一种较好的修改方式,如下:

1     static int min(int firstArg,int...remainingArgs) {2         int min = firstArgs;3         for (int arg : remainingArgs) {4             if (arg < min)5                 min = arg;6         }7         return min;8     }

      由此可见,当你真正需要让一个方法带有不定数量的参数时,可变参数就非常有效。
      有的时候在重视性能的情况下,使用可变参数机制要特别小心。可变参数方法的每次调用都会导致进行一次数组分配和初始化。如果确定确实无法承受这一成本,但又需要可变参数的灵活性,还有一种模式可以弥补这一不足。假设确定对某个方法95%的调用会有3个或者更少的参数,就声明该方法的5个重载,每个重载方法带有0个至3个普通参数,当参数的数目超过3个时,就使用一个可变参数方法:

1     public void foo() {}2     public void foo(int a1) {}3     public void foo(int a1,int a2) {}4     public void foo(int a1,int a2,int a3) {}5     public void foo(int a1,int a2,int a3,int...rest) {}

      所有调用中只有5%参数数量超过3个的调用需要创建数组。就像大多数的性能优化一样,这种方法通常不恰当,但是一旦真正需要它时,还是非常有用处的。
    
四十三、返回零长度的数组或者集合,而不是null:

      见如下代码:

1     public class CheesesShop {2         private final List<Cheese> cheesesInStock = new List<Cheese>();3         public Cheese[] getCheeses() {4             if (cheesesInStock.size() == 0)5                 return null;6             return cheeseInStock.toArray(null);7         }8     }

      从以上代码可以看出,当没有Cheese的时候,getCheeses()函数返回一种特例情况null。这样做的结果会使所有的调用代码在使用前均需对返回值数组做null的判断,如下:

1     public void testGetCheeses(CheesesShop shop) {2         Cheese[] cheeses = shop.getCheeses();3         if (cheese != null && Array.asList(cheeses).contains(Cheese.STILTON))4             System.out.println("Jolly good, just the thing.");5     }

      对于一个返回null而不是零长度数组或者集合的方法,几乎每次用到该方法时都需要这种曲折的处理方式。很显然,这样是比较容易出错的。如果我们使getCheeses()函数在没有Cheese的时候不再返回null,而是返回一个零长度的数组,那么我的调用代码将会变得更加简洁,如下:

1     public void testGetCheeses2(CheesesShop shop) {2         if (Array.asList(shop.getCheeses()).contains(Cheese.STILTON))3             System.out.println("Jolly good, just the thing.");4     }

      相比于数组,集合亦是如此。

作者: Stephen_Liu 发表于 2012-01-25 08:03 原文链接

评论: 0 查看评论 发表评论


最新新闻:
· 心灵学家宣称阿波罗16号宇航员发现外星飞船(2012-01-25 22:59)
· 哪些软件应该强制开源?(2012-01-25 22:58)
· 俄罗斯科学家声称在金星发现生命(2012-01-25 22:57)
· Kelihos僵尸网络作者之一为杀毒软件公司工作(2012-01-25 22:56)
· 前谷歌员工谈离职前需做的14件事(2012-01-25 19:55)

编辑推荐:我是如何学习计算机编程的

网站导航:博客园首页  我的园子  新闻  闪存  博问  知识库

SyndContentImpl.interface=interface com.sun.syndication.feed.synd.SyndContent
SyndContentImpl.type=text/plain

声明:本文采用 BY-NC-SA 协议进行授权,本文链接:Effective Java (方法)