我学习使用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层次结构不熟悉，而滥用重载导致。