不良代码展示-很多判断的写法-给新手

原创文章，如有转载，请注明出处：http://blog.youkuaiyun.com/yihui823/article/details/6801695

今天Review代码，看到了一串很“漂亮”的代码。这里把代码改了一下，贴给新手们，作为一个教训吧。
漂亮的意思，是有很工整的缩进。

package com.hoperun; import android.view.View; import android.view.View.OnClickListener; /** * 数字键盘监听器 */ public class NumListener implements OnClickListener { public void onClick(View v) { if (v.equals(btnNum0)) { //按键0 } else { if (v.equals(btnNum1)) { //按键1 } else { if (v.equals(btnNum2)) { //按键2 } else { if (v.equals(btnNum3)) { //按键3 } else { if (v.equals(btnNum4)) { //按键4 } else { if (v.equals(btnNum5)) { //按键5 } else { if (v.equals(btnNum6)) { //按键6 } else { if (v.equals(btnNum7)) { //按键7 } else { if (v.equals(btnNum8)) { //按键8 } else { if (v.equals(btnNum9)) { //按键9 } else { if (v.equals(btnNumAdd)) { //按键加法 } else { if (v.equals(btnNumSubtraction)) { //按键减法 } else { if (v.equals(btnNumEquals)) { //按键等于 } } } } } } } } } } } } } } }
其实，这么写是完全没有必要的。这么多的else，其实可以叠在else if里面，如下：

package com.hoperun; import android.view.View; import android.view.View.OnClickListener; /** * 数字键盘监听器 */ public class NumListener implements OnClickListener { public void onClick(View v) { if (v.equals(btnNum0)) { //按键0 } else { if (v.equals(btnNum1)) { //按键1 } else if (v.equals(btnNum2)) { //按键2 } else if (v.equals(btnNum3)) { //按键3 } else if (v.equals(btnNum4)) { //按键4 } else if (v.equals(btnNum5)) { //按键5 } else if (v.equals(btnNum6)) { //按键6 } else if (v.equals(btnNum7)) { //按键7 } else if (v.equals(btnNum8)) { //按键8 } else if (v.equals(btnNum9)) { //按键9 } else if (v.equals(btnNumAdd)) { //按键加法 } else if (v.equals(btnNumSubtraction)) { //按键减法 } else if (v.equals(btnNumEquals)) { //按键等于 } } }

其实这个效率不高。其实传入的参数View，就是被按住的那个按钮，不需要用equals判断，直接用“ == ”来判断就可以了。

当然，最好的还不是用“ == ”，而是用switch - case.

package com.hoperun; import android.view.View; import android.view.View.OnClickListener; /** * 数字键盘监听器 */ public class NumListener implements OnClickListener { public void onClick(View v) { switch (v.getId()) { case R.id.key_0: //按键0 break; case R.id.key_1: //按键1 break; case R.id.key_2: //按键2 break; case R.id.key_3: //按键3 break; case R.id.key_4: //按键4 break; case R.id.key_5: //按键0 break; case R.id.key_6: //按键6 break; case R.id.key_7: //按键7 break; case R.id.key_8: //按键8 break; } } }
仅给新人学习，高手勿喷。