什麽是重构
可以运作的程式跟可以维护的程式之间，还有一道难以言说的鸿沟。
一个程式设计之初，是用来解决特定问题。就像在前面章节的学习中，我们也已经写好了一个可以运作的 BMI 程式。但是对程式设计来说，当我们写越多程式，我们会希望可以从这些程式之中，找到一个更广泛适用的法则，让每个程式都清晰易读，从而变得更好修改与维护。
让程式清晰易读有什麽好处呢？当一段程式被写出来，之后我们所要做的事，就是修改它与维护它。一旦程式越长越複杂，混乱到无法维护的境界时，就只好砍掉重练。 所以若我们能透过某些方式，例如重新组织或部分改写程式码，好让程式容易维护，那麽我们就可以为自己省下许多时间，以从容迎接新的挑战。
我们回过头来看看前面所写的 Android 程式。Android 平台的开发者已经先依照 MVC 模式，为我们将显示介面所用的 XML 描述档、显示资源所用的 XML 描述档从程式码中区隔开来。将与程式流程无关的部份分开来组织，让程式流程更清楚，相对易于维护。
而在主要程式码(Bmi.java)方面，虽然程式码量很少，还算好读，但整体上并不那麽令人满意。例如，假使我们要在这段程式码中再多加上按键、适用于多种萤幕显示模式、或是再加入选单等等内容，很快地程式码就开始变得複杂，变得不容易阅读，也开始越来越不容易维护。
因此，在继续新的主题之前，我们先来重构这个 BMI 应用程式。在重构的过程中，也许我们能学到的东西，比学任何新主题还重要哩。
MVC
我们打算重构 BMI 程式的部份 java 程式码。既然我们已经照著 Android 平台的作法，套用 MVC 模式在我们的程式组织上，那麽，我们不妨也试著套用同样的 MVC 模式在 Bmi.java 程式码上。
如何套用 MVC 模式到Bmi.java 程式码上哩？
原来的程式片段是这样的
[java] view plaincopyprint?
@Override
public void onCreate(Bundle savedInstanceState) {
super.onCreate(savedInstanceState);
setContentView(R.layout.main);
//Listen for button clicks
Button button = (Button) findViewById(R.id.submit);
button.setOnClickListener(calcBMI);
}
上面的程式片段中，包含了所有 Android 程式共用的标准内容， 整个程式的大致架构在前面章节中已经讲解过，现在我们从中取出我们感兴趣的部分来讨论：
[java] view plaincopyprint?
Button button = (Button) findViewById(R.id.submit);
button.setOnClickListener(calcBMI);
在第7行我们看到一段程式码来宣告按钮物件，与针对该按钮物件作动作的程式码。 button.setOnClickListener 程式码的意义是指定一个函式，来负责处理"按下"这个"按钮"后的动作。
我们可以想像，在同一个画面中，多加入一些按钮与栏位后，"onCreate" 这段程式将变得壅肿，我们来试著先对此稍作修改：
首先，我们可以套用 MVC 模式，将宣告介面元件(按钮、数字栏位)、指定负责函式等动作抽取出来，将 onCreate 函式改写如下
[java] view plaincopyprint?
@Override
public void onCreate(Bundle savedInstanceState) {
super.onCreate(savedInstanceState);
setContentView(R.layout.main);
findViews();
setListeners();
}
接著我们将宣告介面元件的部份写成一个独立的「findViews」函式：
[java] view plaincopyprint?
private Button calcbutton;
private EditText fieldheight;
private EditText fieldweight;
private void findViews()
{
calcbutton = (Button) findViewById(R.id.submit);
fieldheight = (EditText) findViewById(R.id.height);
fieldweight = (EditText) findViewById(R.id.weight);
}
顺便将原本很没个性的按钮识别参数「button」改名成「calcbutton」，以后在程式中一看到「calcbutton」，就知道是一个按下后将开始处理计算工作的按钮。
同样地，我们也将指定特定动作(按按钮)的负责函式独立出来：
[java] view plaincopyprint?
//Listen for button clicks
private void setListeners() {
calcbutton.setOnClickListener(calcBMI);
}
如此一来，我们就将程式逻辑与介面元件的宣告分离开来，达成我们重构的目的。
完整程式如下：
[java] view plaincopyprint?
package com.demo.android.bmi;
import java.text.DecimalFormat;
import android.app.Activity;
import android.os.Bundle;
import android.view.View;
import android.widget.Button;
import android.widget.EditText;
import android.widget.TextView;
public class Bmi extends Activity {
/** Called when the activity is first created. */
@Override
public void onCreate(Bundle savedInstanceState) {
super.onCreate(savedInstanceState);
setContentView(R.layout.main);
findViews();
setListeners();
}
private Button button_calc;
private EditText field_height;
private EditText field_weight;
private TextView view_result;
private TextView view_suggest;
private void findViews()
{
button_calc = (Button) findViewById(R.id.submit);
field_height = (EditText) findViewById(R.id.height);
field_weight = (EditText) findViewById(R.id.weight);
view_result = (TextView) findViewById(R.id.result);
view_suggest = (TextView) findViewById(R.id.suggest);
}
//Listen for button clicks
private void setListeners() {
button_calc.setOnClickListener(calcBMI);
}
private Button.OnClickListener calcBMI = new Button.OnClickListener()
{
public void onClick(View v)
{
DecimalFormat nf = new DecimalFormat("0.0");
double height = Double.parseDouble(field_height.getText().toString())/100;
double weight = Double.parseDouble(field_weight.getText().toString());
double BMI = weight / (height * height);
//Present result
view_result.setText(getText(R.string.bmi_result) + nf.format(BMI));
//Give health advice
if(BMI>25){
view_suggest.setText(R.string.advice_heavy);
}else if(BMI<20){
view_suggest.setText(R.string.advice_light);
}else{
view_suggest.setText(R.string.advice_average);
}
}
};
}
同样是「calcBMI」 函式，在完整程式中，改将「calcBMI」 函式从原本的「OnClickListener」宣告成 「Button.OnClickListener」。这个改变有什麽差别呢？
阅读原本的程式码，在汇入(import)的部分可以看到，「OnClickListener」是来自于「android.view.View.OnClickListener」函式：
[java] view plaincopyprint?
import android.view.View.OnClickListener;
改成 「Button.OnClickListener」后，「Button.OnClickListener」就变成来自于「android.widget.Button」中的「OnClickListener」函式，在查阅程式时，整个「Button」与「OnClickListener」之间的关系变得更清晰。
另外，我们偷偷将「OnClickListener」中其他会存取到的介面元件识别参数，也补进 findViews 宣告中：
[java] view plaincopyprint?
private void findViews()
{
button_calc = (Button) findViewById(R.id.submit);
field_height = (EditText) findViewById(R.id.height);
field_weight = (EditText) findViewById(R.id.weight);
view_result = (TextView) findViewById(R.id.result);
view_suggest = (TextView) findViewById(R.id.suggest);
}
同时，我们也把识别参数的命名方法做了统一：按钮的识别参数前加上 「button_」前缀，可输入栏位的识别参数前加上 「field_」前缀，用作显示的识别参数前则加上「view_」前缀。将变数名称的命名方法统一有什麽好处呢？ 好处在于以后不管是在命名新变数，或是阅读程式码时，都能以更快速度命名或理解变数的意义，让程式变得更好读。
我们也把原本在程式中直接写进的字串
[java] view plaincopyprint?
TextView result = (TextView) findViewById(R.id.result);
result.setText("Your BMI is "+nf.format(BMI));
改写成
[java] view plaincopyprint?
//Present result
view_result.setText(getText(R.string.bmi_result) + nf.format(BMI));
并将「TextView view_result」宣告改放到 findViews 中一次处理好。
现在，整个程式流程是不是清爽了许多呢？