Android开发，别把Java当Javascript
Android开发中，匿名类的应用使得开发更加灵活，而匿名类的滥用却让程序变得杂乱无章、难以维护。程序员都是喜欢偷懒的，我们常常看到一个Activity类中布满了匿名类，有Thread、有Handler、有Adapter、有各种Listener以及给匿名类传递数据的final变量，给人的感觉就像随地大小便，很不文明。
Java毕竟不是Javascript，没有闭包和没有匿名函数，把面向对象设计的语言用函数式编程的思想来写是很牵强的。下面贴一段JS代码：
MongoClient.connect('mongodb://127.0.0.1:27017/test', function(err, db) {
if(err) throw err;
db.dropDatabase(function(err, done) {
db.createCollection('test_custom_key', function(err, collection) {
collection.insert({'a':1}, function(err, docs) {
collection.find({'_id':new ObjectID("aaaaaaaaaaaa")}).toArray(function(err, items) {
console.dir(items);
// Let's close the db
db.close();
});
});
});
});
});
得益于JS的闭包、匿名函数、函数即变量等特性，JS开发者可以将函数嵌套发挥到令人叹为观止的地步，而且即使套了很多层，也不会显得很臃肿。你能想想如果Java套这么多层会是什么效果吗？或许编辑器都显示不下最长的缩进了，但能多Android开发者就好这口，来看个例子：
public class MyActivity extends Activity { @Override protected void onCreate(Bundle savedInstanceState) { super.onCreate(savedInstanceState); ((ListView) findViewById(R.id.listview)).setAdapter(new BaseAdapter() { @Override public View getView(int arg0, View arg1, ViewGroup arg2) { if (arg1 == null) { arg1 = LayoutInflater.from(MyActivity.this).inflate( R.id.feed, arg2, false); } arg1.findViewById(R.id.subject).setOnClickListener( new OnClickListener() { @Override public void onClick(View v) { final Handler handler = new Handler( new Callback() { @Override public boolean handleMessage( Message msg) { // 此处省略50行 return false; } }); new Thread(new Runnable() { @Override public void run() { // 此处省略100行 handler.sendEmptyMessage(0); } }).start(); } }); return arg1; } // 。。。。。。 }); } }
这是个真实的例子，其中很多习惯很多安卓开发者也有。这个onCreate函数竟然有近千行代码，已经完全丧失可读性了。如果你走了有人来接替你的活儿，看到这样的代码他一定会把你骂死。
下面我就Android开发中常见的匿名类给出自己的替代方案，仅供参考，错误之处还请指证。
用继承接口代替匿名类
这种方式适合各种Listener，这里以OnClickListener为例作说明。绑定事件一般是在onCreate时完成的，所以在onCreate中可能会有很多个OnClickListener的匿名类，比如：
findViewById(R.id.button).setOnClickListener(new OnClickListener() { @Override public void onClick(View arg0) { // .... } });
这时onCreate方法就会变得冗长，而且包含很多不应该出现在onCreate中的处理逻辑，如果我们让Activity去实现OnClickListener，将所有点击处理的listener都设为这个activity，在onClick中再根据view.getId()来分支处理，程序的结构就会紧凑很多：
public class DummyActivity extends Activity implements OnClickListener { //..... @Override protected void onCreate(Bundle savedInstanceState) { for(int id : in new int[] { R.id.button }) { findViewById(id).setOnClickListener(this); } } @Override public void onClick(View arg0) { switch(arg0.getId()) { case R.id.button: // ...... break; } } }
包括onLongClickListener等针对单一View的Listener都可以通过让Activity继承接口的方式来操作。如果是针对列表项的ItemListener，建议让Adapter去继承接口。
用子类代替匿名类
如果一个Activity中有AdapterView的子类，如ListView、GridView，就会设置相应的Adapter，除了最简单的情形可以用SimpleAdapter、ArrayAdapter直接映射，很多情况都都需要继承BaseAdapter之类后重写一些方法，逻辑比较简单时用内部类不失为可取办法。例如很多人喜欢在onCreate中这么写：
((ListView)findViewById(R.id.listView)).setAdapter(new BaseAdapter() { @Override public View getView(int arg0, View arg1, ViewGroup arg2) { //...... } //...... });
为了不在onCreate参杂不相关的逻辑，保持代码结构合理，你可以尝试这样处理：
public class DummyActivity extends Activity { //..... @Override protected void onCreate(Bundle savedInstanceState) { ((ListView)findViewById(R.id.listView)).setAdapter(new MyAdapter(....)); } private static class MyAdapter extends Base Adapter{ MyAdapter(...) { ....... } @Override public View getView(int arg0, View arg1, ViewGroup arg2) { ......... } } }
至于static与否则取决于你是否要用到动态成员变量了，这种方式使得参数传递也更加方便：只需通过构造函数传入，不必通过final关键字。
用外部类代替匿名类
对于Adapter这类逻辑简单的类可以用内部类处理，如果涉及到更加复杂的逻辑，比如数据处理、I/O、数据库操作等，建议用外部类使各个类的业务属性尽可能单一。比如Thread类，跟Activity类相关度本来就很低，为了偷懒非得硬生生塞进Activity中显然是不合适的，无论是匿名类还是子类的形式，这里就不举具体例子了。至于外部类的命名，如果想不到合适的名字，可考虑前缀命名，比如现在有一个DummyActivity，之前在里面塞了一个MyThread，那提出来再叫MyThread显然不合适，就以前缀命名的方式改为DummyActivityThread就一目了然了。
小结
Javascript是门很优秀的语言，拥有函数式编程的诸多特性，合理的灵活嵌套可以使代码紧凑而优雅，而Java是一门严格面向对象的语言，匿名类嵌套的滥用会让你的程序臃肿混乱，建议不要把Java写成了Javascript的风格，那样你会很难维护。
