在一个风和日丽的下午,码农A写了一个可以获取数据的组件,代码是这样的:
package net.gerhut.example.authora;
public class Component {
public void getData()
{
System.out.println("Transferring data...");
};
}
码农B负责整个系统开发,使用码农A设计的组件,认为码农A的获取数据应该有个回调函数,就给它加了一个回调函数,代码是这样的:
package net.gerhut.example.authorb;
import net.gerhut.example.authora.Component;
public class MainSystem {
private Component component;
private class MainComponent extends Component
{
public void getData()
{
super.getData();
getDataHandle();
}
}
private MainSystem()
{
component = new MainComponent();
}
private void getDataHandle()
{
System.out.println("MainSystem Data Refreshed.");
}
private void refresh()
{
component.getData();
}
public static void main(String[] args) {
new MainSystem().refresh();
}
}
运行结果一切正常。
Transferring data...
MainSystem Data Refreshed.
在另一个风和日丽的下午,码农A良心发现,觉得自己的组件没有回调函数太不好了,就给自己的组件加了一个回调函数,于是这个组件的2.0版就变成了这样:
package net.gerhut.example.authora;
public class Component {
public void getDataHandle()
{
System.out.println("Component Data Refreshed.");
}
public void getData()
{
System.out.println("Transferring data...");
getDataHandle();
};
}
码农B已经交差给人干别的去了,结果码农A的看似不影响整个系统的改变却导致了整个系统的瘫痪,运行结果变成了现在这样。
Transferring data...
Component Data Refreshed.
Component Data Refreshed.
也就是说,码农A的一个功能添加的操作居然也能导致整个系统的瘫痪,实在是匪夷所思,问题出现在哪里呢。其实出现在码农B的代码不规范上,我认为这里是Java的一个设计缺陷。内部类和匿名类的this的多义性导致了其指向的改变。
内部类的this有三种指向:
- 本类的成员,即this.getDataHandle
- 父类的成员,即super.getDataHandle
- 所属类的成员,即MainSystem.this.getDataHandle
其优先级是从1最高3最低,如果强制指定低优先级的this需要改成全称。码农B在this的指向上没有考虑到将来可能会有的的升级,直接指代了优先级最低的对象,是一种很不好的习惯,所以说,this一般来说就指本类的成员,其他七七八八的成员最好使用super或者所属类.this的指代方法,避免类似问题不再发生。