这是我今天在工作中碰到的问题,是关于继承和多态的。同事对项目中的一项基础功能进行了重构,可是当我们从CVS服务器上更新了项目源代码并编译了之后,发现这项功能已不能正常工作了。先撇开这个同事所犯的错误(对代码进行重构后没有测试他的新代码就上传到了CVS服务器上)不说,在这里我就说说这个问题所带出来的JAVAkonwhow.
由于我们的项目比较复杂,我在这里采取比较简单的例子来讲解这个问题。首先,我们有2个类:supper.TestSupper.java和sub.TestSub.java。他们的代码是这样的:
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.";
}
}
十分简单,第一眼看上去,你会觉得TestSub继承了TestSupper并且重写getString()方法。
现在我们写一个测试程序:
package supper;
import sub.TestSub;
public class Test {
public static void main(String[] args) {
TestSupper test = new TestSub();
System.out.println(test.getString());
}
}
请注意啊!测试程序和TestSupper在同一个包里。从理论上说,这个测试程序应该输出“This is sub class”,因为test的实体是一个TestSub对象而不是TestSupper。所以,当我们调用test.getString()时,真正被调用的应该是TestSub里的getString()。可是事实如何呢?输出是"This is supper class."!为什么会这样的?
原因很简单,因为在TestSupper里方法getString()的标签(signature)是"default",是默认的不用写出来。这导致这个方法只能在这个包里面可见。TestSub虽然继承了TestSupper,却没有办法“看见”getString()方法,因为TestSub在另外一个包里。所以当我们在测试程序里调用test.getString(),程序首先会寻找TestSub中是否重写了这个方法,在这里请一定要注意标签是default的,当然是没有发现。结果程序就会调用父类的相应方法,故父类中的结果就被输出了。
解决的办法很简单,但凡是要被重写的方法一定不能定义成“default”,最少要定义成"protected”.
如果你是在使用Eclipse的话,你可以在eclipse中进行设置,把这种情况视为Error就可以避免这种错误的产生。
方法是:window->preferences->style->methodsoverriddenbutnotpachagevisible这项选为Error。我用的是eclipse3.0。
后话
别看这是个很小的问题,而且很简单,一看就明白,可是当系统出现了问题,而你要在几百个类中寻找到问题所在的时候,这种不易察觉的错误绝对是致命的。寻找这个错误花了我们2个人天!可怕吧!