我学习使用java的一点体会（4）

      本文的上一篇发表之后，承蒙各位网友关注，发表了很多评论，我感觉很多人对我写得文章有误解，大概是我表述不清楚的原因吧。这篇文章是对上一篇的补充，以一个示例阐述了解collection框架的重要性。

      我在半年以前写过一个函数printAll(Vector vector)，具体代码如下

import java.util.*;

public class UtilTool
{
 public static void printAll ( Vector vector )
 {
  System.out.println( "the Collection is vector" );
  System.out.println( vector.getClass().toString() );
  Iterator iterator = vector.iterator();
  while ( iterator.hasNext() )
  {
   System.out.println(iterator.next().toString());
  }
 }

public static void main( String[] arg )
 {
 Vector vector = new Vector();
  vector.add( new Integer(1));
  vector.add( new Integer(2));
  vector.add( new Integer(3));
  UtilTool.printAll(vector);
}

}

      printAll这个函数设计的很不好——不够通用，假如，还想打印HashSet类型的数据，你就必须重载printAll函数，代码如下

public static void printAll ( HashSet hashSet )
 {
  System.out.println( "the Collection is hashSet" );
  System.out.println( hashSet.getClass().toString() );
  Iterator iterator = hashSet.iterator();
  while ( iterator.hasNext() )
  {
   System.out.println(iterator.next().toString());
  }
 }

      printAll函数的代码重用率低。其实Vector和 HashSet都是Collection的实现，可以将printAll的参数类型改为Collection，而不必重载。代码如下

public static void printAll ( Collection collection )
 {
  System.out.println( "the Collection is collection" );
  System.out.println( collection.getClass().toString() );
  Iterator iterator = collection.iterator();
  while ( iterator.hasNext() )
  {
   System.out.println(iterator.next().toString());
  }
 }
      这样就可以删除printAll(Vector vector)和printAll(HashSet hashSet)函数了。

      在设计函数时，应优先使用接口，而不是类。当然必须了解Vector 是Collection的实现。

      如果对Collection的继承关系不清楚，很容易滥用重载，以下代码是一个有问题的代码（摘自Effective Java Programming Language Guide）

public class CollectionClassifier{

    public static String classify(Set s){

        return "Set";

    }

    public static String classify(List l){

        return "List";

    }

    public static String classify(Collection c){

        return "Unknow Collection";

    }

    public static void main( String[] args )

        Collection[] tests = new Collection[]{

            new HashSet(),

            new ArrayList(),

            new HashMap().values()

       }

       for(int i=0;i<tests.length;i++)

        System.out.println(classify(test[i]));

     }

}

程序输出的是三次"Unknown Collection"，而不是你期望的打印"Set","List"以及"Unknown Collection"。这个程序错误的根源是对Collection层次结构不熟悉，而滥用重载导致。