从android 谈重构

什麽是重构

可以运作的程式跟可以维护的程式之间，还有一道难以言说的鸿沟。

一个程式设计之初，是用来解决特定问题。就像在前面章节的学习中，我们也已经写好了一个可以运作的 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);  

     }      

[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));