(转)小心重写方法，正确实现多态

这是我今天在工作中碰到的问题，是关于继承和多态的。同事对项目中的一项基础功能进行了重构，可是当我们从CVS服务器上更新了项目源代码并编译了之后，发现这项功能已不能正常工作了。先撇开这个同事所犯的错误（对代码进行重构后没有测试他的新代码就上传到了ＣＶＳ服务器上）不说，在这里我就说说这个问题所带出来的ＪＡＶＡ　ｋｏｎｗｈｏｗ．

由于我们的项目比较复杂，我在这里采取比较简单的例子来讲解这个问题。首先，我们有２个类：ｓｕｐｐｅｒ．ＴｅｓｔＳｕｐｐｅｒ．ｊａｖａ　和ｓｕｂ．ＴｅｓｔＳｕｂ．ｊａｖａ。他们的代码是这样的：

package supper;

public class TestSupper {    
    String getString()
    {
        return "This is supper class．";
    }
}
－－－－－－－－－－－－－－－－－－－－－－－－－－－－－－－－－－－
package sub;

import supper.TestSupper;

public class TestSub extends TestSupper {
  public String getString(){
        return "This is sub class.";
    }
}

十分简单，第一眼看上去，你会觉得ＴｅｓｔＳｕｂ继承了ＴｅｓｔＳｕｐｐｅｒ并且重写ｇｅｔＳｔｒｉｎｇ（）方法。
现在我们写一个测试程序：
package supper;

import sub.TestSub;

public class Test {
    public static void main(String[] args) {
        TestSupper test = new TestSub();
        System.out.println(test.getString());
    }
}

请注意啊！测试程序和ＴｅｓｔＳｕｐｐｅｒ在同一个包里。从理论上说，这个测试程序应该输出“This is sub class”，因为ｔｅｓｔ的实体是一个ＴｅｓｔＳｕｂ对象而不是ＴｅｓｔＳｕｐｐｅｒ。所以，当我们调用ｔｅｓｔ．ｇｅｔＳｔｒｉｎｇ（）时，真正被调用的应该是ＴｅｓｔＳｕｂ里的ｇｅｔＳｔｒｉｎｇ（）。可是事实如何呢？输出是"This is supper class．"！为什么会这样的？

原因很简单，因为在ＴｅｓｔＳｕｐｐｅｒ里方法ｇｅｔＳｔｒｉｎｇ（）的标签（ｓｉｇｎａｔｕｒｅ）是＂ｄｅｆａｕｌｔ＂，是默认的不用写出来。这导致这个方法只能在这个包里面可见。ＴｅｓｔＳｕｂ虽然继承了ＴｅｓｔＳｕｐｐｅｒ，却没有办法“看见”ｇｅｔＳｔｒｉｎｇ（）方法，因为ＴｅｓｔＳｕｂ在另外一个包里。所以当我们在测试程序里调用ｔｅｓｔ．ｇｅｔＳｔｒｉｎｇ（），程序首先会寻找ＴｅｓｔＳｕｂ中是否重写了这个方法，在这里请一定要注意标签是ｄｅｆａｕｌｔ的，当然是没有发现。结果程序就会调用父类的相应方法，故父类中的结果就被输出了。

解决的办法很简单，但凡是要被重写的方法一定不能定义成“ｄｅｆａｕｌｔ”，最少要定义成＂ｐｒｏｔｅｃｔｅｄ”．

如果你是在使用Ｅｃｌｉｐｓｅ的话，你可以在ｅｃｌｉｐｓｅ中进行设置，把这种情况视为Ｅｒｒｏｒ就可以避免这种错误的产生。
方法是：ｗｉｎｄｏｗ　－＞ｐｒｅｆｅｒｅｎｃｅｓ　－＞　ｓｔｙｌｅ　－＞ｍｅｔｈｏｄｓ　ｏｖｅｒｒｉｄｄｅｎ　ｂｕｔ　ｎｏｔ　ｐａｃｈａｇｅ　ｖｉｓｉｂｌｅ这项选为Ｅｒｒｏｒ。我用的是ｅｃｌｉｐｓｅ３．０。

后话
别看这是个很小的问题，而且很简单，一看就明白，可是当系统出现了问题，而你要在几百个类中寻找到问题所在的时候，这种不易察觉的错误绝对是致命的。寻找这个错误花了我们２个人天！可怕吧！